RE: [PATCH net-next 3/3] net: stmmac: Introducing support for Page Pool

From: Jose Abreu
Date: Thu Jul 04 2019 - 10:46:08 EST


From: Jesper Dangaard Brouer <brouer@xxxxxxxxxx>

> The page_pool_request_shutdown() API return indication if there are any
> in-flight frames/pages, to know when it is safe to call
> page_pool_free(), which you are also missing a call to.
>
> This page_pool_request_shutdown() is only intended to be called from
> xdp_rxq_info_unreg() code, that handles and schedule a work queue if it
> need to wait for in-flight frames/pages.

So you mean I can't call it or I should implement the same deferred work
?

Notice that in stmmac case there will be no in-flight frames/pages
because we free them all before calling this ...