[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <c4d40afe-b48a-43da-be88-f7ee38dd720c@huawei.com>
Date: Thu, 24 Oct 2024 11:09:22 +0800
From: Jijie Shao <shaojijie@...wei.com>
To: Andrew Lunn <andrew@...n.ch>
CC: <shaojijie@...wei.com>, <davem@...emloft.net>, <edumazet@...gle.com>,
<kuba@...nel.org>, <pabeni@...hat.com>, <andrew+netdev@...n.ch>,
<horms@...nel.org>, <shenjian15@...wei.com>, <wangpeiyang1@...wei.com>,
<liuyonglong@...wei.com>, <chenhao418@...wei.com>, <sudongming1@...wei.com>,
<xujunsheng@...wei.com>, <shiyongbang@...wei.com>, <libaihan@...wei.com>,
<jonathan.cameron@...wei.com>, <shameerali.kolothum.thodi@...wei.com>,
<salil.mehta@...wei.com>, <netdev@...r.kernel.org>,
<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH net-next 3/7] net: hibmcge: Add unicast frame filter
supported in this module
on 2024/10/23 22:05, Andrew Lunn wrote:
>> +static int hbg_add_mac_to_filter(struct hbg_priv *priv, const u8 *addr)
>> +{
>> + u32 index;
>> +
>> + /* already exists */
>> + if (!hbg_get_index_from_mac_table(priv, addr, &index))
>> + return 0;
>> +
>> + for (index = 0; index < priv->filter.table_max_len; index++)
>> + if (is_zero_ether_addr(priv->filter.mac_table[index].addr)) {
>> + hbg_set_mac_to_mac_table(priv, index, addr);
>> + return 0;
>> + }
>> +
>> + if (!priv->filter.table_overflow) {
>> + priv->filter.table_overflow = true;
>> + hbg_update_promisc_mode(priv->netdev);
>> + dev_info(&priv->pdev->dev, "mac table is overflow\n");
>> + }
>> +
>> + return -ENOSPC;
> I _think_ this is wrong. If you run out of hardware resources, you
> should change the interface to promiscuous mode and let the stack do
> the filtering. Offloading it to hardware is just an acceleration,
> nothing more.
>
> Andrew
In hbg_update_promisc_mode():
priv->filter.enabled = !(priv->filter.table_overflow || (netdev->flags & IFF_PROMISC));
hbg_hw_set_mac_filter_enable(priv, priv->filter.enabled);
if table_overflow, and netdev->flags not set IFF_PROMISC,
the priv->filter.enabled will set to false, Then, The MAC filter will be closed.
I think it's probably the same thing you said
In this:
+ if (!priv->filter.table_overflow) {
+ priv->filter.table_overflow = true;
+ hbg_update_promisc_mode(priv->netdev);
+ dev_info(&priv->pdev->dev, "mac table is overflow\n");
+ }
+
+ return -ENOSPC;
When the first overflow occurs, a log is printed, the MAC filter will be disabled, and -ENOSPC is returned.
If continue to add MAC addresses, -ENOSPC is returned only.
Thanks,
Jijie Shao
Powered by blists - more mailing lists