Re: [9fans] [PATCH] fossil: fix a deadlock in the caching logic

2023-04-04 Thread noam
Quoth Anthony Martin : > Noam, can you reproduce your problem with the above patch? Just from reading the patch: yes, with ~95% confidence. The logic it's fixing is completely unrelated. So: thanks! Now I can import that patch and have *another* deadlock fixed! Are there any other patches I shou

Re: [9fans] [PATCH] fossil: fix a deadlock in the caching logic

2023-04-04 Thread Anthony Martin
Charles Forsyth once said: > It's funny that usually "it wasn't me" is used when breaking things. here > it's fixing them, but I'm fairly sure "it wasn't me" that fixed it. It was Richard Miller in 2012. https://9p.io/sources/patch/applied/fossil-snap-deadlock/ https://9p.io/sour

Re: [9fans] [PATCH] fossil: fix a deadlock in the caching logic

2023-04-04 Thread noam
Quoth Charles Forsyth : > Fossil works hard to do certain things that now we probably wouldn't bother > to do. Such as? -- 9fans: 9fans Permalink: https://9fans.topicbox.com/groups/9fans/T354fe702e1e9d5e9-M8a784194389d102772df0a9e Delivery options: https:

Re: [9fans] [PATCH] fossil: fix a deadlock in the caching logic

2023-04-04 Thread Charles Forsyth
One of the nice things about several of the papers at iwp9 was the implied or express reconsidering of secondary storage given various types of modern technology. Fossil works hard to do certain things that now we probably wouldn't bother to do. On Tue, 4 Apr 2023 at 21:44, Charles Forsyth wrote:

Re: [9fans] [PATCH] fossil: fix a deadlock in the caching logic

2023-04-04 Thread Charles Forsyth
It's funny that usually "it wasn't me" is used when breaking things. here it's fixing them, but I'm fairly sure "it wasn't me" that fixed it. -- 9fans: 9fans Permalink: https://9fans.topicbox.com/groups/9fans/T354fe702e1e9d5e9-Md75914502384917e733de7a4 Deli

Re: [9fans] [PATCH] fossil: fix a deadlock in the caching logic

2023-04-04 Thread Skip Tavakkolian
it definitely was not me. My bet would be on rsc, geoff, richard, forsyth, quanstrom or djc. On Tue, Apr 4, 2023 at 11:05 AM Steve Simon wrote: > > > was this hard to reproduce? > > i have not seen fossil deadlocking and have used it since i installed my > first home server in 2004. > > there de

Re: [9fans] [PATCH] fossil: fix a deadlock in the caching logic

2023-04-04 Thread noam
Quoth Steve Simon : > was this hard to reproduce? I've seen it sporadically over the last year, and - when looking for it - was able to trigger it deliberately yesterday with minimal effort the first time I looked for it. ...of course, when I was later trying to reproduce it a third time, I wasn'

Re: [9fans] [PATCH] fossil: fix a deadlock in the caching logic

2023-04-04 Thread Steve Simon
was this hard to reproduce?i have not seen fossil deadlocking and have used it since i installed my first home server in 2004.there definitely _was_ a problem in the snapshot code which was finally resolved around 2015 (roughly), i think perhaps skip, or forsyth found it - i apologise if i have the

[9fans] [PATCH] fossil: fix a deadlock in the caching logic

2023-04-04 Thread noam
I've sporadically encountered a deadlock in fossil. Naturally, when your root file system crashes, it can be hard to debug. My solution: stop having a root file system. Was able to attach acid using mycroft's tooling from ANTS, and get a clean stack trace (https://pixelhero.dev/notebook/fossil/s