public inbox for linux-rdma@vger.kernel.org
 help / color / mirror / Atom feed
From: "gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org" <gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org>
To: Bart Van Assche <Bart.VanAssche-Sjgp3cTcYWE@public.gmane.org>
Cc: "jgg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org"
	<jgg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>,
	"linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org"
	<dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
Subject: Re: [PATCH v3 1/5] kobject: Export kobj_ns_grab_current() and kobj_ns_drop()
Date: Tue, 9 Jan 2018 19:30:47 +0100	[thread overview]
Message-ID: <20180109183047.GA3236@kroah.com> (raw)
In-Reply-To: <1515520784.2721.11.camel-Sjgp3cTcYWE@public.gmane.org>

On Tue, Jan 09, 2018 at 05:59:45PM +0000, Bart Van Assche wrote:
> On Mon, 2018-01-08 at 15:14 -0800, Bart Van Assche wrote:
> > Make it possible to call these two functions from a kernel module.
> > Note: despite their name, these two functions can be used meaningfully
> > independent of kobjects. A later patch will add calls to these
> > functions from the SRP driver because this patch series modifies the
> > SRP driver such that it can hold a reference to a namespace that can
> > last longer than the lifetime of the process through which the
> > namespace reference was obtained.
> > 
> > Signed-off-by: Bart Van Assche <bart.vanassche-Sjgp3cTcYWE@public.gmane.org>
> > Cc: Greg Kroah-Hartman <gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org>
> > ---
> >  lib/kobject.c | 2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/lib/kobject.c b/lib/kobject.c
> > index 763d70a18941..06b849eee0ca 100644
> > --- a/lib/kobject.c
> > +++ b/lib/kobject.c
> > @@ -1039,6 +1039,7 @@ void *kobj_ns_grab_current(enum kobj_ns_type type)
> >  
> >  	return ns;
> >  }
> > +EXPORT_SYMBOL_GPL(kobj_ns_grab_current);
> >  
> >  const void *kobj_ns_netlink(enum kobj_ns_type type, struct sock *sk)
> >  {
> > @@ -1074,3 +1075,4 @@ void kobj_ns_drop(enum kobj_ns_type type, void *ns)
> >  		kobj_ns_ops_tbl[type]->drop_ns(ns);
> >  	spin_unlock(&kobj_ns_type_lock);
> >  }
> > +EXPORT_SYMBOL_GPL(kobj_ns_drop);
> 
> Hi Greg,
> 
> I'd like to repost this series to address a bug in patch 5/5 of this series.
> But before I do that, can you tell me whether I perhaps should widen the
> Cc-list for patch 1/5 of this series?

I have no idea why you need these functions, as I don't see any of these
other patches.  Please resend the whole series please.

I will note that no driver had better be mucking around with a "raw"
kobject, or I am going to be very mad...

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2018-01-09 18:30 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-01-08 23:14 [PATCH v3 0/5] IB/srp: Add RDMA/CM support Bart Van Assche
     [not found] ` <20180108231436.19750-1-bart.vanassche-Sjgp3cTcYWE@public.gmane.org>
2018-01-08 23:14   ` [PATCH v3 1/5] kobject: Export kobj_ns_grab_current() and kobj_ns_drop() Bart Van Assche
     [not found]     ` <20180108231436.19750-2-bart.vanassche-Sjgp3cTcYWE@public.gmane.org>
2018-01-09 17:59       ` Bart Van Assche
     [not found]         ` <1515520784.2721.11.camel-Sjgp3cTcYWE@public.gmane.org>
2018-01-09 18:30           ` gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r [this message]
     [not found]             ` <20180109183047.GA3236-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2018-01-09 18:47               ` Bart Van Assche
     [not found]                 ` <1515523648.2721.21.camel-Sjgp3cTcYWE@public.gmane.org>
2018-01-09 18:57                   ` gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r
     [not found]                     ` <20180109185745.GA13061-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2018-01-17 16:13                       ` Bart Van Assche
2018-01-08 23:14   ` [PATCH v3 2/5] IB/srp: Use kstrtoull() instead of simple_strtoull() Bart Van Assche
2018-01-08 23:14   ` [PATCH v3 3/5] IB/srp: Make the path record query error message more informative Bart Van Assche
2018-01-08 23:14   ` [PATCH v3 4/5] IB/srp: Refactor srp_send_req() Bart Van Assche
2018-01-08 23:14   ` [PATCH v3 5/5] IB/srp: Add RDMA/CM support Bart Van Assche

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180109183047.GA3236@kroah.com \
    --to=gregkh-hqyy1w1ycw8ekmwlsbkhg0b+6bgklq7r@public.gmane.org \
    --cc=Bart.VanAssche-Sjgp3cTcYWE@public.gmane.org \
    --cc=dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=jgg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox