From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753063AbdBAQyI (ORCPT ); Wed, 1 Feb 2017 11:54:08 -0500 Received: from mout.kundenserver.de ([212.227.17.13]:51109 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752702AbdBAQyG (ORCPT ); Wed, 1 Feb 2017 11:54:06 -0500 From: Arnd Bergmann To: Oleg Drokin , Andreas Dilger , James Simmons , Greg Kroah-Hartman Cc: Arnd Bergmann , "John L. Hammond" , Dmitry Eremin , "Kirill A. Shutemov" , lustre-devel@lists.lustre.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: [PATCH] staging: lustre: shut up clang warnings on CLASSERT() Date: Wed, 1 Feb 2017 17:52:38 +0100 Message-Id: <20170201165314.2588373-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 X-Provags-ID: V03:K0:OslkUuTkwM6ZFVd3mdIpr7iLki8YQ4z7iHxQ3GPOgEN+cSbH7Wu v55xYhQrNeR0fUZ0L8g18l54recynwLJcBUvsOG9bkJ08XFjLJRwDFKb1lNn3ly4On9G8vL oovg5qgAFXJRI5V05EyK791yZ1Cp0lvzS6AclMTjxPDOCMDYJw7TjJRg3xxnqqB2D+yye/n aAFNQmP4BlNTZvhKcPULw== X-UI-Out-Filterresults: notjunk:1;V01:K0:qDwMI9V2PUg=:vcaVoAVs5IJTvDzv1XjjQp HuJUfbEMKL4gop8d83rhnQSOWltw2cuLfLnudJTxDrjIQk8NlUSvzR04y73xfzqwaIHfrZ9ng rVeXj1Vv3Lft4O0dRQrdHjlgV3+Zp74eaw6IPtaKhlew78qKt0zSdcF4fdolfmyHI4aJ5nZQ1 d2IijWirh3hG2qa/TBkg6sQ1R3Y9Hzp8SA6w0JN4/+r5cHGG8zFuMb4m1AoBLavgf3ekNsKzT phFJf0/N/nCjh0YWKzC4NJtlBC6QoEqAbbqTAcaj8Ck33cqdfWuiQQETDRpm4C9zZFEAW6Z33 QfDI1jhEMkluz8VBoMAnE2fv9keS55jN8dFlCl0dOAN7hZ5U4/2oKrLmNnwyB/OB7u4R/XPAK KF1xQCcPwPvBgJCUDfP/uZeGtWDJSMaYKLDFzPVOqSzjswY6+oFYako8Xe4QD3B77LSnkR6KC k6QMl4rD8hag3DOrjdAIJZxb/BOvLAMA4JxTD7DCMMJdZry7zCaJasutn/Wg/e3HcJInL/04h zswXzg2zi5/dtUsvoYvInyDMfF7AtvoU/52SXiRnIrPou+fY9mf97zWTuLWIaaO4jYskEtaGw aX+pEMh85fTzlQmY8J+GLLsvWpsoXrG3fC240GqLhYwPH3txfSGhOW0w3hP7g31aykjEPc8Il /F0F/RC4V57GZG1Fsf51b2ItZtgjYF9QfYUhP65mIkRq4LGlDPkm1cwgM1bFd+FUwkN4= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org lustre uses a fake switch() statement as a compile-time assert, but unfortunately each use of that causes a warning when building with clang: drivers/staging/lustre/lnet/klnds/socklnd/socklnd.c:2907:2: warning: no case matching constant switch condition '42' drivers/staging/lustre/lnet/klnds/socklnd/../../../include/linux/libcfs/libcfs_private.h:294:36: note: expanded from macro 'CLASSERT' #define CLASSERT(cond) do {switch (42) {case (cond): case 0: break; } } while (0) Adding a 'default:' label in there shuts up the warning. Signed-off-by: Arnd Bergmann --- drivers/staging/lustre/include/linux/libcfs/libcfs_private.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs_private.h b/drivers/staging/lustre/include/linux/libcfs/libcfs_private.h index aab15d8112a4..2d5435029185 100644 --- a/drivers/staging/lustre/include/linux/libcfs/libcfs_private.h +++ b/drivers/staging/lustre/include/linux/libcfs/libcfs_private.h @@ -291,7 +291,7 @@ do { \ * value after conversion... * */ -#define CLASSERT(cond) do {switch (42) {case (cond): case 0: break; } } while (0) +#define CLASSERT(cond) do {switch (42) {case (cond): case 0: default: break; } } while (0) /* max value for numeric network address */ #define MAX_NUMERIC_VALUE 0xffffffff -- 2.9.0