RE: [PATCH iwl-next v6 4/6] igb: Introduce XSK data structures and helpers

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

 




> -----Original Message-----
> From: Maciej Fijalkowski <maciej.fijalkowski@xxxxxxxxx>
> Sent: Monday, 19 August 2024 16:11
> To: Kurt Kanzenbach <kurt@xxxxxxxxxxxxx>
> Cc: Tony Nguyen <anthony.l.nguyen@xxxxxxxxx>; Przemek Kitszel
> <przemyslaw.kitszel@xxxxxxxxx>; David S. Miller <davem@xxxxxxxxxxxxx>;
> Eric Dumazet <edumazet@xxxxxxxxxx>; Jakub Kicinski <kuba@xxxxxxxxxx>;
> Paolo Abeni <pabeni@xxxxxxxxxx>; Alexei Starovoitov <ast@xxxxxxxxxx>;
> Daniel Borkmann <daniel@xxxxxxxxxxxxx>; Jesper Dangaard Brouer
> <hawk@xxxxxxxxxx>; John Fastabend <john.fastabend@xxxxxxxxx>; Richard
> Cochran <richardcochran@xxxxxxxxx>; Sriram Yagnaraman
> <sriram.yagnaraman@xxxxxxxxxxxx>; Benjamin Steinke
> <benjamin.steinke@xxxxxxxxxxxxxx>; Sebastian Andrzej Siewior
> <bigeasy@xxxxxxxxxxxxx>; intel-wired-lan@xxxxxxxxxxxxxxxx;
> netdev@xxxxxxxxxxxxxxx; bpf@xxxxxxxxxxxxxxx; Sriram Yagnaraman
> <sriram.yagnaraman@xxxxxxxx>
> Subject: Re: [PATCH iwl-next v6 4/6] igb: Introduce XSK data structures and
> helpers
>
> On Mon, Aug 19, 2024 at 03:41:20PM +0200, Kurt Kanzenbach wrote:
> > On Mon Aug 19 2024, Maciej Fijalkowski wrote:
> > > On Fri, Aug 16, 2024 at 11:24:03AM +0200, Kurt Kanzenbach wrote:
> > >> From: Sriram Yagnaraman <sriram.yagnaraman@xxxxxxxx>
> > >>
> > >> Add the following ring flag:
> > >> - IGB_RING_FLAG_TX_DISABLED (when xsk pool is being setup)
> > >>
> > >> Add a xdp_buff array for use with XSK receive batch API, and a
> > >> pointer to xsk_pool in igb_adapter.
> > >>
> > >> Add enable/disable functions for TX and RX rings.
> > >> Add enable/disable functions for XSK pool.
> > >> Add xsk wakeup function.
> > >>
> > >> None of the above functionality will be active until
> > >> NETDEV_XDP_ACT_XSK_ZEROCOPY is advertised in netdev-
> >xdp_features.
> > >>
> > >> Signed-off-by: Sriram Yagnaraman <sriram.yagnaraman@xxxxxxxx>
> > >
> > > Sriram's mail bounces unfortunately, is it possible to grab his
> > > current address?
> >
> > His current email address is in the Cc list. However, i wasn't sure if
> > it's okay to update the From and SoB of these patches?
>
> Okay. Then I believe Sriram should provide a mailmap entry to map his old
> mail to a new one.

Please feel free to remove my "est.tech" address from From: and Signed-of-By:
I am just glad that my work has not gone to waste. Thank you for that.
I will check with my company's *lawyers* to see if I can provide a mailmap to my current address :(

>
> >
> > >
> > > You could also update the copyright date in igb_xsk.c.
> >
> > Ditto for the copyright. It probably has to be something like
> > Copyright(c) 2023 Ericsson?
>
> It says 2018 Intel. I don't think Sriram was working under E/// employment as
> he said he was forbidden to work on this further and that's why you picked it
> up, right?
>
> My intent was not stir up the copyright pot, though. It can be left as-is or have
> something of a Linutronix/Sriram Yagnamaran mix :P
>
> >
> > Thanks,
> > Kurt
>





[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux