On Thu, Nov 21, 2019 at 2:37 AM <jgq516@xxxxxxxxx> wrote: > > From: Guoqing Jiang <guoqing.jiang@xxxxxxxxxxxxxxx> > > 1. The related resources (spin_lock, list and waitqueue) are needed for > address raid1 reorder overlap issue too, so add "is_force" parameter to > funcs (mddev_create/destroy_serial_pool). The parameter is set to true > if we want to enable or disable raid1 io serialization in later patch. Looks like is_force is always the same as "rdev == NULL"? Can we remove the is_force argument and use rdev == NULL instead? I guess it will also simplify the functions. Thanks, Song