On 13/02/24 1:24 pm, Christophe Leroy wrote:


Le 01/02/2024 à 18:12, Hari Bathini a écrit :
With module addresses supported, override bpf_jit_supports_kfunc_call()
to enable kfunc support. Module address offsets can be more than 32-bit
long, so override bpf_jit_supports_far_kfunc_call() to enable 64-bit
pointers.

What's the impact on PPC32 ? There are no 64-bit pointers on PPC32.

Yeah. Not required to return true for PPC32 case and probably not a
good thing to claim support for far kfunc calls for PPC32. Changing to:

+bool bpf_jit_supports_far_kfunc_call(void)
+{
+       return IS_ENABLED(CONFIG_PPC64);
+}


Signed-off-by: Hari Bathini <hbath...@linux.ibm.com>
---

* No changes since v1.


   arch/powerpc/net/bpf_jit_comp.c | 10 ++++++++++
   1 file changed, 10 insertions(+)

diff --git a/arch/powerpc/net/bpf_jit_comp.c b/arch/powerpc/net/bpf_jit_comp.c
index 7b4103b4c929..f896a4213696 100644
--- a/arch/powerpc/net/bpf_jit_comp.c
+++ b/arch/powerpc/net/bpf_jit_comp.c
@@ -359,3 +359,13 @@ void bpf_jit_free(struct bpf_prog *fp)
bpf_prog_unlock_free(fp);
   }
+
+bool bpf_jit_supports_kfunc_call(void)
+{
+       return true;
+}
+
+bool bpf_jit_supports_far_kfunc_call(void)
+{
+       return true;
+}

Reply via email to