[PATCH 1/2] drm: Fix memory leak at error path of drm_read()

2014-12-05 Thread Chris Wilson
On Thu, Dec 04, 2014 at 06:25:08PM +0100, Daniel Vetter wrote: > On Thu, Dec 04, 2014 at 04:31:15PM +, Chris Wilson wrote: > > On Thu, Dec 04, 2014 at 01:28:39PM +0100, Daniel Vetter wrote: > > > On Thu, Dec 04, 2014 at 11:51:14AM +, Chris Wilson wrote: > > > > On Thu, Dec 04, 2014 at 11:56

[PATCH 1/2] drm: Fix memory leak at error path of drm_read()

2014-12-04 Thread Daniel Vetter
On Thu, Dec 04, 2014 at 04:31:15PM +, Chris Wilson wrote: > On Thu, Dec 04, 2014 at 01:28:39PM +0100, Daniel Vetter wrote: > > On Thu, Dec 04, 2014 at 11:51:14AM +, Chris Wilson wrote: > > > On Thu, Dec 04, 2014 at 11:56:42AM +0100, Takashi Iwai wrote: > > > > Signed-off-by: Takashi Iwai >

[PATCH 1/2] drm: Fix memory leak at error path of drm_read()

2014-12-04 Thread Takashi Iwai
At Thu, 4 Dec 2014 16:38:57 +, Chris Wilson wrote: > > On Thu, Dec 04, 2014 at 04:31:15PM +, Chris Wilson wrote: > > On Thu, Dec 04, 2014 at 01:28:39PM +0100, Daniel Vetter wrote: > > > On Thu, Dec 04, 2014 at 11:51:14AM +, Chris Wilson wrote: > > > > On Thu, Dec 04, 2014 at 11:56:42AM

[PATCH 1/2] drm: Fix memory leak at error path of drm_read()

2014-12-04 Thread Chris Wilson
On Thu, Dec 04, 2014 at 04:31:15PM +, Chris Wilson wrote: > On Thu, Dec 04, 2014 at 01:28:39PM +0100, Daniel Vetter wrote: > > On Thu, Dec 04, 2014 at 11:51:14AM +, Chris Wilson wrote: > > > On Thu, Dec 04, 2014 at 11:56:42AM +0100, Takashi Iwai wrote: > > > > Signed-off-by: Takashi Iwai >

[PATCH 1/2] drm: Fix memory leak at error path of drm_read()

2014-12-04 Thread Chris Wilson
On Thu, Dec 04, 2014 at 01:28:39PM +0100, Daniel Vetter wrote: > On Thu, Dec 04, 2014 at 11:51:14AM +, Chris Wilson wrote: > > On Thu, Dec 04, 2014 at 11:56:42AM +0100, Takashi Iwai wrote: > > > Signed-off-by: Takashi Iwai > > > --- > > > drivers/gpu/drm/drm_fops.c | 1 + > > > 1 file changed

[PATCH 1/2] drm: Fix memory leak at error path of drm_read()

2014-12-04 Thread Daniel Vetter
On Thu, Dec 04, 2014 at 11:51:14AM +, Chris Wilson wrote: > On Thu, Dec 04, 2014 at 11:56:42AM +0100, Takashi Iwai wrote: > > Signed-off-by: Takashi Iwai > > --- > > drivers/gpu/drm/drm_fops.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/gpu/drm/drm_fops.c b/drivers

[PATCH 1/2] drm: Fix memory leak at error path of drm_read()

2014-12-04 Thread Takashi Iwai
At Thu, 4 Dec 2014 12:13:46 +, Chris Wilson wrote: > > On Thu, Dec 04, 2014 at 12:56:38PM +0100, Takashi Iwai wrote: > > At Thu, 4 Dec 2014 11:51:14 +, > > Chris Wilson wrote: > > > > > > On Thu, Dec 04, 2014 at 11:56:42AM +0100, Takashi Iwai wrote: > > > > Signed-off-by: Takashi Iwai >

[PATCH 1/2] drm: Fix memory leak at error path of drm_read()

2014-12-04 Thread Takashi Iwai
At Thu, 4 Dec 2014 11:51:14 +, Chris Wilson wrote: > > On Thu, Dec 04, 2014 at 11:56:42AM +0100, Takashi Iwai wrote: > > Signed-off-by: Takashi Iwai > > --- > > drivers/gpu/drm/drm_fops.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/gpu/drm/drm_fops.c b/drivers/gpu

[PATCH 1/2] drm: Fix memory leak at error path of drm_read()

2014-12-04 Thread Takashi Iwai
At Thu, 04 Dec 2014 12:50:04 +0100, Takashi Iwai wrote: > > At Thu, 4 Dec 2014 12:46:16 +0100, > Daniel Vetter wrote: > > > > so maybe we should backport that one once it's landed in 3.19? Speaking about this: yes, it's worth, IMO, as this (unexpectedly) seems really fixing the issue on the curr

[PATCH 1/2] drm: Fix memory leak at error path of drm_read()

2014-12-04 Thread Takashi Iwai
At Thu, 4 Dec 2014 12:46:16 +0100, Daniel Vetter wrote: > > On Thu, Dec 04, 2014 at 11:56:42AM +0100, Takashi Iwai wrote: > > Signed-off-by: Takashi Iwai > > Applied to my drm-misc branch. The second patch seems to be a dupe of > > commit bd008e5b2953186fc0c6633a885ade95e7043800 > Author: Chris

[PATCH 1/2] drm: Fix memory leak at error path of drm_read()

2014-12-04 Thread Daniel Vetter
On Thu, Dec 04, 2014 at 11:56:42AM +0100, Takashi Iwai wrote: > Signed-off-by: Takashi Iwai Applied to my drm-misc branch. The second patch seems to be a dupe of commit bd008e5b2953186fc0c6633a885ade95e7043800 Author: Chris Wilson Date: Tue Oct 7 14:13:51 2014 +0100 drm: Implement O_NONB

[PATCH 1/2] drm: Fix memory leak at error path of drm_read()

2014-12-04 Thread Chris Wilson
On Thu, Dec 04, 2014 at 12:56:38PM +0100, Takashi Iwai wrote: > At Thu, 4 Dec 2014 11:51:14 +, > Chris Wilson wrote: > > > > On Thu, Dec 04, 2014 at 11:56:42AM +0100, Takashi Iwai wrote: > > > Signed-off-by: Takashi Iwai > > > --- > > > drivers/gpu/drm/drm_fops.c | 1 + > > > 1 file changed,

[PATCH 1/2] drm: Fix memory leak at error path of drm_read()

2014-12-04 Thread Takashi Iwai
Signed-off-by: Takashi Iwai --- drivers/gpu/drm/drm_fops.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/drm_fops.c b/drivers/gpu/drm/drm_fops.c index ed7bc68f7e87..a82dc28d54f3 100644 --- a/drivers/gpu/drm/drm_fops.c +++ b/drivers/gpu/drm/drm_fops.c @@ -525,6 +525,7 @@ ssiz

[PATCH 1/2] drm: Fix memory leak at error path of drm_read()

2014-12-04 Thread Chris Wilson
On Thu, Dec 04, 2014 at 11:56:42AM +0100, Takashi Iwai wrote: > Signed-off-by: Takashi Iwai > --- > drivers/gpu/drm/drm_fops.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/gpu/drm/drm_fops.c b/drivers/gpu/drm/drm_fops.c > index ed7bc68f7e87..a82dc28d54f3 100644 > --- a/driver