Commit 44842f64 authored by Jens Axboe's avatar Jens Axboe
Browse files

iomap: use an unsigned type for IOMAP_DIO_* defines 



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>
parent 3486237c

fs/iomap/direct-io.c

+4 −4
Original line number Diff line number Diff line
@@ -20,10 +20,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;