Re: index-pack died on pread

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Robin Rosenberg <robin.rosenberg.lists@...>
Cc: Michal Rokos <michal.rokos@...>, GIT <git@...>
Date: Wednesday, July 25, 2007 - 7:44 pm

On Thu, 26 Jul 2007, Robin Rosenberg wrote:

Interesting.

It's true that pread() is used much less than normal reads, and maybe the 
cygwin pread() is indeed broken. But it's intriguing how apparently both 
HP-UX and Cygwin are showing the same breakage.

But if git was doing something odd with pread(), then NO_PREAD shouldn't 
work either, because that just enables the exact same code, except we now 
supply a pread emulation library function (which does it using "lseek()": 
not a *good* emulation, but good enough for the very limited use that git 
puts it to).

So yeah, it looks like pread() is broken under cygwin and hpux-11.11.

Googling for "pread" "cygwin" does seem to show that it's been buggy at 
times.  Eg:

	Fixes since 1.5.21-1:

	...

	- Fix pread in case current file offset is non-zero. (Hideki Iwamoto)

but I'm a bit surprised that hp-ux has the bug. pread isn't *that* 
complex.

		Linus
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
index-pack died on pread, Michal Rokos, (Mon Jul 23, 8:52 am)
Re: index-pack died on pread, Linus Torvalds, (Mon Jul 23, 1:04 pm)
Re: index-pack died on pread, Robin Rosenberg, (Wed Jul 25, 7:15 pm)
Re: index-pack died on pread, Linus Torvalds, (Wed Jul 25, 7:44 pm)
Re: index-pack died on pread, Alex Riesen, (Thu Jul 26, 8:42 am)
Re: index-pack died on pread, Linus Torvalds, (Thu Jul 26, 12:13 pm)
Re: index-pack died on pread, Junio C Hamano, (Thu Jul 26, 11:43 pm)
Re: index-pack died on pread, Linus Torvalds, (Fri Jul 27, 1:36 am)
Re: index-pack died on pread, Nicolas Pitre, (Fri Jul 27, 9:38 am)
Re: index-pack died on pread, Tomash Brechko, (Fri Jul 27, 5:50 am)
Re: index-pack died on pread, Tomash Brechko, (Fri Jul 27, 6:33 am)
Re: index-pack died on pread, Alex Riesen, (Thu Jul 26, 12:51 pm)
Re: index-pack died on pread, Linus Torvalds, (Thu Jul 26, 2:02 pm)
Re: index-pack died on pread, Nicolas Pitre, (Mon Jul 23, 2:03 pm)
Re: index-pack died on pread, Alex Riesen, (Mon Jul 23, 11:32 am)
Re: index-pack died on pread, Michal Rokos, (Wed Jul 25, 4:07 pm)
Re: index-pack died on pread, Alex Riesen, (Wed Jul 25, 4:48 pm)