Re: [PATCH-for-5.2?] target/mips/translate: Check R6 reserved encoding for Load Linked Word

2020-12-08 Thread Philippe Mathieu-Daudé
On 12/8/20 7:43 PM, Maciej W. Rozycki wrote: > On Tue, 8 Dec 2020, Philippe Mathieu-Daudé wrote: > >> Duh I hit that again, read the patch again, looks correct. I guess >> I got confused myself reviewing the offending patch... >> So I'm applying this patch to mips-next queue, using >> Fixes:

Re: [PATCH-for-5.2?] target/mips/translate: Check R6 reserved encoding for Load Linked Word

2020-12-08 Thread Philippe Mathieu-Daudé
On Tue, Nov 24, 2020 at 5:15 PM Philippe Mathieu-Daudé wrote: > On 11/24/20 4:59 PM, Richard Henderson wrote: > > On 11/24/20 5:45 AM, Philippe Mathieu-Daudé wrote: > >> Release 6 recoded the 'Load Linked Word' using SPECIAL3 opcode, > >> this opcode (0b11) is now reserved. > >> > >> Ref: A.2

Re: [PATCH-for-5.2?] target/mips/translate: Check R6 reserved encoding for Load Linked Word

2020-12-08 Thread Maciej W. Rozycki
On Tue, 8 Dec 2020, Philippe Mathieu-Daudé wrote: > Duh I hit that again, read the patch again, looks correct. I guess > I got confused myself reviewing the offending patch... > So I'm applying this patch to mips-next queue, using > Fixes: d9224450208 ("target-mips: Tighten ISA level checks")

Re: [PATCH-for-5.2?] target/mips/translate: Check R6 reserved encoding for Load Linked Word

2020-11-24 Thread Philippe Mathieu-Daudé
On 11/24/20 4:59 PM, Richard Henderson wrote: > On 11/24/20 5:45 AM, Philippe Mathieu-Daudé wrote: >> Release 6 recoded the 'Load Linked Word' using SPECIAL3 opcode, >> this opcode (0b11) is now reserved. >> >> Ref: A.2 Instruction Bit Encoding Tables: >> >> "6Rm instructions signal a

Re: [PATCH-for-5.2?] target/mips/translate: Check R6 reserved encoding for Load Linked Word

2020-11-24 Thread Richard Henderson
On 11/24/20 5:45 AM, Philippe Mathieu-Daudé wrote: > Release 6 recoded the 'Load Linked Word' using SPECIAL3 opcode, > this opcode (0b11) is now reserved. > > Ref: A.2 Instruction Bit Encoding Tables: > > "6Rm instructions signal a Reserved Instruction exception >when executed by a

[PATCH-for-5.2?] target/mips/translate: Check R6 reserved encoding for Load Linked Word

2020-11-24 Thread Philippe Mathieu-Daudé
Release 6 recoded the 'Load Linked Word' using SPECIAL3 opcode, this opcode (0b11) is now reserved. Ref: A.2 Instruction Bit Encoding Tables: "6Rm instructions signal a Reserved Instruction exception when executed by a Release 6 implementation." The check was added in commit