Commit d28aa030 authored by Jens Axboe's avatar Jens Axboe Committed by Zhihao Cheng
Browse files

iomap: use an unsigned type for IOMAP_DIO_* defines 

mainline inclusion
from mainline-v6.6-rc1
commit 44842f64
category: perf
bugzilla: https://gitee.com/openeuler/kernel/issues/I90ZB5
CVE: NA

Reference:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=44842f647346cac4063b2bb8e9476fad09e363e7



--------------------------------

IOMAP_DIO_DIRTY shifts by 31 bits, which makes UBSAN unhappy. Clean up
all the defines by making the shifted value an unsigned value.

Reviewed-by: default avatarDarrick J. Wong <djwong@kernel.org>
Reported-by: default avatarDarrick J. Wong <djwong@kernel.org>
Reviewed-by: default avatarChristoph Hellwig <hch@lst.de>
Reviewed-by: default avatarDave Chinner <dchinner@redhat.com>
Signed-off-by: default avatarJens Axboe <axboe@kernel.dk>
Signed-off-by: default avatarZhihao Cheng <chengzhihao1@huawei.com>
parent 46070886

fs/iomap/direct-io.c

+4 −4
Original line number Diff line number Diff line
@@ -18,10 +18,10 @@ 
 * Private flags for iomap_dio, must not overlap with the public ones in

 * iomap.h:

 */

#define IOMAP_DIO_WRITE_FUA	(1 << 28)

#define IOMAP_DIO_NEED_SYNC	(1 << 29)

#define IOMAP_DIO_WRITE		(1 << 30)

#define IOMAP_DIO_DIRTY		(1 << 31)

#define IOMAP_DIO_WRITE_FUA	(1U << 28)

#define IOMAP_DIO_NEED_SYNC	(1U << 29)

#define IOMAP_DIO_WRITE		(1U << 30)

#define IOMAP_DIO_DIRTY		(1U << 31)



struct iomap_dio {

	struct kiocb		*iocb;