| Message ID | 20190215223741.16881.84864.stgit@localhost.localdomain (mailing list archive) |
|---|---|
| Headers | show |
| Series | Address recent issues found in netdev page_frag_alloc usage | expand |
From: Alexander Duyck <alexander.duyck@gmail.com> Date: Fri, 15 Feb 2019 14:44:05 -0800 > This patch set addresses a couple of issues that I had pointed out to Jann > Horn in response to a recent patch submission. > > The first issue is that I wanted to avoid the need to read/modify/write the > size value in order to generate the value for pagecnt_bias. Instead we can > just use a fixed constant which reduces the need for memory read operations > and the overall number of instructions to update the pagecnt bias values. > > The other, and more important issue is, that apparently we were letting tun > access the napi_alloc_cache indirectly through netdev_alloc_frag and as a > result letting it create unaligned accesses via unaligned allocations. In > order to prevent this I have added a call to SKB_DATA_ALIGN for the fragsz > field so that we will keep the offset in the napi_alloc_cache > SMP_CACHE_BYTES aligned. Series applied, thanks Alexander.
This patch set addresses a couple of issues that I had pointed out to Jann Horn in response to a recent patch submission. The first issue is that I wanted to avoid the need to read/modify/write the size value in order to generate the value for pagecnt_bias. Instead we can just use a fixed constant which reduces the need for memory read operations and the overall number of instructions to update the pagecnt bias values. The other, and more important issue is, that apparently we were letting tun access the napi_alloc_cache indirectly through netdev_alloc_frag and as a result letting it create unaligned accesses via unaligned allocations. In order to prevent this I have added a call to SKB_DATA_ALIGN for the fragsz field so that we will keep the offset in the napi_alloc_cache SMP_CACHE_BYTES aligned. --- Alexander Duyck (2): mm: Use fixed constant in page_frag_alloc instead of size + 1 net: Do not allocate page fragments that are not skb aligned mm/page_alloc.c | 8 ++++---- net/core/skbuff.c | 4 ++++ 2 files changed, 8 insertions(+), 4 deletions(-) --