On Wed, Mar 17, 2021 at 02:53:27PM +0100, Christoph Hellwig wrote: > On Wed, Mar 17, 2021 at 01:42:07PM +0000, Konrad Rzeszutek Wilk wrote: > > > - alloc_size = PAGE_ALIGN(io_tlb_nslabs * sizeof(size_t)); > > > - io_tlb_alloc_size = memblock_alloc(alloc_size, PAGE_SIZE); > > > - if (!io_tlb_alloc_size) > > > - panic("%s: Failed to allocate %zu bytes align=0x%lx\n", > > > - __func__, alloc_size, PAGE_SIZE); > > > > Shouldn't this be converted to: > > mem->alloc_size = memblock_alloc(alloc_size, PAGE_SIZE); > > if (...) > > > > Seems that it got lost in the search and replace? > > Yes, I messed that up during the rebase. That being said it magically > gets fixed in the next patch..
Yes. However if someone does a bisection they are going to be mighty unhappy with you.