Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 18 Aug 2023 00:22:39 -0500
From:      Doug Moore <unkadoug@gmail.com>
To:        Kyle Evans <kevans@FreeBSD.org>, "Bjoern A. Zeeb" <bz@FreeBSD.org>, src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   Re: git: b80ea452375f - main - LinuxKPI: implement mul_u64_u64_div_u64()
Message-ID:  <77da3b75-5f7e-5c4b-caf3-4285bb87c6fe@gmail.com>
In-Reply-To: <3e99dded-478b-8bb7-cbe0-78587040c3d9@FreeBSD.org>
References:  <202308180121.37I1LPTc024071@gitrepo.freebsd.org> <ac12a699-b98a-de85-d459-e1e205280b40@FreeBSD.org> <3e99dded-478b-8bb7-cbe0-78587040c3d9@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
It remains the case, for me at least, that KASSERT is not defined where 
it is used in math64.h, and so compilation still fails.

On 8/18/23 00:14, Kyle Evans wrote:
> On 8/18/23 00:09, Kyle Evans wrote:
>> On 8/17/23 20:21, Bjoern A. Zeeb wrote:
>>> The branch main has been updated by bz:
>>>
>>> URL: 
>>> https://cgit.FreeBSD.org/src/commit/?id=b80ea452375f52a3ab7d82a9aef10da0d89985d9
>>>
>>> commit b80ea452375f52a3ab7d82a9aef10da0d89985d9
>>> Author:     Bjoern A. Zeeb <bz@FreeBSD.org>
>>> AuthorDate: 2023-05-16 20:55:00 +0000
>>> Commit:     Bjoern A. Zeeb <bz@FreeBSD.org>
>>> CommitDate: 2023-08-18 01:20:39 +0000
>>>
>>>      LinuxKPI: implement mul_u64_u64_div_u64()
>>>      Implement mul_u64_u64_div_u64() for an updated iwlwifi driver 
>>> (though
>>>      we do not yet use it there; it is used for in-kernel ptp on wifi).
>>>      Sponsored by:   The FreeBSD Foundation
>>>      Submitted by:   cperciva
>>>      MFC after:      10 days
>>>      Reviewed by:    cperciva, dwmalone
>>>      Differential Revision: https://reviews.freebsd.org/D40120
>>
>> This broke buildkernel on multiple architectures, it seems: 
>> https://ci.freebsd.org/tinderbox/
>>
>> I've confirmed that reverting this commit fixes it. I'd be lying if I 
>> claimed to understand what's going on:
>
> I lied; I looked again at the context and realized it was just missing 
> an extra closing parentheses. I went ahead and hammered it away in 
> dabbbebcb0f5...
>
> Thanks,
>
> Kyle Evans
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?77da3b75-5f7e-5c4b-caf3-4285bb87c6fe>