Commit 3f85944fe207d0225ef21a2c0951d4946fc9a95d ("net: Add sysfs file
for port number") introduce dev_port to network devices. cxgb4 adapters
have multiple ports on the same PCI function, and used dev_id to
identify those ports. That use was removed by commit
8c367fcbe6549195d2eb11e62bea233f811aad41 ("cxgb4: Do not set
net_device::dev_id to VI index"), since dev_id should be used only when
devices share the same MAC address.
Using dev_port for cxgb4 allows different ports on the same PCI function
to be identified.
Signed-off-by: Thadeu Lima de Souza Cascardo <[email protected]>
---
drivers/net/ethernet/chelsio/cxgb4/t4_hw.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/net/ethernet/chelsio/cxgb4/t4_hw.c b/drivers/net/ethernet/chelsio/cxgb4/t4_hw.c
index bba6768..931478e 100644
--- a/drivers/net/ethernet/chelsio/cxgb4/t4_hw.c
+++ b/drivers/net/ethernet/chelsio/cxgb4/t4_hw.c
@@ -3962,6 +3962,7 @@ int t4_port_init(struct adapter *adap, int mbox, int pf, int vf)
p->lport = j;
p->rss_size = rss_size;
memcpy(adap->port[i]->dev_addr, addr, ETH_ALEN);
+ adap->port[i]->dev_port = j;
ret = ntohl(c.u.info.lstatus_to_modtype);
p->mdio_addr = (ret & FW_PORT_CMD_MDIOCAP) ?
--
1.7.1
From: Thadeu Lima de Souza Cascardo <[email protected]>
Date: Sat, 21 Jun 2014 09:48:08 -0300
> Commit 3f85944fe207d0225ef21a2c0951d4946fc9a95d ("net: Add sysfs file
> for port number") introduce dev_port to network devices. cxgb4 adapters
> have multiple ports on the same PCI function, and used dev_id to
> identify those ports. That use was removed by commit
> 8c367fcbe6549195d2eb11e62bea233f811aad41 ("cxgb4: Do not set
> net_device::dev_id to VI index"), since dev_id should be used only when
> devices share the same MAC address.
>
> Using dev_port for cxgb4 allows different ports on the same PCI function
> to be identified.
>
> Signed-off-by: Thadeu Lima de Souza Cascardo <[email protected]>
You've made exactly zero effort to explain why the IPV6 issue carefully
explained in commit 8c367fcbe6549195d2eb11e62bea233f811aad41 no longer
applies.
As far as I can tell, it absolutely does, and therefore this change
here should not be made.
I'm not applying this patch, sorry.
On Sun, Jun 22, 2014 at 05:23:07PM -0700, David Miller wrote:
> From: Thadeu Lima de Souza Cascardo <[email protected]>
> Date: Sat, 21 Jun 2014 09:48:08 -0300
>
> > Commit 3f85944fe207d0225ef21a2c0951d4946fc9a95d ("net: Add sysfs file
> > for port number") introduce dev_port to network devices. cxgb4 adapters
> > have multiple ports on the same PCI function, and used dev_id to
> > identify those ports. That use was removed by commit
> > 8c367fcbe6549195d2eb11e62bea233f811aad41 ("cxgb4: Do not set
> > net_device::dev_id to VI index"), since dev_id should be used only when
> > devices share the same MAC address.
> >
> > Using dev_port for cxgb4 allows different ports on the same PCI function
> > to be identified.
> >
> > Signed-off-by: Thadeu Lima de Souza Cascardo <[email protected]>
>
> You've made exactly zero effort to explain why the IPV6 issue carefully
> explained in commit 8c367fcbe6549195d2eb11e62bea233f811aad41 no longer
> applies.
>
> As far as I can tell, it absolutely does, and therefore this change
> here should not be made.
>
> I'm not applying this patch, sorry.
>
I have set dev_port, not dev_id. This is not a revert of commit
8c367fcbe6549195d2eb11e62bea233f811aad41.
$ git grep dev_port net/ipv6/
$
I believe dev_port was introduced exactly to solve the problem dev_id
was incorrectly being used for.
Cascardo.
From: Thadeu Lima de Souza Cascardo <[email protected]>
Date: Mon, 23 Jun 2014 09:16:40 -0300
> On Sun, Jun 22, 2014 at 05:23:07PM -0700, David Miller wrote:
>> From: Thadeu Lima de Souza Cascardo <[email protected]>
>> Date: Sat, 21 Jun 2014 09:48:08 -0300
>>
>> > Commit 3f85944fe207d0225ef21a2c0951d4946fc9a95d ("net: Add sysfs file
>> > for port number") introduce dev_port to network devices. cxgb4 adapters
>> > have multiple ports on the same PCI function, and used dev_id to
>> > identify those ports. That use was removed by commit
>> > 8c367fcbe6549195d2eb11e62bea233f811aad41 ("cxgb4: Do not set
>> > net_device::dev_id to VI index"), since dev_id should be used only when
>> > devices share the same MAC address.
>> >
>> > Using dev_port for cxgb4 allows different ports on the same PCI function
>> > to be identified.
>> >
>> > Signed-off-by: Thadeu Lima de Souza Cascardo <[email protected]>
>>
>> You've made exactly zero effort to explain why the IPV6 issue carefully
>> explained in commit 8c367fcbe6549195d2eb11e62bea233f811aad41 no longer
>> applies.
>>
>> As far as I can tell, it absolutely does, and therefore this change
>> here should not be made.
>>
>> I'm not applying this patch, sorry.
>>
>
> I have set dev_port, not dev_id. This is not a revert of commit
> 8c367fcbe6549195d2eb11e62bea233f811aad41.
>
> $ git grep dev_port net/ipv6/
> $
>
> I believe dev_port was introduced exactly to solve the problem dev_id
> was incorrectly being used for.
Aha, that makes much more sense. Thanks for explaining.
Applied, thanks.
On Sat, Jun 21, 2014 at 5:48 AM, Thadeu Lima de Souza Cascardo
<[email protected]> wrote:
> That use was removed by commit
> 8c367fcbe6549195d2eb11e62bea233f811aad41 ("cxgb4: Do not set
> net_device::dev_id to VI index"), since dev_id should be used only when
> devices share the same MAC address.
>
> Using dev_port for cxgb4 allows different ports on the same PCI function
> to be identified.
Can you describe what issues were introduced because of this then?
Luis
On Wed, Jun 25, 2014 at 04:08:21PM -0700, Luis R. Rodriguez wrote:
> On Sat, Jun 21, 2014 at 5:48 AM, Thadeu Lima de Souza Cascardo
> <[email protected]> wrote:
> > That use was removed by commit
> > 8c367fcbe6549195d2eb11e62bea233f811aad41 ("cxgb4: Do not set
> > net_device::dev_id to VI index"), since dev_id should be used only when
> > devices share the same MAC address.
> >
> > Using dev_port for cxgb4 allows different ports on the same PCI function
> > to be identified.
>
> Can you describe what issues were introduced because of this then?
>
> Luis
>
udevd builtin net-id cannot rename the interfaces, since it relies on
the dev_id for interfaces on the same PCI function. I am going to send a
patch to use dev_port instead, and fallback to dev_id, and copy netdev.
Regards.
Cascardo.