Thread (17 messages) flat view 17 messages, 3 authors, 2022-04-25

Re: [net-next 0/4] devlink: add dry run support for flash update

From: Jakub Kicinski <kuba@kernel.org>
Date: 2021-10-08 22:36:24

On Fri, 8 Oct 2021 21:43:32 +0000 Keller, Jacob E wrote:
quoted
quoted
Hmm, old kernel vs. new-userspace, the requested dry-run, won't be
really dry run. I guess that user might be surprised in that case...  
Would it be enough to do a policy dump in user space to check attr is
recognized and add a warning that this is required next to the attr
in the uAPI header?  
Doesn't the policy checks prevent any unknown attributes? 
Or are unknown attributes silently ignored?
Did you test it?

DEVLINK_CMD_FLASH_UPDATE has GENL_DONT_VALIDATE_STRICT set.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help