Re: [PATCH] powerpc/kexec_file: Simplify add_usable_mem()
From: Thorsten Blum
Date: Tue Oct 20 2026 - 08:00:30 EST
Hi Sourabh,
On Fri, Sep 25, 2026 at 01:04:52PM +0530, Sourabh Jain wrote:
> On 16/09/26 21:01, Thorsten Blum wrote:
> > Use max() and min() to calculate loc_base and loc_end, respectively.
> > Skip to the next range if loc_base > loc_end and drop the now-obsolete
> > boolean add variable.
> >
> > Signed-off-by: Thorsten Blum <blum@xxxxxxxxxx>
> > ---
> > arch/powerpc/kexec/file_load_64.c | 29 +++++++++--------------------
> > 1 file changed, 9 insertions(+), 20 deletions(-)
[...]
> So, the changes look good to me. Feel free to add:
>
> Reviewed-by: Sourabh Jain sourabhjain@xxxxxxxxxxxxx
Thank you for your review. This should have been:
Reviewed-by: Sourabh Jain <sourabhjain@xxxxxxxxxxxxx>
Without angle brackets <..>, b4 ignores the tag:
NOTE: some trailers ignored due to from/email mismatches:
! Trailer: Reviewed-by: Sourabh Jain sourabhjain@xxxxxxxxxxxxx
Msg From: Sourabh Jain <sourabhjain@xxxxxxxxxxxxx>
Thanks,
Thorsten