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
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