Re: [PATCH] kernel/resource: make iomem_resource implicit in release_mem_region_adjustable()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Wei Yang <richard.weiyang@xxxxxxxxxxxxxxxxx>
Subject: Re: [PATCH] kernel/resource: make iomem_resource implicit in release_mem_region_adjustable()
From: David Hildenbrand <david@xxxxxxxxxx>
Date: Wed, 16 Sep 2020 12:03:20 +0200
Cc: linux-kernel@xxxxxxxxxxxxxxx, virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx, linux-mm@xxxxxxxxx, linux-hyperv@xxxxxxxxxxxxxxx, xen-devel@xxxxxxxxxxxxxxxxxxxx, linux-acpi@xxxxxxxxxxxxxxx, linux-nvdimm@xxxxxxxxxxxx, linux-s390@xxxxxxxxxxxxxxx, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>, Michal Hocko <mhocko@xxxxxxxx>, Dan Williams <dan.j.williams@xxxxxxxxx>, Jason Gunthorpe <jgg@xxxxxxxx>, Kees Cook <keescook@xxxxxxxxxxxx>, Ard Biesheuvel <ardb@xxxxxxxxxx>, Pankaj Gupta <pankaj.gupta.linux@xxxxxxxxx>, Baoquan He <bhe@xxxxxxxxxx>
In-reply-to: <20200916100223.GA46154@L-31X9LVDL-1304.local>
Organization: Red Hat GmbH
References: <20200911103459.10306-1-david@redhat.com> <20200916073041.10355-1-david@redhat.com> <20200916100223.GA46154@L-31X9LVDL-1304.local>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0

On 16.09.20 12:02, Wei Yang wrote:
> On Wed, Sep 16, 2020 at 09:30:41AM +0200, David Hildenbrand wrote:
>> "mem" in the name already indicates the root, similar to
>> release_mem_region() and devm_request_mem_region(). Make it implicit.
>> The only single caller always passes iomem_resource, other parents are
>> not applicable.
>>
> 
> Looks good to me.
> 
> Reviewed-by: Wei Yang <richard.weiyang@xxxxxxxxxxxxxxxxx>
>

Thanks for the review!

-- 
Thanks,

David / dhildenb

Follow-Ups:
- Re: [PATCH] kernel/resource: make iomem_resource implicit in release_mem_region_adjustable()
  - From: Wei Yang

References:
- [PATCH v4 0/8] selective merging of system ram resources
  - From: David Hildenbrand
- [PATCH] kernel/resource: make iomem_resource implicit in release_mem_region_adjustable()
  - From: David Hildenbrand
- Re: [PATCH] kernel/resource: make iomem_resource implicit in release_mem_region_adjustable()
  - From: Wei Yang

Prev by Date: Re: [PATCH] kernel/resource: make iomem_resource implicit in release_mem_region_adjustable()
Next by Date: Re: Changing vma->vm_file in dma_buf_mmap()
Previous by thread: Re: [PATCH] kernel/resource: make iomem_resource implicit in release_mem_region_adjustable()
Next by thread: Re: [PATCH] kernel/resource: make iomem_resource implicit in release_mem_region_adjustable()
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]