netdev - Re: [PATCH wpan v2 0/6] ieee802154: Scan/Beacon fixes

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: <736c9250-ecfc-f9ce-7367-bd79e930f5c3@datenfreihafen.org>
Date:   Sat, 18 Feb 2023 18:20:22 +0100
From:   Stefan Schmidt <stefan@...enfreihafen.org>
To:     Miquel Raynal <miquel.raynal@...tlin.com>,
        Alexander Aring <alex.aring@...il.com>,
        linux-wpan@...r.kernel.org
Cc:     "David S. Miller" <davem@...emloft.net>,
        Jakub Kicinski <kuba@...nel.org>,
        Paolo Abeni <pabeni@...hat.com>,
        Eric Dumazet <edumazet@...gle.com>, netdev@...r.kernel.org,
        David Girault <david.girault@...vo.com>,
        Romuald Despres <romuald.despres@...vo.com>,
        Frederic Blain <frederic.blain@...vo.com>,
        Nicolas Schodet <nico@...fr.eu.org>,
        Guilhem Imberton <guilhem.imberton@...vo.com>,
        Thomas Petazzoni <thomas.petazzoni@...tlin.com>
Subject: Re: [PATCH wpan v2 0/6] ieee802154: Scan/Beacon fixes

Hello Miquel.

On 17.02.23 10:10, Miquel Raynal wrote:
> Hello Jakub, Stefan, Alexander,
> 
> miquel.raynal@...tlin.com wrote on Tue, 14 Feb 2023 14:50:29 +0100:
> 
>> Hello,
>>
>> Following Jakub's review on Stefan's MR, a number of changes were
>> requested for him in order to pull the patches in net. In the mean time,
>> a couple of discussions happened with Alexander (return codes for
>> monitor scans and transmit helper used for beacons).
>>
>> Hopefully this series addresses everything.
> 
> I know it's only been 3 working days since I sent this series but as we
> are approaching the closing of net-next and Stefan's MR was paused
> until these fixes arrived, I wanted to check whether these changes
> might be satisfying enough, in particular Jakub, if you found the
> answers you asked for.
> 
> I mainly want to avoid the "Stefan waits for Alexander who waits for
> Jakub who waits for Stefan" dependency chain :)

I just reviewed and tested them and have no problem to take them in. For 
patches 1 and 2 I would prefer an ack from Jakub to make sure we covered 
all of this review feedback before. Let's hope we can get these on 
Monday or Tuesday. Once we have them in I will re-spin a new pull 
request for all the changes.

regards
Stefan Schmidt