Re: [PATCH net-next 4/5] bnxt_en: Refactor __bnxt_vf_reps_destroy().

2021-04-12 Thread Leon Romanovsky
On Mon, Apr 12, 2021 at 10:51:25AM -0700, Michael Chan wrote: > On Mon, Apr 12, 2021 at 10:33 AM Leon Romanovsky wrote: > > > > On Mon, Apr 12, 2021 at 09:31:33AM -0700, Michael Chan wrote: > > > On Mon, Apr 12, 2021 at 12:37 AM Leon Romanovsky wrote: > > > > > > > > On Sun, Apr 11, 2021 at 08:18

Re: [PATCH net-next 4/5] bnxt_en: Refactor __bnxt_vf_reps_destroy().

2021-04-12 Thread Michael Chan
On Mon, Apr 12, 2021 at 10:33 AM Leon Romanovsky wrote: > > On Mon, Apr 12, 2021 at 09:31:33AM -0700, Michael Chan wrote: > > On Mon, Apr 12, 2021 at 12:37 AM Leon Romanovsky wrote: > > > > > > On Sun, Apr 11, 2021 at 08:18:14PM -0400, Michael Chan wrote: > > > > Add a new helper function __bnxt_

Re: [PATCH net-next 4/5] bnxt_en: Refactor __bnxt_vf_reps_destroy().

2021-04-12 Thread Leon Romanovsky
On Mon, Apr 12, 2021 at 09:31:33AM -0700, Michael Chan wrote: > On Mon, Apr 12, 2021 at 12:37 AM Leon Romanovsky wrote: > > > > On Sun, Apr 11, 2021 at 08:18:14PM -0400, Michael Chan wrote: > > > Add a new helper function __bnxt_free_one_vf_rep() to free one VF rep. > > > We also reintialize the V

Re: [PATCH net-next 4/5] bnxt_en: Refactor __bnxt_vf_reps_destroy().

2021-04-12 Thread Michael Chan
On Mon, Apr 12, 2021 at 12:37 AM Leon Romanovsky wrote: > > On Sun, Apr 11, 2021 at 08:18:14PM -0400, Michael Chan wrote: > > Add a new helper function __bnxt_free_one_vf_rep() to free one VF rep. > > We also reintialize the VF rep fields to proper initial values so that > > the function can be us

Re: [PATCH net-next 4/5] bnxt_en: Refactor __bnxt_vf_reps_destroy().

2021-04-12 Thread Leon Romanovsky
On Sun, Apr 11, 2021 at 08:18:14PM -0400, Michael Chan wrote: > Add a new helper function __bnxt_free_one_vf_rep() to free one VF rep. > We also reintialize the VF rep fields to proper initial values so that > the function can be used without freeing the VF rep data structure. This > will be used

[PATCH net-next 4/5] bnxt_en: Refactor __bnxt_vf_reps_destroy().

2021-04-11 Thread Michael Chan
Add a new helper function __bnxt_free_one_vf_rep() to free one VF rep. We also reintialize the VF rep fields to proper initial values so that the function can be used without freeing the VF rep data structure. This will be used in subsequent patches to free and recreate VF reps after error recover