9fans - fans of the OS Plan 9 from Bell Labs
 help / color / mirror / Atom feed
From: Anthony Martin <ality@pbrane.org>
To: 9fans <9fans@9fans.net>
Cc: noam@pixelhero.dev
Subject: Re: [9fans] [PATCH] fossil: fix a deadlock in the caching logic
Date: Tue, 4 Apr 2023 15:07:55 -0700	[thread overview]
Message-ID: <ZCyfu4tlVPgnwZ_y@alice> (raw)
In-Reply-To: <CAOw7k5g9xE70VzdSC5hpjs9HVwyhPf=Ao+YN9v32Sb7Ydk9_fg@mail.gmail.com>

Charles Forsyth <charles.forsyth@gmail.com> 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/sources/patch/applied/fossil-archive-corruption/
        https://9p.io/sources/patch/applied/fossil-superblock-write/

He fixed another issue a year or two ago.

        http://9legacy.org/9legacy/patch/fossil-deadlocks.diff

Noam, can you reproduce your problem with the above patch?

Cheers,
  Anthony

------------------------------------------
9fans: 9fans
Permalink: https://9fans.topicbox.com/groups/9fans/T354fe702e1e9d5e9-Mce8ab1dcb5e0d8d96f33af8e
Delivery options: https://9fans.topicbox.com/groups/9fans/subscription

  parent reply	other threads:[~2023-04-04 22:08 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-04 17:15 noam
2023-04-04 18:03 ` Steve Simon
2023-04-04 18:34   ` Skip Tavakkolian
2023-04-04 20:44     ` Charles Forsyth
2023-04-04 20:50       ` Charles Forsyth
2023-04-05  1:59         ` noam
2023-04-05 21:25           ` Charles Forsyth
2023-04-06  3:22             ` noam
2023-04-06  3:57               ` Lucio De Re
2023-04-08  7:50                 ` hiro
2023-04-08 14:30                   ` Charles Forsyth
2023-04-08 14:36                     ` Charles Forsyth
2023-04-08 15:09                       ` Dan Cross
2023-04-08 15:27                         ` Steffen Nurpmeso
2023-04-08 17:12                         ` Bakul Shah
2023-04-04 22:07       ` Anthony Martin [this message]
2023-04-05  2:48         ` noam
2023-04-04 22:15   ` noam

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ZCyfu4tlVPgnwZ_y@alice \
    --to=ality@pbrane.org \
    --cc=9fans@9fans.net \
    --cc=noam@pixelhero.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).