Re: [dm-devel] [PATCH 1/2] dm-mpath: Add mutex to synchronize adding and flushing work

[Kiyoshi Ueda <k-ueda ct jp nec com>]

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 ct jp nec com>

Thanks,
Kiyoshi Ueda