]> git.baikalelectronics.ru Git - kernel.git/commit
Merge branch 'ipa-kill-validation'
authorDavid S. Miller <davem@davemloft.net>
Mon, 26 Jul 2021 21:38:24 +0000 (22:38 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 26 Jul 2021 21:38:24 +0000 (22:38 +0100)
commit6e1e944686339b8b3540e268766b5f5a9ec91294
tree7322b4bc3dcab81b14a86b0a56bc8503b1449d13
parenta24d5d69ec0248cb2f6dffa53bf1b02d5ea9b1dd
parentfe1d9f7489b1a834d07759ae1042ccb58c8042c8
Merge branch 'ipa-kill-validation'

Alex Elder says:

====================
net: ipa: kill IPA_VALIDATION

A few months ago I proposed cleaning up some code that validates
certain things conditionally, arguing that doing so once is enough,
thus doing so always should not be necessary.
  https://lore.kernel.org/netdev/20210320141729.1956732-1-elder@linaro.org/
Leon Romanovsky felt strongly that this was a mistake, and in the
end I agreed to change my plans.

This series finally completes what I said I would do about this,
ultimately eliminating the IPA_VALIDATION symbol and conditional
code entirely.

The first patch both extends and simplifies some validation done for
IPA immediate commands, and performs those tests unconditionally.

The second patch fixes a bug that wasn't normally exposed because of
the conditional compilation (a reason Leon was right about this).
It makes filter and routing table validation occur unconditionally.

The third eliminates the remaining conditionally-defined code and
removes the line in the Makefile used to enable validation.

And the fourth removes all comments containing ipa_assert()
statements, replacing most of them with WARN_ON() calls.
====================

Signed-off-by: David S. Miller <davem@davemloft.net>