Messages in this thread | | | Date | Thu, 28 Jul 2022 07:15:51 -0700 | From | Christoph Hellwig <> | Subject | Re: [PATCH 5/5] md/raid5: Ensure batch_last is released before sleeping for quiesce |
| |
On Wed, Jul 27, 2022 at 03:06:00PM -0600, Logan Gunthorpe wrote: > +static struct stripe_head *__raid5_get_active_stripe(struct r5conf *conf, > + struct stripe_request_ctx *ctx, sector_t sector, > + bool previous, bool noblock, bool noquiesce)
Passing three different bool arguments right after another is a really confusing calling convention, at some point this should become a set of flags. I'd also drop the __raid5_get_active_stripe vs raid5_get_active_stripe distinction.
| |