[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 dev
mailing list