netdev - Re: [patch net-next 1/2] Documentation: devlink: extend reload-reinit description

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: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <bb4a8c25-a45a-7981-9b3f-b67d938150a6@intel.com>
Date: Tue, 28 Nov 2023 13:38:54 +0100
From: Przemek Kitszel <przemyslaw.kitszel@...el.com>
To: Jiri Pirko <jiri@...nulli.us>, <netdev@...r.kernel.org>
CC: <kuba@...nel.org>, <pabeni@...hat.com>, <davem@...emloft.net>,
	<edumazet@...gle.com>, <jacob.e.keller@...el.com>, <corbet@....net>,
	<sachin.bahadur@...el.com>, "Nguyen, Anthony L" <anthony.l.nguyen@...el.com>,
	Michal Swiatkowski <michal.swiatkowski@...ux.intel.com>
Subject: Re: [patch net-next 1/2] Documentation: devlink: extend reload-reinit
 description

On 11/28/23 12:52, Jiri Pirko wrote:
> From: Jiri Pirko <jiri@...dia.com>
> 
> Be more explicit about devlink entities that may stay and that have to
> be removed during reload reinit action.
> 
> Signed-off-by: Jiri Pirko <jiri@...dia.com>
> ---
>   Documentation/networking/devlink/devlink-reload.rst | 13 +++++++++++--
>   1 file changed, 11 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/networking/devlink/devlink-reload.rst b/Documentation/networking/devlink/devlink-reload.rst
> index 505d22da027d..2fb0269b2054 100644
> --- a/Documentation/networking/devlink/devlink-reload.rst
> +++ b/Documentation/networking/devlink/devlink-reload.rst
> @@ -22,8 +22,17 @@ By default ``driver_reinit`` action is selected.
>      * - ``driver-reinit``
>        - Devlink driver entities re-initialization, including applying
>          new values to devlink entities which are used during driver
> -       load such as ``devlink-params`` in configuration mode
> -       ``driverinit`` or ``devlink-resources``
> +       load which are:
> +
> +       * ``devlink-params`` in configuration mode ``driverinit``
> +       * ``devlink-resources``
> +
> +       Other devlink entities may stay over the re-initialization:
> +
> +       * ``devlink-health-reporter``
> +       * ``devlink-region``
> +
> +       The rest of the devlink entities have to be removed and readded.
>      * - ``fw_activate``
>        - Firmware activate. Activates new firmware if such image is stored and
>          pending activation. If no limitation specified this action may involve

Thank you!

Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@...el.com>