broadcast ethernet pkg will be dropped?

Hi Sir

I found your mail address from Linux kernel git repo. I think there is a problem in CPSW driver. If it is not your duty to maintain those code, could you please tell me the ones who are in charge of it?

I found that when a AM3352 board boot, I use a laptop to send ARP request to it, the board do not reponse. Even ifconfig do not show that there is any pkg sent to the CPSW device. I'm sure that the ARP request PKG received by SoC(confirmed by my oscilloscope).

After reading the SoC doc (Tech ref manual) and reading the driver code, i found that:
15.3.2.7 Address Lookup Engine (ALE)
"Broadcast packets will be dropped unless the broadcast address is entered into the table with the super bit set."

and in driver: drivers/net/ether/ti/cpsw_ale.c

int cpsw_ale_add_mcast(struct cpsw_ale *ale, u8 *addr, int port_mask,
int flags, u16 vid, int mcast_state)

   {
          /* ....... */

          cpsw_ale_set_super(ale_entry, (flags & ALE_BLOCKED) ? 1 : 0);

          /* ....... */

}

I have searched all of the calling of that function, and found no ALE_BLOCKED flag was set. Especially, in cpsw_add_dual_emac_def_ale_entries(), only ALE_VLAN was provided for the broadcast address.

After change the flag(brutally), my board can response to the ARP broadcast requests correctly.

Here are some questions I do not found the answer:
1. the macro ALE_BLOCKED seems be used uncorrectly.
If one want the pkg should be "blocked", the "super" bit should be cleared, not be set. So, in cpsw_ale_add_mcast(), we should do like this:
cpsw_ale_set_super(ale_entry, (flags & ALE_BLOCKED) ? 0 : 1);

Do you think the current logic is reversed?

2. broadcast ethernet pkg from "FF:FF:FF:FF:FF:FF" should be alowed.
If by default, it desiged to be filtered out, why not add an interface to support restore it? I do not found any code that can change filtering the broadcast pkg in runtime.

--
Brock Zheng <[email protected]>
郑 祎

北京中科腾越科技发展有限公司
北京市海淀区东北旺西路8号中关村软件园21号楼启明星辰大厦二层六区（邮编：100094）

Can you check the hardware statistics counters in CPSW also?

When Super Bit is set, the packet will never be dropped by any ALE rules or rate limit. It is made sure that the packet will reach cpu port irrespective of any rules in the switch. So this is not the correct fix, need to root cause a bit more.

broadcast is not filtered out, it is always allowed to CPU port in CPSW driver. Can you dump ALE table entry and check whether something else is not correct in your setup.

Regards
Mugunthan V N