On 24/10/25 15:36, Philippe Mathieu-Daudé wrote:
On 24/10/25 11:44, Aditya Gupta wrote:
On 25/10/24 02:56PM, Vishal Chourasia wrote:
Add Error **errp parameter to load_image_targphys(),
load_image_targphys_as(), and get_image_size() to enable better
error reporting when image loading fails.

Pass NULL for errp in all existing call sites to maintain current
behavior. No functional change intended in this patch.

Suggested-by: Peter Maydell <[email protected]>
Signed-off-by: Vishal Chourasia <[email protected]>

All tests pass for me, and changes look good.

Does that imply:

  Tested-by: Aditya Gupta <[email protected]>

?

Yes Philippe, can add Tested-by.


Tested-by: Aditya Gupta <[email protected]>


These are the tests I ran on the series:

* PowerPC tests (manually starting a VM + check-functional-ppc64)
* Build test all patches, so no patch introduces a compile error
* Gitlab pipeline passing


Didn't test all archs, but i don't see much functional changes to any arch other than ppc with the series.


Thanks,

- Aditya G




Reviewed-by: Aditya Gupta <[email protected]>

Thanks,
- Aditya G




Reply via email to