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: Tue, 29 Nov 2022 07:16:27 +0000 From: <Arun.Ramadoss@...rochip.com> To: <olteanv@...il.com>, <UNGLinuxDriver@...rochip.com>, <vivien.didelot@...il.com>, <andrew@...n.ch>, <f.fainelli@...il.com>, <kuba@...nel.org>, <edumazet@...gle.com>, <pabeni@...hat.com>, <o.rempel@...gutronix.de>, <Woojung.Huh@...rochip.com>, <davem@...emloft.net> CC: <linux-kernel@...r.kernel.org>, <netdev@...r.kernel.org>, <kernel@...gutronix.de> Subject: Re: [PATCH v1 07/26] net: dsa: microchip: ksz8_r_dyn_mac_table(): remove timestamp support Hi Oleksij, On Mon, 2022-11-28 at 12:59 +0100, Oleksij Rempel wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you > know the content is safe > > We do not use FDB timestamps. So, drop it. If the FDB timestamps are not used/to be dropped, then do we need to have patch 4 * net: dsa: microchip: ksz8: ksz8_fdb_dump: fix time stamp extraction * which is to fix the offset. > > Signed-off-by: Oleksij Rempel <o.rempel@...gutronix.de> > --- > drivers/net/dsa/microchip/ksz8.h | 2 +- > drivers/net/dsa/microchip/ksz8795.c | 7 ++----- > 2 files changed, 3 insertions(+), 6 deletions(-) > > > diff --git a/drivers/net/dsa/microchip/ksz8795.c > b/drivers/net/dsa/microchip/ksz8795.c > index e0530bc3bec0..d0cfe74d5b13 100644 > --- a/drivers/net/dsa/microchip/ksz8795.c > +++ b/drivers/net/dsa/microchip/ksz8795.c > @@ -395,7 +395,7 @@ static int ksz8_valid_dyn_entry(struct ksz_device > *dev, u8 *data) > } > > int ksz8_r_dyn_mac_table(struct ksz_device *dev, u16 addr, u8 > *mac_addr, > - u8 *fid, u8 *src_port, u8 *timestamp, u16 > *entries) > + u8 *fid, u8 *src_port, u16 *entries) > { > u32 data_hi, data_lo; > const u8 *shifts; > @@ -440,8 +440,6 @@ int ksz8_r_dyn_mac_table(struct ksz_device *dev, > u16 addr, u8 *mac_addr, > shifts[DYNAMIC_MAC_FID]; > *src_port = (data_hi & > masks[DYNAMIC_MAC_TABLE_SRC_PORT]) >> > shifts[DYNAMIC_MAC_SRC_PORT]; > - *timestamp = (data_hi & > masks[DYNAMIC_MAC_TABLE_TIMESTAMP]) >> > - shifts[DYNAMIC_MAC_TIMESTAMP]; The macros DYNAMIC_MAC_TABLE_TIMESTAMP and DYNAMIC_MAC_TIMESTAMP needs to be deleted in ksz8795_shifts/masks and ksz8863_shifts/masks structure in ksz_common.c/h files since it is only in this function. > > mac_addr[5] = (u8)data_lo; > mac_addr[4] = (u8)(data_lo >> 8); > @@ -956,14 +954,13 @@ int ksz8_fdb_dump(struct ksz_device *dev, int > port, > > -- > 2.30.2 >
Powered by blists - more mailing lists