Re: [Intel-wired-lan] [iwl-next v2 05/15] ice: allocate devlink for subfunction

2024-05-13 Thread Kalesh Anakkur Purayil
On Mon, May 13, 2024 at 2:03 PM Michal Swiatkowski wrote: > > From: Piotr Raczynski > > Make devlink allocation function generic to use it for PF and for SF. > > Add function for SF devlink port creation. It will be used in next > patch. > > Create header file for subfunction device. Define subfu

Re: [Intel-wired-lan] [iwl-next v2 05/15] ice: allocate devlink for subfunction

2024-05-13 Thread Kalesh Anakkur Purayil
On Mon, May 13, 2024 at 3:54 PM Michal Swiatkowski wrote: > > On Mon, May 13, 2024 at 02:55:48PM +0530, Kalesh Anakkur Purayil wrote: > > On Mon, May 13, 2024 at 2:03 PM Michal Swiatkowski > > wrote: > > > > > > From: Piotr Raczynski > > > > >

Re: [Intel-wired-lan] [PATCH iwl-net] ice: Add netif_device_attach/detach into PF reset flow

2024-08-13 Thread Kalesh Anakkur Purayil
Hi David, One question in line. On Mon, Aug 12, 2024 at 3:52 PM Dawid Osuchowski wrote: > > Ethtool callbacks can be executed while reset is in progress and try to > access deleted resources, e.g. getting coalesce settings can result in a > NULL pointer dereference seen below. > > Once the drive

Re: [Intel-wired-lan] [PATCH iwl-next v2 14/15] ice: cleanup inconsistent code

2023-12-13 Thread Kalesh Anakkur Purayil
On Wed, Dec 6, 2023 at 6:32 AM Jesse Brandeburg wrote: > It was found while doing further testing of the previous commit > fbf32a9bab91 ("ice: field get conversion") that one of the FIELD_GET > conversions should really be a FIELD_PREP. The previous code was styled > as a match to the FIELD_GET c

Re: [Intel-wired-lan] [PATCH iwl-next v2 14/15] ice: cleanup inconsistent code

2023-12-14 Thread Kalesh Anakkur Purayil
On Wed, Dec 13, 2023 at 11:57 PM Jesse Brandeburg < jesse.brandeb...@intel.com> wrote: > Please don't use HTML email, your reply was likely dropped by most lists > that filter HTML. > Sure, I will check. > > On 12/12/2023 8:06 PM, Kalesh Anakkur Purayil wrote: &

Re: [Intel-wired-lan] [PATCH iwl-next 1/7] i40e: Remove flags field from i40e_veb

2024-03-26 Thread Kalesh Anakkur Purayil
On Mon, Mar 18, 2024 at 8:01 PM Ivan Vecera wrote: > > The field is initialized always to zero and it is never read. > Remove it. > > Signed-off-by: Ivan Vecera > --- LGTM Reviewed-by: Kalesh AP > drivers/net/ethernet/intel/i40e/i40e.h | 3 +-- > drivers/net/ethernet/intel/i40e/i40e_d

Re: [Intel-wired-lan] [PATCH iwl-next v2 3/7] i40e: Refactor argument of i40e_detect_recover_hung()

2024-03-28 Thread Kalesh Anakkur Purayil
On Wed, Mar 27, 2024 at 1:28 PM Ivan Vecera wrote: > > Commit 07d44190a389 ("i40e/i40evf: Detect and recover hung queue > scenario") changes i40e_detect_recover_hung() argument type from > i40e_pf* to i40e_vsi* to be shareable by both i40e and i40evf. > Because the i40evf does not exist anymore an

Re: [Intel-wired-lan] [PATCH net-next v4 3/3] ice: fold ice_ptp_read_time into ice_ptp_gettimex64

2024-03-29 Thread Kalesh Anakkur Purayil
On Tue, Mar 26, 2024 at 6:07 AM Michal Schmidt wrote: > > This is a cleanup. It is unnecessary to have this function just to call > another function. > > Reviewed-by: Przemek Kitszel > Signed-off-by: Michal Schmidt Reviewed-by: Kalesh AP > --- > drivers/net/ethernet/intel/ice/ice_ptp.c | 25 +

Re: [Intel-wired-lan] [iwl-net v1] ice: block SF port creation in legacy mode

2024-10-04 Thread Kalesh Anakkur Purayil
On Fri, Oct 4, 2024 at 12:25 PM Michal Swiatkowski wrote: > > There is no support for SF in legacy mode. Reflect it in the code. > > Reviewed-by: Przemek Kitszel > Fixes: eda69d654c7e ("ice: add basic devlink subfunctions support") > Signed-off-by: Michal Swiatkowski LGTM, Reviewed-by: Kalesh A

Re: [Intel-wired-lan] [PATCH iwl-next v9 3/7] ixgbe: Add link management support for E610 device

2024-10-17 Thread Kalesh Anakkur Purayil
On Thu, Oct 3, 2024 at 7:49 PM Piotr Kwapulinski wrote: > > Add low level link management support for E610 device. Link management > operations are handled via the Admin Command Interface. Add the following > link management operations: > - get link capabilities > - set up link > - get media type

Re: [Intel-wired-lan] [PATCH iwl-next v9 2/7] ixgbe: Add support for E610 device capabilities detection

2024-10-17 Thread Kalesh Anakkur Purayil
On Thu, Oct 3, 2024 at 7:48 PM Piotr Kwapulinski wrote: > > Add low level support for E610 device capabilities detection. The > capabilities are discovered via the Admin Command Interface. Discover the > following capabilities: > - function caps: vmdq, dcb, rss, rx/tx qs, msix, nvm, orom, reset >

Re: [Intel-wired-lan] [PATCH iwl-next 2/4] ice: split ice_init_hw() out from ice_init_dev()

2024-10-02 Thread Kalesh Anakkur Purayil
On Wed, Oct 2, 2024 at 5:23 PM Przemek Kitszel wrote: > > Split ice_init_hw() call out from ice_init_dev(). Such move enables > pulling the former to be even earlier on call path, what would enable > moving ice_adapter init to be between the two (in subsequent commit). > Such move enables ice_adap

Re: [Intel-wired-lan] [PATCH iwl-next 3/4] ice: minor: rename goto labels from err to unroll

2024-10-02 Thread Kalesh Anakkur Purayil
On Wed, Oct 2, 2024 at 5:23 PM Przemek Kitszel wrote: > > Clean up goto labels after previous commit, to conform to single naming > scheme in ice_probe() and ice_init_dev(). > > Reviewed-by: Marcin Szycik > Signed-off-by: Przemek Kitszel LGTM, Reviewed-by: Kalesh AP -- Regards, Kalesh A P

Re: [Intel-wired-lan] [PATCH iwl-next 4/4] ice: ice_probe: init ice_adapter after HW init

2024-10-02 Thread Kalesh Anakkur Purayil
On Wed, Oct 2, 2024 at 5:23 PM Przemek Kitszel wrote: > > Move ice_adapter initialization to be after HW init, so it could use HW > capabilities, like number of PFs. This is needed for devlink-resource > based RSS LUT size management for PF/VF (not in this series). > > Reviewed-by: Marcin Szycik

Re: [Intel-wired-lan] [RFC net-next 5/9] i40e: Remove unused i40e_get_cur_guaranteed_fd_count

2024-12-22 Thread Kalesh Anakkur Purayil
On Sun, Dec 22, 2024 at 12:13 AM wrote: > > From: "Dr. David Alan Gilbert" > > The last use of i40e_get_cur_guaranteed_fd_count() was removed in 2015 by > commit 04294e38a451 ("i40e: FD filters flush policy changes") > > Remove it. > > Signed-off-by: Dr. David Alan Gilbert LGTM, Reviewed-by: Ka

Re: [Intel-wired-lan] [RFC net-next 2/9] i40e: Remove unused i40e_blink_phy_link_led

2024-12-22 Thread Kalesh Anakkur Purayil
On Sun, Dec 22, 2024 at 12:16 AM wrote: > > From: "Dr. David Alan Gilbert" > > i40e_blink_phy_link_led() was added in 2016 by > commit fd077cd3399b ("i40e: Add functions to blink led on 10GBaseT PHY") > > but hasn't been used. > > Remove it. > > Signed-off-by: Dr. David Alan Gilbert LGTM, Revie

Re: [Intel-wired-lan] [RFC net-next 7/9] i40e: Remove unused i40e_commit_partition_bw_setting

2024-12-22 Thread Kalesh Anakkur Purayil
On Sun, Dec 22, 2024 at 12:15 AM wrote: > > From: "Dr. David Alan Gilbert" > > i40e_commit_partition_bw_setting() was added in 2017 by > commit 4fc8c6763957 ("i40e: genericize the partition bandwidth control") > but hasn't been used. > > Remove it. > > Signed-off-by: Dr. David Alan Gilbert LGTM

Re: [Intel-wired-lan] [RFC net-next 9/9] i40e: Remove unused i40e_dcb_hw_get_num_tc

2024-12-22 Thread Kalesh Anakkur Purayil
On Sun, Dec 22, 2024 at 12:15 AM wrote: > > From: "Dr. David Alan Gilbert" > > The last useof i40e_dcb_hw_get_num_tc() was removed in 2022 by > commit fe20371578ef ("Revert "i40e: Fix reset bw limit when DCB enabled > with 1 TC"") > > Remove it. > > Signed-off-by: Dr. David Alan Gilbert LGTM, R

Re: [Intel-wired-lan] [PATCH net-next v3 4/6] bnxt: use napi's irq affinity

2025-01-04 Thread Kalesh Anakkur Purayil
On Sat, Jan 4, 2025 at 6:13 AM Ahmed Zaki wrote: > > Delete the driver CPU affinity info and use the core's napi config > instead. > > Signed-off-by: Ahmed Zaki > --- > drivers/net/ethernet/broadcom/bnxt/bnxt.c | 26 --- > drivers/net/ethernet/broadcom/bnxt/bnxt.h | 2 -- >

Re: [Intel-wired-lan] [PATCH iwl-next v5 01/15] devlink: add value check to devlink_info_version_put()

2025-03-03 Thread Kalesh Anakkur Purayil
On Fri, Feb 21, 2025 at 5:37 PM Jedrzej Jagielski wrote: > > Prevent from proceeding if there's nothing to print. > > Suggested-by: Przemek Kitszel > Reviewed-by: Jiri Pirko > Signed-off-by: Jedrzej Jagielski Reviewed-by: Kalesh AP -- Regards, Kalesh AP smime.p7s Description: S/MIME Crypt