Index | Thread | Search

From:
Kirill A. Korinsky <kirill@korins.ky>
Subject:
Re: sys/iwx: removed dead code by Coverity scan report
To:
tech@openbsd.org
Date:
Mon, 27 Apr 2026 19:16:55 +0200

Download raw body.

Thread
On Mon, 27 Apr 2026 18:27:34 +0200,
Stefan Sperling <stsp@stsp.name> wrote:
> 
> On Mon, Apr 27, 2026 at 06:15:48PM +0200, Kirill A. Korinsky wrote:
> > tech@,
> > 
> > daniel@ forwarded to me a coverity scan report which highligh a now dead
> > code from one of the last commit to iwx made by me.
> 
> I don't agree with Coverity about having to remove this check.
> There is no bug and larger aggsize values might become possible when moving
> to 11ax and 11be. I fear that if we remove this check we might be setting
> ourselves up for trouble once someone tries to get 11ax or 11be working.
> I haven't checked whether the 11ax/11be specs allow for more than 4M but
> it is a possibility.
> 

Make sense, when I withdraw that diff

-- 
wbr, Kirill