Re: [PATCH 0/3] find_next_iomem_res() fixes

From: lijiang
Date: Wed Sep 26 2018 - 09:38:07 EST


å 2018å09æ26æ 17:22, lijiang åé:
>
>
> å 2018å09æ25æ 06:14, Bjorn Helgaas åé:
>> Hi Lianbo,
>>
>> These three patches are a possible replacement for your first patch
>> ("[PATCH 1/3 v3] resource: fix an error which walks through iomem
>> resources").
>>
>> I think the interface of find_next_iomem_res() can be improved to make
>> the code easier to read and also avoid the errors you're fixing.
>>
>> I can't test these, so they've only been compiled. If you can test
>> them and if you like them, feel free to incorporate them into your
>> series. If not, just drop them (but please at least fix the same
>> error in walk_system_ram_range()).
>>
> Hi, Bjorn
> Sorry for my late reply about this. No need to incorporate them into
> my patch series, you might freely submit them to upstream.
>
> I will test your patches whether it can also work well for my case, and
> you will get feedback about my test later.
>
For my case, your patches can also work well.

> Thanks.
> Lianbo
>
>> ---
>>
>> Bjorn Helgaas (3):
>> x86/kexec: Correct KEXEC_BACKUP_SRC_END off-by-one error
>> resource: Include resource end in walk_*() interfaces
>> resource: Fix find_next_iomem_res() iteration issue
>>
>>
>> arch/x86/include/asm/kexec.h | 2 -
>> kernel/resource.c | 96 ++++++++++++++++++------------------------
>> 2 files changed, 43 insertions(+), 55 deletions(-)
>>
>> _______________________________________________
>> kexec mailing list
>> kexec@xxxxxxxxxxxxxxxxxxx
>> http://lists.infradead.org/mailman/listinfo/kexec
>>