On 10/18/24 10:59, Dumitru Ceara wrote:
> This needs proper authorship, my bad:  I'll change the author to:
> 
> Author: Vladislav Odintsov <[email protected]>
> 

I fixed this up and applied the patch to 23.09.

Thanks,
Dumitru

> On 10/18/24 09:51, Dumitru Ceara wrote:
>> This picks up the following relevant OVS changes:
>>   27841b2c184b compiler: Fix errors in Clang 17 ubsan checks.
>>   a622f6cc6324 vlog: Destroy async_append first then close log_fd.
>>   34b4675aa639 hash, jhash: Fix unaligned access to the hash remainder.
>>   ... and others
>>
>> Reported-at: 
>> https://mail.openvswitch.org/pipermail/ovs-dev/2024-October/417627.html
>> Signed-off-by: Vladislav Odintsov <[email protected]>
>> ---
>>  controller/ofctrl.c | 2 +-
>>  ovs                 | 2 +-
>>  tests/test-ovn.c    | 2 +-
>>  3 files changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/controller/ofctrl.c b/controller/ofctrl.c
>> index 497890eed5..718baac18b 100644
>> --- a/controller/ofctrl.c
>> +++ b/controller/ofctrl.c
>> @@ -3059,7 +3059,7 @@ ofctrl_inject_pkt(const struct ovsrec_bridge *br_int, 
>> const char *flow_s,
>>      uint64_t packet_stub[128 / 8];
>>      struct dp_packet packet;
>>      dp_packet_use_stub(&packet, packet_stub, sizeof packet_stub);
>> -    flow_compose(&packet, &uflow, NULL, 64);
>> +    flow_compose(&packet, &uflow, NULL, 64, false);
>>  
>>      uint64_t ofpacts_stub[1024 / 8];
>>      struct ofpbuf ofpacts = OFPBUF_STUB_INITIALIZER(ofpacts_stub);
>> diff --git a/ovs b/ovs
>> index c88a35fc29..14f8765d65 160000
>> --- a/ovs
>> +++ b/ovs
>> @@ -1 +1 @@
>> -Subproject commit c88a35fc29f0c0eb6189853bfc738c2100d4860f
>> +Subproject commit 14f8765d65ef98a8793ed3937003a3bbdfc90dd4
>> diff --git a/tests/test-ovn.c b/tests/test-ovn.c
>> index 16d2d779db..6f38b1493a 100644
>> --- a/tests/test-ovn.c
>> +++ b/tests/test-ovn.c
>> @@ -1238,7 +1238,7 @@ test_expr_to_packets(struct ovs_cmdl_context *ctx 
>> OVS_UNUSED)
>>          uint64_t packet_stub[128 / 8];
>>          struct dp_packet packet;
>>          dp_packet_use_stub(&packet, packet_stub, sizeof packet_stub);
>> -        flow_compose(&packet, &uflow, NULL, 64);
>> +        flow_compose(&packet, &uflow, NULL, 64, false);
>>  
>>          struct ds output = DS_EMPTY_INITIALIZER;
>>          const uint8_t *buf = dp_packet_data(&packet);

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to