Whamcloud - gitweb
LU-18155 lnet: use LASSERT/F instead of if () LBUG() 16/56116/3
authorTimothy Day <timday@amazon.com>
Thu, 22 Aug 2024 01:28:30 +0000 (21:28 -0400)
committerOleg Drokin <green@whamcloud.com>
Mon, 2 Dec 2024 05:47:40 +0000 (05:47 +0000)
commit9f72288799364d85be49710c09784174928dc2aa
treea2490c9d1c6c3b3f2d23f4b85d79df8e647e28af
parentfa7cb6496dcfec5144e7893720c20fb3261e2f2d
LU-18155 lnet: use LASSERT/F instead of if () LBUG()

We should use a proper LASSERT statement rather than
more verbose if/LBUG blocks.

The patch has been generated with the coccinelle script below.
I manually inverted the conditional logic.

@@
expression L;
expression list F;
@@

- if (L) {
(
- CDEBUG(F);
|
- CWARN(F);
|
- CERROR(F);
|
- CEMERG(F);
|
- CNETERR(F);
|
- LCONSOLE(F);
|
- LCONSOLE_INFO(F);
|
- LCONSOLE_WARN(F);
|
- LCONSOLE_ERROR(F);
|
- LCONSOLE_EMERG(F);
)
- LBUG();
- }
+ LASSERTF(L, F);

@@
expression L;
@@

-if (L) LBUG();
+LASSERT(L);

Test-Parameters: trivial
Signed-off-by: Timothy Day <timday@amazon.com>
Change-Id: I73eff22d6fd4199b02d3ae2cc7740aa754d6945d
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/56116
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Arshad Hussain <arshad.hussain@aeoncomputing.com>
Reviewed-by: James Simmons <jsimmons@infradead.org>
Reviewed-by: Chris Horn <chris.horn@hpe.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lnet/klnds/gnilnd/gnilnd_conn.c
lnet/lnet/api-ni.c