[lkml]   [2018]   [Sep]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] ethernet: hnae: add unlikely() to assert()

On 08/09/18 00:46, David Miller wrote:
> From: Igor Stoppa <>
> Date: Fri, 7 Sep 2018 20:26:50 +0300
>> The assert() condition is likely to be true.
> Worse than that is that drivers should not be definiting their own
> private "assert()" macro.
> I'd rather have that fixed instead. We have tons of standard kernel
> facilities that do what they are trying to do here, without overloading
> the standard C namespace in this way.

I've converted into WARN() what passed the compile test.
In one case it didn't even compile :-(

I could only compile-test.


 \ /
  Last update: 2018-09-08 17:01    [W:0.040 / U:6.592 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site