public inbox for netdev@vger.kernel.org 
 help / color / mirror / Atom feed
From: "Bjørn Mork" <bjorn@mork•no>
To: Marcel Holtmann <marcel@holtmann•org>
Cc: netdev@vger•kernel.org
Subject: Re: [PATCH] net: cdc_ether.c: Add SE J105i to device whitelist
Date: Tue, 05 Jan 2010 13:46:58 +0100	[thread overview]
Message-ID: <87eim4vhvh.fsf@nemi.mork.no> (raw)
In-Reply-To: <1262691113.16627.361.camel@localhost.localdomain> (Marcel Holtmann's message of "Tue, 05 Jan 2010 03:31:53 -0800")

Marcel Holtmann <marcel@holtmann•org> writes:

>> I believe the WWAN cards also can be treated like every other USB
>> Ethernet device, as far as the kernel is concerned.  Any differences are
>> easily resolved by userspace applications. 
>
> and this is where you are totally wrong. It is not easy for userspace to
> identify the type of an interface and resolve things.

No?  Well, I probably have a too limited view of this, but I found it
easy enough to create a pre-up/post-down script which checked whether
the network interface being brought up/down belongs to a supported WWAN
card and then do the necessary magic on one of the related cdc-wdm
channels. This was implemented by poking around in the sysfs.  I fail to
see why one needs a special network device name to do that.

But it would have been useful to have the CDC MDLM header exported
somewhere in sysfs.  That's more of a generic usb/cdc thing though, I
guess. 

> Take WiFi for example. We need to actually connect to a network before
> these are useful. Same applies for 3G cards (aka WWAN) devices. You have
> to register with the network, attach to GPRS etc. before any of this
> becomes really useful. If the the USB network interface does automatic
> connect and does tethering then it is not a WWAN device. Then it is an
> Ethernet device.

I can live with that defintion of the difference between a WWAN/3G
device (wwan%d) and a USB Ethernet device (usb%d).

But I still don't see why this doesn't make a phone (supporting the same
commands as a 3G card) a WWAN device.  A phone can't do an automatic
connection any more than a WWAN card can.  Both *must* be configured
with at least one PDP context, register with the network, attach to GPRS
etc.

Yes, some phones can be configured to auto-connect using it's own
UI. But there's really nothing preventing anyone from implementing the
same feature for a WWAN card.  What would that make that card then?

> And that you can use your phone via a TTY and configure a second PDP
> context and then run PPP has nothing to do with its network device.

I was talking about the network device.  

My experience with these devices is limited to the Ericsson F3507g, but
I assume that many of them will behave identically.  It allows you to
define multiple PDP contexts and select which one you connect with,
either you use PPP or the network device.  The list of defined contexts
are in fact shared.

>> So I'm still trying to figure out what makes a WWAN device special wrt
>> the kernel.  Thanks for explaining.
>
> It has nothing to do with the kernel. It classifies the network device
> type for userspace. Like you classify /dev/sda as "disk" and /dev/sda1
> as "partition".
>
> So please modify your patch as outlined in my first response. It should
> take you only like 5 minutes to do so. Then your phone will show up and
> gets a proper classification.

Sorry for the confusion, but the patch was not mine.  I just stumbled
across the discussion and first wondered why the heck the mbm_info stuff
was added (hadn't noticed before) and then why the heck this device
should be treated differenctly if it shows up as a similar one.



Bjørn

  reply	other threads:[~2010-01-05 12:47 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-12-23 20:56 [PATCH] net: cdc_ether.c: Add SE J105i to device whitelist Thomas Loo
2009-12-23 21:48 ` Marcel Holtmann
2010-01-04 14:50   ` Bjørn Mork
2010-01-04 21:10     ` Marcel Holtmann
2010-01-05  7:34       ` Bjørn Mork
2010-01-05 11:31         ` Marcel Holtmann
2010-01-05 12:46           ` Bjørn Mork [this message]
2010-01-05 20:52             ` Marcel Holtmann
2010-01-07  6:38             ` Dan Williams
2010-01-07  8:50               ` Bjørn Mork
2010-01-07 16:21                 ` Dan Williams

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=87eim4vhvh.fsf@nemi.mork.no \
    --to=bjorn@mork$(echo .)no \
    --cc=marcel@holtmann$(echo .)org \
    --cc=netdev@vger$(echo .)kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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