Re: git: 73ee5756dee6 - main - Fixes in the tcp infrastructure with respect to stack changes as well as other infrastructure updates for incoming rack features.

From: Randall Stewart <rrs_at_netflix.com>
Date: Tue, 23 May 2023 13:59:33 UTC
https://reviews.freebsd.org/D40229 <https://reviews.freebsd.org/D40229>

Should fix this :)

R
> On May 21, 2023, at 6:06 PM, Tomoaki AOKI <junchoon@dec.sakura.ne.jp> wrote:
> 
>> On Tue, Apr 4, 2023 at 12:31 PM Randall Stewart <rrs@freebsd.org> wrote:
>>> 
>>> The branch main has been updated by rrs:
>>> 
>>> URL: https://www.google.com/url?q=https://cgit.FreeBSD.org/src/commit/?id%3D73ee5756dee6b2110eb6fb2b2ef3cde39a1fcb4f&source=gmail-imap&ust=1685311614000000&usg=AOvVaw0HmtU6ylNo8jRrPgvtzIB0
>>> 
>>> commit 73ee5756dee6b2110eb6fb2b2ef3cde39a1fcb4f
>>> Author:     Randall Stewart <rrs@FreeBSD.org>
>>> AuthorDate: 2023-04-01 05:46:38 +0000
>>> Commit:     Randall Stewart <rrs@FreeBSD.org>
>>> CommitDate: 2023-04-01 05:46:38 +0000
>>> 
>>>    Fixes in the tcp infrastructure with respect to stack changes as well as other infrastructure updates for incoming rack features.
>>> 
>>>    So stack switching as always been a bit of a issue. We currently use a break before make setup which means that
>>>    if something goes wrong you have to try to get back to a stack. This patch among a lot of other things changes that so
>>>    that it is a make before break. We also expand some of the function blocks in prep for new features in rack that will allow
>>>    more controlled pacing. We also add other abilities such as the pathway for a stack to query a previous stack to acquire from
>>>    it critical state information so things in flight don't get dropped or mis-handled when switching stacks. We also add the
>>>    concept of a timer granularity. This allows an alternate stack to change from the old ticks granularity to microseconds and
>>>    of course this even gives us a pathway to go to nanosecond timekeeping if we need to (something for the data center to consider
>>>    for sure).
>>> 
>>>    Once all this lands I will then update rack to begin using all these new features.
>>> 
>>>    Reviewed by: tuexen
>>>    Sponsored by: Netflix Inc
>>>    Differential Revision: https://www.google.com/url?q=https://reviews.freebsd.org/D39210&source=gmail-imap&ust=1685311614000000&usg=AOvVaw1ZouYiMywAnpdYyYCp9T9O
>> 
>> Hi,
>> 
>> Could you hide some stuff in netinet/tcp.h?
>> struct http_req breaks curl 8.1.0.
>> 
>> Antoine
> 
> Please refer to Bug 271526 [1], especially Comment 4 [2] by Jonathan
> Reynolds.
> A patch is proposed there inline (not as an attachment).
> 
> [1] https://www.google.com/url?q=https://bugs.freebsd.org/bugzilla/show_bug.cgi?id%3D271526&source=gmail-imap&ust=1685311614000000&usg=AOvVaw2e-0D0bz7CAIVj6kvLZOJg
> 
> [2] https://www.google.com/url?q=https://bugs.freebsd.org/bugzilla/show_bug.cgi?id%3D271526%23c4&source=gmail-imap&ust=1685311614000000&usg=AOvVaw154j12MkBXKn2z9_gepR1n
> 
> -- 
> Tomoaki AOKI    <junchoon@dec.sakura.ne.jp>

------
Randall Stewart
rrs@netflix.com