[lvc-project] [PATCH 5.10] net: bridge: mcast: wait for previous gc cycles when removing port

[Fedor Pchelkin]

On Wed, 04. Mar 16:14, Votokina Victoria wrote:
> From: Victoria Votokina <Victoria.Votokina at kaspersky.com>
> 

Строка выше не должна попадать в тело патча.  Её может автоматически
подставлять git send-email, если настоящий почтовый From, который есть

  Votokina Victoria <Victoria.Votokina at kaspersky.com>

отличается от того, что прописано в ~/.gitconfig.  А там судя по всему

  Victoria Votokina <Victoria.Votokina at kaspersky.com>

с другим порядком имени и фамилии.

Избежать этого поможет приведение конфигураций к одному порядку, либо
можно наверно с опциями git send-email повозиться, но здесь не знаю
https://git-scm.com/docs/git-send-email#Documentation/git-send-email.txt---fromaddress

> Signed-off-by: Victoria Votokina <Victoria.Votokina at kaspersky.com>
> ---

Добавил краткий коммент бэкпортёра, который поясняет суть конфликта
апстримного патча в контексте ветки 5.10 и который _предположительно_
имелся вами ввиду по умолчанию.  Это стоит кратко указывать, чтобы
другим разработчикам и ревьюерам было понятно, что пришлось сделать для
переноса патча в целевую ветку, без необходимости с нуля залазить в дебри
и изучать проблемное место.

[ Victoria: minor conflict adaptation due to missing commit 9632233e7de8
  ("net: bridge: multicast: factor out port multicast context") ]

Подобный комментарий стоит оставлять над своей подписью Signed-off-by.

Подправил эти вещи при применении в lvc-ветку.  Просьба учитывать при
будущих патчах и бэкпортах.

> Backport fix for CVE-2024-44934
>  net/bridge/br_multicast.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/net/bridge/br_multicast.c b/net/bridge/br_multicast.c
> index e5328a2777ec..226183dedcbe 100644
> --- a/net/bridge/br_multicast.c
> +++ b/net/bridge/br_multicast.c
> @@ -1618,16 +1618,14 @@ void br_multicast_del_port(struct net_bridge_port *port)
>  {
>  	struct net_bridge *br = port->br;
>  	struct net_bridge_port_group *pg;
> -	HLIST_HEAD(deleted_head);
>  	struct hlist_node *n;
>  
>  	/* Take care of the remaining groups, only perm ones should be left */
>  	spin_lock_bh(&br->multicast_lock);
>  	hlist_for_each_entry_safe(pg, n, &port->mglist, mglist)
>  		br_multicast_find_del_pg(br, pg);
> -	hlist_move_list(&br->mcast_gc_list, &deleted_head);
>  	spin_unlock_bh(&br->multicast_lock);
> -	br_multicast_gc(&deleted_head);
> +	flush_work(&br->mcast_gc_work);
>  	del_timer_sync(&port->multicast_router_timer);
>  	free_percpu(port->mcast_stats);
>  }
> -- 
> 2.43.0