linux-kernel - Re: [PATCH 1/2] powerpc/sstep: Unexport analyze_instr, emulate_vsx_load, emulate_vsx

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <8e6c75db-5c98-4f82-8151-5f86c53f4cf2@csgroup.eu>
Date: Tue, 1 Oct 2024 14:08:18 +0200
From: Christophe Leroy <christophe.leroy@...roup.eu>
To: Michal Suchanek <msuchanek@...e.de>, linuxppc-dev@...ts.ozlabs.org
Cc: Michael Ellerman <mpe@...erman.id.au>, Nicholas Piggin
 <npiggin@...il.com>, Naveen N Rao <naveen@...nel.org>,
 Madhavan Srinivasan <maddy@...ux.ibm.com>,
 "Gustavo A. R. Silva" <gustavoars@...nel.org>,
 Paul Mackerras <paulus@...abs.org>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/2] powerpc/sstep: Unexport analyze_instr,
 emulate_vsx_load, emulate_vsx_store



Le 01/10/2024 à 12:08, Michal Suchanek a écrit :
> There is no modular user of analyze_instr, and the latter two are only
> used by sstep itself.

analyze_instr() is used in arch/powerpc/kvm/emulate_loadstore.c which 
can be a module as far as I can see in Makefile:

common-objs-y += powerpc.o emulate_loadstore.o

kvm-book3s_64-module-objs := \
	$(common-objs-y) \
	book3s.o \
	book3s_rtas.o \
	$(kvm-book3s_64-objs-y)

kvm-objs-$(CONFIG_KVM_BOOK3S_64) := $(kvm-book3s_64-module-objs)

config KVM_BOOK3S_64
	tristate "KVM support for PowerPC book3s_64 processors"


> 
> Signed-off-by: Michal Suchanek <msuchanek@...e.de>
> ---
>   arch/powerpc/lib/sstep.c | 3 ---
>   1 file changed, 3 deletions(-)
> 
> diff --git a/arch/powerpc/lib/sstep.c b/arch/powerpc/lib/sstep.c
> index e65f3fb68d06..a0557b0d9a24 100644
> --- a/arch/powerpc/lib/sstep.c
> +++ b/arch/powerpc/lib/sstep.c
> @@ -863,7 +863,6 @@ void emulate_vsx_load(struct instruction_op *op, union vsx_reg *reg,
>   		break;
>   	}
>   }
> -EXPORT_SYMBOL_GPL(emulate_vsx_load);
>   NOKPROBE_SYMBOL(emulate_vsx_load);
>   
>   void emulate_vsx_store(struct instruction_op *op, const union vsx_reg *reg,
> @@ -955,7 +954,6 @@ void emulate_vsx_store(struct instruction_op *op, const union vsx_reg *reg,
>   		break;
>   	}
>   }
> -EXPORT_SYMBOL_GPL(emulate_vsx_store);
>   NOKPROBE_SYMBOL(emulate_vsx_store);
>   
>   static nokprobe_inline int do_vsx_load(struct instruction_op *op,
> @@ -3172,7 +3170,6 @@ int analyse_instr(struct instruction_op *op, const struct pt_regs *regs,
>   	op->val = SRR1_PROGTRAP;
>   	return 0;
>   }
> -EXPORT_SYMBOL_GPL(analyse_instr);
>   NOKPROBE_SYMBOL(analyse_instr);
>   
>   /*