From patchwork Mon Aug 24 09:37:17 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: chunbo.luo@windriver.com X-Patchwork-Id: 31897 X-Patchwork-Delegate: davem@davemloft.net Return-Path: X-Original-To: patchwork-incoming@bilbo.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from ozlabs.org (ozlabs.org [203.10.76.45]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "mx.ozlabs.org", Issuer "CA Cert Signing Authority" (verified OK)) by bilbo.ozlabs.org (Postfix) with ESMTPS id 98809B7B69 for ; Mon, 24 Aug 2009 19:37:30 +1000 (EST) Received: by ozlabs.org (Postfix) id 8C60ADDD0B; Mon, 24 Aug 2009 19:37:30 +1000 (EST) Delivered-To: patchwork-incoming@ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.176.167]) by ozlabs.org (Postfix) with ESMTP id 289E9DDD04 for ; Mon, 24 Aug 2009 19:37:30 +1000 (EST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751862AbZHXJhU (ORCPT ); Mon, 24 Aug 2009 05:37:20 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751665AbZHXJhU (ORCPT ); Mon, 24 Aug 2009 05:37:20 -0400 Received: from mail.windriver.com ([147.11.1.11]:60859 "EHLO mail.windriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751680AbZHXJhT (ORCPT ); Mon, 24 Aug 2009 05:37:19 -0400 Received: from localhost.localdomain (pek-lpgbuild6.wrs.com [128.224.153.36]) by mail.windriver.com (8.14.3/8.14.3) with ESMTP id n7O9bIYY003941; Mon, 24 Aug 2009 02:37:19 -0700 (PDT) From: Chunbo Luo To: davem@davemloft.net Cc: netdev@vger.kernel.org, linux-sctp@vger.kernel.org, linux-kernel@vger.kernel.org, chunbo.luo@windriver.com Subject: [PATCH v2] sctp: fix the check for path failure detection Date: Mon, 24 Aug 2009 17:37:17 +0800 Message-Id: <1251106637-434-1-git-send-email-chunbo.luo@windriver.com> X-Mailer: git-send-email 1.6.4 Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org The transport error count should be incremented when an outstanding HB is not acknowledged. And the path failure detection should be done before sending out the HB. Signed-off-by: Chunbo Luo --- net/sctp/sm_sideeffect.c | 6 +++++- net/sctp/sm_statefuns.c | 8 ++++---- 2 files changed, 9 insertions(+), 5 deletions(-) diff --git a/net/sctp/sm_sideeffect.c b/net/sctp/sm_sideeffect.c index 86426aa..fbdf4de 100644 --- a/net/sctp/sm_sideeffect.c +++ b/net/sctp/sm_sideeffect.c @@ -446,7 +446,11 @@ static void sctp_do_8_2_transport_strike(struct sctp_association *asoc, if (transport->state != SCTP_UNCONFIRMED) asoc->overall_error_count++; - if (transport->state != SCTP_INACTIVE && + /* + * The transport error count is incremented when an outstanding HB + * is not acknowledged. + */ + if (transport->hb_sent && transport->state != SCTP_INACTIVE && (transport->error_count++ >= transport->pathmaxrxt)) { SCTP_DEBUG_PRINTK_IPADDR("transport_strike:association %p", " transport IP: port:%d failed.\n", diff --git a/net/sctp/sm_statefuns.c b/net/sctp/sm_statefuns.c index 7288192..7f77099 100644 --- a/net/sctp/sm_statefuns.c +++ b/net/sctp/sm_statefuns.c @@ -981,10 +981,6 @@ sctp_disposition_t sctp_sf_sendbeat_8_3(const struct sctp_endpoint *ep, */ if (transport->param_flags & SPP_HB_ENABLE) { - if (SCTP_DISPOSITION_NOMEM == - sctp_sf_heartbeat(ep, asoc, type, arg, - commands)) - return SCTP_DISPOSITION_NOMEM; /* Set transport error counter and association error counter * when sending heartbeat. */ @@ -992,6 +988,10 @@ sctp_disposition_t sctp_sf_sendbeat_8_3(const struct sctp_endpoint *ep, SCTP_TRANSPORT(transport)); sctp_add_cmd_sf(commands, SCTP_CMD_TRANSPORT_HB_SENT, SCTP_TRANSPORT(transport)); + if (SCTP_DISPOSITION_NOMEM == + sctp_sf_heartbeat(ep, asoc, type, arg, + commands)) + return SCTP_DISPOSITION_NOMEM; } sctp_add_cmd_sf(commands, SCTP_CMD_HB_TIMER_UPDATE, SCTP_TRANSPORT(transport));