All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: Ofono CF states not always correct
Date: Tue, 18 Jan 2011 15:50:48 -0600	[thread overview]
Message-ID: <4D360B38.9030107@gmail.com> (raw)
In-Reply-To: <AANLkTinAG0XXmRQXTMuH-ES6VZ7YatP6r_UenLBEfERg@mail.gmail.com>

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

Hi Pekka,

On 01/18/2011 03:32 PM, Pekka Pessi wrote:
> Hi Denis,
> 
> 2011/1/18 Denis Kenzior <denkenz@gmail.com>:
>> I think we can track the cached flag for conditional CFs independently
>> from CFU.  The question is really whether the networks allow us to
>> modify conditional CFs when CFU is active.  In my testing the answer is No.
> 
> You can erase (##67#) or deactivate (#67#) them.
> 

So CFB active.

Activate CFU
Query CFB -> Inactive

Deregister CFB -> OK
Query CFB -> Inactive

Register CFB -> Error

Deactivate CFU
Query CFB -> Inactive

Correct?

Jeez, who thought this stuff up.

So my thinking is that we should:
- Not update conditional cfs when cfu is active (whether via
GetProperties or mmi)
- only clear the conditional cache flag if a conditional operation is
attempted and succeeds while cfu is active
- Report conditional cfs as empty when CFU is active (and emit signals
appropriately when cfu is activated / deactivated)
- Don't bother running conditional cf queries via GetProperties if cfu
is active

Do you see any issues with this?  I'm really trying to minimize the
number of queries we have to do in the most common case (which is
activating / deactivating cfu via settings)

Regards,
-Denis

  reply	other threads:[~2011-01-18 21:50 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-11 10:00 Ofono CF states not always correct Jarko Poutiainen
2011-01-11 15:16 ` Denis Kenzior
2011-01-14 13:20   ` Jarko Poutiainen
2011-01-14 15:59     ` Denis Kenzior
2011-01-18 12:21       ` Jarko Poutiainen
2011-01-18 15:56         ` Denis Kenzior
2011-01-18 21:08           ` Pekka Pessi
2011-01-18 21:24             ` Denis Kenzior
2011-01-18 21:32               ` Pekka Pessi
2011-01-18 21:50                 ` Denis Kenzior [this message]
2011-01-18 22:08                   ` Pekka Pessi
2011-01-26  9:14                   ` Jarko Poutiainen
2011-01-26 19:10                     ` Denis Kenzior

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=4D360B38.9030107@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.