RE: [PATCH v2] ndctl: fix libdaxctl memory leak

2018-04-16 Thread Plewa, Lukasz
On Apr 13, 2018, at 5:09PM, Dave Jiang wrote: > On 4/12/2018 11:57 PM, Plewa, Lukasz wrote: > > On Apr 13, 2018, at 4:40AM, Dave Jiang wrote: > >>> On Apr 12, 2018, at 7:30 PM, Verma, Vishal L > >>> > >> wrote: > On Tue, 2018-04-10 at 10:43 -0700, Dave Jiang wrote: > > > On 04/10/201

Re: [PATCH v2] ndctl: fix libdaxctl memory leak

2018-04-13 Thread Dave Jiang
On 4/12/2018 11:57 PM, Plewa, Lukasz wrote: On Apr 13, 2018, at 4:40AM, Dave Jiang wrote: On Apr 12, 2018, at 7:30 PM, Verma, Vishal L wrote: On Tue, 2018-04-10 at 10:43 -0700, Dave Jiang wrote: On 04/10/2018 10:38 AM, Plewa, Lukasz wrote: On Tue, Apr 10, 2018 at 7:17 PM, Dave Jiang wrote

RE: [PATCH v2] ndctl: fix libdaxctl memory leak

2018-04-12 Thread Plewa, Lukasz
On Apr 13, 2018, at 4:40AM, Dave Jiang wrote: > > On Apr 12, 2018, at 7:30 PM, Verma, Vishal L > wrote: > > > >> On Tue, 2018-04-10 at 10:43 -0700, Dave Jiang wrote: > >> > >>> On 04/10/2018 10:38 AM, Plewa, Lukasz wrote: > >>> On Tue, Apr 10, 2018 at 7:17 PM, Dave Jiang > >>> wrote: > When

Re: [PATCH v2] ndctl: fix libdaxctl memory leak

2018-04-12 Thread Jiang, Dave
> On Apr 12, 2018, at 7:30 PM, Verma, Vishal L wrote: > >> On Tue, 2018-04-10 at 10:43 -0700, Dave Jiang wrote: >> >>> On 04/10/2018 10:38 AM, Plewa, Lukasz wrote: >>> On Tue, Apr 10, 2018 at 7:17 PM, Dave Jiang >>> wrote: When daxctl_unref is releasing the context, we should make sure t

Re: [PATCH v2] ndctl: fix libdaxctl memory leak

2018-04-12 Thread Verma, Vishal L
On Tue, 2018-04-10 at 10:43 -0700, Dave Jiang wrote: > > On 04/10/2018 10:38 AM, Plewa, Lukasz wrote: > > On Tue, Apr 10, 2018 at 7:17 PM, Dave Jiang > > wrote: > > > When daxctl_unref is releasing the context, we should make sure that > > > the > > > regions and devices are also being released.

Re: [PATCH v2] ndctl: fix libdaxctl memory leak

2018-04-10 Thread Dave Jiang
On 04/10/2018 10:38 AM, Plewa, Lukasz wrote: > On Tue, Apr 10, 2018 at 7:17 PM, Dave Jiang wrote: >> When daxctl_unref is releasing the context, we should make sure that the >> regions and devices are also being released. free_region() will free all the >> devices under the region. >> >> Signed-

RE: [PATCH v2] ndctl: fix libdaxctl memory leak

2018-04-10 Thread Plewa, Lukasz
On Tue, Apr 10, 2018 at 7:17 PM, Dave Jiang wrote: > When daxctl_unref is releasing the context, we should make sure that the > regions and devices are also being released. free_region() will free all the > devices under the region. > > Signed-off-by: Dave Jiang > --- > > v2: Use list_for_each_