All of lore.kernel.org
 help / color / mirror / Atom feed
From: Inaky Perez-Gonzalez <inaky@linux.intel.com>
To: ofono@ofono.org
Subject: [patch 00/20] SMS D-Bus support and misc small patches
Date: Fri, 23 Jul 2010 13:59:48 -0700	[thread overview]
Message-ID: <cover.1279918330.git.inaky.perez-gonzalez@intel.com> (raw)

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

From: Inaky Perez-Gonzalez <inaky.perez-gonzalez@intel.com>

This (3rd? 4th?) version of the patchset builds the D-Bus support on
top of the new _txq_submit() callback mechanism.

Note there are still a couple of opens that need discussion:

 - the message ID is generated based on the contents of the message --
   thus, the current way doesn't work. We need the caller to
   _txq_submit() to generate it. It's been left out of the STK
   stc.c:handle_command_send_sms() because I am not sure what is the
   right way to do it -- need feedback on that.

 - The generation of the SMS message ID based on contents still has
   shortcomings: if we submit two messages with the same content and
   destination number, the ID is the same [sms.c:sms_msg_send()]. What
   other factor would make sense to add? time?

The following changes since commit 94344e967b4cd3edd65aa5254ef4b4f5dd037e69:
  Denis Kenzior (1):
        TODO: Major updates to STK related tasks

are available in the git repository at:

  git://gitorious.org/~inakypg/ofono/ofono-inakypg.git master

Patches follow for reviewing convenience.

Inaky Perez-Gonzalez (20):
      bug.h: Add BUILD_BUG_ON() and friends for compile-time assert checking
      write_file: make transaction-safe
      manpage: explain debugging options to -d
      SMS: introduce message ID API
      introduce DECLARE_SMS_ADDR_STR()
      _assembly_encode_address: export and rename
      SMS: implement SHA256-based message IDs [incomplete]
      sms: document the org.ofono.SMSMessage D-Bus interface
      SMS: document handle_sms_status_report()
      sms_text_prepare: document @use_delivery_reports
      SMS: rename create_tx_queue_entry() to tx_queue_entry_new()
      struct tx_queue_entry: add a destructor
      SMS: encapsulate D-Bus specific data in 'struct sms_msg_dbus_data'
      SMS: introduce bare state machine and transitions
      SMS: introduce Wait-for-Status-Report state and infrastructure
      SMS: introduce a state change callback for TX messages
      SMS: export outgoing messages over D-Bus
      SMS: send D-Bus SMS-MSG::PropertyChanged signals when message changes status
      SMS: introduce sms_msg_cancel and its D-Bus wrapper
      SMS: Implement D-Bus SMS-MSG::GetProperties

 HACKING                        |   10 +
 Makefile.am                    |    5 +-
 doc/ofonod.8                   |    5 +-
 doc/sms-api.txt                |   49 ++++-
 src/bug.h                      |   50 ++++
 src/ofono.h                    |   42 +++-
 src/sms.c                      |  598 ++++++++++++++++++++++++++++++++++------
 src/smsutil.c                  |  206 ++++++++++++++-
 src/smsutil.h                  |  122 ++++++++
 src/stk.c                      |   24 ++-
 src/storage.c                  |   42 ++-
 test/test-sms-msg-cancel       |  173 ++++++++++++
 test/test-sms-msg-state-change |   24 ++
 unit/test-sms-msg-id.c         |  212 ++++++++++++++
 14 files changed, 1449 insertions(+), 113 deletions(-)
 create mode 100644 src/bug.h
 create mode 100755 test/test-sms-msg-cancel
 create mode 100755 test/test-sms-msg-state-change
 create mode 100644 unit/test-sms-msg-id.c

             reply	other threads:[~2010-07-23 20:59 UTC|newest]

Thread overview: 54+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-07-23 20:59 Inaky Perez-Gonzalez [this message]
2010-07-23 20:59 ` [patch 01/20] bug.h: Add BUILD_BUG_ON() and friends for compile-time assert checking Inaky Perez-Gonzalez
2010-07-23 21:41   ` Denis Kenzior
2010-07-23 21:57     ` Inaky Perez-Gonzalez
2010-07-23 21:59       ` Denis Kenzior
2010-07-23 20:59 ` [patch 02/20] write_file: make transaction-safe Inaky Perez-Gonzalez
2010-07-23 21:57   ` Denis Kenzior
2010-07-23 22:31     ` Inaky Perez-Gonzalez
2010-07-23 20:59 ` [patch 03/20] manpage: explain debugging options to -d Inaky Perez-Gonzalez
2010-07-23 22:05   ` Denis Kenzior
2010-07-23 20:59 ` [patch 04/20] SMS: introduce message ID API Inaky Perez-Gonzalez
2010-07-27  0:10   ` Denis Kenzior
2010-07-23 20:59 ` [patch 05/20] introduce DECLARE_SMS_ADDR_STR() Inaky Perez-Gonzalez
2010-07-23 22:30   ` Denis Kenzior
2010-07-23 20:59 ` [patch 06/20] _assembly_encode_address: export and rename Inaky Perez-Gonzalez
2010-07-23 22:31   ` Denis Kenzior
2010-07-23 20:59 ` [patch 07/20] SMS: implement SHA256-based message IDs [incomplete] Inaky Perez-Gonzalez
2010-07-27 17:03   ` Denis Kenzior
2010-07-29 21:26     ` Inaky Perez-Gonzalez
2010-07-29 21:37       ` Denis Kenzior
2010-07-31  0:22         ` Inaky Perez-Gonzalez
2010-07-23 20:59 ` [patch 08/20] sms: document the org.ofono.SMSMessage D-Bus interface Inaky Perez-Gonzalez
2010-07-23 23:11   ` Denis Kenzior
2010-07-26 17:19     ` Inaky Perez-Gonzalez
2010-07-26 18:05       ` Denis Kenzior
2010-07-26 20:41         ` Inaky Perez-Gonzalez
2010-07-23 20:59 ` [patch 09/20] SMS: document handle_sms_status_report() Inaky Perez-Gonzalez
2010-07-23 20:59 ` [patch 10/20] sms_text_prepare: document @use_delivery_reports Inaky Perez-Gonzalez
2010-07-23 23:01   ` Denis Kenzior
2010-07-23 20:59 ` [patch 11/20] SMS: rename create_tx_queue_entry() to tx_queue_entry_new() Inaky Perez-Gonzalez
2010-07-23 23:02   ` Denis Kenzior
2010-07-26 20:49     ` Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 12/20] struct tx_queue_entry: add a destructor Inaky Perez-Gonzalez
2010-07-23 23:06   ` Denis Kenzior
2010-07-23 23:11     ` Inaky Perez-Gonzalez
2010-07-23 23:14       ` Denis Kenzior
2010-07-26 18:48         ` Inaky Perez-Gonzalez
2010-07-26 20:49     ` Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 13/20] SMS: encapsulate D-Bus specific data in 'struct sms_msg_dbus_data' Inaky Perez-Gonzalez
2010-07-27 17:08   ` Denis Kenzior
2010-07-29 21:47     ` Inaky Perez-Gonzalez
2010-07-29 22:17       ` Denis Kenzior
2010-07-29 23:23         ` Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 14/20] SMS: introduce bare state machine and transitions Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 15/20] SMS: introduce Wait-for-Status-Report state and infrastructure Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 16/20] SMS: introduce a state change callback for TX messages Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 17/20] SMS: export outgoing messages over D-Bus Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 18/20] SMS: send D-Bus SMS-MSG::PropertyChanged signals when message changes status Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 19/20] SMS: introduce sms_msg_cancel and its D-Bus wrapper Inaky Perez-Gonzalez
2010-07-27 17:16   ` Denis Kenzior
2010-07-30 23:12     ` Inaky Perez-Gonzalez
2010-07-23 21:00 ` [patch 20/20] SMS: Implement D-Bus SMS-MSG::GetProperties Inaky Perez-Gonzalez
2010-07-27 17:18   ` Denis Kenzior
2010-08-02 19:14     ` Inaky Perez-Gonzalez

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=cover.1279918330.git.inaky.perez-gonzalez@intel.com \
    --to=inaky@linux.intel.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.