From: "Andreas Färber" <email@example.com> To: Alexander Aring <firstname.lastname@example.org> Cc: Xue Liu <email@example.com>, Jian-Hong Pan <firstname.lastname@example.org>, "David S . Miller" <email@example.com>, Alan Cox <firstname.lastname@example.org>, email@example.com, firstname.lastname@example.org, email@example.com, firstname.lastname@example.org, Marcel Holtmann <email@example.com>, Dollar Chen <firstname.lastname@example.org>, Ken Yu <email@example.com>, linux-wpan - ML <firstname.lastname@example.org>, Jiri Pirko <email@example.com>, Stefan Schmidt <firstname.lastname@example.org> Subject: Re: [PATCH v5 6/6] net: lorawan: List LORAWAN in menuconfig Date: Sat, 29 Dec 2018 07:28:53 +0100 [thread overview] Message-ID: <email@example.com> (raw) In-Reply-To: <20181228154316.2x2tyfzidnb4emqo@x220t> Am 28.12.18 um 16:43 schrieb Alexander Aring: > On Fri, Dec 28, 2018 at 05:57:53AM +0100, Andreas Färber wrote: >> Am 24.12.18 um 16:32 schrieb Alexander Aring: >>> On Tue, Dec 18, 2018 at 02:50:58PM +0100, Xue Liu wrote: >>>> On Mon, 17 Dec 2018 at 15:19, Andreas Färber <firstname.lastname@example.org> wrote: >>>>> Am 17.12.18 um 09:50 schrieb Xue Liu: >>>>>> I have a question about the architecture of your module. AFAIK LoRaWAN >>>>>> is already the MAC Layer above the LoRa technology. Why do you want to >>>>>> make a new layer called "maclorawan" ? >>>>> >>>>> I had asked Jian-Hong to separate between his soft-MAC implementation >>>>> and the common bits needed to drive hard-MAC implementations found on >>>>> several of the hardware modules made available to me. >>>>> >>>> As a reference Linux 802.11 uses cfg80211 to talk with hard-MAC devices. >>>> We may also use the name “cfglora” for hard-MAC implementation. >>> >>> There exists also a cfg802154. :-) >>> >>> Note that cfg80211 is also for providing a backwardscompatibility to the >>> wireless ioctl() interface. >>> >>> In theory it's simple: >>> >>> netlink API -> SoftMAC (macFOOBAR layer) -> cfgFOOBAR implementation -> driver layer >>> \-> HardMAC (driver layer) -> cfgFOOBAR implementation >> >> So how does cfgFOOBAR relate to nlFOOBAR now? Given that we were told to >> use netlink and pointed to some nl802whatever, I am confused about two >> people now calling for cfg. We have an nllora stubbed in linux-lora.git, >> and I was expecting to see an nllorawan¹ either in this series or on > > Why there is a different between two lora technologies? This sounds you > driving into two lora subsystems without one userspace api to access them, > this getting worse. This had just been explained to Jiri: LoRa PHY vs. LoRaWAN MAC. Similar splits exist for other technologies; and how the LoRaWAN soft-MAC can reuse underlying LoRa facilities is the very subject of this sub-thread! It entirely depends on what the user wants to do - connect to a LoRaWAN network as client, forward LoRaWAN packets as a gateway, communicate peer-to-peer, implement alternative MACs in userspace, ... So if a user is working on LoRaWAN MAC layer I expect him to deal with PF_LORAWAN and nllorawan and not need direct access to nllora, as that would be an implementation detail specific to the soft-MAC; a hard-MAC should not need to implement nllora, as it may not have that control. >> top. If you're suggesting to rename them technology-neutral, then please > > I am sorry, I actually meant that... People tell me that I can't explain > things all the time. > >> say so clearly - otherwise it sounds to me like you didn't actually look >> at the staged code yet or didn't read our previous discussions and lead >> our contributors to reinvent things we already have... > > As example for 802.15.4: > > nl802154 (which is one netlink interface for doesn't matter what > kind 802.15.4 device is behind) -> callback structure of cfg802154 which > goes to a somehow 802.15.4 device as SoftMAC layer or HardMAC driver. Okay, I took a shortcut there for LoRa and assume we always have a LoRa netdev (i.e., no theoretical SDR netdev), so that I could postpone figuring out how to register extra per-netdev cfg structs for quicker PoC of the netlink interface. :) My get_freq callback will need to be moved out from struct lora_dev_priv (lora/netdev.h) into its own struct. >> We really need to complete the layers from the ground up before we get >> lost in more nice-to-have upper layers: For LoRaWAN that means we need >> to have TX and RX working for LoRa _and_ FSK. sx1276 still has lots of >> hardcoded stuff from my own testing that needs to hook into nllora, and >> FSK exists only as ETH_P_FSK constant so far, with no concept for >> switching modes yet (which as mentioned in my presentation¹ needs to go >> via sleep mode, losing most register settings) nor any netlink support. >> Not all drivers need to be at the same implementation level, of course, >> but we need at least one that's far enough to validate such patches. > > Your register behaviour sounds for me like a feature for regmap. Or > either a feature to handle in your subsystem. We don't have regmap caching enabled. sx1276 has no paging implemented, as it's slightly more complicated than sx1301 (and the "youngest" of 3). I assume instead of just writing from netlink/cfg callbacks to regmap we'll need to also save the values in the driver for writing them back after a mode switch or suspend. However it's also a question of who initiates the mode switch and how, presumably via netlink. >> And seeing that I just found a major bug in sx1276 driver's TX path, >> apparently no one apart from me is testing that driver - sx128x and >> sx1301 were not yet complete enough to transmit, and due to the open >> socket address/protocol discussions none can receive yet, so as Jiri >> hinted, this LoRaWAN soft-MAC patch series can't have been >> runtime-tested against any staged driver at all! => [RFC lora-next v5 6/6] > > aha. When I started working on ieee802154 many times I thought nobody > had really tested it. That's somehow the process of upstream > programming, it's growing over the time. The first implementation is > always somehow crappy, but people working on it and get experience over > the time, you cannot have perfect code. This is not about iterative development, it's about discussing about a high-level cfg interface on top of nl on top of a PHY that didn't work; not just some optional offload feature being broken. ;) We may never have perfect code, but we can't merge a driver that doesn't work at all, and as long as I know it's not working sufficiently I am holding back on sending out a v2, in favor of queuing more fixes and cleanups that'll make it worth people's time to review. Having the soft-MAC be further implemented and actually using the functions it implements may avoid some of the declarations Jiri disliked by just making them static locally where they're needed. Earlier reviewers were already getting deep into coding style reviews, but it seems this is not yet a "PATCH" ready for merging after all and should therefore by its author be labeled "RFC" or at least "RFT" if it couldn't be tested yet. If it's a "PATCH", I expect to be able to queue it on my tree if I spot no major design problems or nits the author could help fix upfront. Compare below. >> Therefore I thought in our case some hard-MAC may be easier to validate >> LoRaWAN sockets (patch 1/6), to avoid a dependency on completing the MAC >> implementation first. For example, iM880, RF1276TS and 32001353 are pure >> LoRaWAN modules without raw LoRa support. (Whereas many others support >> both and I'm still looking for input on how to best deal with that - >> currently exposing them as LoRa devices for maximal flexibility.) > > So that means you ignore SoftMAC because HardMAC is easier? No, I don't ignore it, I'm missing infrastructure to evaluate it! As Jiri has pointed out earlier, this series is doing two things, 1) introduce LoRaWAN sockets (that's great, needs review/discussion) and 2) prepare a soft-MAC implementation with a number of functions that are not yet called from anywhere. Plus the series omits to extend my lower-level netlink interfaces and driver(s) with the facilities it'll need to actually work. The default Sync Word on sx1276 is 0x12, for instance. Basically Jian-Hong and I are at odds in which direction the layers should plug together, with me vehemently against coding LoRaWAN stuff in LoRa drivers (layering violation!) and instead wanting that centrally in an optional LoRaWAN soft-MAC module (=maclorawan) on top. Compare 2/6 still reading "driver should implement some of them according to the usage" - for me that should be a cfg struct at the nllorawan layer, which he should implement in maclorawan and I only for hard-MAC drivers, instead of exporting an ABI that's not called by anyone and would seemingly need to be reimplemented in each and every of my LoRa drivers! Note that duplicate copies and forks of LoRaWAN soft-MAC userspace implementations on GitHub were one of the reasons for me to start this kernel project, so I really, really don't want that here. And 5/6 main.c appears to still be based around the idea that he gets his own loraX netdevs, seemingly conflicting by name with mine if they would actually get created by the functions getting called anywhere... Maybe you can now grasp my annoyance with this non-RFC v5 making only cosmetic changes since v2? (and completely lacking any changelog in 0/6) > We actually > go the opposite way to say SoftMAC introduce the most infrastructure and > then say that we will bind HardMAC to it. The very definition of a hard-MAC is that it does not need a soft-MAC. So that doesn't make much sense to me. The netlink interfaces for LoRa (e.g., Join operations for authentication) will be constrained by what ops the hard-MACs need, whereas we have much more freedoms with our own soft-MAC. LoRaWAN provides a meta set of operations compared to LoRa+FSK (e.g., setting the data rate on MAC layer would involve regional awareness and translates to setting frequency+bandwidth+SF on PHY layer, and a LoRa transmission/reception will need to set a Sync Word of 0x34); therefore my saying that for the soft-MAC we need interfaces and implementations on my LoRa PHY layer that are still incomplete today. Once the PHY layer is working, the soft-MAC would just need to properly dispatch - and ideally I would have it provide packets in an ETH_P_LORA format, so that I don't need duplicate (ETH_P_LORA + ETH_P_LORAWAN) implementations in each soft-MAC capable LoRa driver. Thus, if the PHY doesn't transmit (yet), LoRaWAN can't send either. And that brings us back to the big question circling since months of how to design these socket layers: Currently I'm using a PF_LORA and was hoping to extend its sock_addr to cope with the Sync Word, frequency, etc. and set that via skb for each TX (which raised problems for RX). At ELCE 2018 it was suggested that we should rather take the easy route of not defining a PF_LORA and just use PF_PACKET with htons(ETH_P_LORA) and apply every global setting via netlink; that however appears to obsolete my lora/dgram.c code dealing with private ifindex field then, so that we could no longer pass any LoRa-specific per-packet metadata from LoRaWAN layer down to LoRa either. And since I'm unfamiliar with PF_PACKET, I'm focusing on making the basics work first before diving into refactorings that give us no functional benefits. Anyway, I assume we'll need to translate from LoRaWAN to LoRa skbs somewhere in this patch series to add the necessary LoRaWAN headers for SOCK_DGRAM or SOCK_SEQPACKET packets - it's neither in 1/6 nor in 5/6. Additionally, operations such as Join that the user would request via netlink command will need to create LoRa skbs in the soft-MAC directly. So in that way LoRaWAN may be comparable to Wifi connecting to an AP, whereas LoRa is more like shoveling packets into/out an Ethernet cable. Hope that explains better from my side as well. Most of it had been brought up in replies to the cover letter of my original RFC on netdev list: https://patchwork.ozlabs.org/cover/937545/ Regards, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg)
next prev parent reply other threads:[~2018-12-29 6:29 UTC|newest] Thread overview: 70+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-08-23 17:15 [RFC 0/3 net] lorawan: Add LoRaWAN soft MAC module Jian-Hong Pan 2018-08-23 17:15 ` [RFC 1/3 net] lorawan: Add LoRaWAN class module Jian-Hong Pan 2018-08-23 17:43 ` Randy Dunlap 2018-08-24 15:58 ` Jian-Hong Pan 2018-09-23 16:40 ` Andreas Färber 2018-09-26 15:52 ` Jian-Hong Pan 2018-11-05 16:55 ` [PATCH V2 0/7] net: lorawan: Add LoRaWAN soft MAC module Jian-Hong Pan 2018-11-05 16:55 ` [PATCH V2 1/7] net: lorawan: Add macro and definition for LoRaWAN Jian-Hong Pan 2018-11-05 16:55 ` [PATCH V2 2/7] net: lorawan: Add LoRaWAN socket module Jian-Hong Pan 2018-11-05 18:16 ` David Miller 2018-11-06 14:28 ` Jian-Hong Pan 2018-11-14 16:01 ` [PATCH V3 0/7] net: lorawan: Add LoRaWAN soft MAC module Jian-Hong Pan 2018-11-14 16:01 ` [PATCH V3 1/7] net: lorawan: Add macro and definition for LoRaWAN Jian-Hong Pan 2018-11-14 16:12 ` Andreas Färber 2018-11-17 6:47 ` Jian-Hong Pan 2018-11-14 16:01 ` [PATCH V3 2/7] net: lorawan: Add LoRaWAN socket module Jian-Hong Pan 2018-11-17 4:32 ` David Miller 2018-11-17 14:54 ` Jian-Hong Pan 2018-12-04 14:13 ` [PATCH V4 0/6] net: lorawan: Add LoRaWAN soft MAC module Jian-Hong Pan 2018-12-04 14:13 ` [PATCH V4 1/6] net: lorawan: Add LoRaWAN socket module Jian-Hong Pan 2018-12-04 14:13 ` [PATCH V4 2/6] net: lorawan: Add LoRaWAN API declaration for LoRa devices Jian-Hong Pan 2018-12-04 14:13 ` [PATCH V4 3/6] net: maclorawan: Add maclorawan module declaration Jian-Hong Pan 2018-12-04 14:13 ` [PATCH V4 4/6] net: maclorawan: Implement the crypto of maclorawan module Jian-Hong Pan 2018-12-04 14:13 ` [PATCH V4 5/6] net: maclorawan: Implement maclorawan class module Jian-Hong Pan 2018-12-04 20:45 ` Alan Cox 2018-12-09 8:27 ` Jian-Hong Pan 2018-12-16 10:18 ` [PATCH v5 0/6] net: lorawan: Add LoRaWAN soft MAC module Jian-Hong Pan 2018-12-17 13:51 ` Jiri Pirko 2018-12-16 10:18 ` [PATCH v5 1/6] net: lorawan: Add LoRaWAN socket module Jian-Hong Pan 2018-12-29 7:27 ` Andreas Färber 2019-01-07 14:47 ` Jian-Hong Pan 2019-01-13 14:51 ` Jian-Hong Pan 2018-12-16 10:18 ` [PATCH v5 2/6] net: lorawan: Add LoRaWAN API declaration for LoRa devices Jian-Hong Pan 2018-12-16 10:18 ` [PATCH v5 3/6] net: maclorawan: Add maclorawan module declaration Jian-Hong Pan 2018-12-16 10:18 ` [PATCH v5 4/6] net: maclorawan: Implement the crypto of maclorawan module Jian-Hong Pan 2018-12-16 10:18 ` [PATCH v5 5/6] net: maclorawan: Implement maclorawan class module Jian-Hong Pan 2018-12-17 14:02 ` Jiri Pirko 2018-12-18 14:27 ` Jian-Hong Pan 2018-12-18 14:27 ` Jiri Pirko 2018-12-18 15:34 ` Jian-Hong Pan 2018-12-18 18:49 ` Andreas Färber 2018-12-19 11:27 ` Ben Whitten 2018-12-19 16:26 ` Jian-Hong Pan 2018-12-20 10:19 ` Ben Whitten 2018-12-20 15:31 ` Andreas Färber 2018-12-16 10:19 ` [PATCH v5 6/6] net: lorawan: List LORAWAN in menuconfig Jian-Hong Pan 2018-12-17 8:50 ` Xue Liu 2018-12-17 14:19 ` Andreas Färber 2018-12-18 13:50 ` Xue Liu 2018-12-24 15:32 ` Alexander Aring 2018-12-28 4:57 ` Andreas Färber 2018-12-28 15:43 ` Alexander Aring 2018-12-29 6:28 ` Andreas Färber [this message] 2018-12-04 14:13 ` [PATCH V4 " Jian-Hong Pan 2018-11-14 16:01 ` [PATCH V3 3/7] net: lorawan: Add LoRaWAN API declaration for LoRa devices Jian-Hong Pan 2018-11-14 16:01 ` [PATCH V3 4/7] net: maclorawan: Add maclorawan module declaration Jian-Hong Pan 2018-11-17 4:32 ` David Miller 2018-11-17 6:32 ` Jian-Hong Pan 2018-11-14 16:01 ` [PATCH V3 5/7] net: maclorawan: Implement the crypto of maclorawan module Jian-Hong Pan 2018-11-14 16:01 ` [PATCH V3 6/7] net: maclorawan: Implement maclorawan class module Jian-Hong Pan 2018-11-14 16:01 ` [PATCH V3 7/7] net: lorawan: List LORAWAN in menuconfig Jian-Hong Pan 2018-11-05 16:55 ` [PATCH V2 3/7] net: lorawan: Add LoRaWAN API declaration for LoRa devices Jian-Hong Pan 2018-11-05 16:55 ` [PATCH V2 4/7] net: maclorawan: Add maclorawan module declaration Jian-Hong Pan 2018-11-05 16:55 ` [PATCH V2 5/7] net: maclorawan: Implement the crypto of maclorawan module Jian-Hong Pan 2018-11-05 16:55 ` [PATCH V2 6/7] net: maclorawan: Implement maclorawan class module Jian-Hong Pan 2018-11-05 16:55 ` [PATCH V2 7/7] net: lorawan: List LORAWAN in menuconfig Jian-Hong Pan 2018-08-23 17:15 ` [RFC 2/3 net] lorawan: Add macro and definition for LoRaWAN class modlue Jian-Hong Pan 2018-09-23 16:06 ` Andreas Färber 2018-09-26 14:46 ` Jian-Hong Pan 2018-08-23 17:15 ` [RFC 3/3 net] lorawan: List LORAWAN in menuconfig Jian-Hong Pan
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 \ --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 \ --subject='Re: [PATCH v5 6/6] net: lorawan: List LORAWAN in menuconfig' \ /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).