From patchwork Mon Apr 15 15:55:00 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 162201 Delivered-To: patch@linaro.org Received: by 2002:a02:c6d8:0:0:0:0:0 with SMTP id r24csp3092928jan; Mon, 15 Apr 2019 08:55:28 -0700 (PDT) X-Google-Smtp-Source: APXvYqzCWdMtV13OAbJPLCZiOtf0LaXvf9Y8soI47r49R46OLyrrL93YXeaQUYt6oEVSyVh6eIFb X-Received: by 2002:a17:902:1123:: with SMTP id d32mr56756876pla.213.1555343728275; Mon, 15 Apr 2019 08:55:28 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1555343728; cv=none; d=google.com; s=arc-20160816; b=omNl5QgIbQRZeLL9KaThc2IiFTHVV6ZWCI1Q5wQwNadhUrYDsgnD5d3komVBbBwWwz YWb5EEwA44vhM7BNRfXJ+wrh3sXvgn65emVfbZFOP8PL74kzSsDqlP7ExeP2rvyiiV6b lMrV54J4ogAuowoYdH6kGaV1ZV/RNorPam0l89EI9CLCfiaTTDmwnmFuCSiY3KajgmNC IxsLcMIYMNuPbiBoCiemM51uKpJllGKUuMV6es3GyuLT73Hlll0F7qKUXbx9qQbgdBoP GR/PtFmAzYkhJ17bpK8/FnrvPSyBwjG70AeniM84En5fK7kCry67HuIe1XdZ+mcZPxSE hxfw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=K0bLlSvpMBgNLolXdZqWZDzsBWOI2Z7/KBK7/lCsmM8=; b=YNkebN4Mx5Gz9YvFzRuKf0uPTHZAkVioODV8pRcDXfFKX2Ew8n52InP7SBHGuXClUx cvQMedaS32X2/W+IiPiSnz1M1Bh9enFeXHpavmONNZ560SfO9mMp1wr1n47/FX4Ok+t8 1L4UJpjnmWucOPludOfunsBf8QXpqrsCgkekIqdlE/kr7oRV/x6sVm3VsVf+GlXBEjxP 9vL5l9GRwm9nwq41GwSwhrGMAyFf4rJeGi04BDTlt3M46YnVCONVoswNpDSLU6Cw1Xe7 VUGYE/3e7WepkMWRXKI/bBV7v6u6uqDuYy4qPH2xpAL2Q0DyYTNsouMFz/nui33kFOpm fLLQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id z127si46850005pfb.254.2019.04.15.08.55.27; Mon, 15 Apr 2019 08:55:28 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727659AbfDOPz0 (ORCPT + 30 others); Mon, 15 Apr 2019 11:55:26 -0400 Received: from mout.kundenserver.de ([217.72.192.75]:57835 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725972AbfDOPzZ (ORCPT ); Mon, 15 Apr 2019 11:55:25 -0400 Received: from threadripper.lan ([149.172.19.189]) by mrelayeu.kundenserver.de (mreue107 [212.227.15.145]) with ESMTPA (Nemesis) id 1MYvLi-1hKVup0f3I-00Uonv; Mon, 15 Apr 2019 17:55:11 +0200 From: Arnd Bergmann To: Corey Minyard , Greg Kroah-Hartman Cc: Arnd Bergmann , Andy Shevchenko , openipmi-developer@lists.sourceforge.net, linux-kernel@vger.kernel.org Subject: [PATCH] ipmi: avoid atomic_inc in exit function Date: Mon, 15 Apr 2019 17:55:00 +0200 Message-Id: <20190415155509.3565087-1-arnd@arndb.de> X-Mailer: git-send-email 2.20.0 MIME-Version: 1.0 X-Provags-ID: V03:K1:yT5pRj1gEeKNHvX+QSSDe1YcoCPQ/mYdkbY06sbVOQPRdm3tdJR c4dsDP/p/bGLUbCpToY4coCkxWjyEGExWboznimdUtIQ2XrZeLu8JvOou0fC7jZfS+s9KQ9 LZIVNrUPgYHtCiU+914HS21/mjKoSQY62YQW1BXT8cTAkvmCqY3iE5nO/DF2gcE7F9KL8wJ TKmq8196/pridrG/t4UyQ== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1; V03:K0:AZSHc6nS8fI=:ZfSwKGZsGAAhSzusOlnvk/ qGtMhxHiDFOOYHnrH8QKwI6Izx16uMUlQeGWhdUHYMzb+XXL/ZDGNf9jijJGkoYJqbxr+LdfF l1H9n1kqzNSn20tRdp+K2jDhwqsPsbWHb4Ftbqt9AMXdtIIwDm0HB6mj9TbXzmyfnFrQlX1JE vQ3nVnwSKKinD7vmR9UQOvqTgc5vh1NzmYRmHu0A1xVBAolLdUg/uctv4qRUnGL1w7JumTeIQ 6y7BPb6jHAsGP475ZCMg1R+tmpxZtdrTD31d2ZaSrjm9pwZCOKt+eekhTewreZKh095yE4SI3 X5RtQdguB/Yrz201Ycu6j5o4huqz5oLaNLaWZ+Mu2RQWoFxD7dZLAKOj/xU6ZFtf2LOB+6GFH EPqLCzXMlEdCqhOaywvp30FIY2MGPhB0jkonaHAg+Bh2XBr9uhjCThD9EHm426uZmyi7VMXTy kRSx196fy/dkra9kI7fwPXawQIjAiuVFBR6NpzBTcOOJYd8TeDrh+fQUzzOw5Gtk/9kj4gHi2 SuWQLHU1ltRySpwf8Sx37MEeNrg5a6fN3FI4UMCgSKJSCzXY0LVj8/8EFU1rl5SeR4FoFdiP7 GPzV4KovxyHu/0q9LBG69csaCm98jQ53Cv2m1WdsFE4ErNd1MsxQQkorhetkP8erMJ76/FvKF rhZa2g25oiyodCV01m1xeTI1LBBBREonvd8PpSAXfD6/3q8CUZcrm5dus8ITpcDdERyTJwVn/ NjKFqVw1Uq4dKN30U+ZwNKtuCtWQds1fxhPjhQ== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This causes a link failure on ARM in certain configurations, when we reference each atomic operation from .alt.smp.init in order to patch out atomics on non-SMP systems: `.exit.text' referenced in section `.alt.smp.init' of drivers/char/ipmi/ipmi_msghandler.o: defined in discarded section `.exit.text' of drivers/char/ipmi/ipmi_msghandler.o In this case, we can trivially replace the atomic_inc() with an atomic_set() that has the same effect and does not require a fixup. Signed-off-by: Arnd Bergmann --- drivers/char/ipmi/ipmi_msghandler.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.20.0 diff --git a/drivers/char/ipmi/ipmi_msghandler.c b/drivers/char/ipmi/ipmi_msghandler.c index e8ba67834746..c48198eef510 100644 --- a/drivers/char/ipmi/ipmi_msghandler.c +++ b/drivers/char/ipmi/ipmi_msghandler.c @@ -5164,7 +5164,7 @@ static void __exit cleanup_ipmi(void) * avoids problems with race conditions removing the timer * here. */ - atomic_inc(&stop_operation); + atomic_set(&stop_operation, 1); del_timer_sync(&ipmi_timer); initialized = false;