From: Ka-Cheong Poon <ka-cheong.poon@oracle.com>
To: Leon Romanovsky <leon@kernel.org>
Cc: Jason Gunthorpe <jgg@ziepe.ca>, linux-rdma@vger.kernel.org
Subject: Re: Finding the namespace of a struct ib_device
Date: Mon, 7 Sep 2020 17:28:23 +0800 [thread overview]
Message-ID: <27a60d6d-0e86-6fc6-f4e9-2893c824ba56@oracle.com> (raw)
In-Reply-To: <20200907090438.GM55261@unreal>
On 9/7/20 5:04 PM, Leon Romanovsky wrote:
> On Mon, Sep 07, 2020 at 04:24:26PM +0800, Ka-Cheong Poon wrote:
>> On 9/7/20 3:18 PM, Leon Romanovsky wrote:
>>> On Mon, Sep 07, 2020 at 11:33:38AM +0800, Ka-Cheong Poon wrote:
>>>> On 9/6/20 3:44 PM, Leon Romanovsky wrote:
>>>>> On Fri, Sep 04, 2020 at 10:02:10PM +0800, Ka-Cheong Poon wrote:
>>>>>> On 9/4/20 7:32 PM, Jason Gunthorpe wrote:
>>>>>>> On Fri, Sep 04, 2020 at 12:01:12PM +0800, Ka-Cheong Poon wrote:
>>>>>>>> On 9/4/20 1:39 AM, Jason Gunthorpe wrote:
>>>>>>>>> On Thu, Sep 03, 2020 at 10:02:01PM +0800, Ka-Cheong Poon wrote:
>>>>>>>>>> When a struct ib_client's add() function is called. is there a
>>>>>>>>>> supported method to find out the namespace of the passed in
>>>>>>>>>> struct ib_device? There is rdma_dev_access_netns() but it does
>>>>>>>>>> not return the namespace. It seems that it needs to have
>>>>>>>>>> something like the following.
>>>>>>>>>>
>>>>>>>>>> struct net *rdma_dev_to_netns(struct ib_device *ib_dev)
>>>>>>>>>> {
>>>>>>>>>> return read_pnet(&ib_dev->coredev.rdma_net);
>>>>>>>>>> }
>>>>>>>>>>
>>>>>>>>>> Comments?
>>>>>>>>>
>>>>>>>>> I suppose, but why would something need this?
>>>>>>>>
>>>>>>>>
>>>>>>>> If the client needs to allocate stuff for the namespace
>>>>>>>> related to that device, it needs to know the namespace of
>>>>>>>> that device. Then when that namespace is deleted, the
>>>>>>>> client can clean up those related stuff as the client's
>>>>>>>> namespace exit function can be called before the remove()
>>>>>>>> function is triggered in rdma_dev_exit_net(). Without
>>>>>>>> knowing the namespace of that device, coordination cannot
>>>>>>>> be done.
>>>>>>>
>>>>>>> Since each device can only be in one namespace, why would a client
>>>>>>> ever need to allocate at a level more granular than a device?
>>>>>>
>>>>>>
>>>>>> A client wants to have namespace specific info. If the
>>>>>> device belongs to a namespace, it wants to associate those
>>>>>> info with that device. When a namespace is deleted, the
>>>>>> info will need to be deleted. You can consider the info
>>>>>> as associated with both a namespace and a device.
>>>>>
>>>>> Can you be more specific about which info you are talking about?
>>>>
>>>>
>>>> Actually, a lot of info can be both namespace and device specific.
>>>> For example, a client wants to have a different PD allocation policy
>>>> with a device when used in different namespaces.
>>>>
>>>>
>>>>> And what is the client that is net namespace-aware from one side,
>>>>> but from another separate data between them "manually"?
>>>>
>>>>
>>>> Could you please elaborate what is meant by "namespace aware from
>>>> one side but from another separate data between them manually"?
>>>> I understand what namespace aware means. But it is not clear what
>>>> is meant by "separating data manually". Do you mean having different
>>>> behavior in different namespaces? If this is the case, there is
>>>> nothing special here. An admin may choose to have different behavior
>>>> in different namespaces. There is nothing manual going on in the
>>>> client code.
>>>
>>> We are talking about net-namespaces, and as we wrote above, the ib_device
>>> that supports such namespace can exist only in a single one
>>>
>>> The client that implemented such support can check its namespace while
>>> "client->add" is called. It should be equal to be seen by ib_device.
>>>
>>> See:
>>> rdma_dev_change_netns ->
>>> enable_device_and_get ->
>>> add_client_context ->
>>> client->add(device)
>>
>>
>> This is the original question. How does the client's add() function
>> know the namespace of device? What is your suggestion in finding
>> the net namespace of device at add() time?
>
> As I wrote above, "It should be equal to be seen by ib_device.", check net
> namespace of your client.
Could you please be more specific? A client calls ib_register_client() to
register with the RDMA framework. Then when a device is added, the client's
add() function is called with the struct ib_device. How does the client
find out the namespace "seen by the ib_device"? Do you mean that there is
a variant of ib_register_client() which can take a net namespace as parameter?
Or is there a variant of struct ib_client which has a net namespace field?
Or? Thanks.
--
K. Poon
ka-cheong.poon@oracle.com
next prev parent reply other threads:[~2020-09-07 9:28 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-09-03 14:02 Finding the namespace of a struct ib_device Ka-Cheong Poon
2020-09-03 17:39 ` Jason Gunthorpe
2020-09-04 4:01 ` Ka-Cheong Poon
2020-09-04 11:32 ` Jason Gunthorpe
2020-09-04 14:02 ` Ka-Cheong Poon
2020-09-06 7:44 ` Leon Romanovsky
2020-09-07 3:33 ` Ka-Cheong Poon
2020-09-07 7:18 ` Leon Romanovsky
2020-09-07 8:24 ` Ka-Cheong Poon
2020-09-07 9:04 ` Leon Romanovsky
2020-09-07 9:28 ` Ka-Cheong Poon [this message]
2020-09-07 10:22 ` Leon Romanovsky
2020-09-07 13:48 ` Ka-Cheong Poon
2020-09-29 16:57 ` RDMA subsystem namespace related questions (was Re: Finding the namespace of a struct ib_device) Ka-Cheong Poon
2020-09-29 17:40 ` Jason Gunthorpe
2020-09-30 10:32 ` Ka-Cheong Poon
2020-10-02 14:04 ` Jason Gunthorpe
2020-10-05 10:27 ` Ka-Cheong Poon
2020-10-05 13:16 ` Jason Gunthorpe
2020-10-05 13:57 ` Ka-Cheong Poon
2020-10-05 14:25 ` Jason Gunthorpe
2020-10-05 15:02 ` Ka-Cheong Poon
2020-10-05 15:45 ` Jason Gunthorpe
2020-10-06 9:36 ` Ka-Cheong Poon
2020-10-06 12:46 ` Jason Gunthorpe
2020-10-07 8:38 ` Ka-Cheong Poon
2020-10-07 11:16 ` Leon Romanovsky
2020-10-08 10:22 ` Ka-Cheong Poon
2020-10-08 10:36 ` Leon Romanovsky
2020-10-08 11:08 ` Ka-Cheong Poon
2020-10-08 16:08 ` Jason Gunthorpe
2020-10-08 16:21 ` Chuck Lever
2020-10-08 16:46 ` Jason Gunthorpe
2020-10-09 4:49 ` Ka-Cheong Poon
2020-10-09 14:39 ` Jason Gunthorpe
2020-10-09 14:48 ` Chuck Lever
2020-10-09 14:57 ` Jason Gunthorpe
2020-10-09 15:00 ` Chuck Lever
2020-10-09 15:07 ` Jason Gunthorpe
2020-10-09 15:27 ` Chuck Lever
2020-10-09 15:34 ` Jason Gunthorpe
2020-10-09 15:52 ` Chuck Lever
2020-10-12 8:20 ` Ka-Cheong Poon
2020-10-16 18:54 ` Jason Gunthorpe
2020-10-16 20:49 ` Chuck Lever
2020-10-19 18:31 ` Jason Gunthorpe
2020-10-07 12:28 ` Jason Gunthorpe
2020-10-08 10:49 ` Ka-Cheong Poon
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=27a60d6d-0e86-6fc6-f4e9-2893c824ba56@oracle.com \
--to=ka-cheong.poon@oracle.com \
--cc=jgg@ziepe.ca \
--cc=leon@kernel.org \
--cc=linux-rdma@vger.kernel.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;
as well as URLs for NNTP newsgroup(s).