On Tue, May 28, 2019 at 7:37 AM Jason Baron <jba...@akamai.com> wrote: > > On 5/24/19 7:17 PM, Yuchung Cheng wrote: > > On Thu, May 23, 2019 at 4:31 PM Yuchung Cheng <ych...@google.com> wrote: > >> > >> On Thu, May 23, 2019 at 12:14 PM David Miller <da...@davemloft.net> wrote: > >>> > >>> From: Jason Baron <jba...@akamai.com> > >>> Date: Wed, 22 May 2019 16:39:32 -0400 > >>> > >>>> Christoph, Igor, and I have worked on an API that facilitates TFO key > >>>> rotation. This is a follow up to the series that Christoph previously > >>>> posted, with an API that meets both of our use-cases. Here's a > >>>> link to the previous work: > >>>> https://patchwork.ozlabs.org/cover/1013753/ > >>> > >>> I have no objections. > >>> > >>> Yuchung and Eric, please review. > >>> > >>> And anyways there will be another spin of this to fix the typo in the > >>> documentation > >>> patch #5. > >> patch set looks fine. I am testing them w/ our internal TFO packetdrill > >> tests. > >>> > >>> Thanks. > > The patch series pass the packetdrill TFO tests :-) It'd be great to > > support of TCP_FASTOPEN_KEY sock opt additionally. > > > > Acked-by: Yuchung Cheng <ych...@google.com> > > > > Thanks for testing. So patch #3 adds support for the backup key to > TCP_FASTOPEN_KEY sock opt for both set/get. Is this what you are > referring to or something else? yes that support is great. so no more further request. patch looks good. thanks!
> > Thanks, > > -Jason >