Re: [PATCH v5 bpf-next 13/14] bpf: add new frame_length field to the XDP ctx

2021-02-01 Thread Eelco Chaudron
On 20 Jan 2021, at 14:20, Eelco Chaudron wrote: On 18 Jan 2021, at 17:48, Maciej Fijalkowski wrote: On Fri, Jan 15, 2021 at 05:36:23PM +0100, Eelco Chaudron wrote: On 16 Dec 2020, at 15:08, Eelco Chaudron wrote: On 15 Dec 2020, at 19:06, Maciej Fijalkowski wrote: On Tue, Dec 15, 2020

Re: [PATCH v5 bpf-next 13/14] bpf: add new frame_length field to the XDP ctx

2021-01-20 Thread Eelco Chaudron
On 18 Jan 2021, at 17:48, Maciej Fijalkowski wrote: On Fri, Jan 15, 2021 at 05:36:23PM +0100, Eelco Chaudron wrote: On 16 Dec 2020, at 15:08, Eelco Chaudron wrote: On 15 Dec 2020, at 19:06, Maciej Fijalkowski wrote: On Tue, Dec 15, 2020 at 02:28:39PM +0100, Eelco Chaudron wrote: On

Re: [PATCH v5 bpf-next 13/14] bpf: add new frame_length field to the XDP ctx

2021-01-18 Thread Maciej Fijalkowski
On Fri, Jan 15, 2021 at 05:36:23PM +0100, Eelco Chaudron wrote: > > > On 16 Dec 2020, at 15:08, Eelco Chaudron wrote: > > > On 15 Dec 2020, at 19:06, Maciej Fijalkowski wrote: > > > > > On Tue, Dec 15, 2020 at 02:28:39PM +0100, Eelco Chaudron wrote: > > > > > > > > > > > > On 9 Dec 2020, at 1

Re: [PATCH v5 bpf-next 13/14] bpf: add new frame_length field to the XDP ctx

2021-01-15 Thread Eelco Chaudron
On 16 Dec 2020, at 15:08, Eelco Chaudron wrote: On 15 Dec 2020, at 19:06, Maciej Fijalkowski wrote: On Tue, Dec 15, 2020 at 02:28:39PM +0100, Eelco Chaudron wrote: On 9 Dec 2020, at 13:07, Eelco Chaudron wrote: On 9 Dec 2020, at 12:10, Maciej Fijalkowski wrote: + + c

Re: [PATCH v5 bpf-next 13/14] bpf: add new frame_length field to the XDP ctx

2020-12-16 Thread Eelco Chaudron
On 15 Dec 2020, at 19:06, Maciej Fijalkowski wrote: > On Tue, Dec 15, 2020 at 02:28:39PM +0100, Eelco Chaudron wrote: >> >> >> On 9 Dec 2020, at 13:07, Eelco Chaudron wrote: >> >>> On 9 Dec 2020, at 12:10, Maciej Fijalkowski wrote: >> >> >> >>> + >>> + ctx_reg = (si->src_

Re: [PATCH v5 bpf-next 13/14] bpf: add new frame_length field to the XDP ctx

2020-12-15 Thread Maciej Fijalkowski
On Tue, Dec 15, 2020 at 02:28:39PM +0100, Eelco Chaudron wrote: > > > On 9 Dec 2020, at 13:07, Eelco Chaudron wrote: > > > On 9 Dec 2020, at 12:10, Maciej Fijalkowski wrote: > > > > > > > > > + > > > > > > + ctx_reg = (si->src_reg == si->dst_reg) ? scratch_reg - > > > > > > 1 : > >

Re: [PATCH v5 bpf-next 13/14] bpf: add new frame_length field to the XDP ctx

2020-12-15 Thread Eelco Chaudron
On 9 Dec 2020, at 13:07, Eelco Chaudron wrote: On 9 Dec 2020, at 12:10, Maciej Fijalkowski wrote: + + ctx_reg = (si->src_reg == si->dst_reg) ? scratch_reg - 1 : si->src_reg; + while (dst_reg == ctx_reg || scratch_reg == ctx_reg) + ctx_reg

Re: [PATCH v5 bpf-next 13/14] bpf: add new frame_length field to the XDP ctx

2020-12-09 Thread Eelco Chaudron
On 9 Dec 2020, at 12:10, Maciej Fijalkowski wrote: On Wed, Dec 09, 2020 at 11:35:13AM +0100, Eelco Chaudron wrote: On 8 Dec 2020, at 23:17, Maciej Fijalkowski wrote: On Mon, Dec 07, 2020 at 05:32:42PM +0100, Lorenzo Bianconi wrote: From: Eelco Chaudron This patch adds a new field to t

Re: [PATCH v5 bpf-next 13/14] bpf: add new frame_length field to the XDP ctx

2020-12-09 Thread Maciej Fijalkowski
On Wed, Dec 09, 2020 at 11:35:13AM +0100, Eelco Chaudron wrote: > > > On 8 Dec 2020, at 23:17, Maciej Fijalkowski wrote: > > > On Mon, Dec 07, 2020 at 05:32:42PM +0100, Lorenzo Bianconi wrote: > > > From: Eelco Chaudron > > > > > > This patch adds a new field to the XDP context called frame_le

Re: [PATCH v5 bpf-next 13/14] bpf: add new frame_length field to the XDP ctx

2020-12-09 Thread Eelco Chaudron
On 8 Dec 2020, at 23:17, Maciej Fijalkowski wrote: On Mon, Dec 07, 2020 at 05:32:42PM +0100, Lorenzo Bianconi wrote: From: Eelco Chaudron This patch adds a new field to the XDP context called frame_length, which will hold the full length of the packet, including fragments if existing. Th

Re: [PATCH v5 bpf-next 13/14] bpf: add new frame_length field to the XDP ctx

2020-12-08 Thread Maciej Fijalkowski
On Mon, Dec 07, 2020 at 05:32:42PM +0100, Lorenzo Bianconi wrote: > From: Eelco Chaudron > > This patch adds a new field to the XDP context called frame_length, > which will hold the full length of the packet, including fragments > if existing. The approach you took for ctx access conversion is