On Thu, Sep 10, 2020 at 08:48:40PM +0000, sudar...@codeaurora.org wrote: > On 2020-09-10 03:50, Anshuman Khandual wrote: > > The proposed patch tried to mix both approaches. Regardless, the first > > approach > > here seems better and is the case in vmemmap_populate_hugepages() > > implementation > > on x86 as well. > > The 1st approach looks more cleaner compared to bailing out in first > failure, unmapping all previously mapped sections and map entire request > with vmemmap_populate_basepages. Thanks for the review and suggestion - will > send over a cleaner patch soon.
Did you send an updated version of this? The threading has gone wonky in my mail client, so I may have missed it. Will