Re: [PATCH v2 0/4] sysctl: Disallow partial updates of miss-formatted sysctl vectors

On Sat, Aug 15, 2026 at 05:09:30PM +0100, Bradley Morgan wrote:
> 
> Copy it, only write back once every value converted. That's the
> right way to do this. Went through the whole series against the
> tree, notes in the patch replies.
Your feedback was useful, but it was riddled with a *lot* of hand
waving. Please be succinct and to the point. The previous paragraph is
not needed.

> 
> Some nits on the series:
> - "miss-formatted" in the subject. mis-formatted, or just malformed.
Note that this comment is missing the why. It is good to know why you
want the change

> - akpm's scalars nit still applies.
> - Sashiko saw something on this one (akpm linked it).
In my opinion this is just noise. If you see that I have missed a
comment, call me out on it. If you have additional information, answer
to reviewers mail. 

> 
> Otherwise fine by me, the split is clean.
> 
> 
> Reviewed-by: Bradley Morgan <[email protected]>
> also
> 
> Tested-by: Bradley Morgan <[email protected]> for the test
How did you test? And what where your results?

Best

-----BEGIN PGP SIGNATURE-----

iQGzBAABCgAdFiEErkcJVyXmMSXOyyeQupfNUreWQU8FAmqEVDkACgkQupfNUreW
QU+sdQwAiiR0nJg+0a4UBdZxBcSQ1BO7cn+hJgcmdVyUFr+plSJKAcYIJW4gWdJX
2V2MSMvW2RSvPkucsQZJogIPsw7Uf/vSCwtyiy2sOCBPIGxd4KaMJRKunzrZGbSp
BNqLfvKzm8VUWBCA9x0b28LIVuZLp28mDR8tv/+W+4Ah2GcaeDDRccCtZL3TjYV0
ScfIr710yqRMkCUPil0qkEdU5LApmXvIEHz0N+HxksOGddsW4SPBmtmQ91ZNa7+v
X9ldCz7evpB9n22vjGkMI3QXI8IeRB7lbWWDtvQZXeWypFPvyhgBXbmAi4HZqAjj
dNcPj4rPVAR7saqHOm5uFbR5lMYolL3xApT9pZx31h7bDNrkKxn9CUosZ8yJ1L6b
MOxxPxY2DAFU1T377mMnzF4cVrPAaCgAEP4lvzax4ictwZpMJ1n30HMghgmeb2R6
IK2TdWDgavFQLBtUKOCPY0SawLS4ekWMsA0CeqDZoEoWPHtNgaz/jPLw5V8YHu+W
zhJT4INO
=x0ir
-----END PGP SIGNATURE-----