On Mon, Mar 04, 2024 at 06:01:37PM -0800, Mina Almasry wrote: > From: Jakub Kicinski <kuba@xxxxxxxxxx> > > The page providers which try to reuse the same pages will > need to hold onto the ref, even if page gets released from > the pool - as in releasing the page from the pp just transfers > the "ownership" reference from pp to the provider, and provider > will wait for other references to be gone before feeding this > page back into the pool. The word hook always rings a giant warning bell for me, and looking into this series I am concerned indeed. The only provider provided here is the dma-buf one, and that basically is the only sensible one for the documented design. So instead of adding hooks that random proprietary crap can hook into, why not hard code the dma buf provide and just use a flag? That'll also avoid expensive indirect calls.
