[v5] Misc clean ups to target/ppc exception handling

[PATCH v5 0/9] Misc clean ups to target/ppc exception handling

Posted by BALATON Zoltan 10 months, 1 week ago

These are some small clean ups for target/ppc/excp_helper.c trying to
make this code a bit simpler. No functional change is intended. This
series was submitted before but only partially merged due to freeze
and conflicting series os thia was postponed then to avoid conflicts.

v5:
- rebase on master
- keep logging nip pointing to the sc instruction
- add another patch

v4: Rebased on master dropping what was merged

BALATON Zoltan (9):
  target/ppc: Use env_cpu for cpu_abort in excp_helper
  target/ppc: Readability improvements in exception handlers
  target/ppc: Fix gen_sc to use correct nip
  target/ppc: Move patching nip from exception handler to helper_scv
  target/ppc: Simplify syscall exception handlers
  target/ppc: Clean up ifdefs in excp_helper.c, part 1
  target/ppc: Clean up ifdefs in excp_helper.c, part 2
  target/ppc: Clean up ifdefs in excp_helper.c, part 3
  target/ppc: Remove interrupt handler wrapper functions

 target/ppc/cpu.h         |   1 +
 target/ppc/excp_helper.c | 490 +++++++++++++--------------------------
 target/ppc/translate.c   |  16 +-
 3 files changed, 170 insertions(+), 337 deletions(-)

-- 
2.30.9

Re: [PATCH v5 0/9] Misc clean ups to target/ppc exception handling

Posted by BALATON Zoltan 10 months ago

On Thu, 18 Jan 2024, BALATON Zoltan wrote:
> These are some small clean ups for target/ppc/excp_helper.c trying to
> make this code a bit simpler. No functional change is intended. This
> series was submitted before but only partially merged due to freeze
> and conflicting series os thia was postponed then to avoid conflicts.

Ping?

Regards,
BALATON Zoltan

> v5:
> - rebase on master
> - keep logging nip pointing to the sc instruction
> - add another patch
>
> v4: Rebased on master dropping what was merged
>
> BALATON Zoltan (9):
>  target/ppc: Use env_cpu for cpu_abort in excp_helper
>  target/ppc: Readability improvements in exception handlers
>  target/ppc: Fix gen_sc to use correct nip
>  target/ppc: Move patching nip from exception handler to helper_scv
>  target/ppc: Simplify syscall exception handlers
>  target/ppc: Clean up ifdefs in excp_helper.c, part 1
>  target/ppc: Clean up ifdefs in excp_helper.c, part 2
>  target/ppc: Clean up ifdefs in excp_helper.c, part 3
>  target/ppc: Remove interrupt handler wrapper functions
>
> target/ppc/cpu.h         |   1 +
> target/ppc/excp_helper.c | 490 +++++++++++++--------------------------
> target/ppc/translate.c   |  16 +-
> 3 files changed, 170 insertions(+), 337 deletions(-)
>
>

Re: [PATCH v5 0/9] Misc clean ups to target/ppc exception handling

Posted by BALATON Zoltan 9 months, 2 weeks ago

On Tue, 30 Jan 2024, BALATON Zoltan wrote:
> On Thu, 18 Jan 2024, BALATON Zoltan wrote:
>> These are some small clean ups for target/ppc/excp_helper.c trying to
>> make this code a bit simpler. No functional change is intended. This
>> series was submitted before but only partially merged due to freeze
>> and conflicting series os thia was postponed then to avoid conflicts.
>
> Ping?

Ping^2?

Regards,
BALATON Zoltan

>> v5:
>> - rebase on master
>> - keep logging nip pointing to the sc instruction
>> - add another patch
>> 
>> v4: Rebased on master dropping what was merged
>> 
>> BALATON Zoltan (9):
>>  target/ppc: Use env_cpu for cpu_abort in excp_helper
>>  target/ppc: Readability improvements in exception handlers
>>  target/ppc: Fix gen_sc to use correct nip
>>  target/ppc: Move patching nip from exception handler to helper_scv
>>  target/ppc: Simplify syscall exception handlers
>>  target/ppc: Clean up ifdefs in excp_helper.c, part 1
>>  target/ppc: Clean up ifdefs in excp_helper.c, part 2
>>  target/ppc: Clean up ifdefs in excp_helper.c, part 3
>>  target/ppc: Remove interrupt handler wrapper functions
>> 
>> target/ppc/cpu.h         |   1 +
>> target/ppc/excp_helper.c | 490 +++++++++++++--------------------------
>> target/ppc/translate.c   |  16 +-
>> 3 files changed, 170 insertions(+), 337 deletions(-)
>> 
>> 
>
>