Re: [PATCH] vfio/type1: Absorb num_pages_contiguous()

[Alex Williamson <alex.williamson@redhat.com>]

On Tue, 5 Aug 2025 07:56:43 -0600
Alex Williamson <alex.williamson@redhat.com> wrote:

> On Tue, 5 Aug 2025 15:27:35 +0200
> David Hildenbrand <david@redhat.com> wrote:
> 
> > On 05.08.25 03:24, Alex Williamson wrote:  
> > > Objections were raised to adding this helper to common code with only a
> > > single user and dubious generalism.  Pull it back into subsystem code.
> > > 
> > > Link: https://lore.kernel.org/all/CAHk-=whhYRMS7Xc9k_JBdrGvp++JLmU0T2xXEgn046hWrj7q8Q@mail.gmail.com/
> > > Cc: David Hildenbrand <david@redhat.com>
> > > Cc: Jason Gunthorpe <jgg@nvidia.com>
> > > Cc: Li Zhe <lizhe.67@bytedance.com>
> > > Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
> > > ---    
> > 
> > So, we might have a version that Linus should be happy with, that we 
> > could likely place in mm/util.c.
> > 
> > Alex, how would you want to proceed with that?  
> 
> Still reading the thread overnight, but if there's a better solution
> that Linus won't balk at then let's do it.  Thanks,

How soon were you thinking of sending an alternate proposal?  I don't
want to make this any messier, but this patch gives us some breathing
room for the merge window.  Thanks,

Alex