Merge ~paelzer/ubuntu/+source/rdma-core:lp-1823836-clear-cmd-buffer-disco into ubuntu/+source/rdma-core:ubuntu/disco-devel

Proposed by Christian Ehrhardt 
Status: Merged
Approved by: Christian Ehrhardt 
Approved revision: 6461d74097e025ed5cae3bf4613681596f4cdbc3
Merge reported by: Christian Ehrhardt 
Merged at revision: 6461d74097e025ed5cae3bf4613681596f4cdbc3
Proposed branch: ~paelzer/ubuntu/+source/rdma-core:lp-1823836-clear-cmd-buffer-disco
Merge into: ubuntu/+source/rdma-core:ubuntu/disco-devel
Diff against target: 66 lines (+46/-0)
3 files modified
debian/changelog (+7/-0)
debian/patches/lp-1823836-clear-cmd-buffer.patch (+38/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
Andreas Hasenack Approve
Canonical Server packageset reviewers Pending
Canonical Server Pending
Review via email: mp+366380@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

PPA: https://launchpad.net/~paelzer/+archive/ubuntu/bug-1823836-rdma-core-mem-clear/+packages
Bug: https://bugs.launchpad.net/ubuntu/+source/dpdk/+bug/1823836

No Eoan MP as the importer is still stuck, but it would be the same except the version number being 22.1-1ubuntu1 then

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

+1, cool investigation

review: Approve
Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

my checks also pointed out that I forgot u-m, added that as well and waiting to EE to open fully.
Then I'll upload both versions one after the other.

Thanks for the review.

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Tagged and uploaded

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 1b67953..979aa44 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,10 @@
6+rdma-core (22.1-1ubuntu0.1) disco; urgency=medium
7+
8+ * d/p/lp-1823836-clear-cmd-buffer.patch: verbs: clear cmd buffer when
9+ creating indirection table; fixing MLX device initialization (LP: #1823836)
10+
11+ -- Christian Ehrhardt <christian.ehrhardt@canonical.com> Tue, 23 Apr 2019 09:29:45 +0200
12+
13 rdma-core (22.1-1) unstable; urgency=medium
14
15 * New upstream bugfix release.
16diff --git a/debian/patches/lp-1823836-clear-cmd-buffer.patch b/debian/patches/lp-1823836-clear-cmd-buffer.patch
17new file mode 100644
18index 0000000..b6051f5
19--- /dev/null
20+++ b/debian/patches/lp-1823836-clear-cmd-buffer.patch
21@@ -0,0 +1,38 @@
22+From d38817ea554a9fd140e56414a7c74e7eef444c50 Mon Sep 17 00:00:00 2001
23+From: Mark Bloch <markb@mellanox.com>
24+Date: Wed, 20 Feb 2019 20:00:45 +0000
25+Subject: [PATCH] verbs: clear cmd buffer when creating indirection table
26+
27+[ Upstream commit 34225464c1d165f63ffd462364810cc9f374109d ]
28+
29+Make sure we clear the cmd buffer, not doing so will make the kernel to
30+fail the command.
31+
32+Fixes: 75c65bbcadcd ("verbs: Consolidate duplicate code in create_rwq_ind_table")
33+Signed-off-by: Mark Bloch <markb@mellanox.com>
34+Signed-off-by: Yishai Hadas <yishaih@mellanox.com>
35+Signed-off-by: Nicolas Morey-Chaisemartin <nmoreychaisemartin@suse.com>
36+
37+Origin: upstream, https://github.com/linux-rdma/rdma-core/commit/d38817ea554a9fd140e56414a7c74e7eef444c50
38+Bug-Ubuntu: https://bugs.launchpad.net/bugs/1823836
39+Last-Update: 2019-04-23
40+
41+---
42+ libibverbs/cmd.c | 1 +
43+ 1 file changed, 1 insertion(+)
44+
45+diff --git a/libibverbs/cmd.c b/libibverbs/cmd.c
46+index 34c71e56..5817c598 100644
47+--- a/libibverbs/cmd.c
48++++ b/libibverbs/cmd.c
49+@@ -1854,6 +1854,7 @@ int ibv_cmd_create_rwq_ind_table(struct ibv_context *context,
50+ cmd_size = sizeof(*cmd) + num_tbl_entries * sizeof(cmd->wq_handles[0]);
51+ cmd_size = (cmd_size + 7) / 8 * 8;
52+ cmd = alloca(cmd_size);
53++ memset(cmd, 0, cmd_size);
54+
55+ for (i = 0; i < num_tbl_entries; i++)
56+ cmd->wq_handles[i] = init_attr->ind_tbl[i]->handle;
57+--
58+2.17.1
59+
60diff --git a/debian/patches/series b/debian/patches/series
61index 212c6aa..c2a5fdd 100644
62--- a/debian/patches/series
63+++ b/debian/patches/series
64@@ -1 +1,2 @@
65 pyverbs-Add-shebang-to-ib_devices.py-example.patch
66+lp-1823836-clear-cmd-buffer.patch

Subscribers

People subscribed via source and target branches