netdev - RE: [PATCH V2 net-next 0/7] net: wwan: iosm: Bug 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] [day] [month] [year] [list]

Date:   Wed, 8 Dec 2021 10:36:37 +0000
From:   "Kumar, M Chetan" <m.chetan.kumar@...el.com>
To:     Jakub Kicinski <kuba@...nel.org>,
        M Chetan Kumar <m.chetan.kumar@...ux.intel.com>
CC:     "netdev@...r.kernel.org" <netdev@...r.kernel.org>,
        "davem@...emloft.net" <davem@...emloft.net>,
        "johannes@...solutions.net" <johannes@...solutions.net>,
        "ryazanov.s.a@...il.com" <ryazanov.s.a@...il.com>,
        "loic.poulain@...aro.org" <loic.poulain@...aro.org>,
        "Sudi, Krishna C" <krishna.c.sudi@...el.com>,
        linuxwwan <linuxwwan@...el.com>
Subject: RE: [PATCH V2 net-next 0/7] net: wwan: iosm: Bug fixes

> -----Original Message-----
> From: Jakub Kicinski <kuba@...nel.org>
> Sent: Wednesday, December 8, 2021 10:52 AM
> To: M Chetan Kumar <m.chetan.kumar@...ux.intel.com>
> Cc: netdev@...r.kernel.org; davem@...emloft.net;
> johannes@...solutions.net; ryazanov.s.a@...il.com;
> loic.poulain@...aro.org; Sudi, Krishna C <krishna.c.sudi@...el.com>; Kumar,
> M Chetan <m.chetan.kumar@...el.com>; linuxwwan
> <linuxwwan@...el.com>
> Subject: Re: [PATCH V2 net-next 0/7] net: wwan: iosm: Bug fixes
> 
> On Sun,  5 Dec 2021 20:34:48 +0530 M Chetan Kumar wrote:
> > This patch series brings in IOSM driver bug fixes. Patch details are
> > explained below.
> >
> > PATCH1:
> >  * stop sending unnecessary doorbell in IP tx flow.
> > PATCH2:
> >  * set tx queue len.
> > PATCH3:
> >  * Restore the IP channel configuration after fw flash.
> > PATCH4:
> >  * Release data channel if there is no active IP session.
> > PATCH5:
> >  * Removes dead code.
> > PATCH6:
> >  * Removed the unnecessary check around control port TX transfer.
> > PATCH7:
> >  * Correct open parenthesis alignment to fix checkpatch warning.
> 
> Are any of these fixing functional bugs which users may encounter?
> 
> Looks like at least patches 1, 3, and 6 may be?

Right 1, 3, & 6 are fixes. 
2, 4 are improvement and 5 & 7 are cleanup.

> 
> All the fixes for bugs should have Fixes tags and be posted against the
> netdev/net tree with [PATCH net] in the subject (meaning a separate series).
> If there are dependencies between cleanups and fixes - you'll need to defer
> the cleanups for a few days, until net is merged into net-next. It usually
> happens every Thursday.

There is no dependency b/w batches.
I will break into 2 patch series and submit fixes to netdev/net tree & improvement/cleanups into netdev/net-next tree.