Re: [PATCH iproute2-next v3] rdma: Document IB device renaming option

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/18/2018 6:05 AM, Leon Romanovsky wrote:
On Fri, Nov 16, 2018 at 08:10:35PM +0000, Ruhl, Michael J wrote:
-----Original Message-----
From: linux-rdma-owner@xxxxxxxxxxxxxxx [mailto:linux-rdma-
owner@xxxxxxxxxxxxxxx] On Behalf Of Leon Romanovsky
Sent: Sunday, November 4, 2018 2:11 PM
To: David Ahern <dsahern@xxxxxxxxx>
Cc: Leon Romanovsky <leonro@xxxxxxxxxxxx>; netdev
<netdev@xxxxxxxxxxxxxxx>; RDMA mailing list <linux-rdma@xxxxxxxxxxxxxxx>;
Stephen Hemminger <stephen@xxxxxxxxxxxxxxxxxx>
Subject: [PATCH iproute2-next v3] rdma: Document IB device renaming
option

From: Leon Romanovsky <leonro@xxxxxxxxxxxx>

Hi Leon,

After looking at this and Steve Wise's changes for the ADDLINK/DELLINK,
it occurred to me that the driver that handed the name to ib_register_device()
might be interested in knowing that this name change occurred.

Are there plans to include a some kind of notify mechanism so drivers can
find out when things like this occur?

At least for device rename, I don't see any real need for such event,
because drivers are not supposed to rely on names.

It just seems like something they want to know. But agree driver should not rely on a name.

I would say that it is probably driver bug to rely on device name
during its execution.


Is this something that should be done?

I think yes, we can extend ib_event to support more events than now,
but should we?

On one hand it make sense to have an event. In case the driver had something it wanted to do with the name but I can't really think of a compelling reason it does. So I'd say we don't worry about it now but if someone finds a use for it then adding it later should be an option that is on the table.

-Denny






[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux