Re: git: 07c64d74917e - main - acpica: Import ACPICA 20230628

From: Andriy Gapon <avg_at_FreeBSD.org>
Date: Wed, 31 Jan 2024 20:36:28 UTC
On 31/01/2024 19:40, Cy Schubert wrote:
> In message <04c4a0e1-aa79-4d25-a1f7-2196cfa65578@FreeBSD.org>, Jung-uk Kim
> writ
> es:
>> On 24. 1. 31., Baptiste Daroussin wrote:
>>> Hello,
>>>
>>> Either this one or the previous import is breaking arm64 build
>>>
>>> --- acpi_iort.o ---
>>> /home/bapt/worktrees/main/sys/arm64/acpica/acpi_iort.c:103:4: error: field
>>> 'data' with variable sized type 'union (unnamed union at
>>> /home/bapt/worktrees/main/sys/arm64/acpica/acpi_iort.c:98:2
>>> )' not at the end of a struct or class is a GNU extension
>>> [-Werror,-Wgnu-variable-sized-type-not-at-end]
>>>     103 |         } data;
>>>           |           ^
>>
>> Sorry for the breakage.  I will fix it soon.
>>
>> BTW, this code was added by this:
>>
>> https://reviews.freebsd.org/D31267
>>
>> It seems struct iort_named_component was a hack, which duplicated
>> ACPI_IORT_NAMED_COMPONENT but with a fixed length field DeviceName[32].
>> Is it really necessary?
> 
> Though they incorporated the WOL patch I've been using, they've broken
> poweroff.

The poweroff issue could be because of 9cdf326b4f

-- 
Andriy Gapon