Re: [RFC PATCH 05/10] qemu_capabilities: Added capability for qemu's "ebpf_rss_fds".

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,
No problem, those patches are RFC.
If there would be issues with the libvirt interface - qemu patches should not be applied either.

On Fri, Aug 20, 2021 at 3:59 PM Michal Prívozník <mprivozn@xxxxxxxxxx> wrote:
On 8/20/21 2:57 PM, Michal Prívozník wrote:
> On 7/28/21 10:17 AM, Andrew Melnychenko wrote:
>> Added check for qemu virtio-net "ebpf_rss_fds" property.
>> This property allows to pass eBPF program/map file descriptors for RSS program.
>> In other patches, libvirt may launch qemu-ebpf-rss-helper and pass fds
>> to virtio-net qemu.
>>
>> Signed-off-by: Andrew Melnychenko <andrew@xxxxxxxxxx>
>> ---
>>  src/qemu/qemu_capabilities.c | 2 ++
>>  src/qemu/qemu_capabilities.h | 1 +
>>  2 files changed, 3 insertions(+)
>>
>> diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
>> index 7cd6dab259..f16115226d 100644
>> --- a/src/qemu/qemu_capabilities.c
>> +++ b/src/qemu/qemu_capabilities.c
>> @@ -638,6 +638,7 @@ VIR_ENUM_IMPL(virQEMUCaps,
>>                "query-display-options",
>>                "s390-pv-guest",
>>                "virtio-net.rss",
>> +              "virtio-net.ebpf_rss_fds",
>>      );
>> 
>> 
>> @@ -1423,6 +1424,7 @@ static struct virQEMUCapsDevicePropsFlags virQEMUCapsDevicePropsVirtioNet[] = {
>>      { "packed", QEMU_CAPS_VIRTIO_PACKED_QUEUES, NULL },
>>      { "acpi-index", QEMU_CAPS_ACPI_INDEX, NULL },
>>      { "rss", QEMU_CAPS_VIRTIO_RSS, NULL },
>> +    { "ebpf_rss_fds", QEMU_CAPS_VIRTIO_EBPF_RSS_FDS, NULL },
>
> I haven't found this in QEMU. Looking into qemu-devel list it was
> proposed but apparently is not merged yet:
>
> https://lists.nongnu.org/archive/html/qemu-devel/2021-07/msg03536.html

Ooops, hit send too early.

What I wanted to say is that it's perfectly okay to send patches before
QEMU part is merged. However, libvirt part won't be merged before QEMU.
We did this mistake too many times before and learned our lesson.

Michal


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux