Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1393925

Re: [RFC PATCH] livepatch: allow removal of a disabled patch

Show all headers | View raw

On Tue, May 03, 2016 at 09:39:48PM -0500, Josh Poimboeuf wrote:
> On Wed, May 04, 2016 at 12:31:12AM +0200, Jiri Kosina wrote:
> > On Tue, 3 May 2016, Josh Poimboeuf wrote:
> > 
> > > > 1. Do we really need a completion? If I am not missing something
> > > > kobject_del() always waits for sysfs callers to leave thanks to kernfs
> > > > active protection.
> > > 
> > > What do you mean by "kernfs active protection"?  I see that
> > > kernfs_remove() gets the kernfs_mutex lock, but I can't find anywhere
> > > that a write to a sysfs file uses that lock.
> > > 
> > > I'm probably missing something...
> > 
> > I don't want to speak on Miroslav's behalf, but I'm pretty sure that what 
> > he has on mind is per-kernfs_node active refcounting kernfs does (see 
> > kernfs_node->active, and especially it's usage in __kernfs_remove()).
> > 
> > More specifically, execution of store() and show() sysfs callbacks is 
> > guaranteed (by kernfs) to happen with that particular attribute's active 
> > reference held for reading (and that makes it impossible for that 
> > attribute to vanish prematurely).
> 
> Thanks, that makes sense.
> 
> So what exactly is the problem the completion is trying to solve?  Is it
> to ensure that the kobject has been cleaned up before it returns to the
> caller, in case the user wants to call klp_register() again after
> unregistering?
> 
> If so, that's quite an unusual use case which I think we should just
> consider unsupported.  In fact, if you try to do it, kobject_init()
> complains loudly because kobj->state_initialized is still 1 because
> kobjects aren't meant to be reused like that.

... and now I realize the point is actually to prevent the caller from
freeing klp_patch before kobject_cleanup() runs.

So yeah, it looks like we need the completion in case
CONFIG_DEBUG_KOBJECT_RELEASE is enabled.

Or alternatively we could convert patch->kobj to be dynamically
allocated instead of embedded in klp_patch.

-- 
Josh

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[RFC PATCH] livepatch: allow removal of a disabled patch Miroslav Benes <mbenes@suse.cz> - 2016-05-02 14:00 +0200
  Re: [RFC PATCH] livepatch: allow removal of a disabled patch Josh Poimboeuf <jpoimboe@redhat.com> - 2016-05-02 17:10 +0200
    Re: [RFC PATCH] livepatch: allow removal of a disabled patch Miroslav Benes <mbenes@suse.cz> - 2016-05-03 10:20 +0200
  Re: [RFC PATCH] livepatch: allow removal of a disabled patch Josh Poimboeuf <jpoimboe@redhat.com> - 2016-05-03 23:40 +0200
    Re: [RFC PATCH] livepatch: allow removal of a disabled patch Jiri Kosina <jikos@kernel.org> - 2016-05-04 00:40 +0200
      Re: [RFC PATCH] livepatch: allow removal of a disabled patch Josh Poimboeuf <jpoimboe@redhat.com> - 2016-05-04 04:40 +0200
        Re: [RFC PATCH] livepatch: allow removal of a disabled patch Josh Poimboeuf <jpoimboe@redhat.com> - 2016-05-04 05:40 +0200
          Re: [RFC PATCH] livepatch: allow removal of a disabled patch Miroslav Benes <mbenes@suse.cz> - 2016-05-04 14:00 +0200
            Re: [RFC PATCH] livepatch: allow removal of a disabled patch Josh Poimboeuf <jpoimboe@redhat.com> - 2016-05-04 15:20 +0200
              Re: [RFC PATCH] livepatch: allow removal of a disabled patch Miroslav Benes <mbenes@suse.cz> - 2016-05-04 16:40 +0200
                Re: [RFC PATCH] livepatch: allow removal of a disabled patch Josh Poimboeuf <jpoimboe@redhat.com> - 2016-05-04 18:20 +0200
                Re: [RFC PATCH] livepatch: allow removal of a disabled patch Miroslav Benes <mbenes@suse.cz> - 2016-05-05 10:30 +0200
                Re: [RFC PATCH] livepatch: allow removal of a disabled patch Josh Poimboeuf <jpoimboe@redhat.com> - 2016-05-05 15:30 +0200
                Re: [RFC PATCH] livepatch: allow removal of a disabled patch Miroslav Benes <mbenes@suse.cz> - 2016-05-05 16:30 +0200
                Re: [RFC PATCH] livepatch: allow removal of a disabled patch Josh Poimboeuf <jpoimboe@redhat.com> - 2016-05-05 17:10 +0200
                Re: [RFC PATCH] livepatch: allow removal of a disabled patch Jiri Kosina <jikos@kernel.org> - 2016-05-05 23:10 +0200
                Re: livepatch: allow removal of a disabled patch Jessica Yu <jeyu@redhat.com> - 2016-05-06 02:50 +0200
                Re: livepatch: allow removal of a disabled patch Miroslav Benes <mbenes@suse.cz> - 2016-05-06 10:00 +0200

csiph-web