netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Subash Abhinov Kasiviswanathan <subashab@codeaurora.org>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: "Dan Williams" <dcbw@redhat.com>, "Bjørn Mork" <bjorn@mork.no>,
	netdev@vger.kernel.org,
	"Sean Tranchetti" <stranche@codeaurora.org>,
	"Daniele Palmas" <dnlplm@gmail.com>,
	"Aleksander Morgado" <aleksander@aleksander.es>,
	netdev-owner@vger.kernel.org
Subject: Re: cellular modem driver APIs
Date: Sun, 14 Apr 2019 13:09:27 -0600	[thread overview]
Message-ID: <a8832dce4fc90b24a5da998fef2724f3@codeaurora.org> (raw)
In-Reply-To: <7980f3d8a567a16f94e03b4c90f14362fc901560.camel@sipsolutions.net>

>> Currently, iproute2 can be used to add the underlying dev as real_dev
>> and create rmnet links over it (ip link add link rmnet_ipa0 name 
>> rmnet0 type
>> rmnet mux_id 1). Would this continue to work if -
>> 1. the rmnet library were to be included directly as part of the
>> underlying driver itself
>> 2. there is no underlying dev at all
> 
> Yeah, this is the big question.
> 
> If there's no underlying netdev at all, then no, it wouldn't work.
> Though, it could be "faked" in a sense, by doing two things:
> 
> a) having the driver/infra always create a default channel interface,
>    say mux_id 0?
> b) by treating
> 
> 	ip link add link rmnet_ipa0 name rmnet0 type rmnet mux_id 1
> 
>    as not creating a new netdev *on top of* but rather *as sibling to*
>    "rmnet0".
> 
> 
> The alternative is to just keep rmnet and the underlying driver, but 
> tie
> them together a little more closely so that they can - together -
> register with a hypothetical new WWAN framework.
> 
> See - even if we create such a framework in a way that it doesn't
> require an underlying netdev (which I think is the better choice for
> various reasons, particularly related to 5G), then there's nothing that
> says that you *cannot* have it anyway and keep the exact same rmnet +
> underlying driver model.
> 
> Hmm, not sure I understand this. If you do RPS/RSS then that's a
> hardware function, and the netdev doesn't really come into play
> immediately? If the underlying driver directly deals with multiple
> netdevs that's actually an *advantage*, no?

RPS is in SW only though.

I think this shouldn't be a concern if the existing underlying netdev 
model
could co-exist with the new framework.

-- 
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

  parent reply	other threads:[~2019-04-14 19:09 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-03 21:15 cellular modem driver APIs Johannes Berg
2019-04-04  8:51 ` Bjørn Mork
2019-04-04  9:00   ` Johannes Berg
2019-04-04 15:52     ` Dan Williams
2019-04-04 19:16       ` Subash Abhinov Kasiviswanathan
2019-04-04 20:38         ` Johannes Berg
2019-04-04 21:00           ` Johannes Berg
2019-04-05  4:45           ` Subash Abhinov Kasiviswanathan
2019-04-06 17:22             ` Daniele Palmas
2019-04-08 19:49             ` Johannes Berg
2019-04-11  3:54               ` Subash Abhinov Kasiviswanathan
2019-04-12 12:01                 ` Johannes Berg
2019-04-12 14:27                   ` Bjørn Mork
2019-04-12 17:04                     ` Johannes Berg
2019-04-14 19:09                   ` Subash Abhinov Kasiviswanathan [this message]
2019-04-15  8:27                     ` Johannes Berg
2019-04-06 17:20     ` Daniele Palmas
2019-04-08 19:51       ` Johannes Berg

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=a8832dce4fc90b24a5da998fef2724f3@codeaurora.org \
    --to=subashab@codeaurora.org \
    --cc=aleksander@aleksander.es \
    --cc=bjorn@mork.no \
    --cc=dcbw@redhat.com \
    --cc=dnlplm@gmail.com \
    --cc=johannes@sipsolutions.net \
    --cc=netdev-owner@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=stranche@codeaurora.org \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).