Received: by 2002:a25:ad19:0:0:0:0:0 with SMTP id y25csp3853325ybi; Mon, 29 Jul 2019 13:58:50 -0700 (PDT) X-Google-Smtp-Source: APXvYqy2mBCpwpwpZ9bfACqSQ61+uSGjPwadzdwhoYz7+EmtZN5bLLJ4Iarf7xPNgqrVePgiSpcN X-Received: by 2002:a62:7695:: with SMTP id r143mr39674779pfc.173.1564433930081; Mon, 29 Jul 2019 13:58:50 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1564433930; cv=none; d=google.com; s=arc-20160816; b=g5UVbyt1BRqP6F9iZe/RAzwiheFIwHcTf1A7ObEcEOXSaafpYnzOvYpxNZV3FjaPvV HhFYpMH7fztNEaETaayUepzwh9d27bpEGmMjFrFTBYkm5s3V3qAI1DhVc8lM1lVl2nKT 5LccyUwhTxIpjaTtgJ/Ua62yNtEFJnb/jWu4ltQjI3bZaCI6u/9hHb5G2lMoYofdh7qG 750WF9AWhCW9gHx+FLoRvybYJZCjaxfB6SaI9dmrJkr4W+HsgCnsmquOd5dUUTMhC+Kg cwwj+qqGtLcZfqnAfAvFQpam+pgf9JpOdQtURclzGGSYx+9xmm9GnUjNnjIVbqyMR9eF r6oA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=OCBg8SJ3+ECtlnFhkK0FZXvrbbwZJ3DFrryZcy3S/yo=; b=QROhwroFHwLkSNh1gtLA0Fp01GOTKSRQnK2M7DGiUfKkSpIYAMvaowSvdFLHE6obRL AG4Z3LiTZtylD1ltjtFWrtTVtbB9dyMiLKW5a1NeCFRt1FpinXn0mmHgZCfcWekMBW0E 2Ccj7R+rRndtw/krJu1rc+BpBtVLQD2pVumzgoM4cw0zNVw7OCVq6eRtT9E42FjflBbm kNJ5Es+AfJUdltRiYCsTrrxFmeNBHpzdukkLwKI+biT1iu64O5725KrEYwsbWrlJ0K19 fEYkOM2hm4r9wJDH3q7h1i2pq9U8NZr1Zz80ZXdmYgs0pd9UVw0LpTUA3eSsYrvev9sL MVPw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@chromium.org header.s=google header.b=JdsUH2cU; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id l62si26614783pgd.24.2019.07.29.13.58.34; Mon, 29 Jul 2019 13:58:50 -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; dkim=pass header.i=@chromium.org header.s=google header.b=JdsUH2cU; 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; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=chromium.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729738AbfG2U5u (ORCPT + 99 others); Mon, 29 Jul 2019 16:57:50 -0400 Received: from mail-pl1-f194.google.com ([209.85.214.194]:36910 "EHLO mail-pl1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729659AbfG2U5t (ORCPT ); Mon, 29 Jul 2019 16:57:49 -0400 Received: by mail-pl1-f194.google.com with SMTP id b3so27989768plr.4 for ; Mon, 29 Jul 2019 13:57:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=OCBg8SJ3+ECtlnFhkK0FZXvrbbwZJ3DFrryZcy3S/yo=; b=JdsUH2cUj009zj5qFs6JLBlNIHzixuWaj4EX++fiyjyjAUYk0q1QkquWeGOmV28RdT g7AJHn7fhYhZUcEsJzj1uZMyAB9I1Hm4II76KnAIyVmBHwpyO1Wyk1q8s7iwbWMsJq00 4qnIMSFw5yF77nQ0KJOIIuq3CXf6FrmnxUH9I= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=OCBg8SJ3+ECtlnFhkK0FZXvrbbwZJ3DFrryZcy3S/yo=; b=d3+RmMOBMxMVKL8KAiv/FrVUhiW2QNmhr+z6POA2KH6g9R+kochpSjw4x61Z+Q59R6 aE41aPIZ2Kj9d/cXawywO0N5FY5UUaw/rsfDr/UCFgS3Gq+/WFFP9gcm4XvYH/9f+i2R joYJ+m8HQCxpUEf50scCTBhQzSxYZsvmaoo45wjYAlYUJ2dmqKfSY4EZBm6CQ35WQRi4 5yuCfnL4KH0HHxHq4DXWj41BkOr2LbwTQ4Ln993nYYlVUUMPl4Ckaxz5Q1j6iDyyaUpb xI5TBve1lAyA4mLAtHzBRS7x/c5qHLXLjX6khOWGt1Sjr5FBris5WVeE8qEBWlIVAkM6 9RPw== X-Gm-Message-State: APjAAAWE+w+iBvEaUkKnUhvmA6Jgf6VREsJ5G6kyLzticMCUggxGN4YW 3lozv9wXXvknWymy4ImqO1woOg== X-Received: by 2002:a17:902:aa8a:: with SMTP id d10mr114252213plr.154.1564433869179; Mon, 29 Jul 2019 13:57:49 -0700 (PDT) Received: from localhost ([2620:15c:202:1:75a:3f6e:21d:9374]) by smtp.gmail.com with ESMTPSA id q21sm48521374pgb.48.2019.07.29.13.57.48 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 29 Jul 2019 13:57:48 -0700 (PDT) Date: Mon, 29 Jul 2019 13:57:46 -0700 From: Matthias Kaehlcke To: Greg Kroah-Hartman Cc: linux-kernel@vger.kernel.org, stable@vger.kernel.org, Douglas Anderson , Nick Desaulniers , Alexander Duyck , "David S. Miller" , Sasha Levin , Toralf =?utf-8?Q?F=C3=B6rster?= , Nathan Huckleberry Subject: Re: [PATCH 5.2 083/215] net/ipv4: fib_trie: Avoid cryptic ternary expressions Message-ID: <20190729205746.GI250418@google.com> References: <20190729190739.971253303@linuxfoundation.org> <20190729190753.998851246@linuxfoundation.org> <20190729205422.GH250418@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20190729205422.GH250418@google.com> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jul 29, 2019 at 01:54:22PM -0700, Matthias Kaehlcke wrote: > Hi Greg, > > Toralf just pointed out in another thread that the commit message and > the content of this patch don't match (https://lkml.org/lkml/2019/7/29/1475) > > I did some minor digging, the content of the queued patch is: > > commit 4df607cc6fe8e46b258ff2a53d0a60ca3008ffc7 > Author: Nathan Huckleberry > Date: Mon Jun 17 10:28:29 2019 -0700 > > kbuild: Remove unnecessary -Wno-unused-value > > > however the commit message is from: > > commit 25cec756891e8733433efea63b2254ddc93aa5cc > Author: Matthias Kaehlcke > Date: Tue Jun 18 14:14:40 2019 -0700 > > net/ipv4: fib_trie: Avoid cryptic ternary expressions > > > It seems this hasn't been commited to -stable yet, so we are probably > in time to remove it from the queue before it becomes git history and > have Nathan re-spin the patch(es). s/Nathan/Sasha/ For some reason I thought Nathan backported this and wondered that his SOB is missing. The correct SOB is actually there. > On Mon, Jul 29, 2019 at 09:21:19PM +0200, Greg Kroah-Hartman wrote: > > [ Upstream commit 25cec756891e8733433efea63b2254ddc93aa5cc ] > > > > empty_child_inc/dec() use the ternary operator for conditional > > operations. The conditions involve the post/pre in/decrement > > operator and the operation is only performed when the condition > > is *not* true. This is hard to parse for humans, use a regular > > 'if' construct instead and perform the in/decrement separately. > > > > This also fixes two warnings that are emitted about the value > > of the ternary expression being unused, when building the kernel > > with clang + "kbuild: Remove unnecessary -Wno-unused-value" > > (https://lore.kernel.org/patchwork/patch/1089869/): > > > > CC net/ipv4/fib_trie.o > > net/ipv4/fib_trie.c:351:2: error: expression result unused [-Werror,-Wunused-value] > > ++tn_info(n)->empty_children ? : ++tn_info(n)->full_children; > > > > Fixes: 95f60ea3e99a ("fib_trie: Add collapse() and should_collapse() to resize") > > Signed-off-by: Matthias Kaehlcke > > Reviewed-by: Douglas Anderson > > Reviewed-by: Nick Desaulniers > > Acked-by: Alexander Duyck > > Signed-off-by: David S. Miller > > Signed-off-by: Sasha Levin > > --- > > scripts/Makefile.extrawarn | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/scripts/Makefile.extrawarn b/scripts/Makefile.extrawarn > > index 3ab8d1a303cd..b293246e48fe 100644 > > --- a/scripts/Makefile.extrawarn > > +++ b/scripts/Makefile.extrawarn > > @@ -68,7 +68,6 @@ else > > > > ifdef CONFIG_CC_IS_CLANG > > KBUILD_CFLAGS += -Wno-initializer-overrides > > -KBUILD_CFLAGS += -Wno-unused-value > > KBUILD_CFLAGS += -Wno-format > > KBUILD_CFLAGS += -Wno-sign-compare > > KBUILD_CFLAGS += -Wno-format-zero-length