From: Stephan Gerhold <firstname.lastname@example.org> To: Loic Poulain <email@example.com> Cc: "David S. Miller" <firstname.lastname@example.org>, Jakub Kicinski <email@example.com>, Sergey Ryazanov <firstname.lastname@example.org>, Johannes Berg <email@example.com>, Bjorn Andersson <firstname.lastname@example.org>, Andy Gross <email@example.com>, Vinod Koul <firstname.lastname@example.org>, Rob Herring <email@example.com>, Aleksander Morgado <firstname.lastname@example.org>, Network Development <email@example.com>, linux-arm-msm <firstname.lastname@example.org>, email@example.com, devicetree <firstname.lastname@example.org>, open list <email@example.com>, firstname.lastname@example.org, ~email@example.com, Jeffrey Hugo <firstname.lastname@example.org> Subject: Re: [RFC PATCH net-next 4/4] net: wwan: Add Qualcomm BAM-DMUX WWAN network driver Date: Thu, 22 Jul 2021 17:40:32 +0200 [thread overview] Message-ID: <YPmRcBXpRtKKSDl8@gerhold.net> (raw) In-Reply-To: <CAMZdPi8oxRMo0erfd0wrUPzD2UsbexoR=86u2N75Fd9RpXHoKg@mail.gmail.com> Hi Loic, On Mon, Jul 19, 2021 at 06:01:33PM +0200, Loic Poulain wrote: > On Mon, 19 Jul 2021 at 17:01, Stephan Gerhold <email@example.com> wrote: > > > > I'm not sure how to integrate the driver with the WWAN subsystem yet. > > At the moment the driver creates network interfaces for all channels > > announced by the modem, it does not make use of the WWAN link management > > yet. Unfortunately, this is a bit complicated: > > > > Both QMAP and the built-in multiplexing layer might be needed at some point. > > There are firmware versions that do not support QMAP and the other way around > > (the built-in multiplexing was disabled on very recent firmware versions). > > Only userspace can check if QMAP is supported in the firmware (via QMI). > > > > I could ignore QMAP completely for now but I think someone will show up > > who will need this eventually. And if there is going to be common code for > > QMAP/rmnet link management it would be nice if BAM-DMUX could also make > > use of it. > > I have this on my TODO list for mhi-net QMAP. > Great, thanks! > > But the question is, how could this look like? How do we know if we should > > create a link for QMAP or a BAM-DMUX channel? Does it even make sense > > to manage the 1-8 channels via the WWAN link management? > > Couldn't it be specified via dts (property or different compatible > string)? It would probably work in most cases, but I have to admit that I would prefer to avoid this for the following reason: This driver is used on some smartphones that have different variants for different parts of the world. As far as Linux is concerned the hardware is pretty much identical, but the modem firmware is often somewhat device-specific. This means that the same device tree is often used with different firmware versions. Perhaps we are lucky enough that the firmware versions have the same capabilities, but I'm not fully sure about that. I think at the end the situation is fairly similar to qmi_wwan/USB. There the kernel also does not know if the modem supports QMAP or not. The way it's solved there at the moment is that ModemManager tries to enable it from user space and then the mode of the network interface can be switched through a sysfs file ("qmi/pass_through"). Something like this should probably also work in my case. This should also allow me to ignore QMAP for now and deal with it if someone really needs it at some point since it's quite complicated for BAM-DMUX. (I tried QMAP again today and listed the problems in  for reference, but it's all BAM-DMUX specific...)  https://lore.kernel.org/netdev/YPmF8bzevuabO2K9@gerhold.net/ > > would it make sense to have two drivers (with common core) to > manage either the multi-bam channel or newer QMAP based single > bam-channel modems. > There should be fairly little difference between those two usage modes, so I don't think it's worth splitting the driver for this. Actually right now (ignoring the link management of the WWAN subsystem), it's already possible to use both. I can use the network interfaces as-is in Raw-IP mode or I do "sudo ip link add link rmnet0 name rmnet0_qmap type rmnet mux_id 1" on top and use QMAP. The BAM-DMUX driver does not care, because it just hands over sent/received packets as-is and the modem data format must be always configured via QMI from user space. > > > > Another problem is that the WWAN subsystem currently creates all network > > interfaces below the common WWAN device. This means that userspace like > > ModemManager has no way to check which driver provides them. This is > > necessary though to decide how to set it up via QMI (ModemManager uses it). > > Well, I have quite a similar concern since I'm currently porting > mhi-net mbim to wwan framework, and I was thinking about not making > wwan device parent of the network link/netdev (in the same way as > wlan0 is not child of ieee80211 device), but not sure if it's a good > idea or not since we can not really consider driver name part of the > uapi. > Hm, I think the main disadvantage of that would be that the network interface is no longer directly related to the WWAN device, right? Userspace would then need some special matching to find the network interfaces that belong to a certain control port. With the current setup, e.g. ModemManager can simply match the WWAN device and then look at its children and find the control port and network interfaces. How would it find the network interfaces if they are no longer below the WWAN device? > The way links are created is normally abstracted, so if you know which > bam variant you have from wwan network driver side (e.g. via dts), you > should have nothing to check on the user side, except the session id. In a perfect world it would probably be like this, but I'm afraid the Qualcomm firmware situation isn't as simple. User space needs to know which setup it is dealing with because all the setup happens via QMI. Let's take the BAM-DMUX channels vs QMAP mux-IDs for example: First, user space needs to configure the data format. This happens with the QMI WDA (Wireless Data Administrative Service) "Set Data Format" message. Parameter would be link layer format (Raw-IP in both cases) but also the uplink/downlink data aggregation protocol. This is either one of many QMAP versions (qmap|qmapv2|qmapv3|qmapv4|qmapv5), or simply "none" when using BAM-DMUX without QMAP. Then, the "session ID" (= BAM-DMUX channel or QMAP mux-ID) must be bound to a WDS (Wireless Data Service) session. The QMI message for that is different for BAM-DMUX and QMAP: - BAM-DMUX: WDS "Bind Data Port" (Parameter: SIO port number, can be derived from channel ID) - QMAP: WDS "Bind MUX Data Port" (note the "MUX", different message!) (Parameter: MUX ID, port type (USB/embedded/...), port number) My point here: Since userspace is responsible for QMI at the moment we will definitely need to make it aware of the setup that it needs to apply. Just having an abstract "session ID" won't be enough to set up the connection properly. :/ Thanks! Stephan
next prev parent reply other threads:[~2021-07-22 15:40 UTC|newest] Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-07-19 14:53 [RFC PATCH net-next 0/4] " Stephan Gerhold 2021-07-19 14:53 ` [RFC PATCH net-next 1/4] dt-bindings: dmaengine: bam_dma: Add remote power collapse mode Stephan Gerhold 2021-07-29 19:36 ` Rob Herring 2021-07-29 19:50 ` Stephan Gerhold 2021-07-19 14:53 ` [RFC PATCH net-next 2/4] dmaengine: qcom: " Stephan Gerhold 2021-07-19 14:53 ` [RFC PATCH net-next 3/4] dt-bindings: net: Add schema for Qualcomm BAM-DMUX Stephan Gerhold 2021-07-19 14:53 ` [RFC PATCH net-next 4/4] net: wwan: Add Qualcomm BAM-DMUX WWAN network driver Stephan Gerhold 2021-07-19 16:01 ` Loic Poulain 2021-07-22 15:40 ` Stephan Gerhold [this message] 2021-07-24 11:25 ` Sergey Ryazanov 2021-07-26 8:10 ` Aleksander Morgado 2021-07-26 22:40 ` Sergey Ryazanov 2021-07-20 9:10 ` Sergey Ryazanov 2021-07-21 12:17 ` Stephan Gerhold 2021-07-24 10:22 ` Sergey Ryazanov 2021-07-19 15:43 ` [RFC PATCH net-next 0/4] " Jeffrey Hugo 2021-07-19 18:23 ` Stephan Gerhold 2021-07-19 23:13 ` Jeffrey Hugo 2021-07-22 14:51 ` Stephan Gerhold 2021-07-26 14:58 ` Jeffrey Hugo
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=YPmRcBXpRtKKSDl8@gerhold.net \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --subject='Re: [RFC PATCH net-next 4/4] net: wwan: Add Qualcomm BAM-DMUX WWAN network driver' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).