[Bug tdep/26916] New: [s390] Error recording "store on condition" insns

[Bug tdep/26916] New: [s390] Error recording "store on condition" insns

[arnez at linux dot ibm.com]

https://sourceware.org/bugzilla/show_bug.cgi?id=26916

            Bug ID: 26916
           Summary: [s390] Error recording "store on condition" insns
           Product: gdb
           Version: HEAD
            Status: NEW
          Severity: normal
          Priority: P2
         Component: tdep
          Assignee: unassigned at sourceware dot org
          Reporter: arnez at linux dot ibm.com
  Target Milestone: ---

On s390x, when a program contains one of the "store on condition" instructions
STOC, STOCG, or STOCFH, the recording usually fails with an error like this:

   Process record: failed to record execution log.

   Program stopped.

In fact, these instructions are recorded as if their instruction formats
resembled that of STG, which they don't.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug tdep/26916] [s390] Error recording "store on condition" insns

[arnez at linux dot ibm.com]

https://sourceware.org/bugzilla/show_bug.cgi?id=26916

Andreas Arnez <arnez at linux dot ibm.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|unassigned at sourceware dot org   |arnez at linux dot ibm.com
             Status|NEW                         |ASSIGNED
                 CC|                            |uweigand at gcc dot gnu.org

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug tdep/26916] [s390] Error recording "store on condition" insns

[arnez at linux dot ibm.com]

https://sourceware.org/bugzilla/show_bug.cgi?id=26916

--- Comment #1 from Andreas Arnez <arnez at linux dot ibm.com> ---
Posted a fix:
 https://sourceware.org/pipermail/gdb-patches/2020-November/173399.html

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug tdep/26916] [s390] Error recording "store on condition" insns

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=26916

--- Comment #2 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Andreas Arnez <arnez@sourceware.org>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=d5ef21c3ecf2d7a97272bf5a5881f4e71289a1af

commit d5ef21c3ecf2d7a97272bf5a5881f4e71289a1af
Author: Andreas Arnez <arnez@linux.ibm.com>
Date:   Thu Nov 19 19:10:58 2020 +0100

    gdb/s390: Correct recording of "store on condition" insns

    The "store on condition" instructions STOC, STOCG, and STOCFH are recorded
    as if their instruction formats resembled that of STG.  This is wrong,
    usually resulting in "failed to record execution log" errors when trying
    to record code with any of these instructions.

    This patch fixes the recording of these instructions.

    gdb/ChangeLog:

            PR tdep/26916
            * s390-tdep.c (s390_process_record): Fix recording of STOC, STOCG,
            and STOCFH.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug tdep/26916] [s390] Error recording "store on condition" insns

[arnez at linux dot ibm.com]

https://sourceware.org/bugzilla/show_bug.cgi?id=26916

Andreas Arnez <arnez at linux dot ibm.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|ASSIGNED                    |RESOLVED

--- Comment #3 from Andreas Arnez <arnez at linux dot ibm.com> ---
With the commit mentioned in comment #2 the bug should be fixed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.