Re: [PATCH 1/6] block: ensure we don't truncate top bits of the request command flags

From: Jens Axboe
Date: Tue Mar 22 2016 - 15:01:58 EST


On 03/22/2016 12:59 PM, Christoph Hellwig wrote:
On Tue, Mar 22, 2016 at 11:55:15AM -0600, Jens Axboe wrote:
Some of the flags that we want to use from the make_request_fn path
are now larger than 32-bit, so change the functions involved to
accept an u64 instead of an unsigned int.

When did we start doing that? We really should merge Mike's split
of the operation style flags into the cmd_type before making things
even worse in the flags area.

Just now, and I ran into it last week as well, for a test patch on cfq that passed in higher flags for get_request -> may_queue() as well. We can do Mike's split first, I think it's a good cleanup. As a standalone series, I needed it though.

--
Jens Axboe