Re: [PATCH net-next RFC 00/13] Add devlink reload level option

2020-08-12 Thread Vasundhara Volam
On Wed, Aug 5, 2020 at 1:51 PM Moshe Shemesh wrote: > > > On 8/5/2020 9:55 AM, Vasundhara Volam wrote: > > On Wed, Aug 5, 2020 at 12:02 PM Moshe Shemesh wrote: > >> > >> On 8/4/2020 1:13 PM, Vasundhara Volam wrote: > >>> On Mon, Aug 3, 2020 at 7:23

Re: [PATCH net-next RFC 00/13] Add devlink reload level option

2020-08-03 Thread Vasundhara Volam
On Tue, Jul 28, 2020 at 10:13 PM Jacob Keller wrote: > > > > On 7/27/2020 10:25 PM, Vasundhara Volam wrote: > > On Mon, Jul 27, 2020 at 4:36 PM Moshe Shemesh wrote: > >> > >> Introduce new option on devlink reload API to enable the user to select the > >

Re: [PATCH net-next RFC 00/13] Add devlink reload level option

2020-08-03 Thread Vasundhara Volam
On Mon, Aug 3, 2020 at 5:47 PM Moshe Shemesh wrote: > > > On 8/3/2020 1:24 PM, Vasundhara Volam wrote: > > On Tue, Jul 28, 2020 at 10:13 PM Jacob Keller > > wrote: > >> > >> > >> On 7/27/2020 10:25 PM, Vasundhara Volam wrote: > >&

Re: [PATCH V2 3/3] bnxt_en: Add support to collect crash dump via ethtool

2019-10-17 Thread Vasundhara Volam
On Fri, Oct 18, 2019 at 12:52 AM Jakub Kicinski wrote: > > On Thu, 17 Oct 2019 17:31:22 +0530, Sheetal Tigadoli wrote: > > From: Vasundhara Volam > > > > Driver supports 2 types of core dumps. > > > > 1. Live dump - Firmware dump when system is up and runnin

Re: [PATCH V2 3/3] bnxt_en: Add support to collect crash dump via ethtool

2019-10-20 Thread Vasundhara Volam
On Fri, Oct 18, 2019 at 10:31 PM Jakub Kicinski wrote: > > On Fri, 18 Oct 2019 12:04:35 +0530, Vasundhara Volam wrote: > > On Fri, Oct 18, 2019 at 12:52 AM Jakub Kicinski wrote: > > > On Thu, 17 Oct 2019 17:31:22 +0530, Sheetal Tigadoli wrote: > > > > diff --gi

Re: [PATCH net-next v2 03/16] devlink: Add devlink reload limit option

2020-10-07 Thread Vasundhara Volam
On Wed, Oct 7, 2020 at 11:32 AM Moshe Shemesh wrote: > > Add reload limit to demand restrictions on reload actions. > Reload limits supported: > no_reset: No reset allowed, no down time allowed, no link flap and no > configuration is lost. > > By default reload limit is unspecified and s

Re: [PATCH v2 04/15] bnxt: use new module_firmware_crashed()

2020-05-15 Thread Vasundhara Volam
ere is no crash. I will propose a patch to use this wrapper in the error recovery path, where the driver may not be able to recover. > #ifdef CONFIG_TEE_BNXT_FW > return tee_bnxt_copy_coredump(buf, 0, dump->len); > #endif > -- > 2.26.2 > Nacked-by: Vasundhara Volam

Re: [PATCH net-next RFC 00/13] Add devlink reload level option

2020-08-04 Thread Vasundhara Volam
On Mon, Aug 3, 2020 at 7:23 PM Moshe Shemesh wrote: > > > On 8/3/2020 3:47 PM, Vasundhara Volam wrote: > > On Mon, Aug 3, 2020 at 5:47 PM Moshe Shemesh wrote: > >> > >> On 8/3/2020 1:24 PM, Vasundhara Volam wrote: > >>> On Tue, Jul 28

Re: [PATCH net-next RFC 00/13] Add devlink reload level option

2020-08-04 Thread Vasundhara Volam
On Wed, Aug 5, 2020 at 12:02 PM Moshe Shemesh wrote: > > > On 8/4/2020 1:13 PM, Vasundhara Volam wrote: > > On Mon, Aug 3, 2020 at 7:23 PM Moshe Shemesh wrote: > >> > >> On 8/3/2020 3:47 PM, Vasundhara Volam wrote: > >>> On Mon, Aug 3, 2020 at 5:47

Re: [PATCH net-next RFC 00/13] Add devlink reload level option

2020-07-27 Thread Vasundhara Volam
On Mon, Jul 27, 2020 at 4:36 PM Moshe Shemesh wrote: > > Introduce new option on devlink reload API to enable the user to select the > reload level required. Complete support for all levels in mlx5. > The following reload levels are supported: > driver: Driver entities re-instantiation only. >

Re: [PATCH net-next RFC v3 01/14] devlink: Add reload action option to devlink reload command

2020-09-09 Thread Vasundhara Volam
On Wed, Sep 9, 2020 at 10:51 PM Moshe Shemesh wrote: > > > On 9/7/2020 8:58 PM, Jakub Kicinski wrote: > > On Mon, 7 Sep 2020 16:46:01 +0300 Moshe Shemesh wrote: > >>> In that sense I don't like --live because it doesn't really say much. > >>> AFAIU it means 1) no link flap; 2) < 2 sec datapath dow

Re: [PATCH net-next RFC v4 01/15] devlink: Add reload action option to devlink reload command

2020-09-14 Thread Vasundhara Volam
On Mon, Sep 14, 2020 at 11:39 AM Moshe Shemesh wrote: > > Add devlink reload action to allow the user to request a specific reload > action. The action parameter is optional, if not specified then devlink > driver re-init action is used (backward compatible). > Note that when required to do firmwa

Re: [PATCH net-next RFC v4 01/15] devlink: Add reload action option to devlink reload command

2020-09-14 Thread Vasundhara Volam
On Mon, Sep 14, 2020 at 3:02 PM Jiri Pirko wrote: > > Mon, Sep 14, 2020 at 09:08:58AM CEST, vasundhara-v.vo...@broadcom.com wrote: > >On Mon, Sep 14, 2020 at 11:39 AM Moshe Shemesh wrote: > > [...] > > > >> @@ -1126,15 +1126,24 @@ mlxsw_devlink_core_bus_device_reload_down(struct > >> devlink *de