[PATCH 0/1] Limits the send sms retry to network timeout

[PATCH 0/1] Limits the send sms retry to network timeout

[Jeevaka Badrappan]

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

Hi,

  If sending of SMS fails, then oFono core will try to resend it after
n * 5 seconds(n = retry count). Due to this, conformance test case
27.22.8 sequence 1.4 fails(refer 31.124 spec). Plan is to limit the
failue codes for which the retry will be done. This patch allows the
retry if the failure is only due to network timeout. For all the other
failure cases, retry is not done.

Regards,
Jeevaka

Jeevaka Badrappan (1):
  sms: limit the sms sending retry

 src/sms.c |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

[PATCH 1/1] sms: limit the sms sending retry

[Jeevaka Badrappan]

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

---
 src/sms.c |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

diff --git a/src/sms.c b/src/sms.c
index 01f54dd..da810f8 100644
--- a/src/sms.c
+++ b/src/sms.c
@@ -47,6 +47,7 @@
 #define SETTINGS_GROUP "Settings"
 
 #define TXQ_MAX_RETRIES 4
+#define NETWORK_TIMEOUT 332
 
 static gboolean tx_next(gpointer user_data);
 
@@ -630,6 +631,11 @@ static void tx_finished(const struct ofono_error *error, int mr, void *data)
 		if (sms->registered == FALSE)
 			return;
 
+		/* Retry done only for Network Timeout failure */
+		if (error->type == OFONO_ERROR_TYPE_CMS &&
+				error->error != NETWORK_TIMEOUT)
+			goto next_q;
+
 		if (!(entry->flags & OFONO_SMS_SUBMIT_FLAG_RETRY))
 			goto next_q;
 
-- 
1.7.0.4

Re: [PATCH 0/1] Limits the send sms retry to network timeout

[Denis Kenzior]

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

Hi Jeevaka,

On 03/17/2011 12:20 PM, Jeevaka Badrappan wrote:
> Hi,
> 
>   If sending of SMS fails, then oFono core will try to resend it after
> n * 5 seconds(n = retry count). Due to this, conformance test case
> 27.22.8 sequence 1.4 fails(refer 31.124 spec). Plan is to limit the
> failue codes for which the retry will be done. This patch allows the
> retry if the failure is only due to network timeout. For all the other
> failure cases, retry is not done.

Patch has been applied, thanks.

Regards,
-Denis

[PATCH 0/1] Limits the send sms retry to network timeout

[Jeevaka Badrappan]

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

Hi,

  If sending of SMS fails, then oFono core will try to resend it after
n * 5 seconds(n = retry count). Due to this, conformance test case
27.22.8 sequence 1.4 fails(refer 31.124 spec). Plan is to limit the
failue codes for which the retry will be done. This patch allows the
retry if the failure is only due to network timeout. For all the other
failure cases, retry is not done.

Regards,
Jeevaka

Jeevaka Badrappan (1):
  sms: limit the sms sending retry

 src/sms.c |    5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)