git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: What to expect after 0.99.8

From
Daniel Barkalow <barkalow@iabervon.org>
Date
Oct 4, 2005, 07:31 UTC
Message-ID
<Pine.LNX.4.63.0510040321170.23242@iabervon.org>
In-Reply-To
<20051004071210.GA18716@localdomain>
On Tue, 4 Oct 2005, Dan Aloni wrote:
Show 33 quoted lines
> On Mon, Oct 03, 2005 at 04:16:27PM -0700, Linus Torvalds wrote:
> > 
> > 
> > On Mon, 3 Oct 2005, Junio C Hamano wrote:
> > > 
> > > This reminds me of one patch:
> > > 
> > >     From: Dan Aloni <da-x@monatomic.org>
> > >     Subject: [PATCH] Fix git+ssh's indefinite halts during long fetches
> > >     Date: Sat, 1 Oct 2005 21:39:42 +0300
> > >     Message-ID: <20051001183942.GA2099@localdomain>
> > > 
> > > I'd appreciate it if you had a chance to take a look at it and
> > > comment on it.
> > 
> > I personally hate it.
> > 
> > It adds horrible patches to fairly core stuff, all because the prefetching 
> > is not limited.
> 
> Well it can be reworked to be more clean...
>  
> > As far as I can tell, it should be much easier to just limit the 
> > prefetching to some reasonable limit (say, a few objects deep), which 
> > guarantees that the prefetching doesn't fill up the write queues on the 
> > fetching side.
> 
> I'm not sure how this will be completely reliable, even if you limit the
> prefetching to one object.
> 
> Suppose that this one object's size is larger than the receiving queues of 
> the receiving end (like 1 MB?) and the bandwidth is high, wouldn't that 
> break?

It shouldn't cause any problem, unless there isn't a 4K buffer between the git-ssh-fetch and ssh; the fetch side would have to fill this buffer before getting stuck, even if ssh can't send out any more data until the object has been read, and 100 requests (each 21 bytes) wouldn't be enough. I remember that there's a lot that depends on being able to put 4K into an empty pipe without blocking, and I'd guess that UNIX sockets have a similar capacity (although I'm not going to look it up tonight).

	-Daniel
*This .sig left intentionally blank*
Previous: Dan AloniNext: Matthias Urlichs
Message 19 of 39 in “What to expect after 0.99.8”
  1. Junio C HamanoOct 3, 2005
  2. A Large Angry SCMOct 3, 2005
  3. Junio C HamanoOct 3, 2005
  4. Enable and fix support for base less merges.Fredrik Kuivinen, Oct 3, 2005
  5. Josef WeidendorferOct 3, 2005
  6. Junio C HamanoOct 4, 2005
  7. Josef WeidendorferOct 4, 2005
  8. Junio C HamanoOct 4, 2005
  9. Random documentation fixesJonas Fonseca, Oct 3, 2005
  10. Daniel BarkalowOct 3, 2005
  11. Martin CoxallOct 3, 2005
  12. Nick HengeveldOct 3, 2005
  13. Daniel BarkalowOct 3, 2005
  14. Junio C HamanoOct 3, 2005
  15. Daniel BarkalowOct 3, 2005
  16. Junio C HamanoOct 3, 2005
  17. Linus TorvaldsOct 3, 2005
  18. Dan AloniOct 4, 2005
  19. Daniel BarkalowOct 4, 2005
  20. Matthias UrlichsOct 4, 2005
  21. H. Peter AnvinOct 4, 2005
  22. Matthias UrlichsOct 4, 2005
  23. H. Peter AnvinOct 4, 2005
  24. Junio C HamanoOct 4, 2005
  25. Linus TorvaldsOct 5, 2005
  26. H. Peter AnvinOct 5, 2005
  27. Daniel BarkalowOct 4, 2005
  28. H. Peter AnvinOct 4, 2005
  29. Daniel BarkalowOct 4, 2005
  30. Alan ChandlerOct 3, 2005
  31. H. Peter AnvinOct 3, 2005
  32. Greg KHOct 4, 2005
  33. H. Peter AnvinOct 5, 2005
  34. Matthias UrlichsOct 3, 2005
  35. Chuck LeverOct 4, 2005
  36. Junio C HamanoOct 4, 2005
  37. Fredrik KuivinenOct 4, 2005
  38. Fredrik KuivinenOct 5, 2005
  39. Junio C HamanoOct 5, 2005

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.