Skip to contentSkip to conversation

IB/srp patches for Linux kernel v4.3

PATCH In review Bart Van Assche 3 participants 5 messages Jul 31, 2015 – Aug 1, 2015 1 message not in the archive

Subsystems (3)
lore
Share & tools
lore
#0 lore raw

cover IB/srp patches for Linux kernel v4.3

Hello Doug,

Please apply the following three patches at your earliest convenience:

0001-IB-srp-Constify-a-function-argument.patch 0002-IB-srp-Handle-partial-connection-success-correctly.patch 0003-IB-srp-Bump-driver-version-and-release-date.patch

Thanks,

Bart.

#1 lore raw

PATCH 1/3 IB/srp: Constify a function argument

This patch does not change any functionality.

Cc Sagi Grimberg <[email protected]>
Cc Sebastian Parschauer <[email protected]>
Cc Christoph Hellwig <[email protected]>
---
 drivers/infiniband/ulp/srp/ib_srp.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/infiniband/ulp/srp/ib_srp.c +1 -1
index 31a20b4..61dadac 100644
@@ -2171,7 +2171,7 @@ static uint32_t srp_compute_rq_tmo(struct ib_qp_attr *qp_attr, int attr_mask)
 }
 
 static void srp_cm_rep_handler(struct ib_cm_id *cm_id,
-			       struct srp_login_rsp *lrsp,
+			       const struct srp_login_rsp *lrsp,
 			       struct srp_rdma_ch *ch)
 {
 	struct srp_target_port *target = ch->target;
Signature · 1 line
2.1.4

A message that is not in the archive (sent off-list or not yet received): [email protected]

#3 lore raw

PATCH 2/3 IB/srp: Handle partial connection success correctly

Avoid that the following kernel warning is reported if the SRP target system accepts fewer channels per connection than what was requested by the initiator system:

WARNING: at drivers/infiniband/ulp/srp/ib_srp.c:617 srp_destroy_qp+0xb1/0x120 [ib_srp]()
Call Trace:
[<ffffffff8105d67f>] warn_slowpath_common+0x7f/0xc0
[<ffffffff8105d6da>] warn_slowpath_null+0x1a/0x20
[<ffffffffa05419e1>] srp_destroy_qp+0xb1/0x120 [ib_srp]
[<ffffffffa05445fb>] srp_create_ch_ib+0x19b/0x420 [ib_srp]
[<ffffffffa0545257>] srp_create_target+0x7d7/0xa94 [ib_srp]
[<ffffffff8138dac0>] dev_attr_store+0x20/0x30
[<ffffffff812079ef>] sysfs_write_file+0xef/0x170
[<ffffffff81191fc4>] vfs_write+0xb4/0x130
[<ffffffff8119276f>] sys_write+0x5f/0xa0
[<ffffffff815a0a59>] system_call_fastpath+0x16/0x1b
Cc Sagi Grimberg <sagig-VPRAkNaXOzVWk0Htik3J/[email protected]>
Cc Sebastian Parschauer <[email protected]>
Cc Christoph Hellwig <[email protected]>
---
 drivers/infiniband/ulp/srp/ib_srp.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
drivers/infiniband/ulp/srp/ib_srp.c +2 -1
index 61dadac..ab18665 100644
@@ -3262,7 +3262,7 @@ static ssize_t srp_create_target(struct device *dev,
 					srp_free_ch_ib(target, ch);
 					srp_free_req_data(target, ch);
 					target->ch_count = ch - target->ch;
-					break;
+					goto connected;
 				}
 			}
 
@@ -3272,6 +3272,7 @@ static ssize_t srp_create_target(struct device *dev,
 		node_idx++;
 	}
 
+connected:
 	target->scsi_host->nr_hw_queues = target->ch_count;
 
 	ret = srp_add_target(host, target);
Signature · 6 lines
2.1.4

--
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
#4 lore raw
In reply to a message not in the archive
Quoted 7 lines · Bart Van Assche

Hello Doug,

Please apply the following three patches at your earliest convenience:

0001-IB-srp-Constify-a-function-argument.patch 0002-IB-srp-Handle-partial-connection-success-correctly.patch 0003-IB-srp-Bump-driver-version-and-release-date.patch

Got 'em, thanks!

Signature · 2 lines
Doug Ledford <[email protected]>
              GPG KeyID: 0E572FDD
#5 lore raw

PATCH 3/3 IB/srp: Bump driver version and release date

In reply to Bart Van Assche

Since version 1.0 e.g. scsi-mq has been added. Since this is a significant change, bump the driver version and release date.

Cc Sagi Grimberg <[email protected]>
Cc Sebastian Parschauer <[email protected]>
Cc Christoph Hellwig <[email protected]>
---
 drivers/infiniband/ulp/srp/ib_srp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/infiniband/ulp/srp/ib_srp.c +2 -2
index ab18665..e67e6039 100644
@@ -55,8 +55,8 @@
 
 #define DRV_NAME	"ib_srp"
 #define PFX		DRV_NAME ": "
-#define DRV_VERSION	"1.0"
-#define DRV_RELDATE	"July 1, 2013"
+#define DRV_VERSION	"2.0"
+#define DRV_RELDATE	"July 26, 2015"
 
 MODULE_AUTHOR("Roland Dreier");
 MODULE_DESCRIPTION("InfiniBand SCSI RDMA Protocol initiator");
Signature · 1 line
2.1.4