netdev - Re:[RFC PATCH v5 0/7] net: wwan: add NMEA port type support

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: <DF8AF3F7-9A3F-4DCB-963C-DCAE46309F7B@gmail.com>
Date: Fri, 09 Jan 2026 09:11:58 +0200
From: Sergey Ryazanov <ryazanov.s.a@...il.com>
To: Slark Xiao <slark_xiao@....com>
CC: Loic Poulain <loic.poulain@....qualcomm.com>,
 Johannes Berg <johannes@...solutions.net>,
 Andrew Lunn <andrew+netdev@...n.ch>, Eric Dumazet <edumazet@...gle.com>,
 "David S . Miller" <davem@...emloft.net>, Jakub Kicinski <kuba@...nel.org>,
 Paolo Abeni <pabeni@...hat.com>, netdev@...r.kernel.org,
 Muhammad Nuzaihan <zaihan@...ealasia.net>, Daniele Palmas <dnlplm@...il.com>,
 Qiang Yu <quic_qianyu@...cinc.com>, Manivannan Sadhasivam <mani@...nel.org>,
 Johan Hovold <johan@...nel.org>
Subject: Re:[RFC PATCH v5 0/7] net: wwan: add NMEA port type support

On January 9, 2026 5:21:34 AM, Slark Xiao <slark_xiao@....com> wrote:
>At 2026-01-09 09:09:02, "Sergey Ryazanov" <ryazanov.s.a@...il.com> wrote:
>>The series introduces a long discussed NMEA port type support for the
>>WWAN subsystem. There are two goals. From the WWAN driver perspective,
>>NMEA exported as any other port type (e.g. AT, MBIM, QMI, etc.). From
>>user space software perspective, the exported chardev belongs to the
>>GNSS class what makes it easy to distinguish desired port and the WWAN
>>device common to both NMEA and control (AT, MBIM, etc.) ports makes it
>>easy to locate a control port for the GNSS receiver activation.
>>
>>Done by exporting the NMEA port via the GNSS subsystem with the WWAN
>>core acting as proxy between the WWAN modem driver and the GNSS
>>subsystem.
>>
>>The series starts from a cleanup patch. Then three patches prepares the
>>WWAN core for the proxy style operation. Followed by a patch introding a
>>new WWNA port type, integration with the GNSS subsystem and demux. The
>>series ends with a couple of patches that introduce emulated EMEA port
>>to the WWAN HW simulator.
>>
>>The series is the product of the discussion with Loic about the pros and
>>cons of possible models and implementation. Also Muhammad and Slark did
>>a great job defining the problem, sharing the code and pushing me to
>>finish the implementation. Daniele has caught an issue on driver
>>unloading and suggested an investigation direction. What was concluded
>>by Loic. Many thanks.
>>
>>Slark, if this series with the unregister fix suits you, please bundle
>>it with your MHI patch, and (re-)send for final inclusion.
>>
>>Changes RFCv1->RFCv2:
>>* Uniformly use put_device() to release port memory. This made code less
>>  weird and way more clear. Thank you, Loic, for noticing and the fix
>>  discussion!
>>Changes RFCv2->RFCv5:
>>* Fix premature WWAN device unregister; new patch 2/7, thus, all
>>  subsequent patches have been renumbered
>>* Minor adjustments here and there
>>
>Shall I keep these RFC changes info in my next commit?
>Also these RFC changes info in these single patch.

Generally, yeah, it's a good idea to keep information about changes, especially per item patch. Keeping the cover latter changelog is up to you.

>And I want to know whether  v5 or v6 shall be used for my next serial?

Any of them will work. If you asking me, then I would suggest to send it as v6 to continue numbering.

>Is there a review progress for these RFC patches ( for patch 2/7 and 
>3/7 especially). If yes, I will hold my commit until these review progress
>finished. If not, I will combine these changes with my MHI patch and send
>them out asap.

I have collected all the feedback. E.g., minor number leak was fixed. Fixed one long noticed mistype. And collected two new review tags given by Loic. So, my advice is to use these patches as base and put your MHI patch on top of them.

>>CC: Slark Xiao <slark_xiao@....com>
>>CC: Muhammad Nuzaihan <zaihan@...ealasia.net>
>>CC: Daniele Palmas <dnlplm@...il.com>
>>CC: Qiang Yu <quic_qianyu@...cinc.com>
>>CC: Manivannan Sadhasivam <mani@...nel.org>
>>CC: Johan Hovold <johan@...nel.org>
>>
>>Sergey Ryazanov (7):
>>  net: wwan: core: remove unused port_id field
>>  net: wwan: core: explicit WWAN device reference counting
>>  net: wwan: core: split port creation and registration
>>  net: wwan: core: split port unregister and stop
>>  net: wwan: add NMEA port support
>>  net: wwan: hwsim: refactor to support more port types
>>  net: wwan: hwsim: support NMEA port emulation
>>
>> drivers/net/wwan/Kconfig      |   1 +
>> drivers/net/wwan/wwan_core.c  | 280 +++++++++++++++++++++++++++-------
>> drivers/net/wwan/wwan_hwsim.c | 201 +++++++++++++++++++-----
>> include/linux/wwan.h          |   2 +
>> 4 files changed, 396 insertions(+), 88 deletions(-)
>>
>>-- 
>>2.52.0

Hi Slark