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 kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9E8FCC54E41 for ; Tue, 27 Feb 2024 09:10:18 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 29FBF6B0074; Tue, 27 Feb 2024 04:10:18 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 2014E94002F; Tue, 27 Feb 2024 04:10:18 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id F1F866B0078; Tue, 27 Feb 2024 04:10:17 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id DC1A36B0072 for ; Tue, 27 Feb 2024 04:10:17 -0500 (EST) Received: from smtpin01.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay06.hostedemail.com (Postfix) with ESMTP id B43EAA0FEA for ; Tue, 27 Feb 2024 09:10:17 +0000 (UTC) X-FDA: 81837012474.01.9B590E6 Received: from eu-smtp-delivery-151.mimecast.com (eu-smtp-delivery-151.mimecast.com [185.58.85.151]) by imf06.hostedemail.com (Postfix) with ESMTP id 7D8A4180003 for ; Tue, 27 Feb 2024 09:10:15 +0000 (UTC) Authentication-Results: imf06.hostedemail.com; dkim=none; spf=pass (imf06.hostedemail.com: domain of david.laight@aculab.com designates 185.58.85.151 as permitted sender) smtp.mailfrom=david.laight@aculab.com; dmarc=pass (policy=none) header.from=aculab.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1709025015; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=u52Fmg8OQED/NJc/WN2CEt4vjCDgNYVblYn1YhGiuMI=; b=EixubnL/UNmaI+HxMvh7f1k/Ev+zWgiDyukSGlyTbX353Pk3/sGRXxZeLNIMsK5aX2ZZhB nFm7cTzw1ZkZTrObQgjEIj3Yj5ivX9n8objOiGHlqc1aldzzOOShIJMCWIgdCtEW0f1ifb ASsdoxL8W+CmGHe7tJUTw5K/30ZQJms= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1709025015; a=rsa-sha256; cv=none; b=i53YVUwtiGZfL1MgJcmFAchptATgRaIRcatkWEgDsDRzrghYHZeJJgrRDZV1V2dUGJUyvq Fe1D6pbIVUyYzEmShdt28fTg7a+kKvIwHuDKk9GmNn+sJhdlIqiVDRhzXsFEmige5ol+1a a0P+BbPqt6p+hacwMUQ6mFsApWgdEVU= ARC-Authentication-Results: i=1; imf06.hostedemail.com; dkim=none; spf=pass (imf06.hostedemail.com: domain of david.laight@aculab.com designates 185.58.85.151 as permitted sender) smtp.mailfrom=david.laight@aculab.com; dmarc=pass (policy=none) header.from=aculab.com Received: from AcuMS.aculab.com (156.67.243.121 [156.67.243.121]) by relay.mimecast.com with ESMTP with both STARTTLS and AUTH (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id uk-mta-128-E934HnI0OCyPXoda2PTIuw-1; Tue, 27 Feb 2024 09:10:11 +0000 X-MC-Unique: E934HnI0OCyPXoda2PTIuw-1 Received: from AcuMS.Aculab.com (10.202.163.6) by AcuMS.aculab.com (10.202.163.6) with Microsoft SMTP Server (TLS) id 15.0.1497.48; Tue, 27 Feb 2024 09:10:09 +0000 Received: from AcuMS.Aculab.com ([::1]) by AcuMS.aculab.com ([::1]) with mapi id 15.00.1497.048; Tue, 27 Feb 2024 09:10:09 +0000 From: David Laight To: 'kernel test robot' , "'linux-kernel@vger.kernel.org'" , 'Linus Torvalds' , 'Netdev' , "'dri-devel@lists.freedesktop.org'" CC: "oe-kbuild-all@lists.linux.dev" , "'Jens Axboe'" , "'Matthew Wilcox (Oracle)'" , 'Christoph Hellwig' , "'linux-btrfs@vger.kernel.org'" , 'Andrew Morton' , Linux Memory Management List , 'Andy Shevchenko' , "'David S . Miller'" , 'Dan Carpenter' , "'Jani Nikula'" Subject: RE: [PATCH next v2 03/11] minmax: Simplify signedness check Thread-Topic: [PATCH next v2 03/11] minmax: Simplify signedness check Thread-Index: AdpoCqfCgp/0gHjwSqumBl0qZkMqdgBEnDuAAA9N+TA= Date: Tue, 27 Feb 2024 09:10:09 +0000 Message-ID: <291975e1412548daa70abfe747dfd893@AcuMS.aculab.com> References: <8657dd5c2264456f8a005520a3b90e2b@AcuMS.aculab.com> <202402270937.9kmO5PFt-lkp@intel.com> In-Reply-To: <202402270937.9kmO5PFt-lkp@intel.com> Accept-Language: en-GB, en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.202.205.107] MIME-Version: 1.0 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: aculab.com Content-Language: en-US Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Rspamd-Queue-Id: 7D8A4180003 X-Rspam-User: X-Rspamd-Server: rspam11 X-Stat-Signature: xbasjr36wz5bo7kohd6shptjs87ynsjy X-HE-Tag: 1709025015-316105 X-HE-Meta: U2FsdGVkX1/Y7ec5m52zJVtG74QAf4Ai+LDPtjhEaJYcjG0OZppo/oE+vOJpmvS2yJHPz26+KNi47nleiiLd/ZaoHLxbDbb8DXa5lmy7PdoprgbPDXoSv6aRUBqBXNPF1zdmwQqj7PELJmhpqVIpGnBpTgy0ZkI9tRcZMEYGZy5hH8trmqRArze4T+kDroqdSZZst1Hydb3IyvLiIUIhFsgWcAh7e1I1wKAK1pt7JJ+zhj9J/6b97Gq+a7CU2stLZFCFuCzIycwEvGeM2yhAEtv7uqaOKKKt7imoKwqDjWX9+JZqGjMiLFhzP2hWqz5CaJRwZuVn94LoStlBLqqyh9jpVJ2H+4SLQZTqoEO9trbLhT978ODnui2be6Obw+ohyvcN0iS9sW4lnt/ZdI0IIykivl62QEH4Kah9nnfXnNoAxlfxk+ASB8wy2+PdLcQwnR+GmkjK/0h8i37eXuLBh6cChoTsuZZUdsxw2/szp6PZJ7cVLtIA/Bqpyyg8F13M80HnQBmz7C1IaRRs4C7SvlIUpYRtLIeaKi+1mRjJEAeR1t/e0OVBfqk7mPLzT5k00GUC3oWNaoOFtqvklkd+i8Fa0mMMJTImQtxk84wIFTqpWg8brISgcma90tPOuW8NX1qyExhPCNgwBhqj/CnQNhhGxcJULbgDB5kTj+PZjdKKYdoYNH35eaq4WHA7wCxPB1MUNK5ONrv8NK5Jd1fUUbPzMQgJQzxVoFjZlRd7h9Wz2ZX1KJHADgHOUAQ48z0LHDzZYqlEnfLgGuW8npTkQcXbAjzpov5/FRQuWVL7k01vVdqCEFMmCOzI1uQ/pM8spKAQFBMDzzgCEwhU6qt/tOAHb86fN7+R6E4/v3XZoTzkGe9BdNIjSEkNhzso9Nm+ X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: From: kernel test robot > Sent: 27 February 2024 01:34 >=20 > kernel test robot noticed the following build warnings: >=20 > [auto build test WARNING on drm-misc/drm-misc-next] > [also build test WARNING on linux/master mkl-can-next/testing kdave/for-n= ext akpm-mm/mm-nonmm-unstable > axboe-block/for-next linus/master v6.8-rc6 next-20240226] > [cannot apply to next-20240223 dtor-input/next dtor-input/for-linus horms= -ipvs/master] > [If your patch is applied to the wrong git tree, kindly drop us a note. > And when submitting patch, we suggest to use '--base' as documented in > https://git-scm.com/docs/git-format-patch#_base_tree_information] >=20 > url: https://github.com/intel-lab-lkp/linux/commits/David-Laight/minma= x-Put-all-the-clamp- > definitions-together/20240226-005902 > base: git://anongit.freedesktop.org/drm/drm-misc drm-misc-next > patch link: https://lore.kernel.org/r/8657dd5c2264456f8a005520a3b90e2b= %40AcuMS.aculab.com > patch subject: [PATCH next v2 03/11] minmax: Simplify signedness check > config: alpha-defconfig (https://download.01.org/0day-ci/archive/20240227= /202402270937.9kmO5PFt- > lkp@intel.com/config) > compiler: alpha-linux-gcc (GCC) 13.2.0 > reproduce (this is a W=3D1 build): (https://download.01.org/0day- > ci/archive/20240227/202402270937.9kmO5PFt-lkp@intel.com/reproduce) >=20 > If you fix the issue in a separate patch/commit (i.e. not just a new vers= ion of > the same patch/commit), kindly add following tags > | Reported-by: kernel test robot > | Closes: https://lore.kernel.org/oe-kbuild-all/202402270937.9kmO5PFt-lkp= @intel.com/ >=20 > All warnings (new ones prefixed by >>): >=20 > In file included from include/linux/kernel.h:28, > from include/linux/cpumask.h:10, > from include/linux/smp.h:13, > from include/linux/lockdep.h:14, > from include/linux/spinlock.h:63, > from include/linux/swait.h:7, > from include/linux/completion.h:12, > from include/linux/crypto.h:15, > from include/crypto/aead.h:13, > from include/crypto/internal/aead.h:11, > from crypto/skcipher.c:12: > crypto/skcipher.c: In function 'skcipher_get_spot': > >> include/linux/minmax.h:31:70: warning: ordered comparison of pointer w= ith integer zero [-Wextra] > 31 | (is_unsigned_type(typeof(x)) || (__is_constexpr(x) ? (= x) + 0 >=3D 0 : 0)) Hmmm -Wextra isn't normally set. But I do wish the compiler would do dead code elimination before these warnings. Apart from stopping code using min()/max() for pointer types (all the type checking is pointless) I think that __is_constextr() can be implemented using _Generic (instead of sizeof(type)) and then the true/false return values can be specified and need not be the same types. That test can then be: =09(__if_constexpr(x, x, -1) >=3D 0) (The '+ 0' is there to convert bool to int and won't be needed for non-constant bool.) I may drop the last few patches until MIN/MAX have been removed from everywhere else to free up the names. =09David - Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1= PT, UK Registration No: 1397386 (Wales)