Received: by 2002:a05:6a10:9e8c:0:0:0:0 with SMTP id y12csp476554pxx; Mon, 26 Oct 2020 12:52:53 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyRMjKvp9E/lFBH+aNlR0GfwH9T8stIvzpkx8pg2ngx76XCv48zUKTer6peaZd0wC6z0B3y X-Received: by 2002:a17:906:3e48:: with SMTP id t8mr17136478eji.104.1603741973535; Mon, 26 Oct 2020 12:52:53 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1603741973; cv=none; d=google.com; s=arc-20160816; b=gpDWGrVB8zhs1NhH+sLtgy4IDvyi1wwZOwPOKrbGvkfmjYIvB9MXVLGK/ptMCN1DC2 zlHdGluc+RCbzau3HuFyjozhDkM/Jb51Ffw1oyodKXM3CIrs3Z8Tn/FckpFOrE4GjUFN /hOq+vKfYgDv1m3Uw6QQt8ucBrJyym4hFcvf5aZR4tpQCXzZU7UrUaphMIKHeypMqtGC twCZar+fUIaUFHTciu7y+kImCClgqiEtBR+hq32pWkOohO/+CZQBp808WQR3BjszG9mF gYsljPclOWiTtjyNzSwkSqub4g5es/dd322N34fs72GORBMsQmtU6Fx2QKi5jaPO7VmS mjIQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=//lu6xMbJlxbiFG8DRyGUvg9gve889/4bYoNxrBB48c=; b=uLNTMRCavj+xCmp7S10YbrMJ1jPN16Pr+bwznxat+LBToQNsO+CM+xHW+FwmW4BD7N p7nyr4Eb8k3eTZh7B/wjb1sC9GcoO/wrEGR7X62lM1vpDfWmNQADiuwFuBcr7qAKKtQm JU3OY657jt/J5MEbxUhcJBhX6X3kkC+umZp7/VdP01Z1QHcrvsbiVc2qHcq5XsUchK8T db85uI/9urPws6wEs3n4SyvIWfATXOtaKmJAIToSjDG42U1jpA3Kt25lSYLa+AJpJ5yE 86f44Ygq8NliS6dGLzmIzG7VVY8hKqDxcFiaq/273HfeaYQn1Nmq1yTo+te0A41a+RmC y7Xg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@infradead.org header.s=merlin.20170209 header.b=JMHMEFbE; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id e18si7864466eji.106.2020.10.26.12.52.30; Mon, 26 Oct 2020 12:52:53 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@infradead.org header.s=merlin.20170209 header.b=JMHMEFbE; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1786799AbgJZQ6N (ORCPT + 99 others); Mon, 26 Oct 2020 12:58:13 -0400 Received: from merlin.infradead.org ([205.233.59.134]:44842 "EHLO merlin.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2440702AbgJZQ6M (ORCPT ); Mon, 26 Oct 2020 12:58:12 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=merlin.20170209; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=//lu6xMbJlxbiFG8DRyGUvg9gve889/4bYoNxrBB48c=; b=JMHMEFbEpproyn2wjMxZuEGOhe WhXYpL+5+r0u+Ske6gIIi6D1V4WoNRj73FkUw/3EdIN7eEb29IUzx0ufBP6ZMFdNnvJxHafZ+6Hao Ffhvlc47br02RxFj3v8L4w0F8R0MJMbtOrgwAElsbLjdAZaJS6MF6rG5w7oTemEh3Kmc4frLA3AY9 3Yerlm5icl/LXE+NNNtthdZmwyk4zLjNjiQfhFsrxY+KoB0p6nDWRv9NpS6+2+IZYe3eEnCpTMhxs uooEEQ2s98d+AIsrht3MCkgKtUWY35VpjxO81PhUfBLy5c38aQS+fpSrwl88qjHZ6L+Xu1xEyZVKt Aufp6mIg==; Received: from j217100.upc-j.chello.nl ([24.132.217.100] helo=noisy.programming.kicks-ass.net) by merlin.infradead.org with esmtpsa (Exim 4.92.3 #3 (Red Hat Linux)) id 1kX5ot-0005au-24; Mon, 26 Oct 2020 16:58:07 +0000 Received: from hirez.programming.kicks-ass.net (hirez.programming.kicks-ass.net [192.168.1.225]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by noisy.programming.kicks-ass.net (Postfix) with ESMTPS id AD469301179; Mon, 26 Oct 2020 17:58:05 +0100 (CET) Received: by hirez.programming.kicks-ass.net (Postfix, from userid 1000) id 92C16203BE3CE; Mon, 26 Oct 2020 17:58:05 +0100 (CET) Date: Mon, 26 Oct 2020 17:58:05 +0100 From: Peter Zijlstra To: Arnd Bergmann Cc: Ingo Molnar , Will Deacon , "Ahmed S. Darwish" , Arnd Bergmann , "Paul E. McKenney" , Marco Elver , linux-kernel@vger.kernel.org Subject: Re: [PATCH] seqlock: avoid -Wshadow warnings Message-ID: <20201026165805.GS2594@hirez.programming.kicks-ass.net> References: <20201026165044.3722931-1-arnd@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20201026165044.3722931-1-arnd@kernel.org> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Oct 26, 2020 at 05:50:38PM +0100, Arnd Bergmann wrote: > - unsigned seq; \ > + unsigned __seq; \ > - unsigned seq = __read_seqcount_begin(s); \ > + unsigned _seq = __read_seqcount_begin(s); \ > - unsigned seq = __seqcount_sequence(s); \ > + unsigned __seq = __seqcount_sequence(s); \ Can we have a consistent number of leading '_' ? Also, I suppose you're going to find the explicit shadow in ___wait_event(), that one's not going to be trivial to fix.