[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <b47d1950-add0-6449-4160-d5e2f7a8d7f7@iogearbox.net>
Date: Sat, 28 Jan 2023 00:02:33 +0100
From: Daniel Borkmann <daniel@...earbox.net>
To: Jamal Hadi Salim <hadi@...atatu.com>, Jiri Pirko <jiri@...nulli.us>
Cc: Jakub Kicinski <kuba@...nel.org>,
Jamal Hadi Salim <jhs@...atatu.com>, netdev@...r.kernel.org,
kernel@...atatu.com, deb.chatterjee@...el.com,
anjali.singhai@...el.com, namrata.limaye@...el.com,
khalidm@...dia.com, tom@...anda.io, pratyush@...anda.io,
xiyou.wangcong@...il.com, davem@...emloft.net, edumazet@...gle.com,
pabeni@...hat.com, vladbu@...dia.com, simon.horman@...igine.com,
stefanc@...vell.com, seong.kim@....com, mattyk@...dia.com,
dan.daly@...el.com, john.andy.fingerhut@...el.com
Subject: Re: [PATCH net-next RFC 00/20] Introducing P4TC
On 1/27/23 9:04 PM, Jamal Hadi Salim wrote:
> On Fri, Jan 27, 2023 at 1:26 PM Jiri Pirko <jiri@...nulli.us> wrote:
>> Fri, Jan 27, 2023 at 12:30:22AM CET, kuba@...nel.org wrote:
>>> On Tue, 24 Jan 2023 12:03:46 -0500 Jamal Hadi Salim wrote:
>>>> There have been many discussions and meetings since about 2015 in regards to
>>>> P4 over TC and now that the market has chosen P4 as the datapath specification
>>>> lingua franca
>>>
>>> Which market?
>>>
>>> Barely anyone understands the existing TC offloads. We'd need strong,
>>> and practical reasons to merge this. Speaking with my "have suffered
>>> thru the TC offloads working for a vendor" hat on, not the "junior
>>> maintainer" hat.
>>
>> You talk about offload, yet I don't see any offload code in this RFC.
>> It's pure sw implementation.
>>
>> But speaking about offload, how exactly do you plan to offload this
>> Jamal? AFAIK there is some HW-specific compiler magic needed to generate
>> HW acceptable blob. How exactly do you plan to deliver it to the driver?
>> If HW offload offload is the motivation for this RFC work and we cannot
>> pass the TC in kernel objects to drivers, I fail to see why exactly do
>> you need the SW implementation...
>
> Our rule in TC is: _if you want to offload using TC you must have a
> s/w equivalent_.
> We enforced this rule multiple times (as you know).
> P4TC has a sw equivalent to whatever the hardware would do. We are pushing that
> first. Regardless, it has value on its own merit:
> I can run P4 equivalent in s/w in a scriptable (as in no compilation
> in the same spirit as u32 and pedit),
`62001 insertions(+), 45 deletions(-)` and more to come for a software
datapath which in the end no-one will use (assuming you'll have the hw
offloads) is a pretty heavy lift.. imo the layer of abstraction is wrong
here as Stan hinted. What if tomorrow P4 programming language is not the
'lingua franca' anymore and something else comes along? Then all of it is
still baked into uapi instead of having a generic/versatile intermediate
later.
Powered by blists - more mailing lists