All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: GPRS support for Ofono
Date: Wed, 02 Sep 2009 10:00:22 -0500	[thread overview]
Message-ID: <200909021000.23069.denkenz@gmail.com> (raw)
In-Reply-To: <a4e57c0e0909020430k5affae8ex85949269c4730a57@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 3851 bytes --]

Hi Ismo,

> This is a philosophical difference from the provisioning point of
> view. An operator will likely support multiple configured access
> points, and the provisioning program should know their purpose from
> the configuration file (internet/MMS/WAP/...). This metadata is not
> there in the Ofono context, meaning that the provisioning program
> needs to store somewhere else the mapping between Ofono context object
> paths and the neccessary metadata. This begs the question: why not
> store all connection data (APN, authentication data, other metadata)
> to this external store in the first place?

Actually this one is missing from the API proposal.  Marcel already wanted the 
context type (internet, mms, wap, etc) information.  I've updated the proposed 
API in git.

As discussed previously, we want oFono to manage this data, since it can do 
this by using the IMSI.  So if you insert a different operator SIM your APN 
settings are magically updated for that operator.  

> In my proposal the "Status" variable was a three-state variable,
> having the states "attached", "detached" and "suspended". "Suspended"
> was specified to mean that the GPRS was attached, but temporarily not
> available (for instance because of a 2G phone call or temporary loss
> of cellular connectivity). In the IRC discussion someone said that
> this property would not be part of Denis's API because the tri-state
> variables were bad and the "suspended" status was not supported by

So the general rule is that if a feature isn't explicitly allowed in 27.007, 
we do not support it.  There are exceptions to this of course.  27.007 only 
supports two states for context: activated and deactivated.

> spec 27.007. However, the problem is that Maemo 5 already has UI
> support for indicating that the connection is suspended, and that
> makes it harder to drop the feature. Could the "suspended" status enum
> still be left there, and just have the modems that don't support the
> feature to never go to "suspended" mode?

I'd rather not.

If this feature is really required, then some sort of heuristic can be 
applied.  (e.g. Powered on, Attached on, but context is deactivated most 
likely means temporary unavailability of resources)

>
> Still one difference to my proposal was the dropping of the TX and RX
> byte counts, and the explanation that those statistics would be
> handled by the upper layers by reading the values from the network
> interface. I mentioned the problem where the connection was terminated
> by the network and the network interface was brought down before the
> upper layer had a chance to read the values. After giving the matter
> some thought, I still feel that for this reason the modem driver would
> need to get the TX/RX data and send it to the upper levels, if at all
> possible. Since many operators charge by the amount of data
> transferred, not losing the TX/RX data is quite crucial. At least
> Nokia phones have a GPRS data counter feature, that is supposed to
> show the transferred data. This said, I know that not all modem
> drivers can get the data from a connection that has been terminated by
> the network, but that does not mean that the modem drivers that can
> get the data should just discard it.

This really belong in the kernel.  Only the kernel can reliably know when a 
network interface has been brought down and notify the interested applications 
with the statistics.  Nokia hardware supports this explicitly, but many others 
don't.  I don't want nasty kludges inside oFono to enable this.

>
> Btw, is the context API missing the PDP type (IPv4 or IPv6)?

I left this out explicitly since I've never seen anything besides "IP" being 
used.  But we can add this easily enough if there's need.

Regards,
-Denis

  parent reply	other threads:[~2009-09-02 15:00 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-01 11:09 GPRS support for Ofono Ismo Puustinen
2009-09-01 19:02 ` Jean-Christian de Rivaz
2009-09-01 19:25   ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2009-09-01 20:17     ` Jean-Christian de Rivaz
2009-09-01 20:26       ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2009-09-01 20:30       ` Christensen, Mikkel
2009-09-01 19:27   ` Christensen, Mikkel
2009-09-01 21:36 ` Denis Kenzior
2009-09-01 22:42   ` Marcel Holtmann
2009-09-01 22:50     ` Denis Kenzior
2009-09-02  6:39     ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2009-09-02  9:16       ` Marcel Holtmann
2009-09-02  9:22         ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2009-09-02 10:43           ` Aki Niemi
2009-09-02 11:03             ` Marcel Holtmann
2009-09-02 11:19               ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2009-09-02 11:30   ` Ismo Puustinen
2009-09-02 12:02     ` Marcel Holtmann
2009-09-02 12:34       ` Aki Niemi
2009-09-02 12:46         ` Marcel Holtmann
2009-09-02 12:51           ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2009-09-02 15:28         ` Denis Kenzior
2009-09-02 15:42           ` Aki Niemi
2009-09-02 20:37             ` Marcel Holtmann
2009-09-02 20:36               ` Denis Kenzior
2009-09-02 21:09                 ` Marcel Holtmann
2009-09-02 12:46       ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2009-09-02 13:01         ` Marcel Holtmann
2009-09-02 17:51           ` Bastian, Waldo
2009-09-02 20:40             ` Marcel Holtmann
2009-09-02 15:00     ` Denis Kenzior [this message]
2009-09-02 15:32       ` Aki Niemi
2009-09-02 15:36         ` Denis Kenzior
2009-09-02 15:38       ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2009-09-02 16:26         ` Denis Kenzior
2009-09-02 17:39           ` Bastian, Waldo
2009-09-02 17:46             ` Denis Kenzior
2009-09-02 18:41               ` Bastian, Waldo
2009-09-02 21:01                 ` Marcel Holtmann
2009-09-02 21:10                   ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2009-09-02 21:18                     ` Marcel Holtmann
2009-09-02 20:53         ` Marcel Holtmann

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=200909021000.23069.denkenz@gmail.com \
    --to=denkenz@gmail.com \
    --cc=ofono@ofono.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.