| 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
| ||
|
Message-ID: <a29c40ff-7209-4b60-a17f-2aab09318dc1@oracle.com>
Date: Mon, 10 Nov 2025 22:21:40 +0530
From: ALOK TIWARI <alok.a.tiwari@...cle.com>
To: Parvathi Pudi <parvathi@...thit.com>, andrew+netdev@...n.ch,
davem@...emloft.net, edumazet@...gle.com, kuba@...nel.org,
pabeni@...hat.com, danishanwar@...com, rogerq@...nel.org,
pmohan@...thit.com, basharath@...thit.com, afd@...com
Cc: linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, pratheesh@...com,
j-rameshbabu@...com, vigneshr@...com, praneeth@...com, srk@...com,
rogerq@...com, krishna@...thit.com, mohan@...thit.com
Subject: Re: [External] : [PATCH net-next v4 1/3] net: ti: icssm-prueth: Adds
helper functions to configure and maintain FDB
> +static int icssm_prueth_sw_insert_fdb_entry(struct prueth_emac *emac,
> + const u8 *mac, u8 is_static)
> +{
> + struct fdb_index_tbl_entry __iomem *bucket_info;
> + struct fdb_mac_tbl_entry __iomem *mac_info;
> + struct prueth *prueth = emac->prueth;
> + struct prueth_emac *other_emac;
> + enum prueth_port other_port_id;
> + u8 hash_val, mac_tbl_idx;
> + struct fdb_tbl *fdb;
> + u8 flags;
> + u16 val;
> + s16 ret;
> + int err;
> +
> + fdb = prueth->fdb_tbl;
> + other_port_id = (emac->port_id == PRUETH_PORT_MII0) ?
> + PRUETH_PORT_MII1 : PRUETH_PORT_MII0;
> +
> + other_emac = prueth->emac[other_port_id - 1];
> +
> + if (fdb->total_entries == FDB_MAC_TBL_MAX_ENTRIES)
> + return -ENOMEM;
> +
> + if (ether_addr_equal(mac, emac->mac_addr) ||
> + ether_addr_equal(mac, other_emac->mac_addr)) {
> + /* Don't insert fdb of own mac addr */
> + return -EINVAL;
> + }
> +
> + /* Get the bucket that the mac belongs to */
> + hash_val = icssm_prueth_sw_fdb_hash(mac);
> + bucket_info = FDB_IDX_TBL_ENTRY(hash_val);
> +
> + if (!readw(&bucket_info->bucket_entries)) {
> + mac_tbl_idx = icssm_prueth_sw_fdb_find_open_slot(fdb);
> + writew(mac_tbl_idx, &bucket_info->bucket_idx);
> + }
> +
> + ret = icssm_prueth_sw_find_fdb_insert(fdb, prueth, bucket_info, mac,
> + emac->port_id - 1);
> + if (ret < 0)
> + /* mac is already in fdb table */
> + return 0;
> +
> + mac_tbl_idx = ret;
If ret == -1 mac_tbl_idx wraps to 255 silently.
> +
> + err = icssm_prueth_sw_fdb_spin_lock(fdb);
> + if (err) {
> + dev_err(prueth->dev, "PRU lock timeout %d\n", ret);
wrong var ret print.
return ret or err here ?
> + return ret;
> + }
> +
> + mac_info = icssm_prueth_sw_find_free_mac(prueth, bucket_info,
> + mac_tbl_idx, NULL,
> + mac);
> + if (!mac_info) {
> + /* Should not happen */
> + dev_warn(prueth->dev, "OUT of FDB MEM\n");
> + return -ENOMEM;
> + }
> +
> + memcpy_toio(mac_info->mac, mac, ETH_ALEN);
> + writew(0, &mac_info->age);
> + writeb(emac->port_id - 1, &mac_info->port);
> +
> + flags = readb(&mac_info->flags);
> + if (is_static)
> + flags |= FLAG_IS_STATIC;
> + else
> + flags &= ~FLAG_IS_STATIC;
> +
> + /* bit 1 - active */
> + flags |= FLAG_ACTIVE;
> + writeb(flags, &mac_info->flags);
> +
> + val = readw(&bucket_info->bucket_entries);
> + val++;
> + writew(val, &bucket_info->bucket_entries);
> +
> + fdb->total_entries++;
> +
> + icssm_prueth_sw_fdb_spin_unlock(fdb);
> +
> + dev_dbg(prueth->dev, "added fdb: %pM port=%d total_entries=%u\n",
> + mac, emac->port_id, fdb->total_entries);
> +
> + return 0;
> +}
> +
> +static int icssm_prueth_sw_delete_fdb_entry(struct prueth_emac *emac,
> + const u8 *mac, u8 is_static)
> +{
> + struct fdb_index_tbl_entry __iomem *bucket_info;
> + struct fdb_mac_tbl_entry __iomem *mac_info;
> + struct fdb_mac_tbl_array __iomem *mt;
> + u8 hash_val, mac_tbl_idx;
> + u16 idx, entries, val;
> + struct prueth *prueth;
> + s16 ret, left, right;
> + struct fdb_tbl *fdb;
> + u8 flags;
> + int err;
> +
> + prueth = emac->prueth;
> + fdb = prueth->fdb_tbl;
> + mt = fdb->mac_tbl_a;
> +
> + if (fdb->total_entries == 0)
> + return 0;
> +
> + /* Get the bucket that the mac belongs to */
> + hash_val = icssm_prueth_sw_fdb_hash(mac);
> + bucket_info = FDB_IDX_TBL_ENTRY(hash_val);
> +
> + ret = icssm_prueth_sw_fdb_search(mt, bucket_info, mac);
> + if (ret < 0)
> + return ret;
> +
> + mac_tbl_idx = ret;
> + mac_info = FDB_MAC_TBL_ENTRY(mac_tbl_idx);
> +
> + err = icssm_prueth_sw_fdb_spin_lock(fdb);
> + if (err) {
> + dev_err(prueth->dev, "PRU lock timeout %d\n", ret);
> + return ret;
> + }
> +
> + /* Shift all elements in bucket to the left. No need to
> + * update index table since only shifting within bucket.
> + */
> + left = mac_tbl_idx;
> + idx = readw(&bucket_info->bucket_idx);
> + entries = readw(&bucket_info->bucket_entries);
> + right = idx + entries - 1;
> + icssm_prueth_sw_fdb_move_range_left(prueth, left, right);
> +
> + /* Remove end of bucket from table */
> + mac_info = FDB_MAC_TBL_ENTRY(right);
> + flags = readb(&mac_info->flags);
> + /* active = 0 */
> + flags &= ~FLAG_ACTIVE;
> + writeb(flags, &mac_info->flags);
> + val = readw(&bucket_info->bucket_entries);
> + val--;
> + writew(val, &bucket_info->bucket_entries);
> + fdb->total_entries--;
> +
> + icssm_prueth_sw_fdb_spin_unlock(fdb);
> +
> + dev_dbg(prueth->dev, "del fdb: %pM total_entries=%u\n",
> + mac, fdb->total_entries);
> +
> + return 0;
> +}
> +
> +int icssm_prueth_sw_do_purge_fdb(struct prueth_emac *emac)
> +{
> + struct fdb_index_tbl_entry __iomem *bucket_info;
> + struct prueth *prueth = emac->prueth;
> + struct fdb_tbl *fdb;
> + u8 hash_val, flags;
> + u8 mac[ETH_ALEN];
> + u16 val;
> +
> + int ret;
> + s16 i;
> +
> + fdb = prueth->fdb_tbl;
> + if (fdb->total_entries == 0)
> + return 0;
> +
> + ret = icssm_prueth_sw_fdb_spin_lock(fdb);
> + if (ret) {
> + dev_err(prueth->dev, "PRU lock timeout %d\n", ret);
> + return ret;
> + }
> +
> + for (i = 0; i < FDB_MAC_TBL_MAX_ENTRIES; i++) {
> + flags = readb(&fdb->mac_tbl_a->mac_tbl_entry[i].flags);
> + if ((flags & FLAG_ACTIVE) && !(flags & FLAG_IS_STATIC)) {
> + /* Get the bucket that the mac belongs to */
> + memcpy_fromio(mac, FDB_MAC_TBL_ENTRY(i)->mac,
> + ETH_ALEN);
> + hash_val = icssm_prueth_sw_fdb_hash(mac);
> + bucket_info = FDB_IDX_TBL_ENTRY(hash_val);
> + flags &= ~FLAG_ACTIVE;
> + writeb(flags,
> + &fdb->mac_tbl_a->mac_tbl_entry[i].flags);
> + val = readw(&bucket_info->bucket_entries);
> + val--;
> + writew(val, &bucket_info->bucket_entries);
> + fdb->total_entries--;
> + }
> + }
> +
> + icssm_prueth_sw_fdb_spin_unlock(fdb);
\n
> + return 0;
> +}
> +
> +int icssm_prueth_sw_init_fdb_table(struct prueth *prueth)
> +{
> + if (prueth->emac_configured)
> + return 0;
> +
> + prueth->fdb_tbl = kmalloc(sizeof(*prueth->fdb_tbl), GFP_KERNEL);
> + if (!prueth->fdb_tbl)
> + return -ENOMEM;
> +
> + icssm_prueth_sw_fdb_tbl_init(prueth);
> +
> + return 0;
> +}
Thanks,
Alok
Powered by blists - more mailing lists