netdev - [PATCH v2] net: cpsw: fix dual

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-Id: <1397048023-10522-1-git-send-email-yegorslists@googlemail.com>
Date:	Wed,  9 Apr 2014 14:53:43 +0200
From:	yegorslists@...glemail.com
To:	netdev@...r.kernel.org
Cc:	Yegor Yefremov <yegorslists@...glemail.com>,
	Mugunthan V N <mugunthanvnm@...com>
Subject: [PATCH v2] net: cpsw: fix dual_emac VLAN issue

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);
 	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);
 }
 
 static void soft_reset_slave(struct cpsw_slave *slave)
-- 
1.7.7

--
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