On Mon, Mar 23, 2020 at 10:14:53PM -0300, Jason Gunthorpe wrote: > From: Jason Gunthorpe <jgg@xxxxxxxxxxxx> > > This code can be compiled when CONFIG_TRANSPARENT_HUGEPAGE is off, so > remove the ifdef. > > The function is only ever called under > > if (pmd_devmap(pmd) || pmd_trans_huge(pmd)) > > Which is statically false if !CONFIG_TRANSPARENT_HUGEPAGE, so the compiler > reliably eliminates all of this code. > > Reviewed-by: Ralph Campbell <rcampbell@xxxxxxxxxx> > Signed-off-by: Jason Gunthorpe <jgg@xxxxxxxxxxxx> Looks good, Reviewed-by: Christoph Hellwig <hch@xxxxxx>