netdev - Re: [PATCH wpan-next v2 0/2] IEEE 802.15.4 PAN discovery handling

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] [day] [month] [year] [list]

Message-ID: <20221129141114.2b78240b@xps-13>
Date:   Tue, 29 Nov 2022 14:11:14 +0100
From:   Miquel Raynal <miquel.raynal@...tlin.com>
To:     Stefan Schmidt <stefan@...enfreihafen.org>
Cc:     Alexander Aring <aahringo@...hat.com>,
        Alexander Aring <alex.aring@...il.com>,
        linux-wpan@...r.kernel.org,
        "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-next v2 0/2] IEEE 802.15.4 PAN discovery handling

Hi Stefan,

stefan@...enfreihafen.org wrote on Tue, 29 Nov 2022 13:41:32 +0100:

> Hello.
> 
> On 29.11.22 09:03, Miquel Raynal wrote:
> > Hi Alexander,
> > 
> > aahringo@...hat.com wrote on Mon, 28 Nov 2022 17:11:38 -0500:
> >   
> >> Hi,
> >>
> >> On Fri, Nov 18, 2022 at 5:13 PM Miquel Raynal <miquel.raynal@...tlin.com> wrote:  
> >>>
> >>> Hello,
> >>>
> >>> Last preparation step before the introduction of the scanning feature
> >>> (really): generic helpers to handle PAN discovery upon beacon
> >>> reception. We need to tell user space about the discoveries.
> >>>
> >>> In all the past, current and future submissions, David and Romuald from
> >>> Qorvo are credited in various ways (main author, co-author,
> >>> suggested-by) depending of the amount of rework that was involved on
> >>> each patch, reflecting as much as possible the open-source guidelines we
> >>> follow in the kernel. All this effort is made possible thanks to Qorvo
> >>> Inc which is pushing towards a featureful upstream WPAN support.  
> >>>   >>  
> >> Acked-by: Alexander Aring <aahringo@...hat.com>
> >>
> >> I am sorry, I saw this series today. Somehow I mess up my mails if we
> >> are still writing something on v1 but v2 is already submitted. I will
> >> try to keep up next time.  
> > 
> > Haha I was asking myself wether or not you saw it, no problem :) I did
> > send it after your main review but we continued discussing on v1 (about
> > the preambles) so I did not ping for the time the discussion would
> > settle.  
> 
> I was trying to apply these two patches, but the first one does not apply:
> 
> 
> Failed to apply patch:
> error: patch failed: include/net/nl802154.h:58
> error: include/net/nl802154.h: patch does not apply
> hint: Use 'git am --show-current-patch=diff' to see the failed patch
> Applying: ieee802154: Advertize coordinators discovery
> Patch failed at 0001 ieee802154: Advertize coordinators discovery
> 
> It seems you need a rebase as there is commit 8254393663f9b8cb8b84cdce1abb118833c22a54 which touches this area of the file and removes a comment and ifdef. Should be fine to go in after the rebase.

Oh crap, this is gonna conflict with a dozen of my patches /o\. Not hard
to solve though. Let me fix this by moving all new commands in this
header above the security commands.

I'll also have to update the wpan tools to use the same commands.

Thanks,
Miquèl