netdev - Re: [PATCH] mac802154_hwsim: allow users to specify the number of simulated radios dinamically instead of the previously hardcoded value of 2

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAK8U23aRrvC4wC6WvcBRPA4YuyC1MPvOj8FO=cWfi43_Fdh4Zw@mail.gmail.com>
Date: Sat, 7 Jun 2025 18:42:09 -0300
From: Ramon Fontes <ramonreisfontes@...il.com>
To: Andrew Lunn <andrew@...n.ch>
Cc: Alexander Aring <aahringo@...hat.com>, davem@...emloft.net, kuba@...nel.org, 
	pabeni@...hat.com, linux-wpan@...r.kernel.org, alex.aring@...il.com, 
	miquel.raynal@...tlin.com, netdev@...r.kernel.org
Subject: Re: [PATCH] mac802154_hwsim: allow users to specify the number of
 simulated radios dinamically instead of the previously hardcoded value of 2

Ok! Just a last question:

Shouldn't we define a maximum number of supported radios when using
unsigned? A value like -1 would wrap around and result in thousands of
radios, right?
That said, wouldn't it be simpler and safer to just use int instead?


Em sáb., 7 de jun. de 2025 às 16:22, Andrew Lunn <andrew@...n.ch> escreveu:
>
> On Sat, Jun 07, 2025 at 02:54:57PM -0300, Ramon Fontes wrote:
> > > handle as unsigned then this check would not be necessary?
> >
> > Yeah, it does make sense. However, I have a bit of an embarrassing
> > question. How do I submit an updated patch in this same thread?
>
> You don't. You should submit a new patchset, as a new thread.
>
> The CI tooling works on one patchset per thread, and if you deviate
> from that, the CI won't test it. And if it is not tested, it is pretty
> much an automatic reject.
>
> https://www.kernel.org/doc/html/latest/process/maintainer-netdev.html
>
> It is also the merge windows right now, so anything you post will get
> rejected anyway, so please post as RFC.
>
>         Andrew