diff options
author | Christoph Hellwig <hch@lst.de> | 2017-11-23 13:53:40 -0500 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2018-05-02 13:52:38 -0400 |
commit | c213dc826c591cec9e1a458f17ae474af6c8e00c (patch) | |
tree | e40edd9c2959f8113c2f364b6b3cbf15ca1f0b2d /fs/aio.c | |
parent | 01a658e1e92745255bfcae1977adc2f563a159ac (diff) |
aio: remove an outdated BUG_ON and comment in aio_complete
These days we don't treat sync iocbs special in the aio completion code as
they never use it. Remove the old comment and BUG_ON given that the
current definition of is_sync_kiocb makes it impossible to hit.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Diffstat (limited to 'fs/aio.c')
-rw-r--r-- | fs/aio.c | 9 |
1 files changed, 0 insertions, 9 deletions
@@ -1107,15 +1107,6 @@ static void aio_complete(struct kiocb *kiocb, long res, long res2) | |||
1107 | file_end_write(file); | 1107 | file_end_write(file); |
1108 | } | 1108 | } |
1109 | 1109 | ||
1110 | /* | ||
1111 | * Special case handling for sync iocbs: | ||
1112 | * - events go directly into the iocb for fast handling | ||
1113 | * - the sync task with the iocb in its stack holds the single iocb | ||
1114 | * ref, no other paths have a way to get another ref | ||
1115 | * - the sync task helpfully left a reference to itself in the iocb | ||
1116 | */ | ||
1117 | BUG_ON(is_sync_kiocb(kiocb)); | ||
1118 | |||
1119 | if (iocb->ki_list.next) { | 1110 | if (iocb->ki_list.next) { |
1120 | unsigned long flags; | 1111 | unsigned long flags; |
1121 | 1112 | ||