Hi Mike, On 11/16/2009 04:38 PM +0900, Mike Anderson wrote: > Add a mutex to allow possible creators of new work to synchronize with > flushing work queues. <snip> > @@ -1268,7 +1271,11 @@ static void multipath_presuspend(struct dm_target *ti) > > static void multipath_postsuspend(struct dm_target *ti) > { > + struct multipath *m = (struct multipath *) ti->private; Please drop the cast when it's from void. For others, Acked-by: Kiyoshi Ueda <k-ueda@xxxxxxxxxxxxx> Thanks, Kiyoshi Ueda -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel