On Tue, Feb 24, 2009 at 12:53 AM, Mikulas Patocka <mpatocka at redhat.com> wrote: <snip> > > +static int dm_flush(struct mapped_device *md) > +{ > + dm_wait_for_completion(md, TASK_UNINTERRUPTIBLE); > + return 0; > +} > + Always returns zero! Why not void? <snip> Thanks Nikanth