Hi Bala,
On 22/06/20 12:39 pm, Balamuruhan S wrote:
> add provision to declare test is a negative scenario, verify
> whether emulation fails and avoid executing it.
>
> Signed-off-by: Balamuruhan S <bal...@linux.ibm.com>
> ---
> arch/powerpc/lib/test_emulate_step.c | 46 ++++++++++++++++++++++------
> 1 file changed, 36 insertions(+), 10 deletions(-)
>
> diff --git a/arch/powerpc/lib/test_emulate_step.c
> b/arch/powerpc/lib/test_emulate_step.c
> index e3b1797adfae..79acc899a618 100644
> --- a/arch/powerpc/lib/test_emulate_step.c
> +++ b/arch/powerpc/lib/test_emulate_step.c
> @@ -703,6 +703,7 @@ struct compute_test {
> unsigned long flags;
> struct ppc_inst instr;
> struct pt_regs regs;
> + bool negative;
> } subtests[MAX_SUBTESTS + 1];
> };
>
Bits of 'flags' are currently used to specify if parts of the resulting pt_regs
are to be ignored. Instead of adding a new member to the struct, can we not do
this using a bit in 'flags'?
- Sandipan
