WARNING: line length of 81 exceeds 80 columns #63: FILE: net/core/page_pool.c:47: +#define alloc_stat_inc(pool, __stat) \ CHECK: Macro argument 'pool' may be better as '(pool)' to avoid precedence issues #63: FILE: net/core/page_pool.c:47: +#define alloc_stat_inc(pool, __stat) \ + do { \ + struct page_pool_alloc_stats *s = &pool->alloc_stats; \ + u64_stats_update_begin(&s->syncp); \ + u64_stats_inc(&s->__stat); \ + u64_stats_update_end(&s->syncp); \ + } while (0) CHECK: Macro argument '__stat' may be better as '(__stat)' to avoid precedence issues #63: FILE: net/core/page_pool.c:47: +#define alloc_stat_inc(pool, __stat) \ + do { \ + struct page_pool_alloc_stats *s = &pool->alloc_stats; \ + u64_stats_update_begin(&s->syncp); \ + u64_stats_inc(&s->__stat); \ + u64_stats_update_end(&s->syncp); \ + } while (0) WARNING: line length of 81 exceeds 80 columns #64: FILE: net/core/page_pool.c:48: + do { \ WARNING: line length of 81 exceeds 80 columns #65: FILE: net/core/page_pool.c:49: + struct page_pool_alloc_stats *s = &pool->alloc_stats; \ WARNING: line length of 81 exceeds 80 columns #66: FILE: net/core/page_pool.c:50: + u64_stats_update_begin(&s->syncp); \ WARNING: line length of 81 exceeds 80 columns #67: FILE: net/core/page_pool.c:51: + u64_stats_inc(&s->__stat); \ WARNING: line length of 81 exceeds 80 columns #68: FILE: net/core/page_pool.c:52: + u64_stats_update_end(&s->syncp); \ total: 0 errors, 6 warnings, 2 checks, 127 lines checked NOTE: For some of the reported defects, checkpatch may be able to mechanically convert to the typical style using --fix or --fix-inplace. Commit a3f3b5ce0382 ("page_pool: Convert page_pool_alloc_stats to u64_stats_t.") has style problems, please review. NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT BAD_REPORTED_BY_LINK CAMELCASE COMMIT_LOG_LONG_LINE GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. total: 0 errors, 6 warnings, 2 checks, 127 lines checked