netdev - Re: [PATCH wpan-next v5] mac802154: Ensure proper scan-level filtering

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: <20221019093433.76f07627@xps-13>
Date:   Wed, 19 Oct 2022 09:34:33 +0200
From:   Miquel Raynal <miquel.raynal@...tlin.com>
To:     Alexander Aring <aahringo@...hat.com>
Cc:     Alexander Aring <alex.aring@...il.com>,
        Stefan Schmidt <stefan@...enfreihafen.org>,
        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 v5] mac802154: Ensure proper scan-level
 filtering

Hi Alexander,

aahringo@...hat.com wrote on Tue, 18 Oct 2022 18:56:49 -0400:

> Hi,
> 
> On Tue, Oct 18, 2022 at 6:03 PM Miquel Raynal <miquel.raynal@...tlin.com> wrote:
> >
> > Hi Alexander,
> >
> > aahringo@...hat.com wrote on Tue, 18 Oct 2022 16:54:13 -0400:
> >  
> > > Hi,
> > >
> > > On Tue, Oct 18, 2022 at 2:35 PM Miquel Raynal <miquel.raynal@...tlin.com> wrote:  
> > > >
> > > > We now have a fine grained filtering information so let's ensure proper
> > > > filtering in scan mode, which means that only beacons are processed.
> > > >  
> > >
> > > Is this a fixup? Can you resend the whole series please?  
> >
> > Hmm no? Unless I understood things the wrong way, Stefan applied
> > patches 1 to 7 of my v4, and asked me to make a change on the 8th
> > patch.
> >
> > This is v5 just for patch 8/8 of the previous series, I just changed
> > a debug string actually...
> >  
> 
> Okay, I see there are multiple new patches on the list, can you resend
> them in one series? Then we have the right order how they need to be
> applied without figuring it "somehow" out.

The order should not matter much, that's why I posted them as individual
patches. But no problem, I'll resent the three "followup" patches in a
series.

However, the patch for allowing coordinator interfaces should be
considered something else, I've sent it to start discussing it as the
other patches should not require a lot of review time I guess. I didn't
think it made sense to wrap it in the followup series as this is
mainly something new, but I'm fine doing it, the first patches can go
in while we still discuss it further.

> > There was a conflict when he applied it but I believe this is because
> > wpan-next did not contain one of the fixes which made it to Linus' tree
> > a month ago. So in my branch I still have this fix prior to this patch,
> > because otherwise there will be a conflict when merging v6.1-rc1 (which
> > I believe was not done yet).
> >  
> 
> I see. Thanks.
> 
> - Alex
> 

Thanks,
Miquèl