Re: [Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-05-02 Thread Dixit, Ashutosh
On Sun, 01 May 2022 23:22:02 -0700, Andrzej Hajda wrote: > On 29.04.2022 06:25, Dixit, Ashutosh wrote: > > On Thu, 28 Apr 2022 07:36:14 -0700, Andrzej Hajda wrote: > >> See [1], it is quite old, so maybe it is not valid anymore, but I see no > >> code proving sth has changed. > > Hi Andrzej, > > >

Re: [Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-05-02 Thread Andrzej Hajda
On 29.04.2022 06:25, Dixit, Ashutosh wrote: On Thu, 28 Apr 2022 07:36:14 -0700, Andrzej Hajda wrote: On 27.04.2022 22:46, Dixit, Ashutosh wrote: On Sun, 24 Apr 2022 15:36:23 -0700, Andi Shyti wrote: Hi Andrzej and Ashutosh, b/drivers/gpu/drm/i915/gt/intel_gt_types.h index

Re: [Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-28 Thread Dixit, Ashutosh
On Thu, 28 Apr 2022 07:36:14 -0700, Andrzej Hajda wrote: > On 27.04.2022 22:46, Dixit, Ashutosh wrote: > > On Sun, 24 Apr 2022 15:36:23 -0700, Andi Shyti wrote: > >> Hi Andrzej and Ashutosh, > >> > >> b/drivers/gpu/drm/i915/gt/intel_gt_types.h > >> index 937b2e1a305e..4c72b4f983a6 100644 >

[Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-28 Thread Ashutosh Dixit
All kmalloc'd kobjects need a kobject_put() to free memory. For example in previous code, kobj_gt_release() never gets called. The requirement of kobject_put() now results in a slightly different code organization. v2: s/gtn/gt/ (Andi) Cc: Andi Shyti Cc: Andrzej Hajda Fixes: b770bcfae9ad

Re: [Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-28 Thread Andrzej Hajda
On 27.04.2022 22:46, Dixit, Ashutosh wrote: On Sun, 24 Apr 2022 15:36:23 -0700, Andi Shyti wrote: Hi Andrzej and Ashutosh, b/drivers/gpu/drm/i915/gt/intel_gt_types.h index 937b2e1a305e..4c72b4f983a6 100644 --- a/drivers/gpu/drm/i915/gt/intel_gt_types.h +++

Re: [Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-27 Thread Dixit, Ashutosh
On Wed, 27 Apr 2022 04:45:03 -0700, Andi Shyti wrote: > > Hi Ashutosh, Hi Andi, > > > > -static struct kobj_type kobj_gt_type = { > > > > - .release = kobj_gt_release, > > > > +static struct kobj_type kobj_gtn_type = { > > > > > > what does it mean GTN? Or is it GTn? Please use just GT,

Re: [Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-27 Thread Dixit, Ashutosh
On Sun, 24 Apr 2022 15:36:23 -0700, Andi Shyti wrote: > > Hi Andrzej and Ashutosh, > > > > > > b/drivers/gpu/drm/i915/gt/intel_gt_types.h > > > > > index 937b2e1a305e..4c72b4f983a6 100644 > > > > > --- a/drivers/gpu/drm/i915/gt/intel_gt_types.h > > > > > +++

Re: [Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-27 Thread Andi Shyti
Hi Ashutosh, > > > -static struct kobj_type kobj_gt_type = { > > > - .release = kobj_gt_release, > > > +static struct kobj_type kobj_gtn_type = { > > > > what does it mean GTN? Or is it GTn? Please use just GT, gtn is > > confusing. > > > > Same for all the rest of the gtn's you have used below.

Re: [Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-26 Thread Dixit, Ashutosh
On Sun, 24 Apr 2022 15:30:59 -0700, Andi Shyti wrote: > > Hi Ashutosh, > Hi Andi, > [...] > > > -static struct kobj_type kobj_gt_type = { > > - .release = kobj_gt_release, > > +static struct kobj_type kobj_gtn_type = { > > what does it mean GTN? Or is it GTn? Please use just GT, gtn is >

Re: [Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-24 Thread Andi Shyti
Hi Andrzej and Ashutosh, > > > > b/drivers/gpu/drm/i915/gt/intel_gt_types.h > > > > index 937b2e1a305e..4c72b4f983a6 100644 > > > > --- a/drivers/gpu/drm/i915/gt/intel_gt_types.h > > > > +++ b/drivers/gpu/drm/i915/gt/intel_gt_types.h > > > > @@ -222,6 +222,9 @@ struct intel_gt { > > > > }

Re: [Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-24 Thread Andi Shyti
Hi Ashutosh, [...] > -static struct kobj_type kobj_gt_type = { > - .release = kobj_gt_release, > +static struct kobj_type kobj_gtn_type = { what does it mean GTN? Or is it GTn? Please use just GT, gtn is confusing. Same for all the rest of the gtn's you have used below. Thanks, Andi

Re: [Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-20 Thread Andrzej Hajda
On 20.04.2022 18:12, Dixit, Ashutosh wrote: On Wed, 20 Apr 2022 05:17:57 -0700, Andrzej Hajda wrote: Hi Ashutosh, Hi Andrzej, On 20.04.2022 07:21, Ashutosh Dixit wrote: All kmalloc'd kobjects need a kobject_put() to free memory. For example in previous code, kobj_gt_release() never gets

Re: [Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-20 Thread Dixit, Ashutosh
On Wed, 20 Apr 2022 05:17:57 -0700, Andrzej Hajda wrote: > > Hi Ashutosh, Hi Andrzej, > On 20.04.2022 07:21, Ashutosh Dixit wrote: > > All kmalloc'd kobjects need a kobject_put() to free memory. For example in > > previous code, kobj_gt_release() never gets called. The requirement of > >

Re: [Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-20 Thread Dixit, Ashutosh
On Wed, 20 Apr 2022 05:17:57 -0700, Andrzej Hajda wrote: > > Hi Ashutosh, Hi Andrzej, > On 20.04.2022 07:21, Ashutosh Dixit wrote: > > All kmalloc'd kobjects need a kobject_put() to free memory. For example in > > previous code, kobj_gt_release() never gets called. The requirement of > >

Re: [Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-20 Thread Andrzej Hajda
Hi Ashutosh, On 20.04.2022 07:21, Ashutosh Dixit wrote: All kmalloc'd kobjects need a kobject_put() to free memory. For example in previous code, kobj_gt_release() never gets called. The requirement of kobject_put() now results in a slightly different code organization. Cc: Andi Shyti Cc:

[Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-20 Thread Ashutosh Dixit
All kmalloc'd kobjects need a kobject_put() to free memory. For example in previous code, kobj_gt_release() never gets called. The requirement of kobject_put() now results in a slightly different code organization. Cc: Andi Shyti Cc: Andrzej Hajda Cc: Rodrigo Vivi Fixes: b770bcfae9ad

[Intel-gfx] [PATCH 7/9] drm/i915/gt: Fix memory leaks in per-gt sysfs

2022-04-19 Thread Ashutosh Dixit
All kmalloc'd kobjects need a kobject_put() to free memory. For example in previous code, kobj_gt_release() never gets called. The requirement of kobject_put() now results in a slightly different code organization. Cc: Andi Shyti Cc: Andrzej Hajda Cc: Rodrigo Vivi Fixes: b770bcfae9ad