* [PATCH 0/1 v2] ib_srp mr allocation failure
@ 2017-01-03 13:45 Max Gurtovoy
[not found] ` <1483451133-6308-1-git-send-email-maxg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
0 siblings, 1 reply; 4+ messages in thread
From: Max Gurtovoy @ 2017-01-03 13:45 UTC (permalink / raw)
To: bvanassche-HInyCGIudOg, linux-rdma-u79uwXL29TY76Z2rM5mHXA
Cc: israelr-VPRAkNaXOzVWk0Htik3J/w
From: Israel Rukshin <israelr-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Changes from v1:
- added "Fixes:"
- Cc'ed stable kernel 4.7+
Israel Rukshin (1):
IB/srp: fix mr allocation when the device supports sg gaps
drivers/infiniband/ulp/srp/ib_srp.c | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
--
1.8.4.3
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCHv1 1/1] IB/srp: fix invalid indirect_sg_entries parameter value
[not found] ` <1483451133-6308-1-git-send-email-maxg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
@ 2017-01-03 13:45 ` Max Gurtovoy
2017-01-03 13:49 ` [PATCH 0/1 v2] ib_srp mr allocation failure Max Gurtovoy
1 sibling, 0 replies; 4+ messages in thread
From: Max Gurtovoy @ 2017-01-03 13:45 UTC (permalink / raw)
To: bvanassche-HInyCGIudOg, linux-rdma-u79uwXL29TY76Z2rM5mHXA
Cc: israelr-VPRAkNaXOzVWk0Htik3J/w, Max Gurtovoy
From: Israel Rukshin <israelr-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
After setting indirect_sg_entries module_param to huge value (e.g 500,000),
srp_alloc_req_data() fails to allocate indirect descriptors for the request
ring (kmalloc fails). This commit enforces the maximum value of indirect_sg_entries
to be SG_MAX_SEGMENTS as signified in module param description.
Signed-off-by: Israel Rukshin <israelr-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Signed-off-by: Max Gurtovoy <maxg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
---
drivers/infiniband/ulp/srp/ib_srp.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/drivers/infiniband/ulp/srp/ib_srp.c b/drivers/infiniband/ulp/srp/ib_srp.c
index 0f67cf9..ccdd2c2 100644
--- a/drivers/infiniband/ulp/srp/ib_srp.c
+++ b/drivers/infiniband/ulp/srp/ib_srp.c
@@ -3676,6 +3676,7 @@ static struct srp_function_template ib_srp_transport_functions = {
static int __init srp_init_module(void)
{
int ret;
+ unsigned int max_indirect_sg_entries = SG_MAX_SEGMENTS;
if (srp_sg_tablesize) {
pr_warn("srp_sg_tablesize is deprecated, please use cmd_sg_entries\n");
@@ -3699,6 +3700,12 @@ static int __init srp_init_module(void)
indirect_sg_entries = cmd_sg_entries;
}
+ if (indirect_sg_entries > max_indirect_sg_entries) {
+ pr_warn("Clamping indirect_sg_entries to %u\n",
+ max_indirect_sg_entries);
+ indirect_sg_entries = max_indirect_sg_entries;
+ }
+
srp_remove_wq = create_workqueue("srp_remove");
if (!srp_remove_wq) {
ret = -ENOMEM;
--
2.4.3
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 0/1 v2] ib_srp mr allocation failure
[not found] ` <1483451133-6308-1-git-send-email-maxg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2017-01-03 13:45 ` [PATCHv1 1/1] IB/srp: fix invalid indirect_sg_entries parameter value Max Gurtovoy
@ 2017-01-03 13:49 ` Max Gurtovoy
1 sibling, 0 replies; 4+ messages in thread
From: Max Gurtovoy @ 2017-01-03 13:49 UTC (permalink / raw)
To: bvanassche-HInyCGIudOg, linux-rdma-u79uwXL29TY76Z2rM5mHXA
Cc: israelr-VPRAkNaXOzVWk0Htik3J/w
please ignore the cover letter.
was sent by mistake.
On 1/3/2017 3:45 PM, Max Gurtovoy wrote:
> From: Israel Rukshin <israelr-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
>
> Changes from v1:
> - added "Fixes:"
> - Cc'ed stable kernel 4.7+
>
> Israel Rukshin (1):
> IB/srp: fix mr allocation when the device supports sg gaps
>
> drivers/infiniband/ulp/srp/ib_srp.c | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 0/1 v2] ib_srp mr allocation failure
@ 2016-12-28 10:48 Max Gurtovoy
0 siblings, 0 replies; 4+ messages in thread
From: Max Gurtovoy @ 2016-12-28 10:48 UTC (permalink / raw)
To: dledford-H+wXaHxf7aLQT0dZR+AlfA, bvanassche-HInyCGIudOg,
linux-rdma-u79uwXL29TY76Z2rM5mHXA
Cc: Israel Rukshin
From: Israel Rukshin <israelr-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
Changes from v1:
- added "Fixes:"
- Cc'ed stable kernel 4.7+
Israel Rukshin (1):
IB/srp: fix mr allocation when the device supports sg gaps
drivers/infiniband/ulp/srp/ib_srp.c | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)
--
1.8.4.3
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-01-03 13:49 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-03 13:45 [PATCH 0/1 v2] ib_srp mr allocation failure Max Gurtovoy
[not found] ` <1483451133-6308-1-git-send-email-maxg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
2017-01-03 13:45 ` [PATCHv1 1/1] IB/srp: fix invalid indirect_sg_entries parameter value Max Gurtovoy
2017-01-03 13:49 ` [PATCH 0/1 v2] ib_srp mr allocation failure Max Gurtovoy
-- strict thread matches above, loose matches on Subject: below --
2016-12-28 10:48 Max Gurtovoy
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.