All of lore.kernel.org
 help / color / mirror / Atom feed
From: =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont <remi.denis-courmont@nokia.com>
To: ofono@ofono.org
Subject: Re: [RFCv2] doc: Proposal for LTE/IMS API
Date: Wed, 09 Feb 2011 09:02:43 +0200	[thread overview]
Message-ID: <201102090902.43928.remi.denis-courmont@nokia.com> (raw)
In-Reply-To: <AANLkTi=Jt0w-UqJ8MtqDae83A6U6baAbzhh6YtR5+mc5@mail.gmail.com>

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

On Tuesday 08 February 2011 18:22:43 ext Sjur Brændeland, you wrote:
> > > +             boolean PreConditionCheck(string Type, string
> > > PeerAddress, +                                     uint16 PeerPort,
> > >  uint16 LocalPort) +
> > 
> > That stuff is per context. Should it not be in the context object rather
> > than in the IMS manager?
> 
> oFono should only consider QoS information from the IMS
> Default and Dedicated bearers when checking the precondition.
> 
> In theory there might be more than one IMS APN for a operator,
> but I really don't see this as a real life scenario for this.
> If anyone disagrees please speak out and explain...

Well, I don't care if it is part of the normal context interface, or if it is 
an extra IMS-specific interface on the IMS context object path. But this is 
clearly a per-context thing so it should be in the context.

I don't want to end up in a 'woops' situation down the line. And I don't trust 
operators not to actually implement those useless in real life scenarii...

> > > +             array{string} PcscfAddresses [readonly]
> > 
> > Should this be in the context object? The AT command is per CID. Would it
> > make any sense for a network to have different P-CSCF based on the
> > bearer?
> 
> As mentioned above, I don't see why an operator would ever have more than
> one IMS APN. If we only have one IMS APN, then we might as well present
> this information here in the IMS API.

How do you cope with two connections to the same APN?

-- 
Rémi Denis-Courmont
Nokia Devices R&D, Maemo Software, Helsinki

  reply	other threads:[~2011-02-09  7:02 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-08 14:29 [RFCv2] doc: Proposal for LTE/IMS API Sjur =?unknown-8bit?q?Br=C3=A6ndeland?=
2011-02-08 14:47 ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2011-02-08 16:22   ` Sjur =?unknown-8bit?q?Br=C3=A6ndeland?=
2011-02-09  7:02     ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont [this message]
2011-02-09 16:35       ` Sjur =?unknown-8bit?q?Br=C3=A6ndeland?=
2011-02-10  9:30   ` Kjetil ASDAL
2011-02-08 15:20 ` Soum, RedouaneX
2011-02-08 16:42   ` Sjur =?unknown-8bit?q?Br=C3=A6ndeland?=
2011-02-14 21:35 Fallon, Michael F
2011-02-14 21:39 ` 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=201102090902.43928.remi.denis-courmont@nokia.com \
    --to=remi.denis-courmont@nokia.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.