Thread (41 messages) flat view 41 messages, 7 authors, 2016-06-15

Re: [PATCH] Wrap inflateInit to retry allocation after releasing pack memory

From: Shawn O. Pearce <hidden>
Date: 2016-06-15 22:45:53

Linus Torvalds [off-list ref] wrote:
Let's do this (more complete) wrapping instead, ok?
Ack.
 
This one _just_ wraps things, btw - it doesn't do the "retry on low memory 
error" part, at least not yet. I think that's an independent issue from 
the reporting.
I still think we should try to reduce pack memory usage when we get
oom from zlib and retry the current operation once.  We do it almost
everywhere else and it works relatively well.

We may also want to consider dropping (e.g. halving) the window
size and/or limit when we run out of memory.  We'll run slower but
if the OS has denied us further resources it may be a ulimit thing
on a shared system, and we should try harder to work with what we
have available to us.

-- 
Shawn.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help