[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] ppc/pnv: Add initial P9/10 SBE model
From: |
Nicholas Piggin |
Subject: |
Re: [PATCH] ppc/pnv: Add initial P9/10 SBE model |
Date: |
Mon, 29 May 2023 11:51:11 +1000 |
On Fri May 26, 2023 at 10:20 PM AEST, Cédric Le Goater wrote:
> On 8/11/22 10:04, Cédric Le Goater wrote:
> > On 8/11/22 09:54, Nicholas Piggin wrote:
[snip]
> >> +
> >> +#define sbe_cmd_dbg(fmt, ...) do { \
> >> + if (DEBUG_SBE_CMD) { \
> >> + fprintf(stderr, "SBE cmd: %s: " fmt, __func__, ## __VA_ARGS__); \
> >> + } \
> >> +} while (0)
> >
> > Please use trace events instead, with a pnv_sbe_ prefix.
> >
> > Apart from that, LGTM !
> >
> > Reviewed-by: Cédric Le Goater <clg@kaod.org>
>
>
>
>
> This looked like an interesting extension to the baremetal models.
> Did you abandon the idea ? if not, it would be good material for
> QEMU 8.1. Could you resend please ? (with the TOD patches)
Hey Cedric,
Yeah I kind of forgot about these for a while, sorry :( Too much
stuff going on... But by an amazing coincidence I have just started to
look at these again in the past few days, so I should try to get it
merged.
Thanks,
Nick