On 05/26, Jiri Olsa wrote:
>
> which means we need to allow 0x2e prefix which maps to INAT_PFX_CS
> attribute in is_prefix_bad function.
...
> --- a/arch/x86/kernel/uprobes.c
> +++ b/arch/x86/kernel/uprobes.c
> @@ -266,7 +266,6 @@ static bool is_prefix_bad(struct insn *insn)
> attr = inat_get_opcode_attribute(p);
> switch (attr) {
> case INAT_MAKE_PREFIX(INAT_PFX_ES):
> - case INAT_MAKE_PREFIX(INAT_PFX_CS):
I know nothing about how x86 CPU works, so let me ask...
What if insn->x86_64 is false? Is it safe to allow the CS prefix in
this case?
Oleg.
