Skip to content

Commit 0aaf08d

Browse files
author
Al Viro
committed
__blockdev_direct_IO(): get rid of submit_io callback
always NULL... Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
1 parent fe15c26 commit 0aaf08d

3 files changed

Lines changed: 5 additions & 10 deletions

File tree

fs/direct-io.c

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,6 @@ struct dio_submit {
8686
sector_t final_block_in_request;/* doesn't change */
8787
int boundary; /* prev block is at a boundary */
8888
get_block_t *get_block; /* block mapping function */
89-
dio_submit_t *submit_io; /* IO submition function */
9089

9190
loff_t logical_offset_in_bio; /* current first logical block in bio */
9291
sector_t final_block_in_bio; /* current final block in bio + 1 */
@@ -431,10 +430,7 @@ static inline void dio_bio_submit(struct dio *dio, struct dio_submit *sdio)
431430

432431
dio->bio_disk = bio->bi_bdev->bd_disk;
433432

434-
if (sdio->submit_io)
435-
sdio->submit_io(bio, dio->inode, sdio->logical_offset_in_bio);
436-
else
437-
submit_bio(bio);
433+
submit_bio(bio);
438434

439435
sdio->bio = NULL;
440436
sdio->boundary = 0;
@@ -1098,7 +1094,7 @@ static inline int drop_refcount(struct dio *dio)
10981094
ssize_t __blockdev_direct_IO(struct kiocb *iocb, struct inode *inode,
10991095
struct block_device *bdev, struct iov_iter *iter,
11001096
get_block_t get_block, dio_iodone_t end_io,
1101-
dio_submit_t submit_io, int flags)
1097+
int flags)
11021098
{
11031099
unsigned i_blkbits = READ_ONCE(inode->i_blkbits);
11041100
unsigned blkbits = i_blkbits;
@@ -1215,7 +1211,6 @@ ssize_t __blockdev_direct_IO(struct kiocb *iocb, struct inode *inode,
12151211

12161212
sdio.get_block = get_block;
12171213
dio->end_io = end_io;
1218-
sdio.submit_io = submit_io;
12191214
sdio.final_block_in_bio = -1;
12201215
sdio.next_block_for_io = -1;
12211216

fs/ocfs2/aops.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2448,7 +2448,7 @@ static ssize_t ocfs2_direct_IO(struct kiocb *iocb, struct iov_iter *iter)
24482448

24492449
return __blockdev_direct_IO(iocb, inode, inode->i_sb->s_bdev,
24502450
iter, get_block,
2451-
ocfs2_dio_end_io, NULL, 0);
2451+
ocfs2_dio_end_io, 0);
24522452
}
24532453

24542454
const struct address_space_operations ocfs2_aops = {

include/linux/fs.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2778,7 +2778,7 @@ enum {
27782778
ssize_t __blockdev_direct_IO(struct kiocb *iocb, struct inode *inode,
27792779
struct block_device *bdev, struct iov_iter *iter,
27802780
get_block_t get_block,
2781-
dio_iodone_t end_io, dio_submit_t submit_io,
2781+
dio_iodone_t end_io,
27822782
int flags);
27832783

27842784
static inline ssize_t blockdev_direct_IO(struct kiocb *iocb,
@@ -2787,7 +2787,7 @@ static inline ssize_t blockdev_direct_IO(struct kiocb *iocb,
27872787
get_block_t get_block)
27882788
{
27892789
return __blockdev_direct_IO(iocb, inode, inode->i_sb->s_bdev, iter,
2790-
get_block, NULL, NULL, DIO_LOCKING | DIO_SKIP_HOLES);
2790+
get_block, NULL, DIO_LOCKING | DIO_SKIP_HOLES);
27912791
}
27922792
#endif
27932793

0 commit comments

Comments
 (0)