Received: by 2002:a25:c593:0:0:0:0:0 with SMTP id v141csp510975ybe; Wed, 4 Sep 2019 03:26:22 -0700 (PDT) X-Google-Smtp-Source: APXvYqztDPtRJV75XdfKjujPe9I67lUuPtvu8/bsnRlm1ZGcbRcN9QBFD/Of2kiEEluk96+SNbof X-Received: by 2002:a17:90a:fa89:: with SMTP id cu9mr4080686pjb.139.1567592782026; Wed, 04 Sep 2019 03:26:22 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1567592782; cv=none; d=google.com; s=arc-20160816; b=G7DhYBPNFUyPJumYi6Gcn2FSbG8xzVIIDfAHoU5xpu7JFQpJrci/vCuwjC4lDsfIhi RoFNQ/5Cd6//4RQWHH1rW7Cfe4fke+fOVh8I+LUmevQuOIVNPKBebiHoG4FJcvCkQgXl T2hkNCxHXhx85VMb4t+3pnzcutYH2uMA8XQfrEUeMOPaPlqL6zALju7+4BRwgMd1i1ra 6poUZBX/rTxKn8QDcFZyhMZF8ok7R1J8S8HoBbmPYMcRd28Fp6ZGpbx8WKetO26BDqCf U0AccQWv78L4cQ1/4DGEAkl+xazJ+HYYmTNvJRZ7exOqYqUOQcXyz7KIM+xNqqcgKiRN yzjA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject; bh=2fjBomlGOT2DSvbIlCPFTLHPCacTcJEKFrxENuXhtYo=; b=VLJRlaXbiHhnC035Pb5BN/KcIjsmM6TKwJU8qKbd7g9EzgONd1s45nYSUKyxArH5WN 2V29Xvhr6TadsxY3L9S883MMmQHC7++9X73kD2+M+JOi0B58MH/71PnqiGJI9yntT9Ps mdN/TTXIj63nP+9Ku6JpoT5XhazjanXxi4pdhiSPio9NrWK+0kz+Z+Zydk5PNtoab1GZ o34D2x+qtiKYTIpW3bh8IZ8OnOwZYJksRvPxThDBlRogVuZy4HS8WWxrSkDaBUvjkyqx Yl+m2qCZZJbKe3hLLfjIA1jvqBPu4Sa449sEn0z2GvuTf+85WVYaKP99A/jK1bEbUfHK ssYw== ARC-Authentication-Results: i=1; mx.google.com; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id h38si17099573pgi.327.2019.09.04.03.26.05; Wed, 04 Sep 2019 03:26:22 -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; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727929AbfIDKZS (ORCPT + 99 others); Wed, 4 Sep 2019 06:25:18 -0400 Received: from foss.arm.com ([217.140.110.172]:51282 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725966AbfIDKZS (ORCPT ); Wed, 4 Sep 2019 06:25:18 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 6388B337; Wed, 4 Sep 2019 03:25:17 -0700 (PDT) Received: from [10.1.194.37] (e113632-lin.cambridge.arm.com [10.1.194.37]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 258AD3F246; Wed, 4 Sep 2019 03:25:16 -0700 (PDT) Subject: Re: [PATCH] sched: make struct task_struct::state 32-bit To: David Laight , 'Alexey Dobriyan' Cc: "mingo@redhat.com" , "peterz@infradead.org" , "linux-kernel@vger.kernel.org" , "rcu@vger.kernel.org" , "linux-block@vger.kernel.org" , "dm-devel@redhat.com" , "axboe@kernel.dk" , "aarcange@redhat.com" References: <20190902210558.GA23013@avx2> <20190903181920.GA22358@avx2> <651c419e45e043e9be8d0877b5a5406d@AcuMS.aculab.com> From: Valentin Schneider Message-ID: Date: Wed, 4 Sep 2019 11:25:14 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.8.0 MIME-Version: 1.0 In-Reply-To: <651c419e45e043e9be8d0877b5a5406d@AcuMS.aculab.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 04/09/2019 10:43, David Laight wrote: > From: Alexey Dobriyan >> Sent: 03 September 2019 19:19 > ... >>> How did you come up with this changeset, did you pickaxe for some regexp? >> >> No, manually, backtracking up to the call chain. >> Maybe I missed a few places. > > Renaming the structure field and getting the compiler to find all the uses can help. > > David > It's a good idea but sadly doesn't cover whatever the config doesn't compile. A safer starting point could be --- @state_var@ struct task_struct *p; identifier var; @@ ( var = p->state | p->state = var ) @@ identifier state_var.var; @@ - var + FIXME --- But I'm hoping we can get something even better from coccinelle, stay tuned... > - > Registered Address Lakeside, Bramley Road, Mount Farm, Milton Keynes, MK1 1PT, UK > Registration No: 1397386 (Wales) >