lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 9 Apr 2014 18:39:38 +0530 From: Mugunthan V N <mugunthanvnm@...com> To: <yegorslists@...glemail.com>, <netdev@...r.kernel.org> Subject: Re: [PATCH v2] net: cpsw: fix dual_emac VLAN issue On Wednesday 09 April 2014 06:23 PM, yegorslists@...glemail.com wrote: > From: Yegor Yefremov <yegorslists@...glemail.com> > > dual_emac mode was first introduced in TI's PSP kernel 3.2. > See repository git://arago-project.org/git/projects/linux-am33x.git > commit 6afa003a0b06a0376c941436b38204ba2edc4a98. > > Since commit d9ba8f9e6298af71ec1c1fd3d88c3ef68abd0ec3 dual_emac > feature was upstreamed, but with some changes compared to the original > source, that led to following behavior: > > Some stations in LAN don't answer CPSW's ARP request. PSP kernel 3.2 > could talk to all stations in the same LAN. > > Use the same cpsw_ale_add_vlan procedure as in PSP kernel 3.2 to fix > this regression. > > Signed-off-by: Yegor Yefremov <yegorslists@...glemail.com> > CC: Mugunthan V N <mugunthanvnm@...com> > --- > Changes: > v2: make more detailed description > > drivers/net/ethernet/ti/cpsw.c | 4 +++- > 1 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c > index 5d5fec6..aa06370 100644 > --- a/drivers/net/ethernet/ti/cpsw.c > +++ b/drivers/net/ethernet/ti/cpsw.c > @@ -1040,11 +1040,13 @@ static inline void cpsw_add_dual_emac_def_ale_entries( > else > slave_write(slave, slave->port_vlan, CPSW2_PORT_VLAN); > cpsw_ale_add_vlan(priv->ale, slave->port_vlan, port_mask, > - port_mask, port_mask, 0); > + 0, port_mask, port_mask); Dual EMAC VLAN must be untagged on egress and why you need unregistered multicast packet to be passed to CPU? > cpsw_ale_add_mcast(priv->ale, priv->ndev->broadcast, > port_mask, ALE_VLAN, slave->port_vlan, 0); > cpsw_ale_add_ucast(priv->ale, priv->mac_addr, > priv->host_port, ALE_VLAN, slave->port_vlan); > + cpsw_ale_add_vlan(priv->ale, 0, ALE_ALL_PORTS << priv->host_port, > + ALE_ALL_PORTS << priv->host_port, priv->host_port, 0); Vlan ID 0 must not be used, cpsw default vlan id should be used, currently 0 is used as default VLAN for internal packet switching, in case some one using different default vlan id, then this fix will break. I have fixed the issue already and submitted a patch. http://patchwork.ozlabs.org/patch/337769/ Regards Mugunthan V N -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists