Thread (3 messages) flat view 3 messages, 2 authors, 2018-09-08

Re: [PATCH] ethernet: hnae: add unlikely() to assert()

From: Igor Stoppa <hidden>
Date: 2018-09-08 15:00:34
Also in: lkml


On 08/09/18 00:46, David Miller wrote:
From: Igor Stoppa <redacted>
Date: Fri,  7 Sep 2018 20:26:50 +0300
quoted
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.

--
igor
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help