Thread (5 messages) flat view 5 messages, 3 authors, 2016-06-16

Re: Segfault in the attr stack

From: Duy Nguyen <hidden>
Date: 2016-06-16 02:19:42

On Thu, Jun 2, 2016 at 8:02 AM, Duy Nguyen [off-list ref] wrote:
On Thu, Jun 2, 2016 at 5:17 AM, Junio C Hamano [off-list ref] wrote:
quoted
Gaah, of course.

This is coming from the cache preload codepath, where multiple threads
try to run ce_path_match().
It used to be OK because pathspec magic never looked at attributes,
but now it does, and attribute system is not thread-safe.
Off topic. I wonder if we can annotate which functions are thread-safe
(minority currently in git code base, I believe) and make gcc, clang
or sparse spot them in threaded code?
By them I mean unannotated (i.e. not thread-safe) functions, of course.
-- 
Duy
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help