Renà Scharfe wrote: > Am 06.11.2010 19:42, schrieb Jonathan Nieder: >> Maybe add_packed_git()? >> >> use_pack() is called by: [...] >> some of which would make me worry about thread-safety. > > Possibly, but if that's the case then we have a thread-safety issue > already: use_pack() updates pack_mapped, which is used by > release_pack_memory() to see how much can be freed. Good point. The threaded callers are probably protected by read_lock. My other worry would be that try_to_free_pack_memory should not override try_to_free_from_threads. I guess some time documenting these code paths will be needed. :( Thanks for a sanity check, Jonathan -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html