Re: [PATCH] ppc/pnv: Add initial P9/10 SBE model

["Nicholas Piggin" <npiggin@gmail.com>]

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