Re: [Intel-gfx] [PATCH] drm/i915: do not return invalid pointers as a *dentry

2019-02-05 Thread Rodrigo Vivi
On Mon, Feb 04, 2019 at 08:37:01PM +0100, Greg Kroah-Hartman wrote: > On Mon, Feb 04, 2019 at 10:13:25AM -0800, Rodrigo Vivi wrote: > > On Thu, Jan 31, 2019 at 07:17:02PM +0100, Greg Kroah-Hartman wrote: > > > On Thu, Jan 31, 2019 at 09:59:26AM -0800, Rodrigo Vivi wrote: > > > > On Thu, Jan 31,

Re: [Intel-gfx] [PATCH] drm/i915: do not return invalid pointers as a *dentry

2019-02-05 Thread Greg Kroah-Hartman
On Mon, Feb 04, 2019 at 10:13:25AM -0800, Rodrigo Vivi wrote: > On Thu, Jan 31, 2019 at 07:17:02PM +0100, Greg Kroah-Hartman wrote: > > On Thu, Jan 31, 2019 at 09:59:26AM -0800, Rodrigo Vivi wrote: > > > On Thu, Jan 31, 2019 at 02:15:07PM +0100, Greg Kroah-Hartman wrote: > > > > When calling

Re: [Intel-gfx] [PATCH] drm/i915: do not return invalid pointers as a *dentry

2019-02-04 Thread Rodrigo Vivi
On Thu, Jan 31, 2019 at 07:17:02PM +0100, Greg Kroah-Hartman wrote: > On Thu, Jan 31, 2019 at 09:59:26AM -0800, Rodrigo Vivi wrote: > > On Thu, Jan 31, 2019 at 02:15:07PM +0100, Greg Kroah-Hartman wrote: > > > When calling debugfs functions, they can now return error values if > > > something went

Re: [PATCH] drm/i915: do not return invalid pointers as a *dentry

2019-01-31 Thread Greg Kroah-Hartman
On Thu, Jan 31, 2019 at 09:59:26AM -0800, Rodrigo Vivi wrote: > On Thu, Jan 31, 2019 at 02:15:07PM +0100, Greg Kroah-Hartman wrote: > > When calling debugfs functions, they can now return error values if > > something went wrong. If that happens, return a NULL as a *dentry to > > the relay core

Re: [PATCH] drm/i915: do not return invalid pointers as a *dentry

2019-01-31 Thread Rodrigo Vivi
On Thu, Jan 31, 2019 at 02:15:07PM +0100, Greg Kroah-Hartman wrote: > When calling debugfs functions, they can now return error values if > something went wrong. If that happens, return a NULL as a *dentry to > the relay core instead of passing it an illegal pointer. > > The relay core should be

[PATCH] drm/i915: do not return invalid pointers as a *dentry

2019-01-31 Thread Greg Kroah-Hartman
When calling debugfs functions, they can now return error values if something went wrong. If that happens, return a NULL as a *dentry to the relay core instead of passing it an illegal pointer. The relay core should be able to handle an illegal pointer, but add this check to be safe. Cc: Jani