Re: [Intel-wired-lan] [PATCH iwl-next v2 1/1] igc: Improve XDP_SETUP_PROG process

2024-12-16 Thread Song, Yoong Siang
On Friday, December 13, 2024 4:10 PM, Paul Menzel wrote: >Dear Song, > > >Thank you for your patch. Maybe for the summary/title you could be more >specific: > >igc: Avoid unnecessary link down event in XDP_SETUP_PROG process > > >Am 11.12.24 um 14:45 schrieb Song Yoong Siang: >> Improve XDP_SETUP_

Re: [Intel-wired-lan] [PATCH iwl-next v2 1/1] igc: Improve XDP_SETUP_PROG process

2024-12-13 Thread Paul Menzel
Dear Song, Thank you for your patch. Maybe for the summary/title you could be more specific: igc: Avoid unnecessary link down event in XDP_SETUP_PROG process Am 11.12.24 um 14:45 schrieb Song Yoong Siang: Improve XDP_SETUP_PROG process by avoiding unnecessary link down event. This patch i

Re: [Intel-wired-lan] [PATCH iwl-next v2 1/1] igc: Improve XDP_SETUP_PROG process

2024-12-12 Thread Song, Yoong Siang
On Thursday, December 12, 2024 10:09 PM, Fijalkowski, Maciej wrote: >On Wed, Dec 11, 2024 at 09:45:32PM +0800, Song Yoong Siang wrote: >> Improve XDP_SETUP_PROG process by avoiding unnecessary link down event. >> [...] > >I agree we could do better than igc_close/igc_open pair, but have you >trie

Re: [Intel-wired-lan] [PATCH iwl-next v2 1/1] igc: Improve XDP_SETUP_PROG process

2024-12-12 Thread Maciej Fijalkowski
On Wed, Dec 11, 2024 at 09:45:32PM +0800, Song Yoong Siang wrote: > Improve XDP_SETUP_PROG process by avoiding unnecessary link down event. > > This patch is tested by using ip link set xdpdrv command to attach a simple > XDP program which always return XDP_PASS. > > Before this patch, attaching

[Intel-wired-lan] [PATCH iwl-next v2 1/1] igc: Improve XDP_SETUP_PROG process

2024-12-11 Thread Song Yoong Siang
Improve XDP_SETUP_PROG process by avoiding unnecessary link down event. This patch is tested by using ip link set xdpdrv command to attach a simple XDP program which always return XDP_PASS. Before this patch, attaching xdp program will cause ptp4l to lost sync for few seconds, as shown in ptp4l l