Re: [PATCH net-next 0/8] Use max/min to simplify the code

On Tue, 27 Aug 2024 07:45:02 +0300 Kalle Valo wrote:
> > Do you mean some patches will go to other branches (such as mac80211)? =
=20
>=20
> Jakub means that your patchset had compilation errors, see the red on
> patchwork:
>=20
> https://patchwork.kernel.org/project/netdevbpf/list/?series=3D882901&stat=
e=3D*&order=3Ddate

FWIW I prefer not to point noobs to the patchwork checks, lest they
think it's a public CI and they can fling broken code at the list :(
But yes, in case "code doesn't build" needs a further explanation:

net/core/pktgen.c: In function =E2=80=98pktgen_finalize_skb=E2=80=99:
./../include/linux/compiler_types.h:510:45: error: call to =E2=80=98__compi=
letime_assert_928=E2=80=99 declared with attribute error: min(datalen/frags=
, ((1UL) << 12)) signedness error
  510 |         _compiletime_assert(condition, msg, __compiletime_assert_, =
__COUNTER__)
      |                                             ^
./../include/linux/compiler_types.h:491:25: note: in definition of macro =
=E2=80=98__compiletime_assert=E2=80=99
  491 |                         prefix ## suffix();                        =
     \
      |                         ^~~~~~
./../include/linux/compiler_types.h:510:9: note: in expansion of macro =E2=
=80=98_compiletime_assert=E2=80=99
  510 |         _compiletime_assert(condition, msg, __compiletime_assert_, =
__COUNTER__)
      |         ^~~~~~~~~~~~~~~~~~~
../include/linux/build_bug.h:39:37: note: in expansion of macro =E2=80=98co=
mpiletime_assert=E2=80=99
   39 | #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
      |                                     ^~~~~~~~~~~~~~~~~~
../include/linux/minmax.h:100:9: note: in expansion of macro =E2=80=98BUILD=
_BUG_ON_MSG=E2=80=99
  100 |         BUILD_BUG_ON_MSG(!__types_ok(x,y,ux,uy),        \
      |         ^~~~~~~~~~~~~~~~
../include/linux/minmax.h:105:9: note: in expansion of macro =E2=80=98__car=
eful_cmp_once=E2=80=99
  105 |         __careful_cmp_once(op, x, y, __UNIQUE_ID(x_), __UNIQUE_ID(y=
_))
      |         ^~~~~~~~~~~~~~~~~~
../include/linux/minmax.h:129:25: note: in expansion of macro =E2=80=98__ca=
reful_cmp=E2=80=99
  129 | #define min(x, y)       __careful_cmp(min, x, y)
      |                         ^~~~~~~~~~~~~
../net/core/pktgen.c:2796:28: note: in expansion of macro =E2=80=98min=E2=
=80=99
 2796 |                 frag_len =3D min(datalen/frags, PAGE_SIZE);
      |                            ^~~
make[5]: *** [../scripts/Makefile.build:244: net/core/pktgen.o] Error 1