Thread (30 messages) 30 messages, 4 authors, 2022-08-05

RE: [net-next PATCH 1/2] devlink: add dry run attribute to flash update

From: "Keller, Jacob E" <jacob.e.keller@intel.com>
Date: 2022-08-05 16:32:37

Hi,
-----Original Message-----
From: Jakub Kicinski <kuba@kernel.org>
Sent: Monday, July 25, 2022 6:14 PM
To: Keller, Jacob E <jacob.e.keller@intel.com>
Cc: Jiri Pirko <jiri@resnulli.us>; netdev@vger.kernel.org
Subject: Re: [net-next PATCH 1/2] devlink: add dry run attribute to flash update

Hm, yes. Don't invest too much effort into rendering per-cmd policies
right now, tho. I've started working on putting the parsing policies
in YAML last Friday. This way we can auto-gen the policy for the kernel
and user space can auto-gen the parser/nl TLV writer. Long story short
we can kill two birds with one stone if you hold off until I have the
format ironed out. For now maybe just fork the policies into two -
with and without dry run attr. We'll improve the granularity later
when doing the YAML conversion.
Any update on this?

FWIW I started looking at iproute2 code to dump policy and check whether a specific attribute is accepted by the kernel.

Thanks,
Jake 
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help