Re: [PATCH v2 06/15] ppc/vof: Fix unaligned FDT property access

2024-07-08 Thread Nicholas Piggin
On Sun Jul 7, 2024 at 9:46 AM AEST, David Gibson wrote: > On Sat, Jul 06, 2024 at 11:37:08AM +0100, Peter Maydell wrote: > > On Fri, 5 Jul 2024 at 06:13, David Gibson > > wrote: > > > > > > On Fri, Jul 05, 2024 at 02:40:19PM +1000, Nicholas Piggin wrote: >

Re: [PATCH v2 06/15] ppc/vof: Fix unaligned FDT property access

2024-07-05 Thread Nicholas Piggin
On Fri Jul 5, 2024 at 3:12 PM AEST, David Gibson wrote: > On Fri, Jul 05, 2024 at 02:40:19PM +1000, Nicholas Piggin wrote: > > On Fri Jul 5, 2024 at 11:41 AM AEST, David Gibson wrote: > > > On Fri, Jul 05, 2024 at 11:18:47AM +1000, Nicholas Piggin wrote: > > > > On Th

Re: [PATCH v2 06/15] ppc/vof: Fix unaligned FDT property access

2024-07-04 Thread Nicholas Piggin
On Fri Jul 5, 2024 at 11:41 AM AEST, David Gibson wrote: > On Fri, Jul 05, 2024 at 11:18:47AM +1000, Nicholas Piggin wrote: > > On Thu Jul 4, 2024 at 10:15 PM AEST, Peter Maydell wrote: > > > On Sat, 29 Jun 2024 at 04:17, David Gibson > > > wrote: > > > >

Re: [PATCH v2 06/15] ppc/vof: Fix unaligned FDT property access

2024-07-04 Thread Nicholas Piggin
On Thu Jul 4, 2024 at 10:15 PM AEST, Peter Maydell wrote: > On Sat, 29 Jun 2024 at 04:17, David Gibson > wrote: > > > > On Fri, Jun 28, 2024 at 04:20:02PM +0100, Peter Maydell wrote: > > > On Thu, 27 Jun 2024 at 14:39, Akihiko Odaki > > > wrote: > > > > > > > > FDT properties are aligned by 4

Re: [PATCH v2 06/15] ppc/vof: Fix unaligned FDT property access

2024-07-04 Thread Nicholas Piggin
On Sat Jun 29, 2024 at 1:16 PM AEST, David Gibson wrote: > On Fri, Jun 28, 2024 at 04:20:02PM +0100, Peter Maydell wrote: > > On Thu, 27 Jun 2024 at 14:39, Akihiko Odaki > > wrote: > > > > > > FDT properties are aligned by 4 bytes, not 8 bytes. > > > > > > Signed-off-by: Akihiko Odaki > > > ---

Re: [PATCH v2 00/15] Fix check-qtest-ppc64 sanitizer errors

2024-07-04 Thread Nicholas Piggin
On Tue Jul 2, 2024 at 4:23 PM AEST, Thomas Huth wrote: > On 02/07/2024 00.23, BALATON Zoltan wrote: > > On Mon, 1 Jul 2024, Michael S. Tsirkin wrote: > >> On Thu, Jun 27, 2024 at 10:37:43PM +0900, Akihiko Odaki wrote: > >>> Based-on: > >>>

Re: [PATCH 07/13] RFC migration: icp/server is a mess

2023-10-20 Thread Nicholas Piggin
On Fri Oct 20, 2023 at 6:33 PM AEST, Greg Kurz wrote: > On Fri, 20 Oct 2023 17:49:38 +1000 > "Nicholas Piggin" wrote: > > > On Fri Oct 20, 2023 at 7:39 AM AEST, Greg Kurz wrote: > > > On Thu, 19 Oct 2023 21:08:25 +0200 > > > Juan Quint

Re: [PATCH v2 07/13] migration: Hack to maintain backwards compatibility for ppc

2023-10-20 Thread Nicholas Piggin
2_10_vmstate_dummy_icp > * unregister pre_2_10_vmstate_dummy_icp > * register real vmstate_icp > > Created vmstate_replace_hack_for_ppc() with warnings left and right > that it is a hack. Thanks. We'll look at deprecating 2.9 soon so this can all be removed. Reviewed-by: Nicholas Pig

Re: [PATCH 07/13] RFC migration: icp/server is a mess

2023-10-20 Thread Nicholas Piggin
On Fri Oct 20, 2023 at 7:39 AM AEST, Greg Kurz wrote: > On Thu, 19 Oct 2023 21:08:25 +0200 > Juan Quintela wrote: > > > Current code does: > > - register pre_2_10_vmstate_dummy_icp with "icp/server" and instance > > dependinfg on cpu number > > - for newer machines, it register vmstate_icp with

Re: [PATCH v3 0/9] bulk: Replace CONFIG_SOFTMMU by !CONFIG_USER_ONLY/CONFIG_SYSTEM_ONLY

2023-06-14 Thread Nicholas Piggin
On Tue Jun 13, 2023 at 11:33 PM AEST, Philippe Mathieu-Daudé wrote: > Missing review: 1, 7, 8 > > Since v2: > - Rebased > - Added R-b tags > - Rework i386_tr_init_disas_context() patch (Richard) > - Dropped RFC prefix > > This series aims to clarify the CONFIG_[USER|SYSTEM] vs CONFIG_SOFTMMU >

Re: [PATCH v3 4/9] target/ppc: Check for USER_ONLY definition instead of SOFTMMU one

2023-06-14 Thread Nicholas Piggin
On Tue Jun 13, 2023 at 11:33 PM AEST, Philippe Mathieu-Daudé wrote: > Since we *might* have user emulation with softmmu, > replace the system emulation check by !user emulation one. > > Signed-off-by: Philippe Mathieu-Daudé > Reviewed-by: Richard Henderson Reviewed-by: