Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 06 Dec 2019 22:21:58 +0000
From:      "Bjoern A. Zeeb" <bz@FreeBSD.org>
To:        "Gleb Smirnoff" <glebius@freebsd.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r355449 - head/sys/netinet
Message-ID:  <128F09FF-8F61-4967-9558-1C7F8976204F@FreeBSD.org>
In-Reply-To: <20191206220925.GJ2706@FreeBSD.org>
References:  <201912061635.xB6GZmpS042391@repo.freebsd.org> <20191206220925.GJ2706@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 6 Dec 2019, at 22:09, Gleb Smirnoff wrote:

>    Hi,
>
> there is a functional change here:
>
> On Fri, Dec 06, 2019 at 04:35:48PM +0000, Bjoern A. Zeeb wrote:
> B> URL: https://svnweb.freebsd.org/changeset/base/355449
> B>
> B> Log:
> B>   carp: replace caddr_t with char *
> B>
> B>   Change the remaining caddr_t usages to char * following the 
> removal
> B>   of the KAME macros
> B>
> B>   No functional change.
> ...
> B> @@ -978,6 +978,7 @@ carp_send_ad_locked(struct carp_softc *sc)
> B>  #endif /* INET */
> B>  #ifdef INET6
> B>  	if (sc->sc_naddrs6) {
> B> +		struct epoch_tracker et;
> B>  		struct ip6_hdr *ip6;
> B>
> B>  		m = m_gethdr(M_NOWAIT, MT_DATA);
> B> @@ -1031,8 +1032,10 @@ carp_send_ad_locked(struct carp_softc *sc)
> B>
> B>  		CARPSTATS_INC(carps_opackets6);
> B>
> B> +		NET_EPOCH_ENTER(et);
> B>  		carp_send_ad_error(sc, ip6_output(m, NULL, NULL, 0,
> B>  		    &sc->sc_carpdev->if_carp->cif_im6o, NULL, NULL));
> B> +		NET_EPOCH_EXIT(et);
> B>  	}
> B>  #endif /* INET6 */
>
> This could be correct change, just needs some explanation.

Sorry.  My bad.  I had a patch from a different branch applied in 
between review and commit and forgot to clean up before committing.

I’ll put the epoch changes up for review the next hours/days.  I have 
reverted this part in the meantime in 355466.

Thanks for catching this.

/bz





Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?128F09FF-8F61-4967-9558-1C7F8976204F>