Re: [ovs-dev] [PATCH v3] tc: fix crash on EAGAIN return from recvmsg on netlink socket.

2023-05-22 Thread Frode Nordahl
On Mon, May 22, 2023 at 4:31 PM Eelco Chaudron wrote: > > > > On 22 May 2023, at 12:32, Frode Nordahl wrote: > > > On Mon, May 22, 2023 at 11:33 AM Eelco Chaudron wrote: > >> > >> > >> > >> On 19 May 2023, at 12:31, Frode Nordahl wrote: > >> > >>> On Fri, May 19, 2023 at 11:55 AM Eelco Chaudron

Re: [ovs-dev] [PATCH v3] tc: fix crash on EAGAIN return from recvmsg on netlink socket.

2023-05-22 Thread Eelco Chaudron
On 22 May 2023, at 12:32, Frode Nordahl wrote: > On Mon, May 22, 2023 at 11:33 AM Eelco Chaudron wrote: >> >> >> >> On 19 May 2023, at 12:31, Frode Nordahl wrote: >> >>> On Fri, May 19, 2023 at 11:55 AM Eelco Chaudron wrote: On 15 May 2023, at 10:04, Frode Nordahl wrote: >>

Re: [ovs-dev] [PATCH v3] tc: fix crash on EAGAIN return from recvmsg on netlink socket.

2023-05-22 Thread Frode Nordahl
On Mon, May 22, 2023 at 11:33 AM Eelco Chaudron wrote: > > > > On 19 May 2023, at 12:31, Frode Nordahl wrote: > > > On Fri, May 19, 2023 at 11:55 AM Eelco Chaudron wrote: > >> > >> > >> > >> On 15 May 2023, at 10:04, Frode Nordahl wrote: > >> > >>> The tc module combines the use of the `tc_transa

Re: [ovs-dev] [PATCH v3] tc: fix crash on EAGAIN return from recvmsg on netlink socket.

2023-05-22 Thread Eelco Chaudron
On 19 May 2023, at 12:31, Frode Nordahl wrote: > On Fri, May 19, 2023 at 11:55 AM Eelco Chaudron wrote: >> >> >> >> On 15 May 2023, at 10:04, Frode Nordahl wrote: >> >>> The tc module combines the use of the `tc_transact` helper >>> function for communication with the in-kernel tc infrastructur

Re: [ovs-dev] [PATCH v3] tc: fix crash on EAGAIN return from recvmsg on netlink socket.

2023-05-19 Thread Frode Nordahl
On Fri, May 19, 2023 at 11:55 AM Eelco Chaudron wrote: > > > > On 15 May 2023, at 10:04, Frode Nordahl wrote: > > > The tc module combines the use of the `tc_transact` helper > > function for communication with the in-kernel tc infrastructure > > with assertions on the reply data by `ofpbuf_at_ass

Re: [ovs-dev] [PATCH v3] tc: fix crash on EAGAIN return from recvmsg on netlink socket.

2023-05-19 Thread Eelco Chaudron
On 15 May 2023, at 10:04, Frode Nordahl wrote: > The tc module combines the use of the `tc_transact` helper > function for communication with the in-kernel tc infrastructure > with assertions on the reply data by `ofpbuf_at_assert` on the > received data prior to further processing. > > `tc_tran

Re: [ovs-dev] [PATCH v3] tc: fix crash on EAGAIN return from recvmsg on netlink socket.

2023-05-17 Thread Simon Horman
On Mon, May 15, 2023 at 10:04:48AM +0200, Frode Nordahl wrote: > The tc module combines the use of the `tc_transact` helper > function for communication with the in-kernel tc infrastructure > with assertions on the reply data by `ofpbuf_at_assert` on the > received data prior to further processing.

Re: [ovs-dev] [PATCH v3] tc: fix crash on EAGAIN return from recvmsg on netlink socket.

2023-05-15 Thread Roi Dayan via dev
On 15/05/2023 11:04, Frode Nordahl wrote: > The tc module combines the use of the `tc_transact` helper > function for communication with the in-kernel tc infrastructure > with assertions on the reply data by `ofpbuf_at_assert` on the > received data prior to further processing. > > `tc_transact

[ovs-dev] [PATCH v3] tc: fix crash on EAGAIN return from recvmsg on netlink socket.

2023-05-15 Thread Frode Nordahl
The tc module combines the use of the `tc_transact` helper function for communication with the in-kernel tc infrastructure with assertions on the reply data by `ofpbuf_at_assert` on the received data prior to further processing. `tc_transact` in turn calls `nl_transact`, which via `nl_transact_mul