Re: [PATCH 5/5] qmp: Added qemu-ebpf-rss-path command.

[Markus Armbruster]

Andrew Melnichenko <andrew@daynix.com> writes:

> Hi,
>
>> Got something I could git-pull?
>>
> I can share some links for tests:
> https://github.com/daynix/qemu/tree/HelperEBPFv3 - qemu with helper
> https://github.com/daynix/libvirt/tree/RSS_RFC_v1 - libvirt with RSS

Thanks!

I get a few warnings.  I'm copying the ones from Clang:

../hw/net/virtio-net.c:1274:63: warning: array index 2 is past the end of the 
array (which contains 2 elements) [-Warray-bounds]
        ret = ebpf_rss_load_fds(&n->ebpf_rss, fds[0], fds[1], fds[2], fds[3]);
                                                              ^   ~
../hw/net/virtio-net.c:1257:5: note: array 'fds' declared here
    int fds[EBPF_RSS_MAX_FDS];
    ^
../hw/net/virtio-net.c:1274:71: warning: array index 3 is past the end of the 
array (which contains 2 elements) [-Warray-bounds]
        ret = ebpf_rss_load_fds(&n->ebpf_rss, fds[0], fds[1], fds[2], fds[3]);
                                                                      ^   ~
../hw/net/virtio-net.c:1257:5: note: array 'fds' declared here
    int fds[EBPF_RSS_MAX_FDS];
    ^

../ebpf/qemu-ebpf-rss-helper.c:30:6: warning: no previous prototype for 
function 'qemuHelperStamp_a59ff8818a819adc85c28504b7961bdf89f0d9c0' 
[-Wmissing-prototypes]
void QEMU_HELPER_STAMP(void) {}
     ^
./qemu-helper-stamp.h:1:27: note: expanded from macro 'QEMU_HELPER_STAMP'
#define QEMU_HELPER_STAMP 
qemuHelperStamp_a59ff8818a819adc85c28504b7961bdf89f0d9c0
                          ^
../ebpf/qemu-ebpf-rss-helper.c:30:1: note: declare 'static' if the function is 
not intended to be used outside of this translation unit
void QEMU_HELPER_STAMP(void) {}
^
static