Messages in this thread | | | From | Todd Poynor <> | Date | Mon, 16 Jul 2018 14:14:27 -0700 | Subject | Re: [PATCH 05/18] staging: gasket: sysfs mapping creation fixups |
| |
On Fri, Jul 13, 2018 at 11:32 PM, Greg Kroah-Hartman <gregkh@linuxfoundation.org> wrote: > On Fri, Jul 13, 2018 at 10:58:03PM -0700, Todd Poynor wrote: >> From: Todd Poynor <toddpoynor@google.com> >> >> Return EBUSY for attempt to create a mapping already in use. > > Why?
The existing code returns EINVAL which often means something bogus was requested, whereas EBUSY is sometimes used for requesting something already in use. If that's not an error typically returned in this situation I'm happy to drop it.
> >> Remove stale pointers on error allocating attr array. > > This should be a different patch.
Will do.
> > thanks, > > greg k-h
-- Todd
| |