netdev - Re: [PATCH v2 bpf-next 1/2] bpf: Allow bpf_map_lookup

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4AC230DE-6529-4798-BE32-C0AD873855CB@gmail.com>
Date:   Fri, 31 May 2019 09:27:06 -0700
From:   "Jonathan Lemon" <jonathan.lemon@...il.com>
To:     "Björn Töpel" <bjorn.topel@...el.com>
Cc:     netdev@...r.kernel.org, magnus.karlsson@...el.com,
        kernel-team@...com, bpf <bpf@...r.kernel.org>
Subject: Re: [PATCH v2 bpf-next 1/2] bpf: Allow bpf_map_lookup_elem() on an
 xskmap

On 31 May 2019, at 4:49, Björn Töpel wrote:

> On 2019-05-30 20:57, Jonathan Lemon wrote:
>> Currently, the AF_XDP code uses a separate map in order to
>> determine if an xsk is bound to a queue.  Instead of doing this,
>> have bpf_map_lookup_elem() return the queue_id, as a way of
>> indicating that there is a valid entry at the map index.
>>
>> Rearrange some xdp_sock members to eliminate structure holes.
>>
>> Signed-off-by: Jonathan Lemon <jonathan.lemon@...il.com>
>> ---
>>   include/net/xdp_sock.h                            |  6 +++---
>>   kernel/bpf/verifier.c                             |  6 +++++-
>>   kernel/bpf/xskmap.c                               |  4 +++-
>>   .../selftests/bpf/verifier/prevent_map_lookup.c   | 15 ---------------
>>   4 files changed, 11 insertions(+), 20 deletions(-)
>>
>> diff --git a/include/net/xdp_sock.h b/include/net/xdp_sock.h
>> index d074b6d60f8a..7d84b1da43d2 100644
>> --- a/include/net/xdp_sock.h
>> +++ b/include/net/xdp_sock.h
>> @@ -57,12 +57,12 @@ struct xdp_sock {
>>   	struct net_device *dev;
>>   	struct xdp_umem *umem;
>>   	struct list_head flush_node;
>> -	u16 queue_id;
>> -	struct xsk_queue *tx ____cacheline_aligned_in_smp;
>> -	struct list_head list;
>> +	u32 queue_id;
>
> Why the increase of size?

Currently xskmaps are defined as size=4, so the returned pointer for
bpf must have that many bytes.  I was kicking around the idea of using
a union and returning { qid, zc }, but Alexei pointed out that doesn't
make a good API.

Besides, queue_index in struct xdp_rxq_info is already a u32.
-- 
Jonathan