Bug 443035

Summary: The copy, paste and cpbort test does not work reliably.
Product: [Developer tools] valgrind Reporter: Carl Love <cel>
Component: vexAssignee: Julian Seward <jseward>
Status: CLOSED FIXED    
Severity: normal CC: tuliom, will_schmidt
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Other   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:
Attachments: Add .machine directives for scv, copy, paste, cpabort instructions

Description Carl Love 2021-09-27 21:24:17 UTC
The test for the copy, paste and cpabort regression test is not reliably working on an ISA 3.1 system.
Comment 1 Carl Love 2021-09-30 22:10:42 UTC
Created attachment 142050 [details]
Add .machine directives for scv, copy, paste,  cpabort instructions

Add .machine directives for the scv, copy, paste, cpabort assembly instructions.

Hardware sync instruction (hwsync) added after the copy, paste and cpabort
instructions to improve the reliability of the test.
Comment 2 Carl Love 2021-09-30 22:26:07 UTC
*** Bug 441506 has been marked as a duplicate of this bug. ***
Comment 3 Carl Love 2021-09-30 22:42:36 UTC
Updated fix has been committed

commit a63e9c28b53f20ce1c4bdc0b81da59957a17f4f0
Author: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
Date:   Fri Aug 20 19:07:09 2021 -0300

    powerpc: Add .machine directives for scv, copy, paste, cpabort instructions
    
    GCC is no longer passing the "-many" flag to the assembler.  So, the
    inline assembly instructions statements need to use the .machine directives
    for the specific platform.
    
    (gcc commit e154242724b084380e3221df7c08fcdbd8460674 ; "[RS6000] Don't
    pass -many to the assembler".
    
    Hardware sync instruction (hwsync) added after the copy, paste and cpabort
    instructions to improve the reliability of the test.