Re: [PATCH] block: Make write_hints[] big enough

2018-10-18 Thread Avri Altman
>>> Your commit message doesn't really explain what you think the issue >>> is. We have 4 write hint types, with a fifth being "none". Where >>> are we exceeding these 5 hints? >> WRITE_LIFE_EXTREME = RWH_WRITE_LIFE_EXTREME = 5, >> will be ignored by queue_write_hint_{show_store}. >Hmm, wonder

Re: [PATCH] block: Make write_hints[] big enough

2018-10-18 Thread Avri Altman
>>> Your commit message doesn't really explain what you think the issue >>> is. We have 4 write hint types, with a fifth being "none". Where >>> are we exceeding these 5 hints? >> WRITE_LIFE_EXTREME = RWH_WRITE_LIFE_EXTREME = 5, >> will be ignored by queue_write_hint_{show_store}. >Hmm, wonder

Re: [PATCH] block: Make write_hints[] big enough

2018-10-18 Thread Jens Axboe
On 10/18/18 9:30 AM, Avri Altman wrote: >> -Original Message- >> From: Jens Axboe >> Sent: Thursday, October 18, 2018 5:38 PM >> To: Avri Altman ; linux-block@vger.kernel.org >> Cc: Avi Shchislowski ; Alex Lemberg >> >> Subject: Re: [P

RE: [PATCH] block: Make write_hints[] big enough

2018-10-18 Thread Avri Altman
> -Original Message- > From: Jens Axboe > Sent: Thursday, October 18, 2018 5:38 PM > To: Avri Altman ; linux-block@vger.kernel.org > Cc: Avi Shchislowski ; Alex Lemberg > > Subject: Re: [PATCH] block: Make write_hints[] big enough > > On 10/17/18 10:51 AM,

Re: [PATCH] block: Make write_hints[] big enough

2018-10-18 Thread Jens Axboe
On 10/17/18 10:51 AM, Avri Altman wrote: > Just stumbled over this. > Looks like the write hints array in the request queue is not allotted > with the required space to accommodate all the write hint types. > > fixes: f793dfd3f39a (blk-mq: expose write hints through debugfs) Your commit message

[PATCH] block: Make write_hints[] big enough

2018-10-17 Thread Avri Altman
Just stumbled over this. Looks like the write hints array in the request queue is not allotted with the required space to accommodate all the write hint types. fixes: f793dfd3f39a (blk-mq: expose write hints through debugfs) Signed-off-by: Avri Altman --- include/linux/blkdev.h | 2 +- 1 file