From bf6bf606e545cb31c29499b354c13b2621acd649 Mon Sep 17 00:00:00 2001 From: Al Viro Date: Tue, 16 Feb 2016 20:06:19 -0500 Subject: [PATCH] orangefs_copy_{to,from}_bufmap(): don't pass bufmap pointer it's always __orangefs_bufmap Signed-off-by: Al Viro Signed-off-by: Mike Marshall --- fs/orangefs/file.c | 18 ++++++------------ fs/orangefs/orangefs-bufmap.c | 13 ++++++------- fs/orangefs/orangefs-bufmap.h | 6 ++---- 3 files changed, 14 insertions(+), 23 deletions(-) diff --git a/fs/orangefs/file.c b/fs/orangefs/file.c index d4a00ad26f6e..db9dd6ebcc3f 100644 --- a/fs/orangefs/file.c +++ b/fs/orangefs/file.c @@ -21,8 +21,7 @@ * can futher be kernel-space or user-space addresses. * or it can pointers to struct page's */ -static int precopy_buffers(struct orangefs_bufmap *bufmap, - int buffer_index, +static int precopy_buffers(int buffer_index, struct iov_iter *iter, size_t total_size) { @@ -34,8 +33,7 @@ static int precopy_buffers(struct orangefs_bufmap *bufmap, if (total_size) { - ret = orangefs_bufmap_copy_from_iovec(bufmap, - iter, + ret = orangefs_bufmap_copy_from_iovec(iter, buffer_index, total_size); if (ret < 0) @@ -58,8 +56,7 @@ static int precopy_buffers(struct orangefs_bufmap *bufmap, * can futher be kernel-space or user-space addresses. * or it can pointers to struct page's */ -static int postcopy_buffers(struct orangefs_bufmap *bufmap, - int buffer_index, +static int postcopy_buffers(int buffer_index, struct iov_iter *iter, size_t total_size) { @@ -70,8 +67,7 @@ static int postcopy_buffers(struct orangefs_bufmap *bufmap, * struct page pointers. */ if (total_size) { - ret = orangefs_bufmap_copy_to_iovec(bufmap, - iter, + ret = orangefs_bufmap_copy_to_iovec(iter, buffer_index, total_size); if (ret < 0) @@ -138,8 +134,7 @@ populate_shared_memory: * precopy_buffers only pertains to writes. */ if (type == ORANGEFS_IO_WRITE) { - ret = precopy_buffers(bufmap, - buffer_index, + ret = precopy_buffers(buffer_index, iter, total_size); if (ret < 0) @@ -242,8 +237,7 @@ populate_shared_memory: * postcopy_buffers only pertains to reads. */ if (type == ORANGEFS_IO_READ) { - ret = postcopy_buffers(bufmap, - buffer_index, + ret = postcopy_buffers(buffer_index, iter, new_op->downcall.resp.io.amt_complete); if (ret < 0) diff --git a/fs/orangefs/orangefs-bufmap.c b/fs/orangefs/orangefs-bufmap.c index 44d437dbfce0..97689c6cab17 100644 --- a/fs/orangefs/orangefs-bufmap.c +++ b/fs/orangefs/orangefs-bufmap.c @@ -511,19 +511,18 @@ void orangefs_readdir_index_put(int buffer_index) * we've been handed an iovec, we need to copy it to * the shared memory descriptor at "buffer_index". */ -int orangefs_bufmap_copy_from_iovec(struct orangefs_bufmap *bufmap, - struct iov_iter *iter, +int orangefs_bufmap_copy_from_iovec(struct iov_iter *iter, int buffer_index, size_t size) { - struct orangefs_bufmap_desc *to = &bufmap->desc_array[buffer_index]; + struct orangefs_bufmap_desc *to; int i; gossip_debug(GOSSIP_BUFMAP_DEBUG, "%s: buffer_index:%d: size:%zu:\n", __func__, buffer_index, size); - + to = &__orangefs_bufmap->desc_array[buffer_index]; for (i = 0; size; i++) { struct page *page = to->page_array[i]; size_t n = size; @@ -542,14 +541,14 @@ int orangefs_bufmap_copy_from_iovec(struct orangefs_bufmap *bufmap, * we've been handed an iovec, we need to fill it from * the shared memory descriptor at "buffer_index". */ -int orangefs_bufmap_copy_to_iovec(struct orangefs_bufmap *bufmap, - struct iov_iter *iter, +int orangefs_bufmap_copy_to_iovec(struct iov_iter *iter, int buffer_index, size_t size) { - struct orangefs_bufmap_desc *from = &bufmap->desc_array[buffer_index]; + struct orangefs_bufmap_desc *from; int i; + from = &__orangefs_bufmap->desc_array[buffer_index]; gossip_debug(GOSSIP_BUFMAP_DEBUG, "%s: buffer_index:%d: size:%zu:\n", __func__, buffer_index, size); diff --git a/fs/orangefs/orangefs-bufmap.h b/fs/orangefs/orangefs-bufmap.h index 0be62be373f7..babdc713c5e0 100644 --- a/fs/orangefs/orangefs-bufmap.h +++ b/fs/orangefs/orangefs-bufmap.h @@ -27,13 +27,11 @@ int orangefs_readdir_index_get(struct orangefs_bufmap **mapp, int *buffer_index) void orangefs_readdir_index_put(int buffer_index); -int orangefs_bufmap_copy_from_iovec(struct orangefs_bufmap *bufmap, - struct iov_iter *iter, +int orangefs_bufmap_copy_from_iovec(struct iov_iter *iter, int buffer_index, size_t size); -int orangefs_bufmap_copy_to_iovec(struct orangefs_bufmap *bufmap, - struct iov_iter *iter, +int orangefs_bufmap_copy_to_iovec(struct iov_iter *iter, int buffer_index, size_t size);