On 9/21/2015 7:39 PM, Jason Gunthorpe wrote:
On Sun, Sep 20, 2015 at 12:52:45PM +0300, Sagi Grimberg wrote:
This module parameter forces memory registration even for
a continuous memory region. It is true by default as sending
an all-physical rkey with remote permissions might be insecure.
Signed-off-by: Sagi Grimberg <[email protected]>
drivers/infiniband/ulp/iser/iscsi_iser.c | 5 +++++
drivers/infiniband/ulp/iser/iscsi_iser.h | 1 +
drivers/infiniband/ulp/iser/iser_memory.c | 18 ++++++++++++------
drivers/infiniband/ulp/iser/iser_verbs.c | 21 +++++++++++++--------
4 files changed, 31 insertions(+), 14 deletions(-)
diff --git a/drivers/infiniband/ulp/iser/iscsi_iser.c
b/drivers/infiniband/ulp/iser/iscsi_iser.c
index 1ace5d83a4d7..bad9dd701d3c 100644
+++ b/drivers/infiniband/ulp/iser/iscsi_iser.c
@@ -97,6 +97,11 @@ unsigned int iser_max_sectors = ISER_DEF_MAX_SECTORS;
module_param_named(max_sectors, iser_max_sectors, uint, S_IRUGO | S_IWUSR);
MODULE_PARM_DESC(max_sectors, "Max number of sectors in a single scsi command
(default:1024");
+bool iser_always_reg = true;
+module_param_named(always_register, iser_always_reg, bool, S_IRUGO);
+MODULE_PARM_DESC(always_register,
+ "Always register memory, even for continuous memory regions
(default:false)");
^^^^^^^^^^^^^^^
Description doesn't match implementation?
Yea... I noticed that too...
v1 is already on the list.
Thanks for commenting.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
