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 lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (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 6CEC3C001B0 for ; Mon, 14 Aug 2023 08:28:39 +0000 (UTC) Authentication-Results: lists.ozlabs.org; dkim=fail reason="signature verification failed" (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20221208 header.b=RSyyiuay; dkim-atps=neutral Received: from boromir.ozlabs.org (localhost [IPv6:::1]) by lists.ozlabs.org (Postfix) with ESMTP id 4RPSGY6Pqsz3cHS for ; Mon, 14 Aug 2023 18:28:37 +1000 (AEST) Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20221208 header.b=RSyyiuay; dkim-atps=neutral Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=gmail.com (client-ip=2607:f8b0:4864:20::62e; helo=mail-pl1-x62e.google.com; envelope-from=npiggin@gmail.com; receiver=lists.ozlabs.org) Received: from mail-pl1-x62e.google.com (mail-pl1-x62e.google.com [IPv6:2607:f8b0:4864:20::62e]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 4RPSFZ5yfbz2yVm for ; Mon, 14 Aug 2023 18:27:46 +1000 (AEST) Received: by mail-pl1-x62e.google.com with SMTP id d9443c01a7336-1bc8045e09dso24320635ad.0 for ; Mon, 14 Aug 2023 01:27:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1692001664; x=1692606464; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-transfer-encoding:mime-version:from:to:cc:subject:date :message-id:reply-to; bh=M6ZLVmVU72z//BrT7M800C5eC3p9HVcgPrMw+dRpxbc=; b=RSyyiuay19Gb/Kevghl48H2c6NvlQVfdswzcqOls6h+pv464rgudsOF847zc2kj2+0 NkTNJvv8i0pJldDZWVsnpo9jlyl7rYDksBhio0AN+WBW7i/SoWXc0dZrBWs3v/y/rtim JnB0ydKuJ4xXbDkYLiMUDQhDUm5XwRft6jt75J7Bs8tlQXKpUfYT7fiWQFlnl2G5TXGm mREE2eQx6c0AOXrF38XfFTbOs2NO3bX8yYLjlZGEAidTebi9dm8jrIJCMOmq/+ofl1i5 iwTSQ7ry3daxeUEPiYzDcDi0SkAP/1e4UqCYC/IC4Pbni7M2kTTB32mvLWISEbEabNvC 06Cw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1692001664; x=1692606464; h=in-reply-to:references:to:from:subject:cc:message-id:date :content-transfer-encoding:mime-version:x-gm-message-state:from:to :cc:subject:date:message-id:reply-to; bh=M6ZLVmVU72z//BrT7M800C5eC3p9HVcgPrMw+dRpxbc=; b=WMQi5fmYjWPanOY+yt3XsvPHBozWO2Na5bmB2SKHwoGbxp8+CpK2AUnb4KjnqBtuir A7Ec0KcpiT5tlPk7yls2alMUsZn4uxCUqyDM0yhzyLMvG04+9k4C4A5LC2b/NInkXKzm Lnm6FugmnWp4Pw11MtvxkfWhyHaxuOYXc9Gmm4ghzjiY9bW5ik/iig98Hp7K8AFEBhGm jCv6gCKJYK009C8mUxoExOmHVf5232J/GF9jXzpnrR96Pg6Jw0sfoJUNYbmW3p1Fb14m gW9f1hQNr4w0raSYKxI8ekXVaxAw7RGvHRdyQa8XYXW8pD2QUab0loFYloCA+XSZNI9i RCLA== X-Gm-Message-State: AOJu0YzoyvkxRZ43vTQjAOUjobE1oohe+2eVPgcd2XsGcQDurieoN6HM 9z1mN1N9+prGUBYYkEQHzqk= X-Google-Smtp-Source: AGHT+IGYexny14RMiwrfYyJPyf9+GWafcn3Gl1drSRhbJL5TyoTsymt15Fg+PZzBlRE1l+Rsw6Bp8A== X-Received: by 2002:a17:903:41c2:b0:1bb:a55d:c6e7 with SMTP id u2-20020a17090341c200b001bba55dc6e7mr7664776ple.55.1692001664017; Mon, 14 Aug 2023 01:27:44 -0700 (PDT) Received: from localhost ([61.68.161.249]) by smtp.gmail.com with ESMTPSA id j4-20020a170902c3c400b001bdccf6b8c9sm4172836plj.127.2023.08.14.01.27.39 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 14 Aug 2023 01:27:43 -0700 (PDT) Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset=UTF-8 Date: Mon, 14 Aug 2023 18:27:37 +1000 Message-Id: Subject: Re: [PATCH v3 2/6] KVM: PPC: Rename accessor generator macros From: "Nicholas Piggin" To: "Jordan Niethe" , X-Mailer: aerc 0.15.2 References: <20230807014553.1168699-1-jniethe5@gmail.com> <20230807014553.1168699-3-jniethe5@gmail.com> In-Reply-To: <20230807014553.1168699-3-jniethe5@gmail.com> X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: mikey@neuling.org, kvm@vger.kernel.org, sbhat@linux.ibm.com, amachhiw@linux.vnet.ibm.com, gautam@linux.ibm.com, kvm-ppc@vger.kernel.org, kconsul@linux.vnet.ibm.com, vaibhav@linux.ibm.com Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Sender: "Linuxppc-dev" On Mon Aug 7, 2023 at 11:45 AM AEST, Jordan Niethe wrote: > More "wrapper" style accessor generating macros will be introduced for > the nestedv2 guest support. Rename the existing macros with more > descriptive names now so there is a consistent naming convention. > > Signed-off-by: Jordan Niethe > --- > v3: > - New to series > --- > arch/powerpc/include/asm/kvm_ppc.h | 60 +++++++++++++++--------------- > 1 file changed, 30 insertions(+), 30 deletions(-) > > diff --git a/arch/powerpc/include/asm/kvm_ppc.h b/arch/powerpc/include/as= m/kvm_ppc.h > index d16d80ad2ae4..b66084a81dd0 100644 > --- a/arch/powerpc/include/asm/kvm_ppc.h > +++ b/arch/powerpc/include/asm/kvm_ppc.h > @@ -927,19 +927,19 @@ static inline bool kvmppc_shared_big_endian(struct = kvm_vcpu *vcpu) > #endif > } > =20 > -#define SPRNG_WRAPPER_GET(reg, bookehv_spr) \ > +#define KVMPPC_BOOKE_HV_SPRNG_ACESSOR_GET(reg, bookehv_spr) \ > static inline ulong kvmppc_get_##reg(struct kvm_vcpu *vcpu) \ > { \ > return mfspr(bookehv_spr); \ > } \ > =20 > -#define SPRNG_WRAPPER_SET(reg, bookehv_spr) \ > +#define KVMPPC_BOOKE_HV_SPRNG_ACESSOR_SET(reg, bookehv_spr) \ > static inline void kvmppc_set_##reg(struct kvm_vcpu *vcpu, ulong val) \ > { \ > mtspr(bookehv_spr, val); \ > } \ > =20 > -#define SHARED_WRAPPER_GET(reg, size) \ > +#define KVMPPC_VCPU_SHARED_REGS_ACESSOR_GET(reg, size) \ > static inline u##size kvmppc_get_##reg(struct kvm_vcpu *vcpu) \ > { \ > if (kvmppc_shared_big_endian(vcpu)) \ > @@ -948,7 +948,7 @@ static inline u##size kvmppc_get_##reg(struct kvm_vcp= u *vcpu) \ > return le##size##_to_cpu(vcpu->arch.shared->reg); \ > } \ > =20 > -#define SHARED_WRAPPER_SET(reg, size) \ > +#define KVMPPC_VCPU_SHARED_REGS_ACESSOR_SET(reg, size) \ > static inline void kvmppc_set_##reg(struct kvm_vcpu *vcpu, u##size val) = \ > { \ > if (kvmppc_shared_big_endian(vcpu)) \ > @@ -957,36 +957,36 @@ static inline void kvmppc_set_##reg(struct kvm_vcpu= *vcpu, u##size val) \ > vcpu->arch.shared->reg =3D cpu_to_le##size(val); \ > } \ > =20 > -#define SHARED_WRAPPER(reg, size) \ > - SHARED_WRAPPER_GET(reg, size) \ > - SHARED_WRAPPER_SET(reg, size) \ > +#define KVMPPC_VCPU_SHARED_REGS_ACESSOR(reg, size) \ > + KVMPPC_VCPU_SHARED_REGS_ACESSOR_GET(reg, size) \ > + KVMPPC_VCPU_SHARED_REGS_ACESSOR_SET(reg, size) \ > =20 > -#define SPRNG_WRAPPER(reg, bookehv_spr) \ > - SPRNG_WRAPPER_GET(reg, bookehv_spr) \ > - SPRNG_WRAPPER_SET(reg, bookehv_spr) \ > +#define KVMPPC_BOOKE_HV_SPRNG_ACESSOR(reg, bookehv_spr) \ > + KVMPPC_BOOKE_HV_SPRNG_ACESSOR_GET(reg, bookehv_spr) \ > + KVMPPC_BOOKE_HV_SPRNG_ACESSOR_SET(reg, bookehv_spr) \ > =20 > #ifdef CONFIG_KVM_BOOKE_HV > =20 > -#define SHARED_SPRNG_WRAPPER(reg, size, bookehv_spr) \ > - SPRNG_WRAPPER(reg, bookehv_spr) \ > +#define KVMPPC_BOOKE_HV_SPRNG_OR_VCPU_SHARED_REGS_ACCESSOR(reg, size, bo= okehv_spr) \ > + KVMPPC_BOOKE_HV_SPRNG_ACESSOR(reg, bookehv_spr) \ > =20 > #else > =20 > -#define SHARED_SPRNG_WRAPPER(reg, size, bookehv_spr) \ > - SHARED_WRAPPER(reg, size) \ > +#define KVMPPC_BOOKE_HV_SPRNG_OR_VCPU_SHARED_REGS_ACCESSOR(reg, size, bo= okehv_spr) \ > + KVMPPC_VCPU_SHARED_REGS_ACESSOR(reg, size) \ Not the greatest name I've ever seen :D Hard to be concice and consistent though, this is an odd one. Reviewed-by: Nicholas Piggin > =20 > #endif > =20 > -SHARED_WRAPPER(critical, 64) > -SHARED_SPRNG_WRAPPER(sprg0, 64, SPRN_GSPRG0) > -SHARED_SPRNG_WRAPPER(sprg1, 64, SPRN_GSPRG1) > -SHARED_SPRNG_WRAPPER(sprg2, 64, SPRN_GSPRG2) > -SHARED_SPRNG_WRAPPER(sprg3, 64, SPRN_GSPRG3) > -SHARED_SPRNG_WRAPPER(srr0, 64, SPRN_GSRR0) > -SHARED_SPRNG_WRAPPER(srr1, 64, SPRN_GSRR1) > -SHARED_SPRNG_WRAPPER(dar, 64, SPRN_GDEAR) > -SHARED_SPRNG_WRAPPER(esr, 64, SPRN_GESR) > -SHARED_WRAPPER_GET(msr, 64) > +KVMPPC_VCPU_SHARED_REGS_ACESSOR(critical, 64) > +KVMPPC_BOOKE_HV_SPRNG_OR_VCPU_SHARED_REGS_ACCESSOR(sprg0, 64, SPRN_GSPRG= 0) > +KVMPPC_BOOKE_HV_SPRNG_OR_VCPU_SHARED_REGS_ACCESSOR(sprg1, 64, SPRN_GSPRG= 1) > +KVMPPC_BOOKE_HV_SPRNG_OR_VCPU_SHARED_REGS_ACCESSOR(sprg2, 64, SPRN_GSPRG= 2) > +KVMPPC_BOOKE_HV_SPRNG_OR_VCPU_SHARED_REGS_ACCESSOR(sprg3, 64, SPRN_GSPRG= 3) > +KVMPPC_BOOKE_HV_SPRNG_OR_VCPU_SHARED_REGS_ACCESSOR(srr0, 64, SPRN_GSRR0) > +KVMPPC_BOOKE_HV_SPRNG_OR_VCPU_SHARED_REGS_ACCESSOR(srr1, 64, SPRN_GSRR1) > +KVMPPC_BOOKE_HV_SPRNG_OR_VCPU_SHARED_REGS_ACCESSOR(dar, 64, SPRN_GDEAR) > +KVMPPC_BOOKE_HV_SPRNG_OR_VCPU_SHARED_REGS_ACCESSOR(esr, 64, SPRN_GESR) > +KVMPPC_VCPU_SHARED_REGS_ACESSOR_GET(msr, 64) > static inline void kvmppc_set_msr_fast(struct kvm_vcpu *vcpu, u64 val) > { > if (kvmppc_shared_big_endian(vcpu)) > @@ -994,12 +994,12 @@ static inline void kvmppc_set_msr_fast(struct kvm_v= cpu *vcpu, u64 val) > else > vcpu->arch.shared->msr =3D cpu_to_le64(val); > } > -SHARED_WRAPPER(dsisr, 32) > -SHARED_WRAPPER(int_pending, 32) > -SHARED_WRAPPER(sprg4, 64) > -SHARED_WRAPPER(sprg5, 64) > -SHARED_WRAPPER(sprg6, 64) > -SHARED_WRAPPER(sprg7, 64) > +KVMPPC_VCPU_SHARED_REGS_ACESSOR(dsisr, 32) > +KVMPPC_VCPU_SHARED_REGS_ACESSOR(int_pending, 32) > +KVMPPC_VCPU_SHARED_REGS_ACESSOR(sprg4, 64) > +KVMPPC_VCPU_SHARED_REGS_ACESSOR(sprg5, 64) > +KVMPPC_VCPU_SHARED_REGS_ACESSOR(sprg6, 64) > +KVMPPC_VCPU_SHARED_REGS_ACESSOR(sprg7, 64) > =20 > static inline u32 kvmppc_get_sr(struct kvm_vcpu *vcpu, int nr) > {