[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Thu, 6 Jun 2019 10:16:34 -0700
From: Jakub Kicinski <jakub.kicinski@...ronome.com>
To: "Jubran, Samih" <sameehj@...zon.com>
Cc: David Woodhouse <dwmw2@...radead.org>,
"Bshara, Nafea" <nafea@...zon.com>, Andrew Lunn <andrew@...n.ch>,
"Kiyanovski, Arthur" <akiyano@...zon.com>,
"Bshara, Saeed" <saeedb@...zon.com>,
"Tzalik, Guy" <gtzalik@...zon.com>,
"Matushevsky, Alexander" <matua@...zon.com>,
"Liguori, Anthony" <aliguori@...zon.com>,
"Saidi, Ali" <alisaidi@...zon.com>,
"Machulsky, Zorik" <zorik@...zon.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
"Wilson, Matt" <msw@...zon.com>,
"davem@...emloft.net" <davem@...emloft.net>,
"Belgazal, Netanel" <netanel@...zon.com>,
"Herrenschmidt, Benjamin" <benh@...zon.com>
Subject: Re: [PATCH V2 net 00/11] Extending the ena driver to support new
features and enhance performance
Hi Samih!
Please don't top post on Linux kernel mailing lists.
On Thu, 6 Jun 2019 10:23:40 +0000, Jubran, Samih wrote:
> As of today there are no flags exposed by ENA NIC device, however, we
> are planning to use them in the near future. We want to provide
> customers with extra methods to identify (and differentiate) multiple
> network interfaces that can be attached to a single VM. Currently,
> customers can identify a specific network interface (ENI) only by MAC
> address, and later look up this MAC among other multiple ENIs that
> they have. In some cases it might not be convenient. Using these
> flags will let us inform the customer about ENI`s specific
> properties.
Oh no :( You're using private _feature_ flags as labels or tags.
> It's not finalized, but tentatively it can look like this:
> primary-eni: on /* Differentiate between primary and secondary ENIs */
Did you consider using phys_port_name for this use case?
If the intent is to have those interfaces bonded, even better would
be to extend the net_failover module or work on user space ABI for VM
failover. That might be a bigger effort, though.
> has-associated-efa: on /* Will specify ENA device has another associated EFA device */
IDK how your ENA/EFA thing works, but sounds like something that should
be solved in the device model.
Powered by blists - more mailing lists