]> git.baikalelectronics.ru Git - kernel.git/commit
net: ipa: don't break build on large transaction size
authorAlex Elder <elder@linaro.org>
Mon, 9 Nov 2020 16:56:32 +0000 (10:56 -0600)
committerJakub Kicinski <kuba@kernel.org>
Wed, 11 Nov 2020 22:07:00 +0000 (14:07 -0800)
commit7dcc72dd697b8de3195e30dd933e653eac80fc0e
treeb0dc7c54b5ab244bfce2ad0438e343daa21c1497
parentfeffc62780738ba3d996730373409abd1c785d45
net: ipa: don't break build on large transaction size

The following call in ipa_validate_build() is erroneous:

    BUILD_BUG_ON(sizeof(struct gsi_trans) > 128);

The fact is, it is not a bug for the size of a GSI transaction to be
bigger than 128 bytes.  The correct operation of the driver is not
dependent on the size of this structure.  The only consequence of
the transaction being large is that the amount of memory required
is larger.

The problem this was trying to flag is that a *slight* increase in
the size of this structure will have a disproportionate effect on
the amount of memory used.  E.g. if the structure grew to 132 bytes
the memory requirement for the transaction arrays would be about
double.

With various debugging build flags enabled, the size grows to 160
bytes.  But there's no reason to treat that as a build-time bug.

Signed-off-by: Alex Elder <elder@linaro.org>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ipa/ipa_main.c