On Wed, Dec 02, 2015 at 05:10:37PM +0100, Christoph Hellwig wrote: > This allows us to make guaranteed forward progress. > > Signed-off-by: Christoph Hellwig <hch@xxxxxx> > --- > drivers/md/raid5-cache.c | 16 +++++++++++++++- > 1 file changed, 15 insertions(+), 1 deletion(-) > > diff --git a/drivers/md/raid5-cache.c b/drivers/md/raid5-cache.c > index 668e973..ef59564 100644 > --- a/drivers/md/raid5-cache.c > +++ b/drivers/md/raid5-cache.c > @@ -34,6 +34,12 @@ > #define RECLAIM_MAX_FREE_SPACE (10 * 1024 * 1024 * 2) /* sector */ > #define RECLAIM_MAX_FREE_SPACE_SHIFT (2) > > +/* > + * We only need 2 bios per I/O unit to make progress, but ensure we > + * have a few more available to not get too tight. > + */ > +#define R5L_POOL_SIZE 1024 Looks reasonable, the recent arbitrary size bio makes things simpler. I think we don't need to be that conservative. One metadata page can hold at most (PAGE_SIZE - sizeof(struct r5l_meta_block)) / sizeof(struct r5l_payload_data_parity) pages. Thanks, Shaohua -- To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
