* [PATCH v3] IB/umem: Release pid in error and ODP flow @ 2016-12-30 10:18 ` Kenneth Lee 0 siblings, 0 replies; 14+ messages in thread From: Kenneth Lee @ 2016-12-30 10:18 UTC (permalink / raw) To: dledford, sean.hefty, hal.rosenstock Cc: robin.murphy, jroedel, egtvedt, vgupta, liguozhu, dave.hansen, lstoakes, krzk, sebott, markb, linux-rdma, linux-kernel There are two bugfixes in this patch: Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get") This patch introduce the get_task_pid but not put it back on all error path Fixes: 8ada2c1c0c1d ("IB/core: Add support for on demand paging regions") This patch introduce a ODP flow without release pid before enter it Signed-off-by: Kenneth Lee <liguozhu@hisilicon.com> Reviewed-by: Haggai Eran <haggaie@mellanox.com> --- Change from v1 to v2: Correcting the patch title and description Change from v2 to v3: Update the title and add "Fixes" fields in the description drivers/infiniband/core/umem.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c index 1e62a5f..4609b92 100644 --- a/drivers/infiniband/core/umem.c +++ b/drivers/infiniband/core/umem.c @@ -134,6 +134,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_MW_BIND)); if (access & IB_ACCESS_ON_DEMAND) { + put_pid(umem->pid); ret = ib_umem_odp_get(context, umem); if (ret) { kfree(umem); @@ -149,6 +150,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, page_list = (struct page **) __get_free_page(GFP_KERNEL); if (!page_list) { + put_pid(umem->pid); kfree(umem); return ERR_PTR(-ENOMEM); } -- 1.9.1 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v3] IB/umem: Release pid in error and ODP flow @ 2016-12-30 10:18 ` Kenneth Lee 0 siblings, 0 replies; 14+ messages in thread From: Kenneth Lee @ 2016-12-30 10:18 UTC (permalink / raw) To: dledford, sean.hefty, hal.rosenstock Cc: robin.murphy, jroedel, egtvedt, vgupta, liguozhu, dave.hansen, lstoakes, krzk, sebott, markb, linux-rdma, linux-kernel There are two bugfixes in this patch: Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get") This patch introduce the get_task_pid but not put it back on all error path Fixes: 8ada2c1c0c1d ("IB/core: Add support for on demand paging regions") This patch introduce a ODP flow without release pid before enter it Signed-off-by: Kenneth Lee <liguozhu@hisilicon.com> Reviewed-by: Haggai Eran <haggaie@mellanox.com> --- Change from v1 to v2: Correcting the patch title and description Change from v2 to v3: Update the title and add "Fixes" fields in the description drivers/infiniband/core/umem.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c index 1e62a5f..4609b92 100644 --- a/drivers/infiniband/core/umem.c +++ b/drivers/infiniband/core/umem.c @@ -134,6 +134,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_MW_BIND)); if (access & IB_ACCESS_ON_DEMAND) { + put_pid(umem->pid); ret = ib_umem_odp_get(context, umem); if (ret) { kfree(umem); @@ -149,6 +150,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, page_list = (struct page **) __get_free_page(GFP_KERNEL); if (!page_list) { + put_pid(umem->pid); kfree(umem); return ERR_PTR(-ENOMEM); } -- 1.9.1 ^ permalink raw reply related [flat|nested] 14+ messages in thread
[parent not found: <1483093109-188428-1-git-send-email-liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org>]
* Re: [PATCH v3] IB/umem: Release pid in error and ODP flow 2016-12-30 10:18 ` Kenneth Lee @ 2017-01-01 6:47 ` Leon Romanovsky -1 siblings, 0 replies; 14+ messages in thread From: Leon Romanovsky @ 2017-01-01 6:47 UTC (permalink / raw) To: Kenneth Lee Cc: dledford-H+wXaHxf7aLQT0dZR+AlfA, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, robin.murphy-5wv7dgnIgG8, jroedel-l3A5Bk7waGM, egtvedt-BrfabpQBY5qlHtIdYg32fQ, vgupta-HKixBCOQz3hWk0Htik3J/w, dave.hansen-VuQAYsv1563Yd54FQh9/CA, lstoakes-Re5JQEeQqe8AvxtiuMwx3w, krzk-DgEjT+Ai2ygdnm+yROfE0A, sebott-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8, markb-VPRAkNaXOzVWk0Htik3J/w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA [-- Attachment #1: Type: text/plain, Size: 2062 bytes --] On Fri, Dec 30, 2016 at 06:18:29PM +0800, Kenneth Lee wrote: > There are two bugfixes in this patch: > > Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get") > This patch introduce the get_task_pid but not put it back on all error > path > > Fixes: 8ada2c1c0c1d ("IB/core: Add support for on demand paging regions") > This patch introduce a ODP flow without release pid before enter it > > > Signed-off-by: Kenneth Lee <liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org> > Reviewed-by: Haggai Eran <haggaie-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org> > --- > Change from v1 to v2: > Correcting the patch title and description > Change from v2 to v3: > Update the title and add "Fixes" fields in the description OK, I see that you still didn't read Documentation/SubmittingPatches. You must read that document before you are sending patches. But I'll stop here, the code is correct (it fixes bugs) and commit message more usefull than before. > > drivers/infiniband/core/umem.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c > index 1e62a5f..4609b92 100644 > --- a/drivers/infiniband/core/umem.c > +++ b/drivers/infiniband/core/umem.c > @@ -134,6 +134,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_MW_BIND)); > > if (access & IB_ACCESS_ON_DEMAND) { > + put_pid(umem->pid); > ret = ib_umem_odp_get(context, umem); > if (ret) { > kfree(umem); > @@ -149,6 +150,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > page_list = (struct page **) __get_free_page(GFP_KERNEL); > if (!page_list) { > + put_pid(umem->pid); > kfree(umem); > return ERR_PTR(-ENOMEM); > } > -- > 1.9.1 > > -- > 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 [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v3] IB/umem: Release pid in error and ODP flow @ 2017-01-01 6:47 ` Leon Romanovsky 0 siblings, 0 replies; 14+ messages in thread From: Leon Romanovsky @ 2017-01-01 6:47 UTC (permalink / raw) To: Kenneth Lee Cc: dledford, sean.hefty, hal.rosenstock, robin.murphy, jroedel, egtvedt, vgupta, dave.hansen, lstoakes, krzk, sebott, markb, linux-rdma, linux-kernel [-- Attachment #1: Type: text/plain, Size: 1985 bytes --] On Fri, Dec 30, 2016 at 06:18:29PM +0800, Kenneth Lee wrote: > There are two bugfixes in this patch: > > Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get") > This patch introduce the get_task_pid but not put it back on all error > path > > Fixes: 8ada2c1c0c1d ("IB/core: Add support for on demand paging regions") > This patch introduce a ODP flow without release pid before enter it > > > Signed-off-by: Kenneth Lee <liguozhu@hisilicon.com> > Reviewed-by: Haggai Eran <haggaie@mellanox.com> > --- > Change from v1 to v2: > Correcting the patch title and description > Change from v2 to v3: > Update the title and add "Fixes" fields in the description OK, I see that you still didn't read Documentation/SubmittingPatches. You must read that document before you are sending patches. But I'll stop here, the code is correct (it fixes bugs) and commit message more usefull than before. > > drivers/infiniband/core/umem.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c > index 1e62a5f..4609b92 100644 > --- a/drivers/infiniband/core/umem.c > +++ b/drivers/infiniband/core/umem.c > @@ -134,6 +134,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_MW_BIND)); > > if (access & IB_ACCESS_ON_DEMAND) { > + put_pid(umem->pid); > ret = ib_umem_odp_get(context, umem); > if (ret) { > kfree(umem); > @@ -149,6 +150,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > page_list = (struct page **) __get_free_page(GFP_KERNEL); > if (!page_list) { > + put_pid(umem->pid); > kfree(umem); > return ERR_PTR(-ENOMEM); > } > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 14+ messages in thread
[parent not found: <20170101064712.GQ26885-U/DQcQFIOTAAJjI8aNfphQ@public.gmane.org>]
* Re: [PATCH v3] IB/umem: Release pid in error and ODP flow 2017-01-01 6:47 ` Leon Romanovsky @ 2017-01-03 2:11 ` Kenneth Lee -1 siblings, 0 replies; 14+ messages in thread From: Kenneth Lee @ 2017-01-03 2:11 UTC (permalink / raw) To: Leon Romanovsky Cc: dledford-H+wXaHxf7aLQT0dZR+AlfA, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, robin.murphy-5wv7dgnIgG8, jroedel-l3A5Bk7waGM, egtvedt-BrfabpQBY5qlHtIdYg32fQ, vgupta-HKixBCOQz3hWk0Htik3J/w, dave.hansen-VuQAYsv1563Yd54FQh9/CA, lstoakes-Re5JQEeQqe8AvxtiuMwx3w, krzk-DgEjT+Ai2ygdnm+yROfE0A, sebott-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8, markb-VPRAkNaXOzVWk0Htik3J/w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA Thanks, I did read the doc, but maybe I mis-understant some points. Could you please point it out? On Sun, Jan 01, 2017 at 08:47:12AM +0200, Leon Romanovsky wrote: > Date: Sun, 1 Jan 2017 08:47:12 +0200 > From: Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> > To: Kenneth Lee <liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org> > CC: dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, > robin.murphy-5wv7dgnIgG8@public.gmane.org, jroedel-l3A5Bk7waGM@public.gmane.org, egtvedt-BrfabpQBY5qlHtIdYg32fQ@public.gmane.org, > vgupta-HKixBCOQz3hWk0Htik3J/w@public.gmane.org, dave.hansen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org, lstoakes-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, > krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, sebott-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org, markb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org, > linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > Subject: Re: [PATCH v3] IB/umem: Release pid in error and ODP flow > User-Agent: Mutt/1.7.2 (2016-11-26) > Message-ID: <20170101064712.GQ26885-U/DQcQFIOTAAJjI8aNfphQ@public.gmane.org> > > On Fri, Dec 30, 2016 at 06:18:29PM +0800, Kenneth Lee wrote: > > There are two bugfixes in this patch: > > > > Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get") > > This patch introduce the get_task_pid but not put it back on all error > > path > > > > Fixes: 8ada2c1c0c1d ("IB/core: Add support for on demand paging regions") > > This patch introduce a ODP flow without release pid before enter it > > > > > > Signed-off-by: Kenneth Lee <liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org> > > Reviewed-by: Haggai Eran <haggaie-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org> > > --- > > Change from v1 to v2: > > Correcting the patch title and description > > Change from v2 to v3: > > Update the title and add "Fixes" fields in the description > > OK, > > I see that you still didn't read Documentation/SubmittingPatches. You > must read that document before you are sending patches. > > But I'll stop here, the code is correct (it fixes bugs) and commit message > more usefull than before. > > > > > > drivers/infiniband/core/umem.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c > > index 1e62a5f..4609b92 100644 > > --- a/drivers/infiniband/core/umem.c > > +++ b/drivers/infiniband/core/umem.c > > @@ -134,6 +134,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_MW_BIND)); > > > > if (access & IB_ACCESS_ON_DEMAND) { > > + put_pid(umem->pid); > > ret = ib_umem_odp_get(context, umem); > > if (ret) { > > kfree(umem); > > @@ -149,6 +150,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > > page_list = (struct page **) __get_free_page(GFP_KERNEL); > > if (!page_list) { > > + put_pid(umem->pid); > > kfree(umem); > > return ERR_PTR(-ENOMEM); > > } > > -- > > 1.9.1 > > > > -- > > 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 -- -Kenneth(Hisilicon) -- 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] 14+ messages in thread
* Re: [PATCH v3] IB/umem: Release pid in error and ODP flow @ 2017-01-03 2:11 ` Kenneth Lee 0 siblings, 0 replies; 14+ messages in thread From: Kenneth Lee @ 2017-01-03 2:11 UTC (permalink / raw) To: Leon Romanovsky Cc: dledford, sean.hefty, hal.rosenstock, robin.murphy, jroedel, egtvedt, vgupta, dave.hansen, lstoakes, krzk, sebott, markb, linux-rdma, linux-kernel Thanks, I did read the doc, but maybe I mis-understant some points. Could you please point it out? On Sun, Jan 01, 2017 at 08:47:12AM +0200, Leon Romanovsky wrote: > Date: Sun, 1 Jan 2017 08:47:12 +0200 > From: Leon Romanovsky <leon@kernel.org> > To: Kenneth Lee <liguozhu@hisilicon.com> > CC: dledford@redhat.com, sean.hefty@intel.com, hal.rosenstock@gmail.com, > robin.murphy@arm.com, jroedel@suse.de, egtvedt@samfundet.no, > vgupta@synopsys.com, dave.hansen@linux.intel.com, lstoakes@gmail.com, > krzk@kernel.org, sebott@linux.vnet.ibm.com, markb@mellanox.com, > linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org > Subject: Re: [PATCH v3] IB/umem: Release pid in error and ODP flow > User-Agent: Mutt/1.7.2 (2016-11-26) > Message-ID: <20170101064712.GQ26885@mtr-leonro.local> > > On Fri, Dec 30, 2016 at 06:18:29PM +0800, Kenneth Lee wrote: > > There are two bugfixes in this patch: > > > > Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get") > > This patch introduce the get_task_pid but not put it back on all error > > path > > > > Fixes: 8ada2c1c0c1d ("IB/core: Add support for on demand paging regions") > > This patch introduce a ODP flow without release pid before enter it > > > > > > Signed-off-by: Kenneth Lee <liguozhu@hisilicon.com> > > Reviewed-by: Haggai Eran <haggaie@mellanox.com> > > --- > > Change from v1 to v2: > > Correcting the patch title and description > > Change from v2 to v3: > > Update the title and add "Fixes" fields in the description > > OK, > > I see that you still didn't read Documentation/SubmittingPatches. You > must read that document before you are sending patches. > > But I'll stop here, the code is correct (it fixes bugs) and commit message > more usefull than before. > > > > > > drivers/infiniband/core/umem.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c > > index 1e62a5f..4609b92 100644 > > --- a/drivers/infiniband/core/umem.c > > +++ b/drivers/infiniband/core/umem.c > > @@ -134,6 +134,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_MW_BIND)); > > > > if (access & IB_ACCESS_ON_DEMAND) { > > + put_pid(umem->pid); > > ret = ib_umem_odp_get(context, umem); > > if (ret) { > > kfree(umem); > > @@ -149,6 +150,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > > page_list = (struct page **) __get_free_page(GFP_KERNEL); > > if (!page_list) { > > + put_pid(umem->pid); > > kfree(umem); > > return ERR_PTR(-ENOMEM); > > } > > -- > > 1.9.1 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html -- -Kenneth(Hisilicon) ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v3] IB/umem: Release pid in error and ODP flow 2017-01-01 6:47 ` Leon Romanovsky @ 2017-01-03 2:32 ` Kenneth Lee -1 siblings, 0 replies; 14+ messages in thread From: Kenneth Lee @ 2017-01-03 2:32 UTC (permalink / raw) To: Leon Romanovsky Cc: dledford-H+wXaHxf7aLQT0dZR+AlfA, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, robin.murphy-5wv7dgnIgG8, jroedel-l3A5Bk7waGM, egtvedt-BrfabpQBY5qlHtIdYg32fQ, vgupta-HKixBCOQz3hWk0Htik3J/w, dave.hansen-VuQAYsv1563Yd54FQh9/CA, lstoakes-Re5JQEeQqe8AvxtiuMwx3w, krzk-DgEjT+Ai2ygdnm+yROfE0A, sebott-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8, markb-VPRAkNaXOzVWk0Htik3J/w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA On Sun, Jan 01, 2017 at 08:47:12AM +0200, Leon Romanovsky wrote: > Date: Sun, 1 Jan 2017 08:47:12 +0200 > From: Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> > To: Kenneth Lee <liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org> > CC: dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, > robin.murphy-5wv7dgnIgG8@public.gmane.org, jroedel-l3A5Bk7waGM@public.gmane.org, egtvedt-BrfabpQBY5qlHtIdYg32fQ@public.gmane.org, > vgupta-HKixBCOQz3hWk0Htik3J/w@public.gmane.org, dave.hansen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org, lstoakes-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, > krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, sebott-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org, markb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org, > linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > Subject: Re: [PATCH v3] IB/umem: Release pid in error and ODP flow > User-Agent: Mutt/1.7.2 (2016-11-26) > Message-ID: <20170101064712.GQ26885-U/DQcQFIOTAAJjI8aNfphQ@public.gmane.org> > > On Fri, Dec 30, 2016 at 06:18:29PM +0800, Kenneth Lee wrote: > > There are two bugfixes in this patch: > > > > Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get") > > This patch introduce the get_task_pid but not put it back on all error > > path > > > > Fixes: 8ada2c1c0c1d ("IB/core: Add support for on demand paging regions") > > This patch introduce a ODP flow without release pid before enter it > > > > > > Signed-off-by: Kenneth Lee <liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org> > > Reviewed-by: Haggai Eran <haggaie-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org> > > --- > > Change from v1 to v2: > > Correcting the patch title and description > > Change from v2 to v3: > > Update the title and add "Fixes" fields in the description > > OK, > > I see that you still didn't read Documentation/SubmittingPatches. You > must read that document before you are sending patches. > > But I'll stop here, the code is correct (it fixes bugs) and commit message > more usefull than before. > > > > > > drivers/infiniband/core/umem.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c > > index 1e62a5f..4609b92 100644 > > --- a/drivers/infiniband/core/umem.c > > +++ b/drivers/infiniband/core/umem.c > > @@ -134,6 +134,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_MW_BIND)); > > > > if (access & IB_ACCESS_ON_DEMAND) { > > + put_pid(umem->pid); > > ret = ib_umem_odp_get(context, umem); > > if (ret) { > > kfree(umem); > > @@ -149,6 +150,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > > page_list = (struct page **) __get_free_page(GFP_KERNEL); > > if (!page_list) { > > + put_pid(umem->pid); > > kfree(umem); > > return ERR_PTR(-ENOMEM); > > } > > -- > > 1.9.1 > > > > -- > > 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 Thanks, I did read the doc, but maybe I mis-understant some points. Could you please point it out? And sorry. please ignore the last message. I forget to use a bottom-post style. -- -Kenneth(Hisilicon) -- 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] 14+ messages in thread
* Re: [PATCH v3] IB/umem: Release pid in error and ODP flow @ 2017-01-03 2:32 ` Kenneth Lee 0 siblings, 0 replies; 14+ messages in thread From: Kenneth Lee @ 2017-01-03 2:32 UTC (permalink / raw) To: Leon Romanovsky Cc: dledford, sean.hefty, hal.rosenstock, robin.murphy, jroedel, egtvedt, vgupta, dave.hansen, lstoakes, krzk, sebott, markb, linux-rdma, linux-kernel On Sun, Jan 01, 2017 at 08:47:12AM +0200, Leon Romanovsky wrote: > Date: Sun, 1 Jan 2017 08:47:12 +0200 > From: Leon Romanovsky <leon@kernel.org> > To: Kenneth Lee <liguozhu@hisilicon.com> > CC: dledford@redhat.com, sean.hefty@intel.com, hal.rosenstock@gmail.com, > robin.murphy@arm.com, jroedel@suse.de, egtvedt@samfundet.no, > vgupta@synopsys.com, dave.hansen@linux.intel.com, lstoakes@gmail.com, > krzk@kernel.org, sebott@linux.vnet.ibm.com, markb@mellanox.com, > linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org > Subject: Re: [PATCH v3] IB/umem: Release pid in error and ODP flow > User-Agent: Mutt/1.7.2 (2016-11-26) > Message-ID: <20170101064712.GQ26885@mtr-leonro.local> > > On Fri, Dec 30, 2016 at 06:18:29PM +0800, Kenneth Lee wrote: > > There are two bugfixes in this patch: > > > > Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get") > > This patch introduce the get_task_pid but not put it back on all error > > path > > > > Fixes: 8ada2c1c0c1d ("IB/core: Add support for on demand paging regions") > > This patch introduce a ODP flow without release pid before enter it > > > > > > Signed-off-by: Kenneth Lee <liguozhu@hisilicon.com> > > Reviewed-by: Haggai Eran <haggaie@mellanox.com> > > --- > > Change from v1 to v2: > > Correcting the patch title and description > > Change from v2 to v3: > > Update the title and add "Fixes" fields in the description > > OK, > > I see that you still didn't read Documentation/SubmittingPatches. You > must read that document before you are sending patches. > > But I'll stop here, the code is correct (it fixes bugs) and commit message > more usefull than before. > > > > > > drivers/infiniband/core/umem.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c > > index 1e62a5f..4609b92 100644 > > --- a/drivers/infiniband/core/umem.c > > +++ b/drivers/infiniband/core/umem.c > > @@ -134,6 +134,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_MW_BIND)); > > > > if (access & IB_ACCESS_ON_DEMAND) { > > + put_pid(umem->pid); > > ret = ib_umem_odp_get(context, umem); > > if (ret) { > > kfree(umem); > > @@ -149,6 +150,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > > page_list = (struct page **) __get_free_page(GFP_KERNEL); > > if (!page_list) { > > + put_pid(umem->pid); > > kfree(umem); > > return ERR_PTR(-ENOMEM); > > } > > -- > > 1.9.1 > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html Thanks, I did read the doc, but maybe I mis-understant some points. Could you please point it out? And sorry. please ignore the last message. I forget to use a bottom-post style. -- -Kenneth(Hisilicon) ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v3] IB/umem: Release pid in error and ODP flow 2017-01-03 2:32 ` Kenneth Lee @ 2017-01-03 10:12 ` Leon Romanovsky -1 siblings, 0 replies; 14+ messages in thread From: Leon Romanovsky @ 2017-01-03 10:12 UTC (permalink / raw) To: Kenneth Lee Cc: dledford-H+wXaHxf7aLQT0dZR+AlfA, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, robin.murphy-5wv7dgnIgG8, jroedel-l3A5Bk7waGM, egtvedt-BrfabpQBY5qlHtIdYg32fQ, vgupta-HKixBCOQz3hWk0Htik3J/w, dave.hansen-VuQAYsv1563Yd54FQh9/CA, lstoakes-Re5JQEeQqe8AvxtiuMwx3w, krzk-DgEjT+Ai2ygdnm+yROfE0A, sebott-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8, markb-VPRAkNaXOzVWk0Htik3J/w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA [-- Attachment #1: Type: text/plain, Size: 3968 bytes --] On Tue, Jan 03, 2017 at 10:32:50AM +0800, Kenneth Lee wrote: > On Sun, Jan 01, 2017 at 08:47:12AM +0200, Leon Romanovsky wrote: > > Date: Sun, 1 Jan 2017 08:47:12 +0200 > > From: Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> > > To: Kenneth Lee <liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org> > > CC: dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, > > robin.murphy-5wv7dgnIgG8@public.gmane.org, jroedel-l3A5Bk7waGM@public.gmane.org, egtvedt-BrfabpQBY5qlHtIdYg32fQ@public.gmane.org, > > vgupta-HKixBCOQz3hWk0Htik3J/w@public.gmane.org, dave.hansen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org, lstoakes-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, > > krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, sebott-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org, markb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org, > > linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > > Subject: Re: [PATCH v3] IB/umem: Release pid in error and ODP flow > > User-Agent: Mutt/1.7.2 (2016-11-26) > > Message-ID: <20170101064712.GQ26885-U/DQcQFIOTAAJjI8aNfphQ@public.gmane.org> > > > > On Fri, Dec 30, 2016 at 06:18:29PM +0800, Kenneth Lee wrote: > > > There are two bugfixes in this patch: > > > > > > Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get") > > > This patch introduce the get_task_pid but not put it back on all error > > > path > > > > > > Fixes: 8ada2c1c0c1d ("IB/core: Add support for on demand paging regions") > > > This patch introduce a ODP flow without release pid before enter it > > > > > > > > > Signed-off-by: Kenneth Lee <liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org> > > > Reviewed-by: Haggai Eran <haggaie-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org> > > > --- > > > Change from v1 to v2: > > > Correcting the patch title and description > > > Change from v2 to v3: > > > Update the title and add "Fixes" fields in the description > > > > OK, > > > > I see that you still didn't read Documentation/SubmittingPatches. You > > must read that document before you are sending patches. > > > > But I'll stop here, the code is correct (it fixes bugs) and commit message > > more usefull than before. > > > > > > > > > > drivers/infiniband/core/umem.c | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c > > > index 1e62a5f..4609b92 100644 > > > --- a/drivers/infiniband/core/umem.c > > > +++ b/drivers/infiniband/core/umem.c > > > @@ -134,6 +134,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_MW_BIND)); > > > > > > if (access & IB_ACCESS_ON_DEMAND) { > > > + put_pid(umem->pid); > > > ret = ib_umem_odp_get(context, umem); > > > if (ret) { > > > kfree(umem); > > > @@ -149,6 +150,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > > > > page_list = (struct page **) __get_free_page(GFP_KERNEL); > > > if (!page_list) { > > > + put_pid(umem->pid); > > > kfree(umem); > > > return ERR_PTR(-ENOMEM); > > > } > > > -- > > > 1.9.1 > > > > > > -- > > > 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 > > Thanks, > > I did read the doc, but maybe I mis-understant some points. Could you please > point it out? Fixes line should be placed above bottom signatures. As an example of properly written patch, you can take a look on the following patch [1] from Steve. [1] http://marc.info/?l=linux-rdma&m=148244272205411&w=2 > > And sorry. please ignore the last message. I forget to use a bottom-post style. > > > > -- > -Kenneth(Hisilicon) [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v3] IB/umem: Release pid in error and ODP flow @ 2017-01-03 10:12 ` Leon Romanovsky 0 siblings, 0 replies; 14+ messages in thread From: Leon Romanovsky @ 2017-01-03 10:12 UTC (permalink / raw) To: Kenneth Lee Cc: dledford, sean.hefty, hal.rosenstock, robin.murphy, jroedel, egtvedt, vgupta, dave.hansen, lstoakes, krzk, sebott, markb, linux-rdma, linux-kernel [-- Attachment #1: Type: text/plain, Size: 3439 bytes --] On Tue, Jan 03, 2017 at 10:32:50AM +0800, Kenneth Lee wrote: > On Sun, Jan 01, 2017 at 08:47:12AM +0200, Leon Romanovsky wrote: > > Date: Sun, 1 Jan 2017 08:47:12 +0200 > > From: Leon Romanovsky <leon@kernel.org> > > To: Kenneth Lee <liguozhu@hisilicon.com> > > CC: dledford@redhat.com, sean.hefty@intel.com, hal.rosenstock@gmail.com, > > robin.murphy@arm.com, jroedel@suse.de, egtvedt@samfundet.no, > > vgupta@synopsys.com, dave.hansen@linux.intel.com, lstoakes@gmail.com, > > krzk@kernel.org, sebott@linux.vnet.ibm.com, markb@mellanox.com, > > linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org > > Subject: Re: [PATCH v3] IB/umem: Release pid in error and ODP flow > > User-Agent: Mutt/1.7.2 (2016-11-26) > > Message-ID: <20170101064712.GQ26885@mtr-leonro.local> > > > > On Fri, Dec 30, 2016 at 06:18:29PM +0800, Kenneth Lee wrote: > > > There are two bugfixes in this patch: > > > > > > Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get") > > > This patch introduce the get_task_pid but not put it back on all error > > > path > > > > > > Fixes: 8ada2c1c0c1d ("IB/core: Add support for on demand paging regions") > > > This patch introduce a ODP flow without release pid before enter it > > > > > > > > > Signed-off-by: Kenneth Lee <liguozhu@hisilicon.com> > > > Reviewed-by: Haggai Eran <haggaie@mellanox.com> > > > --- > > > Change from v1 to v2: > > > Correcting the patch title and description > > > Change from v2 to v3: > > > Update the title and add "Fixes" fields in the description > > > > OK, > > > > I see that you still didn't read Documentation/SubmittingPatches. You > > must read that document before you are sending patches. > > > > But I'll stop here, the code is correct (it fixes bugs) and commit message > > more usefull than before. > > > > > > > > > > drivers/infiniband/core/umem.c | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c > > > index 1e62a5f..4609b92 100644 > > > --- a/drivers/infiniband/core/umem.c > > > +++ b/drivers/infiniband/core/umem.c > > > @@ -134,6 +134,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_MW_BIND)); > > > > > > if (access & IB_ACCESS_ON_DEMAND) { > > > + put_pid(umem->pid); > > > ret = ib_umem_odp_get(context, umem); > > > if (ret) { > > > kfree(umem); > > > @@ -149,6 +150,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > > > > page_list = (struct page **) __get_free_page(GFP_KERNEL); > > > if (!page_list) { > > > + put_pid(umem->pid); > > > kfree(umem); > > > return ERR_PTR(-ENOMEM); > > > } > > > -- > > > 1.9.1 > > > > > > -- > > > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > > > the body of a message to majordomo@vger.kernel.org > > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > Thanks, > > I did read the doc, but maybe I mis-understant some points. Could you please > point it out? Fixes line should be placed above bottom signatures. As an example of properly written patch, you can take a look on the following patch [1] from Steve. [1] http://marc.info/?l=linux-rdma&m=148244272205411&w=2 > > And sorry. please ignore the last message. I forget to use a bottom-post style. > > > > -- > -Kenneth(Hisilicon) [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 14+ messages in thread
[parent not found: <20170103101224.GH12077-U/DQcQFIOTAAJjI8aNfphQ@public.gmane.org>]
* Re: [PATCH v3] IB/umem: Release pid in error and ODP flow 2017-01-03 10:12 ` Leon Romanovsky @ 2017-01-04 7:07 ` Kenneth Lee -1 siblings, 0 replies; 14+ messages in thread From: Kenneth Lee @ 2017-01-04 7:07 UTC (permalink / raw) To: Leon Romanovsky Cc: dledford-H+wXaHxf7aLQT0dZR+AlfA, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, robin.murphy-5wv7dgnIgG8, jroedel-l3A5Bk7waGM, egtvedt-BrfabpQBY5qlHtIdYg32fQ, vgupta-HKixBCOQz3hWk0Htik3J/w, dave.hansen-VuQAYsv1563Yd54FQh9/CA, lstoakes-Re5JQEeQqe8AvxtiuMwx3w, krzk-DgEjT+Ai2ygdnm+yROfE0A, sebott-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8, markb-VPRAkNaXOzVWk0Htik3J/w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA On Tue, Jan 03, 2017 at 12:12:24PM +0200, Leon Romanovsky wrote: > Date: Tue, 3 Jan 2017 12:12:24 +0200 > From: Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> > To: Kenneth Lee <liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org> > CC: dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, > robin.murphy-5wv7dgnIgG8@public.gmane.org, jroedel-l3A5Bk7waGM@public.gmane.org, egtvedt-BrfabpQBY5qlHtIdYg32fQ@public.gmane.org, > vgupta-HKixBCOQz3hWk0Htik3J/w@public.gmane.org, dave.hansen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org, lstoakes-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, > krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, sebott-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org, markb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org, > linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > Subject: Re: [PATCH v3] IB/umem: Release pid in error and ODP flow > User-Agent: Mutt/1.7.2 (2016-11-26) > Message-ID: <20170103101224.GH12077-U/DQcQFIOTAAJjI8aNfphQ@public.gmane.org> > > On Tue, Jan 03, 2017 at 10:32:50AM +0800, Kenneth Lee wrote: > > On Sun, Jan 01, 2017 at 08:47:12AM +0200, Leon Romanovsky wrote: > > > Date: Sun, 1 Jan 2017 08:47:12 +0200 > > > From: Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> > > > To: Kenneth Lee <liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org> > > > CC: dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, > > > robin.murphy-5wv7dgnIgG8@public.gmane.org, jroedel-l3A5Bk7waGM@public.gmane.org, egtvedt-BrfabpQBY5qlHtIdYg32fQ@public.gmane.org, > > > vgupta-HKixBCOQz3hWk0Htik3J/w@public.gmane.org, dave.hansen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org, lstoakes-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, > > > krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, sebott-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org, markb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org, > > > linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > > > Subject: Re: [PATCH v3] IB/umem: Release pid in error and ODP flow > > > User-Agent: Mutt/1.7.2 (2016-11-26) > > > Message-ID: <20170101064712.GQ26885-U/DQcQFIOTAAJjI8aNfphQ@public.gmane.org> > > > > > > On Fri, Dec 30, 2016 at 06:18:29PM +0800, Kenneth Lee wrote: > > > > There are two bugfixes in this patch: > > > > > > > > Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get") > > > > This patch introduce the get_task_pid but not put it back on all error > > > > path > > > > > > > > Fixes: 8ada2c1c0c1d ("IB/core: Add support for on demand paging regions") > > > > This patch introduce a ODP flow without release pid before enter it > > > > > > > > > > > > Signed-off-by: Kenneth Lee <liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org> > > > > Reviewed-by: Haggai Eran <haggaie-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org> > > > > --- > > > > Change from v1 to v2: > > > > Correcting the patch title and description > > > > Change from v2 to v3: > > > > Update the title and add "Fixes" fields in the description > > > > > > OK, > > > > > > I see that you still didn't read Documentation/SubmittingPatches. You > > > must read that document before you are sending patches. > > > > > > But I'll stop here, the code is correct (it fixes bugs) and commit message > > > more usefull than before. > > > > > > > > > > > > > > drivers/infiniband/core/umem.c | 2 ++ > > > > 1 file changed, 2 insertions(+) > > > > > > > > diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c > > > > index 1e62a5f..4609b92 100644 > > > > --- a/drivers/infiniband/core/umem.c > > > > +++ b/drivers/infiniband/core/umem.c > > > > @@ -134,6 +134,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > > IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_MW_BIND)); > > > > > > > > if (access & IB_ACCESS_ON_DEMAND) { > > > > + put_pid(umem->pid); > > > > ret = ib_umem_odp_get(context, umem); > > > > if (ret) { > > > > kfree(umem); > > > > @@ -149,6 +150,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > > > > > > page_list = (struct page **) __get_free_page(GFP_KERNEL); > > > > if (!page_list) { > > > > + put_pid(umem->pid); > > > > kfree(umem); > > > > return ERR_PTR(-ENOMEM); > > > > } > > > > -- > > > > 1.9.1 > > > > > > > > -- > > > > 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 > > > > Thanks, > > > > I did read the doc, but maybe I mis-understant some points. Could you please > > point it out? > > Fixes line should be placed above bottom signatures. > > As an example of properly written patch, you can take a look on the > following patch [1] from Steve. > > [1] http://marc.info/?l=linux-rdma&m=148244272205411&w=2 Thank you. A sample help a lot. But please allow me to argue a little: Documentation/process/submitting-patches.rst does really not mention where Fixes tags should be put:) > > > > > And sorry. please ignore the last message. I forget to use a bottom-post style. > > > > > > > > -- > > -Kenneth(Hisilicon) -- -Kenneth(Hisilicon) -- 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] 14+ messages in thread
* Re: [PATCH v3] IB/umem: Release pid in error and ODP flow @ 2017-01-04 7:07 ` Kenneth Lee 0 siblings, 0 replies; 14+ messages in thread From: Kenneth Lee @ 2017-01-04 7:07 UTC (permalink / raw) To: Leon Romanovsky Cc: dledford, sean.hefty, hal.rosenstock, robin.murphy, jroedel, egtvedt, vgupta, dave.hansen, lstoakes, krzk, sebott, markb, linux-rdma, linux-kernel On Tue, Jan 03, 2017 at 12:12:24PM +0200, Leon Romanovsky wrote: > Date: Tue, 3 Jan 2017 12:12:24 +0200 > From: Leon Romanovsky <leon@kernel.org> > To: Kenneth Lee <liguozhu@hisilicon.com> > CC: dledford@redhat.com, sean.hefty@intel.com, hal.rosenstock@gmail.com, > robin.murphy@arm.com, jroedel@suse.de, egtvedt@samfundet.no, > vgupta@synopsys.com, dave.hansen@linux.intel.com, lstoakes@gmail.com, > krzk@kernel.org, sebott@linux.vnet.ibm.com, markb@mellanox.com, > linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org > Subject: Re: [PATCH v3] IB/umem: Release pid in error and ODP flow > User-Agent: Mutt/1.7.2 (2016-11-26) > Message-ID: <20170103101224.GH12077@mtr-leonro.local> > > On Tue, Jan 03, 2017 at 10:32:50AM +0800, Kenneth Lee wrote: > > On Sun, Jan 01, 2017 at 08:47:12AM +0200, Leon Romanovsky wrote: > > > Date: Sun, 1 Jan 2017 08:47:12 +0200 > > > From: Leon Romanovsky <leon@kernel.org> > > > To: Kenneth Lee <liguozhu@hisilicon.com> > > > CC: dledford@redhat.com, sean.hefty@intel.com, hal.rosenstock@gmail.com, > > > robin.murphy@arm.com, jroedel@suse.de, egtvedt@samfundet.no, > > > vgupta@synopsys.com, dave.hansen@linux.intel.com, lstoakes@gmail.com, > > > krzk@kernel.org, sebott@linux.vnet.ibm.com, markb@mellanox.com, > > > linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org > > > Subject: Re: [PATCH v3] IB/umem: Release pid in error and ODP flow > > > User-Agent: Mutt/1.7.2 (2016-11-26) > > > Message-ID: <20170101064712.GQ26885@mtr-leonro.local> > > > > > > On Fri, Dec 30, 2016 at 06:18:29PM +0800, Kenneth Lee wrote: > > > > There are two bugfixes in this patch: > > > > > > > > Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get") > > > > This patch introduce the get_task_pid but not put it back on all error > > > > path > > > > > > > > Fixes: 8ada2c1c0c1d ("IB/core: Add support for on demand paging regions") > > > > This patch introduce a ODP flow without release pid before enter it > > > > > > > > > > > > Signed-off-by: Kenneth Lee <liguozhu@hisilicon.com> > > > > Reviewed-by: Haggai Eran <haggaie@mellanox.com> > > > > --- > > > > Change from v1 to v2: > > > > Correcting the patch title and description > > > > Change from v2 to v3: > > > > Update the title and add "Fixes" fields in the description > > > > > > OK, > > > > > > I see that you still didn't read Documentation/SubmittingPatches. You > > > must read that document before you are sending patches. > > > > > > But I'll stop here, the code is correct (it fixes bugs) and commit message > > > more usefull than before. > > > > > > > > > > > > > > drivers/infiniband/core/umem.c | 2 ++ > > > > 1 file changed, 2 insertions(+) > > > > > > > > diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c > > > > index 1e62a5f..4609b92 100644 > > > > --- a/drivers/infiniband/core/umem.c > > > > +++ b/drivers/infiniband/core/umem.c > > > > @@ -134,6 +134,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > > IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_MW_BIND)); > > > > > > > > if (access & IB_ACCESS_ON_DEMAND) { > > > > + put_pid(umem->pid); > > > > ret = ib_umem_odp_get(context, umem); > > > > if (ret) { > > > > kfree(umem); > > > > @@ -149,6 +150,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > > > > > > page_list = (struct page **) __get_free_page(GFP_KERNEL); > > > > if (!page_list) { > > > > + put_pid(umem->pid); > > > > kfree(umem); > > > > return ERR_PTR(-ENOMEM); > > > > } > > > > -- > > > > 1.9.1 > > > > > > > > -- > > > > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > > > > the body of a message to majordomo@vger.kernel.org > > > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > > > Thanks, > > > > I did read the doc, but maybe I mis-understant some points. Could you please > > point it out? > > Fixes line should be placed above bottom signatures. > > As an example of properly written patch, you can take a look on the > following patch [1] from Steve. > > [1] http://marc.info/?l=linux-rdma&m=148244272205411&w=2 Thank you. A sample help a lot. But please allow me to argue a little: Documentation/process/submitting-patches.rst does really not mention where Fixes tags should be put:) > > > > > And sorry. please ignore the last message. I forget to use a bottom-post style. > > > > > > > > -- > > -Kenneth(Hisilicon) -- -Kenneth(Hisilicon) ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v3] IB/umem: Release pid in error and ODP flow 2017-01-04 7:07 ` Kenneth Lee @ 2017-01-04 7:08 ` Leon Romanovsky -1 siblings, 0 replies; 14+ messages in thread From: Leon Romanovsky @ 2017-01-04 7:08 UTC (permalink / raw) To: Kenneth Lee Cc: dledford-H+wXaHxf7aLQT0dZR+AlfA, sean.hefty-ral2JQCrhuEAvxtiuMwx3w, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w, robin.murphy-5wv7dgnIgG8, jroedel-l3A5Bk7waGM, egtvedt-BrfabpQBY5qlHtIdYg32fQ, vgupta-HKixBCOQz3hWk0Htik3J/w, dave.hansen-VuQAYsv1563Yd54FQh9/CA, lstoakes-Re5JQEeQqe8AvxtiuMwx3w, krzk-DgEjT+Ai2ygdnm+yROfE0A, sebott-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8, markb-VPRAkNaXOzVWk0Htik3J/w, linux-rdma-u79uwXL29TY76Z2rM5mHXA, linux-kernel-u79uwXL29TY76Z2rM5mHXA [-- Attachment #1: Type: text/plain, Size: 6058 bytes --] On Wed, Jan 04, 2017 at 03:07:04PM +0800, Kenneth Lee wrote: > On Tue, Jan 03, 2017 at 12:12:24PM +0200, Leon Romanovsky wrote: > > Date: Tue, 3 Jan 2017 12:12:24 +0200 > > From: Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> > > To: Kenneth Lee <liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org> > > CC: dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, > > robin.murphy-5wv7dgnIgG8@public.gmane.org, jroedel-l3A5Bk7waGM@public.gmane.org, egtvedt-BrfabpQBY5qlHtIdYg32fQ@public.gmane.org, > > vgupta-HKixBCOQz3hWk0Htik3J/w@public.gmane.org, dave.hansen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org, lstoakes-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, > > krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, sebott-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org, markb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org, > > linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > > Subject: Re: [PATCH v3] IB/umem: Release pid in error and ODP flow > > User-Agent: Mutt/1.7.2 (2016-11-26) > > Message-ID: <20170103101224.GH12077-U/DQcQFIOTAAJjI8aNfphQ@public.gmane.org> > > > > On Tue, Jan 03, 2017 at 10:32:50AM +0800, Kenneth Lee wrote: > > > On Sun, Jan 01, 2017 at 08:47:12AM +0200, Leon Romanovsky wrote: > > > > Date: Sun, 1 Jan 2017 08:47:12 +0200 > > > > From: Leon Romanovsky <leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> > > > > To: Kenneth Lee <liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org> > > > > CC: dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, sean.hefty-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org, hal.rosenstock-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, > > > > robin.murphy-5wv7dgnIgG8@public.gmane.org, jroedel-l3A5Bk7waGM@public.gmane.org, egtvedt-BrfabpQBY5qlHtIdYg32fQ@public.gmane.org, > > > > vgupta-HKixBCOQz3hWk0Htik3J/w@public.gmane.org, dave.hansen-VuQAYsv1563Yd54FQh9/CA@public.gmane.org, lstoakes-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, > > > > krzk-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, sebott-23VcF4HTsmIX0ybBhKVfKdBPR1lH4CV8@public.gmane.org, markb-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org, > > > > linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org > > > > Subject: Re: [PATCH v3] IB/umem: Release pid in error and ODP flow > > > > User-Agent: Mutt/1.7.2 (2016-11-26) > > > > Message-ID: <20170101064712.GQ26885-U/DQcQFIOTAAJjI8aNfphQ@public.gmane.org> > > > > > > > > On Fri, Dec 30, 2016 at 06:18:29PM +0800, Kenneth Lee wrote: > > > > > There are two bugfixes in this patch: > > > > > > > > > > Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get") > > > > > This patch introduce the get_task_pid but not put it back on all error > > > > > path > > > > > > > > > > Fixes: 8ada2c1c0c1d ("IB/core: Add support for on demand paging regions") > > > > > This patch introduce a ODP flow without release pid before enter it > > > > > > > > > > > > > > > Signed-off-by: Kenneth Lee <liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org> > > > > > Reviewed-by: Haggai Eran <haggaie-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org> > > > > > --- > > > > > Change from v1 to v2: > > > > > Correcting the patch title and description > > > > > Change from v2 to v3: > > > > > Update the title and add "Fixes" fields in the description > > > > > > > > OK, > > > > > > > > I see that you still didn't read Documentation/SubmittingPatches. You > > > > must read that document before you are sending patches. > > > > > > > > But I'll stop here, the code is correct (it fixes bugs) and commit message > > > > more usefull than before. > > > > > > > > > > > > > > > > > > drivers/infiniband/core/umem.c | 2 ++ > > > > > 1 file changed, 2 insertions(+) > > > > > > > > > > diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c > > > > > index 1e62a5f..4609b92 100644 > > > > > --- a/drivers/infiniband/core/umem.c > > > > > +++ b/drivers/infiniband/core/umem.c > > > > > @@ -134,6 +134,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > > > IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_MW_BIND)); > > > > > > > > > > if (access & IB_ACCESS_ON_DEMAND) { > > > > > + put_pid(umem->pid); > > > > > ret = ib_umem_odp_get(context, umem); > > > > > if (ret) { > > > > > kfree(umem); > > > > > @@ -149,6 +150,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > > > > > > > > page_list = (struct page **) __get_free_page(GFP_KERNEL); > > > > > if (!page_list) { > > > > > + put_pid(umem->pid); > > > > > kfree(umem); > > > > > return ERR_PTR(-ENOMEM); > > > > > } > > > > > -- > > > > > 1.9.1 > > > > > > > > > > -- > > > > > 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 > > > > > > Thanks, > > > > > > I did read the doc, but maybe I mis-understant some points. Could you please > > > point it out? > > > > Fixes line should be placed above bottom signatures. > > > > As an example of properly written patch, you can take a look on the > > following patch [1] from Steve. > > > > [1] http://marc.info/?l=linux-rdma&m=148244272205411&w=2 > > Thank you. A sample help a lot. > > But please allow me to argue a little: > Documentation/process/submitting-patches.rst does really not mention where Fixes > tags should be put:) Sure, you can't and don't want to document everything. There is section named "2) Describe your changes" which has very extensive description about commit message. If you had followed it, you would find that it is very natural to place Fixes at the bottom. Thanks > > > > > > > > > And sorry. please ignore the last message. I forget to use a bottom-post style. > > > > > > > > > > > > -- > > > -Kenneth(Hisilicon) > > > > -- > -Kenneth(Hisilicon) [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v3] IB/umem: Release pid in error and ODP flow @ 2017-01-04 7:08 ` Leon Romanovsky 0 siblings, 0 replies; 14+ messages in thread From: Leon Romanovsky @ 2017-01-04 7:08 UTC (permalink / raw) To: Kenneth Lee Cc: dledford, sean.hefty, hal.rosenstock, robin.murphy, jroedel, egtvedt, vgupta, dave.hansen, lstoakes, krzk, sebott, markb, linux-rdma, linux-kernel [-- Attachment #1: Type: text/plain, Size: 5077 bytes --] On Wed, Jan 04, 2017 at 03:07:04PM +0800, Kenneth Lee wrote: > On Tue, Jan 03, 2017 at 12:12:24PM +0200, Leon Romanovsky wrote: > > Date: Tue, 3 Jan 2017 12:12:24 +0200 > > From: Leon Romanovsky <leon@kernel.org> > > To: Kenneth Lee <liguozhu@hisilicon.com> > > CC: dledford@redhat.com, sean.hefty@intel.com, hal.rosenstock@gmail.com, > > robin.murphy@arm.com, jroedel@suse.de, egtvedt@samfundet.no, > > vgupta@synopsys.com, dave.hansen@linux.intel.com, lstoakes@gmail.com, > > krzk@kernel.org, sebott@linux.vnet.ibm.com, markb@mellanox.com, > > linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org > > Subject: Re: [PATCH v3] IB/umem: Release pid in error and ODP flow > > User-Agent: Mutt/1.7.2 (2016-11-26) > > Message-ID: <20170103101224.GH12077@mtr-leonro.local> > > > > On Tue, Jan 03, 2017 at 10:32:50AM +0800, Kenneth Lee wrote: > > > On Sun, Jan 01, 2017 at 08:47:12AM +0200, Leon Romanovsky wrote: > > > > Date: Sun, 1 Jan 2017 08:47:12 +0200 > > > > From: Leon Romanovsky <leon@kernel.org> > > > > To: Kenneth Lee <liguozhu@hisilicon.com> > > > > CC: dledford@redhat.com, sean.hefty@intel.com, hal.rosenstock@gmail.com, > > > > robin.murphy@arm.com, jroedel@suse.de, egtvedt@samfundet.no, > > > > vgupta@synopsys.com, dave.hansen@linux.intel.com, lstoakes@gmail.com, > > > > krzk@kernel.org, sebott@linux.vnet.ibm.com, markb@mellanox.com, > > > > linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org > > > > Subject: Re: [PATCH v3] IB/umem: Release pid in error and ODP flow > > > > User-Agent: Mutt/1.7.2 (2016-11-26) > > > > Message-ID: <20170101064712.GQ26885@mtr-leonro.local> > > > > > > > > On Fri, Dec 30, 2016 at 06:18:29PM +0800, Kenneth Lee wrote: > > > > > There are two bugfixes in this patch: > > > > > > > > > > Fixes: 87773dd56d54 ("IB: ib_umem_release() should decrement mm->pinned_vm from ib_umem_get") > > > > > This patch introduce the get_task_pid but not put it back on all error > > > > > path > > > > > > > > > > Fixes: 8ada2c1c0c1d ("IB/core: Add support for on demand paging regions") > > > > > This patch introduce a ODP flow without release pid before enter it > > > > > > > > > > > > > > > Signed-off-by: Kenneth Lee <liguozhu@hisilicon.com> > > > > > Reviewed-by: Haggai Eran <haggaie@mellanox.com> > > > > > --- > > > > > Change from v1 to v2: > > > > > Correcting the patch title and description > > > > > Change from v2 to v3: > > > > > Update the title and add "Fixes" fields in the description > > > > > > > > OK, > > > > > > > > I see that you still didn't read Documentation/SubmittingPatches. You > > > > must read that document before you are sending patches. > > > > > > > > But I'll stop here, the code is correct (it fixes bugs) and commit message > > > > more usefull than before. > > > > > > > > > > > > > > > > > > drivers/infiniband/core/umem.c | 2 ++ > > > > > 1 file changed, 2 insertions(+) > > > > > > > > > > diff --git a/drivers/infiniband/core/umem.c b/drivers/infiniband/core/umem.c > > > > > index 1e62a5f..4609b92 100644 > > > > > --- a/drivers/infiniband/core/umem.c > > > > > +++ b/drivers/infiniband/core/umem.c > > > > > @@ -134,6 +134,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > > > IB_ACCESS_REMOTE_ATOMIC | IB_ACCESS_MW_BIND)); > > > > > > > > > > if (access & IB_ACCESS_ON_DEMAND) { > > > > > + put_pid(umem->pid); > > > > > ret = ib_umem_odp_get(context, umem); > > > > > if (ret) { > > > > > kfree(umem); > > > > > @@ -149,6 +150,7 @@ struct ib_umem *ib_umem_get(struct ib_ucontext *context, unsigned long addr, > > > > > > > > > > page_list = (struct page **) __get_free_page(GFP_KERNEL); > > > > > if (!page_list) { > > > > > + put_pid(umem->pid); > > > > > kfree(umem); > > > > > return ERR_PTR(-ENOMEM); > > > > > } > > > > > -- > > > > > 1.9.1 > > > > > > > > > > -- > > > > > To unsubscribe from this list: send the line "unsubscribe linux-rdma" in > > > > > the body of a message to majordomo@vger.kernel.org > > > > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > > > > > Thanks, > > > > > > I did read the doc, but maybe I mis-understant some points. Could you please > > > point it out? > > > > Fixes line should be placed above bottom signatures. > > > > As an example of properly written patch, you can take a look on the > > following patch [1] from Steve. > > > > [1] http://marc.info/?l=linux-rdma&m=148244272205411&w=2 > > Thank you. A sample help a lot. > > But please allow me to argue a little: > Documentation/process/submitting-patches.rst does really not mention where Fixes > tags should be put:) Sure, you can't and don't want to document everything. There is section named "2) Describe your changes" which has very extensive description about commit message. If you had followed it, you would find that it is very natural to place Fixes at the bottom. Thanks > > > > > > > > > And sorry. please ignore the last message. I forget to use a bottom-post style. > > > > > > > > > > > > -- > > > -Kenneth(Hisilicon) > > > > -- > -Kenneth(Hisilicon) [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2017-01-04 7:08 UTC | newest] Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2016-12-30 10:18 [PATCH v3] IB/umem: Release pid in error and ODP flow Kenneth Lee 2016-12-30 10:18 ` Kenneth Lee [not found] ` <1483093109-188428-1-git-send-email-liguozhu-C8/M+/jPZTeaMJb+Lgu22Q@public.gmane.org> 2017-01-01 6:47 ` Leon Romanovsky 2017-01-01 6:47 ` Leon Romanovsky [not found] ` <20170101064712.GQ26885-U/DQcQFIOTAAJjI8aNfphQ@public.gmane.org> 2017-01-03 2:11 ` Kenneth Lee 2017-01-03 2:11 ` Kenneth Lee 2017-01-03 2:32 ` Kenneth Lee 2017-01-03 2:32 ` Kenneth Lee 2017-01-03 10:12 ` Leon Romanovsky 2017-01-03 10:12 ` Leon Romanovsky [not found] ` <20170103101224.GH12077-U/DQcQFIOTAAJjI8aNfphQ@public.gmane.org> 2017-01-04 7:07 ` Kenneth Lee 2017-01-04 7:07 ` Kenneth Lee 2017-01-04 7:08 ` Leon Romanovsky 2017-01-04 7:08 ` Leon Romanovsky
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.