lkml.org 
[lkml]   [2022]   [Jul]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v6 00/11] remove msize limit in virtio transport
Date
On Samstag, 16. Juli 2022 01:28:51 CEST Dominique Martinet wrote:
> Dominique Martinet wrote on Sat, Jul 16, 2022 at 07:30:45AM +0900:
> > Christian Schoenebeck wrote on Fri, Jul 15, 2022 at 11:35:26PM +0200:
> > > * Patches 7..11 tremendously reduce unnecessarily huge 9p message sizes
> > > and
> > >
> > > therefore provide performance gain as well. So far, almost all 9p
> > > messages
> > > simply allocated message buffers exactly msize large, even for
> > > messages
> > > that actually just needed few bytes. So these patches make sense by
> > > themselves, independent of this overall series, however for this
> > > series
> > > even more, because the larger msize, the more this issue would have
> > > hurt
> > > otherwise.
> >
> > Unless they got stuck somewhere the mails are missing patches 10 and 11,
> > one too many 0s to git send-email ?
>
> nevermind, they just got in after 1h30... I thought it'd been 1h since
> the first mails because the first ones were already 50 mins late and I
> hadn't noticed! I wonder where they're stuck, that's the time
> lizzy.crudebyte.com received them and it filters earlier headers so
> probably between you and it?

Certainly an outbound SMTP greylisting delay, i.e. lack of karma. Sometimes my
patches make it to lists after 3 hours. I haven't figured out though why some
patches within the same series arrive significantly faster than certain other
ones, which is especially weird when that happens not in order they were sent.

> ohwell.
>
> > I'll do a quick review from github commit meanwhile
>
> Looks good to me, I'll try to get some tcp/rdma testing done this
> weekend and stash them up to next

Great, thanks!

> --
> Dominique




\
 
 \ /
  Last update: 2022-07-16 11:56    [W:0.056 / U:1.020 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site