On 25/10/23 01:46PM, Sourabh Jain wrote:
> 
> 
> > <...snip...>
> > +/*
> > + * Handle the "FADUMP_CMD_REGISTER" command in 'ibm,configure-kernel-dump'
> > + *
> > + * Note: Any changes made by the kernel to the fadump memory struct won't
> > + * reflect in QEMU after the 'ibm,configure-kernel-dump' RTAS call has 
> > returned,
> > + * as we store the passed fadump memory structure passed during fadump
> > + * registration.
> 
> Seems like you forgot to update the above comment.

Yeah, missed to reword here. Reworded in the description.

> 
> Rest all look good to me.
> Reviewed-by: Sourabh Jain <[email protected]>

Thanks for the tag Sourabh !

- Aditya G


Reply via email to