Bug 458845

Summary: PowerPC: The L field for the dcbf and sync instruction should be 3 bits in ISA 3.1
Product: [Developer tools] valgrind Reporter: Carl Love <cel>
Component: vexAssignee: Julian Seward <jseward>
Status: CLOSED FIXED    
Severity: normal CC: shivaprasadbhat
Priority: NOR    
Version First Reported In: 3.18.1   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Carl Love 2022-09-07 16:32:06 UTC
The L field is two bits wide for the sync and dcbf instruction in ISA 3.0 and older.  It changes to three bits in ISA 3.1.  The dcbf and sync instructions are only extracting two bits for ISA 3.1.
Comment 1 Carl Love 2022-09-07 19:30:22 UTC
Patch to fix the issue committed.

commit 6658311aef8c1a604edb34f2763e83b25882e01b (HEAD -> master)
Author: Carl Love <cel@us.ibm.com>
Date:   Wed Sep 7 11:59:03 2022 -0500

    PowerPC: Fix the L field for the sync and dcbf instructions.
    
    The L field for the sync and dcbf instructions are three bits wide
    starting with ISA 3.1.  The L field is two bits wide prior to ISA 3.1.
    
    Patch based on patches from Shivaprasad Bhat <shivaprasadbhat@gmail.com>