Re: [PATCH v4 3/6] block: change ioprio_valid() to an inline function

On 11/08/2021 10:51, Damien Le Moal wrote:

On 2021/08/11 16:56, Johannes Thumshirn wrote:

quoted

On 11/08/2021 05:37, Damien Le Moal wrote:

quoted

Change the ioprio_valid() macro in include/usapi/linux/ioprio.h to an                                        uapi ~^

quoted

inline function declared on the kernel side in include/linux/ioprio.h.
Also improve checks on the class value by checking the upper bound
value.

But I think it needs to stay in include/uapi/linux/ioprio.h as it's there
since the 2.6.x days (I've checked back to v2.6.39.4) so the chance of
user-space using it is quite high.

include/uapi/linux/ioprio.h is being introduced with kernel 5.15. This user
header did not exist now and in previous kernels. include/linux/ioprio.h has
been around for a while though, but that is a kernel header, not an application
header.

Ah ok this was the now or never one, I thought it was about one of the constants.
Sorry for the noise.