From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 104FBC433EF for ; Mon, 14 Feb 2022 05:13:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240408AbiBNFN2 (ORCPT ); Mon, 14 Feb 2022 00:13:28 -0500 Received: from mxb-00190b01.gslb.pphosted.com ([23.128.96.19]:58498 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233236AbiBNFN1 (ORCPT ); Mon, 14 Feb 2022 00:13:27 -0500 Received: from zeniv-ca.linux.org.uk (zeniv-ca.linux.org.uk [IPv6:2607:5300:60:148a::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 65F804DF4B for ; Sun, 13 Feb 2022 21:13:20 -0800 (PST) Received: from viro by zeniv-ca.linux.org.uk with local (Exim 4.94.2 #2 (Red Hat Linux)) id 1nJTfm-001eJX-FG; Mon, 14 Feb 2022 05:13:14 +0000 Date: Mon, 14 Feb 2022 05:13:14 +0000 From: Al Viro To: Alexander Aring Cc: kernel test robot , kbuild-all@lists.01.org, linux-kernel@vger.kernel.org, David Teigland , cluster-devel Subject: Re: fs/dlm/midcomms.c:913:22: sparse: sparse: restricted __le32 degrades to integer Message-ID: References: <202201221028.YKA8kSdm-lkp@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: Al Viro Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, Jan 22, 2022 at 01:28:20PM -0500, Alexander Aring wrote: > Hi, > > On Fri, Jan 21, 2022 at 9:45 PM kernel test robot wrote: > > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master > > head: 9b57f458985742bd1c585f4c7f36d04634ce1143 > > commit: 658bd576f95ed597e519cdadf1c86ac87c17aea5 fs: dlm: move version conversion to compile time > > date: 3 months ago > > config: x86_64-rhel-8.3-kselftests (https://download.01.org/0day-ci/archive/20220122/202201221028.YKA8kSdm-lkp@intel.com/config) > > compiler: gcc-9 (Debian 9.3.0-22) 9.3.0 > > reproduce: > > # apt-get install sparse > > # sparse version: v0.6.4-dirty > > # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=658bd576f95ed597e519cdadf1c86ac87c17aea5 > > git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > git fetch --no-tags linus master > > git checkout 658bd576f95ed597e519cdadf1c86ac87c17aea5 > > # save the config file to linux build tree > > mkdir build_dir > > make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' O=build_dir ARCH=x86_64 SHELL=/bin/bash > > > > If you fix the issue, kindly add following tag as appropriate > > I have it on my list but it isn't easy to make sparse happy here... > this is the second time the robot reported this issue. Is there a way > to turn the robot off in that case? > > Maybe some human who reads that knows the answer? Frankly, these "convert in place" functions (dlm_message_in(), etc.) are asking for trouble. IOW, it's a genuinely fishy code. Saner approach is to keep them in little-endian through the entire thing, but it's quite a bit of massage.