Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 22 Jun 2021 12:36:26 +0100
From:      Jessica Clarke <jrtc27@freebsd.org>
To:        Dmitry Chagin <dchagin@freebsd.org>
Cc:        "src-committers@freebsd.org" <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: e013e36939ac - main - linux(4): Get rid of Linuxulator kernel build options.
Message-ID:  <4C0E8010-EFBF-42A5-90EF-A502E7BBD3D7@freebsd.org>
In-Reply-To: <YNHC7bIN%2BtpRUm0C@heemeyer.club>
References:  <202106211656.15LGuXQU059224@gitrepo.freebsd.org> <C06B43AE-D887-410F-A212-5A16BC28DDB7@freebsd.org> <YNHC7bIN%2BtpRUm0C@heemeyer.club>

next in thread | previous in thread | raw e-mail | index | archive | help
On 22 Jun 2021, at 12:01, Dmitry Chagin <dchagin@freebsd.org> wrote:
> On Mon, Jun 21, 2021 at 10:56:38PM +0100, Jessica Clarke wrote:
>> On 21 Jun 2021, at 17:56, Dmitry Chagin <dchagin@FreeBSD.org> wrote:
>>>=20
>>> The branch main has been updated by dchagin:
>>>=20
>>> URL: =
https://cgit.FreeBSD.org/src/commit/?id=3De013e36939ac87b53195370fb5e29f29=
c1a4b5c6
>>>=20
>>> commit e013e36939ac87b53195370fb5e29f29c1a4b5c6
>>> Author:     Dmitry Chagin <dchagin@FreeBSD.org>
>>> AuthorDate: 2021-06-22 05:32:39 +0000
>>> Commit:     Dmitry Chagin <dchagin@FreeBSD.org>
>>> CommitDate: 2021-06-22 05:32:39 +0000
>>>=20
>>>   linux(4): Get rid of Linuxulator kernel build options.
>>>=20
>>>   Stop confusing people, retire COMPAT_LINUX and COMPAT_LINUX32 =
kernel
>>>   build options. Since we have 32 and 64 bit Linux emulators, we =
can't build both
>>>   emulators together into the kernel. I don't think it matters, =
Linux emulation
>>>   depends on loadable modules (via rc).
>>>=20
>>>   Cut LINPROCFS and LINSYSFS for consistency.
>>=20
>> I don=E2=80=99t see why these two should be deleted? They currently =
build fine,
>> and GNU/kFreeBSD kernels enable them. They might work as modules, but =
I
>> would worry that too many parts of userland would try and read them
>> before /etc/init.d/kldutils (the init script that loads modules) =
loads
>> them, so then we=E2=80=99d have to mess around with GRUB configs to =
preload
>> them. If the options work, please leave them in.
>>=20
>=20
> both FS modules depend on linux.ko on i386 or linux_common.ko on =
amd64,
> so it doesn't make sense to have options for them

But that still worked, and was even in NOTES so being tested by LINT.

>> There=E2=80=99s a separate debate of whether this is the =E2=80=9Cright=
=E2=80=9D fix for
>> COMPAT_LINUX*; arguably that *should* work and it=E2=80=99s a bug =
that they
>> don=E2=80=99t, not a feature, even if it=E2=80=99s not of much =
interest to support=E2=80=A6
>>=20
>> I=E2=80=99d like to see the second half reverted, please, and believe =
the first
>> should be too, but I feel less strongly about that.
>>=20
>=20
> I think that descendants should adapt to upstream, esp since there is =
no
> KBI or ABI breakage. btw, debian wiki says gnu/kFreeBSD unmaintained
> since 2014. That is the reason to worry about dead project?

As the current maintainer this is news to me. I see no such comment on
the port=E2=80=99s wiki page[1].

Jess

[1] https://wiki.debian.org/Debian_GNU/kFreeBSD




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4C0E8010-EFBF-42A5-90EF-A502E7BBD3D7>