[PATCH 2/2] test/hash: fix coverity warning

Medvedkin, Vladimir vladimir.medvedkin at intel.com
Thu Nov 3 19:52:34 CET 2022


Hi Stephen,

On 03/11/2022 18:33, Stephen Hemminger wrote:
> On Thu,  3 Nov 2022 18:13:38 +0000
> Vladimir Medvedkin <vladimir.medvedkin at intel.com> wrote:
>
>> +				if (ret != 0) {
>> +					printf("rte_hash_lookup_with_hash_bulk"
>> +						" failed with %d\n", ret);
>> +					return -1;
>> +				}
> It makes it harder to search for error messages when they are split.
> Ignore any checkpatch warnings about this.

Will fix, thanks!

>
> Also, shouldn't test failures be printed on stderr rather than stdout?

It seems that most of the tests print on stdout.

-- 
Regards,
Vladimir



More information about the stable mailing list