Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 18 Apr 2024 13:54:41 -0600
From:      Warner Losh <imp@bsdimp.com>
To:        John Baldwin <jhb@freebsd.org>
Cc:        Ed Maste <emaste@freebsd.org>, src-committers <src-committers@freebsd.org>,  "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>,  "<dev-commits-src-main@freebsd.org>" <dev-commits-src-main@freebsd.org>
Subject:   Re: git: ede456359e55 - main - CONTRIBUTING: mention sys/crypto as another "contrib" directory
Message-ID:  <CANCZdfqcHEqH8JgRwXpH1x3r81Q6ANa35vj365phAkjQnxAe5w@mail.gmail.com>
In-Reply-To: <f2df2c62-a73f-49d8-b617-17f1a9226d4e@FreeBSD.org>
References:  <202404181857.43IIviHV084762@gitrepo.freebsd.org> <f2df2c62-a73f-49d8-b617-17f1a9226d4e@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
--0000000000004b42250616645837
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Thu, Apr 18, 2024, 1:10=E2=80=AFPM John Baldwin <jhb@freebsd.org> wrote:

> On 4/18/24 11:57 AM, Ed Maste wrote:
> > The branch main has been updated by emaste:
> >
> > URL:
> https://cgit.FreeBSD.org/src/commit/?id=3Dede456359e55958ab17ac665e586767=
4a0c0e81b
> >
> > commit ede456359e55958ab17ac665e5867674a0c0e81b
> > Author:     Ed Maste <emaste@FreeBSD.org>
> > AuthorDate: 2024-04-18 18:44:30 +0000
> > Commit:     Ed Maste <emaste@FreeBSD.org>
> > CommitDate: 2024-04-18 18:57:19 +0000
> >
> >      CONTRIBUTING: mention sys/crypto as another "contrib" directory
> >
> >      Reviewed by:    imp
> >      Sponsored by:   The FreeBSD Foundation
> >      Differential Revision: https://reviews.freebsd.org/D44857
>
> Hmm, sys/crypto is a bit of a mixed bag there.  openssl is a mix of
> upstream
> and local bits.  libsodium is an upstream.  chacha20 and blake2 I think a=
re
> upstreams.  rijndael did come from an upstream but I don't think it's
> alive.
>
> I'd actually prefer if we moved a lot of sys/crypto over to sys/dev, e.g.
> actual device drivers (as opposed to software crypto implementations) lik=
e
> aesni, armv8, via, should definitely move to sys/dev.  For sys/openssl it
> might be nicer then to have the home-grown bits of ossl(4) that aren't fr=
om
> upstream in a sys/dev/ossl leaving the stock openssl bits in
> sys/crypto/openssl.
>

I like this idea... git makes it easier... this area has historical been a
hodgepodge..

Warner

>

--0000000000004b42250616645837
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"auto"><div><br><br><div class=3D"gmail_quote"><div dir=3D"ltr" =
class=3D"gmail_attr">On Thu, Apr 18, 2024, 1:10=E2=80=AFPM John Baldwin &lt=
;<a href=3D"mailto:jhb@freebsd.org">jhb@freebsd.org</a>&gt; wrote:<br></div=
><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1=
px #ccc solid;padding-left:1ex">On 4/18/24 11:57 AM, Ed Maste wrote:<br>
&gt; The branch main has been updated by emaste:<br>
&gt; <br>
&gt; URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3Dede456359e55=
958ab17ac665e5867674a0c0e81b" rel=3D"noreferrer noreferrer" target=3D"_blan=
k">https://cgit.FreeBSD.org/src/commit/?id=3Dede456359e55958ab17ac665e58676=
74a0c0e81b</a><br>
&gt; <br>
&gt; commit ede456359e55958ab17ac665e5867674a0c0e81b<br>
&gt; Author:=C2=A0 =C2=A0 =C2=A0Ed Maste &lt;emaste@FreeBSD.org&gt;<br>
&gt; AuthorDate: 2024-04-18 18:44:30 +0000<br>
&gt; Commit:=C2=A0 =C2=A0 =C2=A0Ed Maste &lt;emaste@FreeBSD.org&gt;<br>
&gt; CommitDate: 2024-04-18 18:57:19 +0000<br>
&gt; <br>
&gt;=C2=A0 =C2=A0 =C2=A0 CONTRIBUTING: mention sys/crypto as another &quot;=
contrib&quot; directory<br>
&gt;=C2=A0 =C2=A0 =C2=A0 <br>
&gt;=C2=A0 =C2=A0 =C2=A0 Reviewed by:=C2=A0 =C2=A0 imp<br>
&gt;=C2=A0 =C2=A0 =C2=A0 Sponsored by:=C2=A0 =C2=A0The FreeBSD Foundation<b=
r>
&gt;=C2=A0 =C2=A0 =C2=A0 Differential Revision: <a href=3D"https://reviews.=
freebsd.org/D44857" rel=3D"noreferrer noreferrer" target=3D"_blank">https:/=
/reviews.freebsd.org/D44857</a><br>
<br>
Hmm, sys/crypto is a bit of a mixed bag there.=C2=A0 openssl is a mix of up=
stream<br>
and local bits.=C2=A0 libsodium is an upstream.=C2=A0 chacha20 and blake2 I=
 think are<br>
upstreams.=C2=A0 rijndael did come from an upstream but I don&#39;t think i=
t&#39;s alive.<br>
<br>
I&#39;d actually prefer if we moved a lot of sys/crypto over to sys/dev, e.=
g.<br>
actual device drivers (as opposed to software crypto implementations) like<=
br>
aesni, armv8, via, should definitely move to sys/dev.=C2=A0 For sys/openssl=
 it<br>
might be nicer then to have the home-grown bits of ossl(4) that aren&#39;t =
from<br>
upstream in a sys/dev/ossl leaving the stock openssl bits in sys/crypto/ope=
nssl.<br></blockquote></div></div><div dir=3D"auto"><br></div><div dir=3D"a=
uto">I like this idea... git makes it easier... this area has historical be=
en a hodgepodge..</div><div dir=3D"auto"><br></div><div dir=3D"auto">Warner=
=C2=A0</div><div dir=3D"auto"><div class=3D"gmail_quote"><blockquote class=
=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;padd=
ing-left:1ex"></blockquote></div></div><div dir=3D"auto"><div class=3D"gmai=
l_quote"><blockquote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;borde=
r-left:1px #ccc solid;padding-left:1ex"></blockquote></div></div><div dir=
=3D"auto"><div class=3D"gmail_quote"><blockquote class=3D"gmail_quote" styl=
e=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div></div>

--0000000000004b42250616645837--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfqcHEqH8JgRwXpH1x3r81Q6ANa35vj365phAkjQnxAe5w>