diff options
author | Jesper Dangaard Brouer <brouer@redhat.com> | 2019-11-16 12:22:43 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-11-18 17:03:18 -0800 |
commit | 7c9e69428da39ed761c9d903c4850368fa4ef7bf (patch) | |
tree | d306499c9d6cff133cbd504135d61c703e53f5a2 /include/net/page_pool.h | |
parent | c491eae8f9c0720520ebdeb4d335671f84b84b71 (diff) | |
download | linux-7c9e69428da39ed761c9d903c4850368fa4ef7bf.tar.bz2 |
page_pool: add destroy attempts counter and rename tracepoint
When Jonathan change the page_pool to become responsible to its
own shutdown via deferred work queue, then the disconnect_cnt
counter was removed from xdp memory model tracepoint.
This patch change the page_pool_inflight tracepoint name to
page_pool_release, because it reflects the new responsability
better. And it reintroduces a counter that reflect the number of
times page_pool_release have been tried.
The counter is also used by the code, to only empty the alloc
cache once. With a stuck work queue running every second and
counter being 64-bit, it will overrun in approx 584 billion
years. For comparison, Earth lifetime expectancy is 7.5 billion
years, before the Sun will engulf, and destroy, the Earth.
Signed-off-by: Jesper Dangaard Brouer <brouer@redhat.com>
Acked-by: Toke Høiland-Jørgensen <toke@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'include/net/page_pool.h')
-rw-r--r-- | include/net/page_pool.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/include/net/page_pool.h b/include/net/page_pool.h index 1121faa99c12..ace881c15dcb 100644 --- a/include/net/page_pool.h +++ b/include/net/page_pool.h @@ -112,6 +112,8 @@ struct page_pool { * refcnt serves purpose is to simplify drivers error handling. */ refcount_t user_cnt; + + u64 destroy_cnt; }; struct page *page_pool_alloc_pages(struct page_pool *pool, gfp_t gfp); |