Re: [Intel-gfx] [PATCH v2] drm/i915: load driver even if debugfs fails
On Mon, Aug 03, 2015 at 12:54:33PM +0200, Daniel Vetter wrote: > On Mon, Aug 03, 2015 at 02:56:42PM +0530, Sudip Mukherjee wrote: > > On Thu, Jul 23, 2015 at 07:36:12PM +0530, Sudip Mukherjee wrote: > > > debugfs files are not necessary for the usual operation of the driver > > > and the device. No need to check for the return values from the debugfs > > > file creation. Even if one debugfs file fails to create we try with the > > > next debugfs file and ultimately return success always so that the > > > driver continues to load. > > Does this even happen? realistically - I don't think it can happen. Even if it happens then there will be more serious problem like memory crunch or filesystem problem and in those cases there is no point in continuing to load the driver. regards sudip ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH v2] drm/i915: load driver even if debugfs fails
On Mon, Aug 03, 2015 at 02:56:42PM +0530, Sudip Mukherjee wrote: > On Thu, Jul 23, 2015 at 07:36:12PM +0530, Sudip Mukherjee wrote: > > debugfs files are not necessary for the usual operation of the driver > > and the device. No need to check for the return values from the debugfs > > file creation. Even if one debugfs file fails to create we try with the > > next debugfs file and ultimately return success always so that the > > driver continues to load. Does this even happen? I'm reluctant to merge patches without real-world justification. -Daniel > > cleanup will clean all the created debugfs files as the list of file > > that are created are maintained in minor->debugfs_list. > > > > Cc: Chris Wilson > > Signed-off-by: Sudip Mukherjee > > --- > A gentle ping. > > regards > sudip > ___ > dri-devel mailing list > dri-de...@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] [PATCH v2] drm/i915: load driver even if debugfs fails
On Thu, Jul 23, 2015 at 07:36:12PM +0530, Sudip Mukherjee wrote: > debugfs files are not necessary for the usual operation of the driver > and the device. No need to check for the return values from the debugfs > file creation. Even if one debugfs file fails to create we try with the > next debugfs file and ultimately return success always so that the > driver continues to load. > cleanup will clean all the created debugfs files as the list of file > that are created are maintained in minor->debugfs_list. > > Cc: Chris Wilson > Signed-off-by: Sudip Mukherjee > --- A gentle ping. regards sudip ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx