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 for Android: free password hash cracker in your pocket
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CH2PR12MB3895BF1DF9793B183AE7E68BD7329@CH2PR12MB3895.namprd12.prod.outlook.com>
Date:   Fri, 28 Oct 2022 15:08:47 +0000
From:   Asmaa Mnebhi <asmaa@...dia.com>
To:     Andrew Lunn <andrew@...n.ch>,
        David Thompson <davthompson@...dia.com>
CC:     "davem@...emloft.net" <davem@...emloft.net>,
        "edumazet@...gle.com" <edumazet@...gle.com>,
        "kuba@...nel.org" <kuba@...nel.org>,
        "pabeni@...hat.com" <pabeni@...hat.com>,
        "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        "cai.huoqing@...ux.dev" <cai.huoqing@...ux.dev>,
        "brgl@...ev.pl" <brgl@...ev.pl>, Liming Sun <limings@...dia.com>
Subject: RE: [PATCH net-next v1 3/4] mlxbf_gige: add BlueField-3 Serdes
 configuration

> +mlxbf_gige_clm_init[] = {
> +	{.addr = 0x001, .wdata = 0x0105},
> +	{.addr = 0x008, .wdata = 0x0001},
> +	{.addr = 0x00B, .wdata = 0x8420},
> +	{.addr = 0x00E, .wdata = 0x0110},
> +	{.addr = 0x010, .wdata = 0x3010},
> +	{.addr = 0x027, .wdata = 0x0104},
> +	{.addr = 0x02F, .wdata = 0x09EA},
> +	{.addr = 0x055, .wdata = 0x0008},
> +	{.addr = 0x058, .wdata = 0x0088},
> +	{.addr = 0x072, .wdata = 0x3222},
> +	{.addr = 0x073, .wdata = 0x7654},
> +	{.addr = 0x074, .wdata = 0xBA98},
> +	{.addr = 0x075, .wdata = 0xDDDC}
> +};
> +
> +#define MLXBF_GIGE_UPHY_CLM_INIT_NUM_ENTRIES \
> +	(sizeof(mlxbf_gige_clm_init) / sizeof(struct 
> +mlxbf_gige_uphy_cfg_reg))

ARRAY_SIZE() ?

You have a lot of completely undocumented black magic here. And these tables are quite large. I'm wondering if it really should be considered firmware, and loaded on demand from /lib/firmware ?

Hi Andrew,
We can add comments to document the "black magic" tables. The NVIDIA serdes has a lot of speed features and lane configurations. The lowest and simplest speed is 1G/1lane (the only one we support in this driver). there is an internal microcontroller which needs to be configured to implement all this. These tables were given to us as the hardware recipes to program serdes to operate in 1G/1 lane. We would prefer to keep all this code in one place i.e. as part of this driver.

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ