git: c6b2d024d7ee - main - Retore the vnet before returning an error.

Kevin Bowling kevin.bowling at kev009.com
Thu Jul 1 18:33:00 UTC 2021


Hi George,

After the git transition we use the commit author field itself to
communicate authorship.  The obtained field should say Netapp, Inc.
See section 9.14
https://docs.freebsd.org/en/articles/committers-guide/#commit-log-message

Regards,
Kevin

On Mon, Jun 21, 2021 at 7:50 AM George V. Neville-Neil <gnn at freebsd.org> wrote:
>
> The branch main has been updated by gnn:
>
> URL: https://cgit.FreeBSD.org/src/commit/?id=c6b2d024d7eedbf32f52a17bc029c92f5a4d1a54
>
> commit c6b2d024d7eedbf32f52a17bc029c92f5a4d1a54
> Author:     George V. Neville-Neil <gnn at FreeBSD.org>
> AuthorDate: 2021-06-21 14:46:20 +0000
> Commit:     George V. Neville-Neil <gnn at FreeBSD.org>
> CommitDate: 2021-06-21 14:46:20 +0000
>
>     Retore the vnet before returning an error.
>
>     Obtained from:  Kanndula, Dheeraj <Dheeraj.Kandula at netapp.com>
>     MFC after:      2 weeks
>     Differential Revision:  https://reviews.freebsd.org/D30741
> ---
>  sys/net/if_vlan.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/sys/net/if_vlan.c b/sys/net/if_vlan.c
> index bd3a5335a97f..39f767c2a603 100644
> --- a/sys/net/if_vlan.c
> +++ b/sys/net/if_vlan.c
> @@ -609,6 +609,7 @@ vlan_setmulti(struct ifnet *ifp)
>                 mc = malloc(sizeof(struct vlan_mc_entry), M_VLAN, M_NOWAIT);
>                 if (mc == NULL) {
>                         IF_ADDR_WUNLOCK(ifp);
> +                       CURVNET_RESTORE();
>                         return (ENOMEM);
>                 }
>                 bcopy(ifma->ifma_addr, &mc->mc_addr, ifma->ifma_addr->sa_len);
> @@ -619,8 +620,10 @@ vlan_setmulti(struct ifnet *ifp)
>         CK_SLIST_FOREACH (mc, &sc->vlan_mc_listhead, mc_entries) {
>                 error = if_addmulti(ifp_p, (struct sockaddr *)&mc->mc_addr,
>                     NULL);
> -               if (error)
> +               if (error) {
> +                       CURVNET_RESTORE();
>                         return (error);
> +               }
>         }
>
>         CURVNET_RESTORE();
> _______________________________________________
> dev-commits-src-main at freebsd.org mailing list
> https://lists.freebsd.org/mailman/listinfo/dev-commits-src-main
> To unsubscribe, send any mail to "dev-commits-src-main-unsubscribe at freebsd.org"


More information about the dev-commits-src-all mailing list