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:

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

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

2018-04-13 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

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

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

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

[PATCH v2] ndctl: fix libdaxctl memory leak

2018-04-10 Thread Dave Jiang
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_safe() for region removal. (Dan)