Re: [PATCH 08/11] replay: introduce a central report point for sync errors

2023-12-07 Thread Pavel Dovgalyuk
On 06.12.2023 19:48, Richard Henderson wrote: On 12/6/23 03:35, Philippe Mathieu-Daudé wrote: Hi Alex, On 5/12/23 21:41, Alex Bennée wrote: Figuring out why replay has failed is tricky at the best of times. Lets centralise the reporting of a replay sync error and add a little bit of extra info

Re: [PATCH 08/11] replay: introduce a central report point for sync errors

2023-12-06 Thread Richard Henderson
On 12/5/23 12:41, Alex Bennée wrote: Figuring out why replay has failed is tricky at the best of times. Lets centralise the reporting of a replay sync error and add a little bit of extra information to help with debugging. Signed-off-by: Alex Bennée --- replay/replay-internal.h | 12 +

Re: [PATCH 08/11] replay: introduce a central report point for sync errors

2023-12-06 Thread Richard Henderson
On 12/6/23 03:35, Philippe Mathieu-Daudé wrote: Hi Alex, On 5/12/23 21:41, Alex Bennée wrote: Figuring out why replay has failed is tricky at the best of times. Lets centralise the reporting of a replay sync error and add a little bit of extra information to help with debugging. Signed-off-by:

Re: [PATCH 08/11] replay: introduce a central report point for sync errors

2023-12-06 Thread Philippe Mathieu-Daudé
Hi Alex, On 5/12/23 21:41, Alex Bennée wrote: Figuring out why replay has failed is tricky at the best of times. Lets centralise the reporting of a replay sync error and add a little bit of extra information to help with debugging. Signed-off-by: Alex Bennée --- replay/replay-internal.h | 12

[PATCH 08/11] replay: introduce a central report point for sync errors

2023-12-05 Thread Alex Bennée
Figuring out why replay has failed is tricky at the best of times. Lets centralise the reporting of a replay sync error and add a little bit of extra information to help with debugging. Signed-off-by: Alex Bennée --- replay/replay-internal.h | 12 replay/replay-char.c | 6 ++---