On Wed, 5 Nov 2025 21:44:50 +1000 Gavin Shan <[email protected]> wrote:
> For one particular error (Error), we can't call error_setg() for twice. > Otherwise, the assert(*errp == NULL) will be triggered unexpectedly in > error_setv(). In ghes_record_cper_errors(), get_ghes_source_offsets() > can return a error initialized by error_setg(). Without bailing on > this error, it can call into the second error_setg() due to the > unexpected value from the read acknowledgement register. > > Bail early in ghes_record_cper_errors() when error is received from > get_ghes_source_offsets() to avoid the exception. > > Signed-off-by: Gavin Shan <[email protected]> Seems reasonable to me. Reviewed-by: Jonathan Cameron <[email protected]>
