[lustre-devel] [PATCH 10/27] lustre: llite: Rename ldp_aio to sdio
James Simmons
jsimmons at infradead.org
Fri Mar 21 06:06:53 PDT 2025
From: Patrick Farrell <pfarrell at whamcloud.com>
ldp_aio is a weird name for a 'sub dio' struct - rename it
to sdio.
WC-bug-id: https://jira.whamcloud.com/browse/LU-13805
Lustre-commit: 7c416273ad1a8f559 ("LU-13805 llite: Rename ldp_aio to sdio")
Signed-off-by: Patrick Farrell <pfarrell at whamcloud.com>
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/50170
Reviewed-by: Qian Yingjin <qian at ddn.com>
Reviewed-by: Sebastien Buisson <sbuisson at ddn.com>
Reviewed-by: Oleg Drokin <green at whamcloud.com>
Reviewed-by: Andreas Dilger <adilger at whamcloud.com>
Signed-off-by: James Simmons <jsimmons at infradead.org>
---
fs/lustre/llite/rw26.c | 24 ++++++++++++------------
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/fs/lustre/llite/rw26.c b/fs/lustre/llite/rw26.c
index 9239e029276e..5e9dcefa84f0 100644
--- a/fs/lustre/llite/rw26.c
+++ b/fs/lustre/llite/rw26.c
@@ -307,7 +307,7 @@ static ssize_t ll_direct_IO(struct kiocb *iocb, struct iov_iter *iter)
struct file *file = iocb->ki_filp;
struct inode *inode = file->f_mapping->host;
struct cl_dio_aio *ll_dio_aio;
- struct cl_sub_dio *ldp_aio;
+ struct cl_sub_dio *sdio;
size_t count = iov_iter_count(iter);
ssize_t tot_bytes = 0, result = 0;
loff_t file_offset = iocb->ki_pos;
@@ -378,21 +378,21 @@ static ssize_t ll_direct_IO(struct kiocb *iocb, struct iov_iter *iter)
* otherwise it is freed on the final call to cl_sync_io_note
* (either in this function or from a ptlrpcd daemon)
*/
- ldp_aio = cl_sub_dio_alloc(ll_dio_aio, sync_submit);
- if (!ldp_aio) {
+ sdio = cl_sub_dio_alloc(ll_dio_aio, sync_submit);
+ if (!sdio) {
result = -ENOMEM;
goto out;
}
- pvec = &ldp_aio->csd_dio_pages;
+ pvec = &sdio->csd_dio_pages;
result = ll_get_user_pages(rw, iter, &pages,
&pvec->ldp_count, count);
if (unlikely(result <= 0)) {
- cl_sync_io_note(env, &ldp_aio->csd_sync, result);
+ cl_sync_io_note(env, &sdio->csd_sync, result);
if (sync_submit) {
- LASSERT(ldp_aio->csd_creator_free);
- cl_sub_dio_free(ldp_aio);
+ LASSERT(sdio->csd_creator_free);
+ cl_sub_dio_free(sdio);
}
goto out;
}
@@ -402,19 +402,19 @@ static ssize_t ll_direct_IO(struct kiocb *iocb, struct iov_iter *iter)
pvec->ldp_pages = pages;
result = ll_direct_rw_pages(env, io, count,
- rw, inode, ldp_aio);
+ rw, inode, sdio);
/* We've submitted pages and can now remove the extra
* reference for that
*/
- cl_sync_io_note(env, &ldp_aio->csd_sync, result);
+ cl_sync_io_note(env, &sdio->csd_sync, result);
if (sync_submit) {
- rc2 = cl_sync_io_wait(env, &ldp_aio->csd_sync,
+ rc2 = cl_sync_io_wait(env, &sdio->csd_sync,
0);
if (result == 0 && rc2)
result = rc2;
- LASSERT(ldp_aio->csd_creator_free);
- cl_sub_dio_free(ldp_aio);
+ LASSERT(sdio->csd_creator_free);
+ cl_sub_dio_free(sdio);
}
if (unlikely(result < 0))
goto out;
--
2.39.3
More information about the lustre-devel
mailing list