Hi Mark, > -----Original Message----- > From: Mark Rutland <mark.rutland@xxxxxxx> > Sent: Wednesday, October 16, 2019 1:25 AM > To: Jianyong Wu (Arm Technology China) <Jianyong.Wu@xxxxxxx> > Cc: netdev@xxxxxxxxxxxxxxx; yangbo.lu@xxxxxxx; john.stultz@xxxxxxxxxx; > tglx@xxxxxxxxxxxxx; pbonzini@xxxxxxxxxx; sean.j.christopherson@xxxxxxxxx; > maz@xxxxxxxxxx; richardcochran@xxxxxxxxx; will@xxxxxxxxxx; Suzuki > Poulose <Suzuki.Poulose@xxxxxxx>; linux-kernel@xxxxxxxxxxxxxxx; linux- > arm-kernel@xxxxxxxxxxxxxxxxxxx; kvmarm@xxxxxxxxxxxxxxxxxxxxx; > kvm@xxxxxxxxxxxxxxx; Steve Capper <Steve.Capper@xxxxxxx>; Kaly Xin > (Arm Technology China) <Kaly.Xin@xxxxxxx>; Justin He (Arm Technology > China) <Justin.He@xxxxxxx>; nd <nd@xxxxxxx> > Subject: Re: [PATCH v5 1/6] psci: Export psci_ops.conduit symbol as modules > will use it. > > On Tue, Oct 15, 2019 at 06:48:17PM +0800, Jianyong Wu wrote: > > If arm_smccc_1_1_invoke used in modules, psci_ops.conduit should be > > export. > > > > Signed-off-by: Jianyong Wu <jianyong.wu@xxxxxxx> > > I have a patch queued [1] in the arm64 tree which adds > arm_smccc_1_1_get_conduit() for this purpose. > > Please use that, adding an EXPORT_SYMBOL() if necessary. > Great, I will apply it next version. Thanks Jianyong Wu > Thanks, > Mark. > > [1] > https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git/commit/?h= > for-next/smccc-conduit- > cleanup&id=6b7fe77c334ae59fed9500140e08f4f896b36871 > > > --- > > drivers/firmware/psci/psci.c | 6 ++++++ > > include/linux/arm-smccc.h | 2 +- > > include/linux/psci.h | 1 + > > 3 files changed, 8 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/firmware/psci/psci.c > > b/drivers/firmware/psci/psci.c index f82ccd39a913..35c4eaab1451 100644 > > --- a/drivers/firmware/psci/psci.c > > +++ b/drivers/firmware/psci/psci.c > > @@ -212,6 +212,12 @@ static unsigned long > psci_migrate_info_up_cpu(void) > > 0, 0, 0); > > } > > > > +enum psci_conduit psci_get_conduit(void) { > > + return psci_ops.conduit; > > +} > > +EXPORT_SYMBOL(psci_get_conduit); > > + > > static void set_conduit(enum psci_conduit conduit) { > > switch (conduit) { > > diff --git a/include/linux/arm-smccc.h b/include/linux/arm-smccc.h > > index 552cbd49abe8..a6e4d3e3d10a 100644 > > --- a/include/linux/arm-smccc.h > > +++ b/include/linux/arm-smccc.h > > @@ -357,7 +357,7 @@ asmlinkage void __arm_smccc_hvc(unsigned long > a0, unsigned long a1, > > * The return value also provides the conduit that was used. > > */ > > #define arm_smccc_1_1_invoke(...) ({ > \ > > - int method = psci_ops.conduit; \ > > + int method = psci_get_conduit(); \ > > switch (method) { \ > > case PSCI_CONDUIT_HVC: > \ > > arm_smccc_1_1_hvc(__VA_ARGS__); > \ > > diff --git a/include/linux/psci.h b/include/linux/psci.h index > > a8a15613c157..e5cedc986049 100644 > > --- a/include/linux/psci.h > > +++ b/include/linux/psci.h > > @@ -42,6 +42,7 @@ struct psci_operations { > > enum smccc_version smccc_version; > > }; > > > > +extern enum psci_conduit psci_get_conduit(void); > > extern struct psci_operations psci_ops; > > > > #if defined(CONFIG_ARM_PSCI_FW) > > -- > > 2.17.1 > >
