From patchwork Mon Jul 16 18:18:00 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Colin Cross X-Patchwork-Id: 1201431 Return-Path: X-Original-To: patchwork-linux-arm@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork2.kernel.org Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) by patchwork2.kernel.org (Postfix) with ESMTP id 77AA5E0038 for ; Mon, 16 Jul 2012 18:25:55 +0000 (UTC) Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.76 #1 (Red Hat Linux)) id 1SqpsZ-0007k1-VP; Mon, 16 Jul 2012 18:18:44 +0000 Received: from mail-ey0-f201.google.com ([209.85.215.201]) by merlin.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1Sqps8-0007jR-Bh for linux-arm-kernel@lists.infradead.org; Mon, 16 Jul 2012 18:18:17 +0000 Received: by eaai13 with SMTP id i13so317635eaa.0 for ; Mon, 16 Jul 2012 11:18:03 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=from:to:cc:subject:date:message-id:x-mailer:x-gm-message-state; bh=RRkvYSL0Odn+joh0d7QLhM2q73QYPxpldupZxl1Z664=; b=iepomkR+Ca+92/ct37BDNozcML1gHgda6AJtHWXuCCh6APv2I8YkB4eeE7zj4IxtQg kvnJpyk/hXwq2eYre+8F9ir1m8aE5OirTFz2tZRK0Q2OQq85pX0ZrqYB0XrzpwZQU4Q6 mjummRmO2Z4bB/o9K5SgdUzcvrbKswhBtaugmg6VbZxHBjGsfre6qKGwXbw0OO8kF9a4 juQ7Y6FxZ7tzF+G9dmuK5RuJ281oCc9Hd6Dr/vr/Yvt8c7YeOf0PWR8S4SPW6SCA5Cla /OmfCtBOU8XKL5lE5GVN/HN1UDrPBNMr0LHxcfrJBlvXHbkUbkRMdT1DkwcTrQUuESQo Cjgw== Received: by 10.14.211.196 with SMTP id w44mr5521843eeo.0.1342462683668; Mon, 16 Jul 2012 11:18:03 -0700 (PDT) Received: by 10.14.211.196 with SMTP id w44mr5521833eeo.0.1342462683603; Mon, 16 Jul 2012 11:18:03 -0700 (PDT) Received: from hpza10.eem.corp.google.com ([74.125.121.33]) by gmr-mx.google.com with ESMTPS id s44si9116570eeo.0.2012.07.16.11.18.03 (version=TLSv1/SSLv3 cipher=AES128-SHA); Mon, 16 Jul 2012 11:18:03 -0700 (PDT) Received: from walnut.mtv.corp.google.com (walnut.mtv.corp.google.com [172.18.104.116]) by hpza10.eem.corp.google.com (Postfix) with ESMTP id 6867C20004E; Mon, 16 Jul 2012 11:18:03 -0700 (PDT) Received: by walnut.mtv.corp.google.com (Postfix, from userid 99897) id D6D30257A1E; Mon, 16 Jul 2012 11:18:02 -0700 (PDT) From: Colin Cross To: linux-arm-kernel@lists.infradead.org Subject: [PATCH] ARM: vfp: only clear vfp state for current cpu in vfp_pm_suspend Date: Mon, 16 Jul 2012 11:18:00 -0700 Message-Id: <1342462680-18548-1-git-send-email-ccross@android.com> X-Mailer: git-send-email 1.7.7.3 X-Gm-Message-State: ALoCoQmBcJlUUxpAB7TwZ67dXILpJnN+i5tHIKqDYlrIi1fHHneQJg1t5H1+ORuRzr/LZXFCU40trxII2ox2bnIJp7M7b5dAxIp3KbnnZC4XGM7hlrSIaAo+D4pBk0rsCm8Q+j4nlZdcicZzriPh4E/1B+j26rmdHWwavvxtpVT2RK8xO8R+Jru32EK0yKQwwpLwKI95P7Xh3PHbaj7a5zek1H+UAwElfA== X-Spam-Note: CRM114 invocation failed X-Spam-Score: -2.6 (--) X-Spam-Report: SpamAssassin version 3.3.2 on merlin.infradead.org summary: Content analysis details: (-2.6 points) pts rule name description ---- ---------------------- -------------------------------------------------- -0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at http://www.dnswl.org/, low trust [209.85.215.201 listed in list.dnswl.org] -0.0 T_RP_MATCHES_RCVD Envelope sender domain matches handover relay domain -0.0 SPF_PASS SPF: sender matches SPF record -1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1% [score: 0.0000] Cc: Nicolas Pitre , Russell King , Catalin Marinas , Will Deacon , linux-kernel@vger.kernel.org, Colin Cross X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Sender: linux-arm-kernel-bounces@lists.infradead.org Errors-To: linux-arm-kernel-bounces+patchwork-linux-arm=patchwork.kernel.org@lists.infradead.org vfp_pm_suspend runs on each cpu, only clear the hardware state pointer for the current cpu. Prevents a possible crash if one cpu clears the hw state pointer when another cpu has already checked if it is valid. Signed-off-by: Colin Cross --- arch/arm/vfp/vfpmodule.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/arm/vfp/vfpmodule.c b/arch/arm/vfp/vfpmodule.c index ce55f05..fb849d0 100644 --- a/arch/arm/vfp/vfpmodule.c +++ b/arch/arm/vfp/vfpmodule.c @@ -466,7 +466,7 @@ static int vfp_pm_suspend(void) } /* clear any information we had about last context state */ - memset(vfp_current_hw_state, 0, sizeof(vfp_current_hw_state)); + vfp_current_hw_state[ti->cpu] = NULL; return 0; }