[Skiboot] [PATCH 3/4] npu2: Remove side effects in assert() calls.
Stewart Smith
stewart at linux.vnet.ibm.com
Tue Nov 14 17:48:05 AEDT 2017
Cyril Bur <cyril.bur at au1.ibm.com> writes:
> This likely looks intentional and will do the correct thing.
>
> The correct behaviour does rely on the fact that skiboot has a custom
> assert() which will always generate code. 'Regular' assert() can be made
> to generate no code. If we were to do that with this code, the results
> would be, interesting!
>
> This patch will also keep Coverity happy which is nice.
>
> Fixes: CID 145242, 172044 and 172045
> Signed-off-by: Cyril Bur <cyril.bur at au1.ibm.com>
> ---
> hw/npu2.c | 10 +++++++---
> 1 file changed, 7 insertions(+), 3 deletions(-)
This patch merged as of 8ca2d61f3b4bac88d9d676bf305f1ee2956ce6ad
--
Stewart Smith
OPAL Architect, IBM.
More information about the Skiboot
mailing list