Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 7 Oct 2021 01:20:05 -0500
From:      Kyle Evans <kevans@freebsd.org>
To:        Alexey Dokuchaev <danfe@freebsd.org>
Cc:        src-committers <src-committers@freebsd.org>,  "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>, dev-commits-src-branches@freebsd.org
Subject:   Re: git: 000aad3d093a - stable/12 - loader: allocate properly aligned buffer for network packet
Message-ID:  <CACNAnaFqymg6QQa1ATd0dyreXBa0-u-5Hb=kjjs2igjASFRwEw@mail.gmail.com>
In-Reply-To: <YV6OVKE6dNy%2BkigE@FreeBSD.org>
References:  <202110070337.1973bgRH073900@gitrepo.freebsd.org> <YV6OVKE6dNy%2BkigE@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Oct 7, 2021 at 1:06 AM Alexey Dokuchaev <danfe@freebsd.org> wrote:
>
> On Thu, Oct 07, 2021 at 03:37:42AM +0000, Kyle Evans wrote:
> > commit 000aad3d093a376bb1104a284b4102149db43155
> >
> >     loader: allocate properly aligned buffer for network packet
> >
> >     Use memalign(4, size) to ensure we have properly aligned buffer.
> >
> >     (cherry picked from commit 659bf32dfc595b6cd6aeda7f05cb57872c64d2d1)
>
> I don't understand, so this is a merge of the commit from master (main) to
> stablle/12 which had then been reverted?  So why do the merge in the first
> place?
>

Because commit + revert pairs are noise in the MFC tracker that
distracts from actual candidates for merging. The only feature of svn
I miss is --record-only merge.

Thanks,

Kyle Evans



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CACNAnaFqymg6QQa1ATd0dyreXBa0-u-5Hb=kjjs2igjASFRwEw>