Re: git: f224591746bd - main - Add ASMC_DEBUG make option
- Reply: Enji Cooper (yaneurabeya): "Re: git: f224591746bd - main - Add ASMC_DEBUG make option"
- Reply: Olivier Certner : "Re: git: f224591746bd - main - Add ASMC_DEBUG make option"
- In reply to: Olivier Certner : "Re: git: f224591746bd - main - Add ASMC_DEBUG make option"
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sun, 11 Jan 2026 05:10:28 UTC
Hi Olivier,
> On Jan 9, 2026, at 8:44 AM, Olivier Certner <olce@freebsd.org> wrote:
>
> Hi Enji,
>
>> The branch main has been updated by ngie:
>>
>> URL: https://cgit.FreeBSD.org/src/commit/?id=f224591746bdaf14ad5f63de4738a3146cc2f55f
>>
>> commit f224591746bdaf14ad5f63de4738a3146cc2f55f
>> Author: Enji Cooper <ngie@FreeBSD.org>
>> AuthorDate: 2026-01-04 08:27:57 +0000
>> Commit: Enji Cooper <ngie@FreeBSD.org>
>> CommitDate: 2026-01-06 01:43:04 +0000
>>
>> Add ASMC_DEBUG make option
>
> This commit broke the i386 build. I've just committed a fix. Also, I've restored 'device vt_efifb' in NOTES (under 'x86/' instead of 'amd64/'), as AFAIK, NOTES should list all possible devices/options.
re: vt_efifb: what I did was removed the duplicate entry, since it was already in the x86 one: https://cgit.freebsd.org/src/tree/sys/x86/conf/NOTES#n332 .
> Please make sure you also MFC the two following commits (67599eef01f5 ("sys/x86/NOTES: Add vt_efifb"), fde9fe18219f i386 ("Fix kernel compilation after introduction of ASMC_DEBUG option")) along with the original one.
Sure — although the correct fix would be to not build the driver at all on i386, since it doesn’t support 32-bit CPUs. Apple sold very few i386-based Intel chips before they transitioned to amd64-only ones (it was only a matter of 8-9 months IIRC) and asmc(4) doesn’t support any of the 32-bit chips (I checked).
Cheers,
-Enji