On Thu, Nov 4, 2021, 11:58 AM Warner Losh <i...@bsdimp.com> wrote:
>
>
> On Thu, Nov 4, 2021 at 11:49 AM Richard Henderson <
> richard.hender...@linaro.org> wrote:
>
>> On 11/4/21 10:05 AM, Warner Losh wrote:
>> > +typedef struct target_mcontext {
>> > + uint32_t __gregs[32];
>>
>> sys/arm/include/ucontext.h has
>>
>> #define _NGREG 17
>> typedef __greg_t __gregset_t[_NGREG];
>>
>
There is already a TARGET__NGREG so I'll use that.
With s/32/17/ here,
>> Reviewed-by: Richard Henderson <richard.hender...@linaro.org>
>>
>
> Will do.
>
> And I'm going to add compile time asserts for the size of mcontext and
> ucontext in a future
> patch series. It won't test for layout, but will keep things like this
> from happening again by accident.
>
> Warner
>
