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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id D7AD7CDB482 for ; Thu, 19 Oct 2023 00:07:06 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=3ZrlrB4nIrR3hXLCTu937ypLPzy5Ekudsch+SR4c0ps=; b=gCnlDKpTu+RG0n 6npnwQZpOTxo80y1k28clsUKQ7bBUnGB17iwtlUdkcCS9l1t9Cqzj+w/jOEnCSUAaKADI+d0n5XpG uqkm5okQPuF+kAT4TWU8NTejPuJMIDGJrY998AWu2fLb1hIUJZlfiVw2HItr5FqOmVnurr0vRfOVE EPPbhNlOU7sYWxhMxult2/yxRSQlScznYcW/71L5OVV8WTubCyfDLGPOvFLcVO8ESEIc3EebrNwvM WYCWFYNYez3tVwbPABy/TTpqNdxcCZuWTA0CFFCbARIdZW2lQuKSLFYFDgU67yGHXYij+Ipuxaya3 jtvdMfL5g4MQ7aAjanzA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qtGYj-00Fz8C-2v; Thu, 19 Oct 2023 00:06:41 +0000 Received: from out-192.mta1.migadu.com ([2001:41d0:203:375::c0]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qtGYg-00Fz7G-0G for linux-arm-kernel@lists.infradead.org; Thu, 19 Oct 2023 00:06:40 +0000 Date: Thu, 19 Oct 2023 00:06:25 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.dev; s=key1; t=1697673992; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=hzjaIWah6RYMVBvI14tFSfFydhLBL9+gD67Q8Qa+q8o=; b=Mu1Ho+y3nPc2gCgNmE0x7g0+OC562KEeI3AkiOGEZgX2KWfHvGXyjVxRRNFtLT8RQB587x 4Z0HeLG3XVWB+f7VAvsy6IORv+ddsOuWlZoCfM4CQc7kVcQ1ytzyuqnVS21I/H/wBru5oY LAAnco/ohZrd5G4pFILAwv7pMXd83bQ= X-Report-Abuse: Please report any abuse attempt to abuse@migadu.com and include these headers. From: Oliver Upton To: Eric Auger Cc: Mark Brown , kvm@vger.kernel.org, kvmarm@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-perf-users@vger.kernel.org, Jing Zhang , Zenghui Yu , Suzuki K Poulose , James Morse , Marc Zyngier , Paolo Bonzini , Adrian Hunter , Ian Rogers , Namhyung Kim , Jiri Olsa , Alexander Shishkin , Mark Rutland , Arnaldo Carvalho de Melo , Ingo Molnar , Peter Zijlstra Subject: Re: [PATCH v3 4/5] tools headers arm64: Update sysreg.h with kernel sources Message-ID: References: <20231011195740.3349631-1-oliver.upton@linux.dev> <20231011195740.3349631-5-oliver.upton@linux.dev> <73b94274-4561-1edd-6b1e-8c6245133af2@redhat.com> <3c5332b0-9035-4cb8-96ce-7a9b8d513c3a@sirena.org.uk> <8baca35a-9154-97e6-d682-032fc69d2da6@redhat.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <8baca35a-9154-97e6-d682-032fc69d2da6@redhat.com> X-Migadu-Flow: FLOW_OUT X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231018_170638_259299_145DF106 X-CRM114-Status: GOOD ( 23.44 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Eric, Thanks for reviewing the series. On Wed, Oct 18, 2023 at 03:06:12PM +0200, Eric Auger wrote: > Hi Mark, Oliver, > > On 10/18/23 14:16, Mark Brown wrote: > > On Wed, Oct 18, 2023 at 01:57:31PM +0200, Eric Auger wrote: > >> On 10/11/23 21:57, Oliver Upton wrote: > > > >>> #define set_pstate_pan(x) asm volatile(SET_PSTATE_PAN(x)) > >>> #define set_pstate_uao(x) asm volatile(SET_PSTATE_UAO(x)) > >>> #define set_pstate_ssbs(x) asm volatile(SET_PSTATE_SSBS(x)) > >>> +#define set_pstate_dit(x) asm volatile(SET_PSTATE_DIT(x)) > > > >> could you comment on the *DIT* addictions, what is it for? > > > > DIT is data independent timing, this tells the processor to ensure that > > instructions take a constant time regardless of the data they are > > handling. > > > > > Note that this file is just a copy of arch/arm64/include/asm/gpr-num.h, > > the main purpose here is to sync with the original. > > Ah thanks. that's helpful for me to understand where this gpr-num.h > comes from. This could be documented in the commit msg though. > > Something like: > > adding tools/arch/arm64/include/asm/gpr-num.h matching linux > arch/arm64/include/asm/gpr-num.h > > and syncing tools/arch/arm64/include/asm/sysreg.h with the fellow header > in the linux tree. Yeah, I could've spelled it out a bit more. I already cracked this off of an even larger patch from before I picked up the series because the diff was massive. > tbh I did not initially understand that all this diffstat was aimed to > match the linux arch/arm64/include/asm/sysreg.h. Now diffing both I have > some diffs. Doesn't it need a refresh? I'm worried it is a fool's errand at this point to keep the two in sync, as I'm sure there will be more in -rc1. The tools copy of sysreg.h isn't a verbatim copy either, there are some deliberate deletions in there as well. I've taken this as is, we can always come back and update the headers afterwards if we find a need for it -- Thanks, Oliver _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel