Re: [PATCH V7 4/5] bnxt_en: Add TPH support in BNXT driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Wei Huang <wei.huang2@xxxxxxx>
Subject: Re: [PATCH V7 4/5] bnxt_en: Add TPH support in BNXT driver
From: Jakub Kicinski <kuba@xxxxxxxxxx>
Date: Tue, 15 Oct 2024 16:45:48 -0700
Cc: "Panicker, Manoj" <Manoj.Panicker2@xxxxxxx>, "linux-pci@xxxxxxxxxxxxxxx" <linux-pci@xxxxxxxxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, "linux-doc@xxxxxxxxxxxxxxx" <linux-doc@xxxxxxxxxxxxxxx>, "netdev@xxxxxxxxxxxxxxx" <netdev@xxxxxxxxxxxxxxx>, "Jonathan.Cameron@xxxxxxxxxx" <Jonathan.Cameron@xxxxxxxxxx>, "helgaas@xxxxxxxxxx" <helgaas@xxxxxxxxxx>, "corbet@xxxxxxx" <corbet@xxxxxxx>, "davem@xxxxxxxxxxxxx" <davem@xxxxxxxxxxxxx>, "edumazet@xxxxxxxxxx" <edumazet@xxxxxxxxxx>, "pabeni@xxxxxxxxxx" <pabeni@xxxxxxxxxx>, "alex.williamson@xxxxxxxxxx" <alex.williamson@xxxxxxxxxx>, "gospo@xxxxxxxxxxxx" <gospo@xxxxxxxxxxxx>, "michael.chan@xxxxxxxxxxxx" <michael.chan@xxxxxxxxxxxx>, "ajit.khaparde@xxxxxxxxxxxx" <ajit.khaparde@xxxxxxxxxxxx>, "somnath.kotur@xxxxxxxxxxxx" <somnath.kotur@xxxxxxxxxxxx>, "andrew.gospodarek@xxxxxxxxxxxx" <andrew.gospodarek@xxxxxxxxxxxx>, "VanTassell, Eric" <Eric.VanTassell@xxxxxxx>, "vadim.fedorenko@xxxxxxxxx" <vadim.fedorenko@xxxxxxxxx>, "horms@xxxxxxxxxx" <horms@xxxxxxxxxx>, "bagasdotme@xxxxxxxxx" <bagasdotme@xxxxxxxxx>, "bhelgaas@xxxxxxxxxx" <bhelgaas@xxxxxxxxxx>, "lukas@xxxxxxxxx" <lukas@xxxxxxxxx>, "paul.e.luse@xxxxxxxxx" <paul.e.luse@xxxxxxxxx>, "jing2.liu@xxxxxxxxx" <jing2.liu@xxxxxxxxx>
In-reply-to: <341139a9-a2d0-465e-bdd3-bdd009b78589@amd.com>
References: <20241002165954.128085-1-wei.huang2@amd.com> <20241002165954.128085-5-wei.huang2@amd.com> <20241008063959.0b073aab@kernel.org> <MN0PR12MB6174E0F2572E7BFC65EA464BAF792@MN0PR12MB6174.namprd12.prod.outlook.com> <341139a9-a2d0-465e-bdd3-bdd009b78589@amd.com>

On Tue, 15 Oct 2024 14:50:39 -0500 Wei Huang wrote:
> Any suggestions on how to proceed? I can send out a V8 patchset if Jakub
> is OK with Manoj's solution? Or only a new patch #4 is needed since the
> rest are intact.

1) y'all need to stop top posting
2) Manoj's reply is AMD internal and I'm not an AMD employee
3) precedent in drivers means relatively little, existing code 
   can be buggy

References:
- [PATCH V7 0/5] TPH and cache direct injection support
  - From: Wei Huang
- [PATCH V7 4/5] bnxt_en: Add TPH support in BNXT driver
  - From: Wei Huang
- Re: [PATCH V7 4/5] bnxt_en: Add TPH support in BNXT driver
  - From: Jakub Kicinski
- RE: [PATCH V7 4/5] bnxt_en: Add TPH support in BNXT driver
  - From: Panicker, Manoj
- Re: [PATCH V7 4/5] bnxt_en: Add TPH support in BNXT driver
  - From: Wei Huang

Prev by Date: Re: [PATCH RESEND v3 2/3] kasan: migrate copy_user_test to kunit
Next by Date: Re: [PATCH v3 1/5] maple_tree: add mas_for_each_rev() helper
Previous by thread: Re: [PATCH V7 4/5] bnxt_en: Add TPH support in BNXT driver
Next by thread: [PATCH V7 5/5] bnxt_en: Pass NQ ID to the FW when allocating RX/RX AGG rings
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Newbies] [Security] [Netfilter] [Bugtraq] [Linux FS] [Yosemite Forum] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Samba] [Video 4 Linux] [Device Mapper] [Linux Resources]