All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marcel Holtmann <marcel@holtmann.org>
To: ofono@ofono.org
Subject: RE: [PATCH 0/5] Call Counters (2nd)
Date: Wed, 08 Dec 2010 23:35:31 +0100	[thread overview]
Message-ID: <1291847731.4795.204.camel@aeonflux> (raw)
In-Reply-To: <725063E4C37388418CDDC5A2C287906A20D08A19F8@NOK-EUMSG-03.mgdnok.nokia.com>

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

Hi Kai,

> > So I am actually thinking that doing that inside PulseAudio is a lot
> > more efficient solution.
> > 
> > The idea is that PA already runs in the user session and has to monitor
> > the uplink/downlink state (and additionally could monitor call states
> > as well if needed). So it knows when a call is active and it is active
> > anyway doing the audio processing. So it could just then go ahead and
> > write your call accounting into Tracker.
> 
> Pulseaudio (or the meego PA modules for voice) isn't really an option 
> for at least the following reasons:
>  - lots of modems still handle all audio and PA will know nothing
>    about calls at all with these modems

actually Denis raised the same question, but then again this is for a
product specific requirement. And for that product we know that PA will
do the audio processing.

>  - PA does not really care about individual calls, but whether 
>    the audio traffic channel is connected or not
> 	- multi-call cases, cases were traffic channel is reconnected 
>         mid-call, etc, etc, PA simply does not know enough (and should
>         not)

And that is my point (see other email). This should be tracking talk
time and not individual call time. And thus it would be fitting nicely
into PA in that case.

> I guess this could be done in higher layers on top of oFono (telepathy-ring,
> dialer app, policy framework, etc), but I don't really know enough about
> the call timer stuff to comment on the pro/cons of this.

Important for me is to weight the extra costs of IO and CPU consumption
when doing this inside oFono. And since ofonod itself would actually not
wake up during a phone call, I think this is not a good fit.

It could be that PA is not a good candidate either, but to me it looks
like a pretty smart place to handle this.

Regards

Marcel



  reply	other threads:[~2010-12-08 22:35 UTC|newest]

Thread overview: 29+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-22 16:25 [PATCH 0/5] Call Counters (2nd) Andras Domokos
2010-11-22 16:25 ` [RFC PATCH 1/5] history: expand history API include file Andras Domokos
2010-11-22 16:26 ` [RFC PATCH 2/5] history: expand history API Andras Domokos
2010-11-22 16:26 ` [RFC PATCH 3/5] voicecall: take into use the new history function Andras Domokos
2010-11-22 16:26 ` [RFC PATCH 4/5] plugins: add call counters Andras Domokos
2010-11-22 16:26 ` [RFC PATCH 5/5] doc: call counters API doc Andras Domokos
2010-12-07 12:57 ` [PATCH 0/5] Call Counters (2nd) Aki Niemi
2010-12-07 18:33   ` Denis Kenzior
2010-12-08  9:03   ` Marcel Holtmann
2010-12-08  9:34     ` Aki Niemi
2010-12-08 12:41       ` Marcel Holtmann
2010-12-08 13:28         ` Mika.Liljeberg
2010-12-08 16:01         ` Andras Domokos
2010-12-08 16:12           ` Marcel Holtmann
2010-12-08 17:30             ` Andras Domokos
2010-12-08 22:32               ` Marcel Holtmann
2010-12-08 21:02             ` Kai.Vehmanen
2010-12-08 22:35               ` Marcel Holtmann [this message]
2010-12-09 12:25                 ` Kai.Vehmanen
2010-12-09 18:10                   ` Marcel Holtmann
2010-12-09 19:52                     ` Kai.Vehmanen
2010-12-09 22:53                       ` Marcel Holtmann
2010-12-10  8:14                         ` Andras Domokos
2010-12-10 17:57                         ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2010-12-10 18:34                           ` Marcel Holtmann
2010-12-10 23:57                             ` =?unknown-8bit?q?R=C3=A9mi?= Denis-Courmont
2010-12-11 14:57                               ` Marcel Holtmann
2010-12-13  7:56                               ` Kai.Vehmanen
2010-12-12 15:46                         ` Kai.Vehmanen

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=1291847731.4795.204.camel@aeonflux \
    --to=marcel@holtmann.org \
    --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.