Re: [PATCH 01/11] fs: add support for an inode to carry write hint related data
From: Jens Axboe <axboe@kernel.dk>
Date: 2017-06-19 14:55:35
Also in:
linux-fsdevel
From: Jens Axboe <axboe@kernel.dk>
Date: 2017-06-19 14:55:35
Also in:
linux-fsdevel
On 06/19/2017 12:26 AM, Christoph Hellwig wrote:
quoted
+/* + * Write life time hint values. + */ +enum rw_hint { + WRITE_LIFE_NONE = 0, + WRITE_LIFE_SHORT, + WRITE_LIFE_MEDIUM, + WRITE_LIFE_LONG, + WRITE_LIFE_EXTREME, +}; + +#define RW_HINT_MASK 0x7 /* 3 bits */FYI, exposing enums in a uapi is always a bit problematic, due to different ABI rules. It might be better to make these explicit defines at least for the uapi. Btw, I think it might make sense to merge this with patch 5.
OK, I'll change the uapi to be defines. -- Jens Axboe