>       int (*startwrite)(struct mddev *mddev, sector_t offset,
> -                       unsigned long sectors);
> +                       unsigned long sectors, bool is_discard);
>       void (*endwrite)(struct mddev *mddev, sector_t offset,
> -                      unsigned long sectors);
> +                      unsigned long sectors, bool is_discard);

a bool discard is not a very good interface.  I'd expect an op enum or a set
of flag to properly describe it.

But is start/end write really the right interface for discard or should it
have it's own set of ops?


Reply via email to