OpenSBI Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Leo Liang <ycliang@andestech.com>
To: opensbi@lists.infradead.org
Subject: [OpenSBI] rfence related question
Date: Tue, 19 Oct 2021 16:55:04 +0800	[thread overview]
Message-ID: <YW6H6BqLTV1UyYto@ubuntu02> (raw)
In-Reply-To: <CAOnJCUJrnMi8zS-+8aAOdGQEn0YRBaUBn3xXH1LPQK2n_56anQ@mail.gmail.com>

On Fri, Oct 15, 2021 at 12:45:19PM -0700, Atish Patra wrote:
> On Thu, Oct 14, 2021 at 2:41 AM Leo Liang <ycliang@andestech.com> wrote:
> >
> > Hi,
> >
> > I am studying the rfence part of OpenSBI recently.
> >
> > From what I understand the mechanism of the IPI framework is
> > to do "update" to update the IPI information for remote harts,
> > and then send IPI through "ipi_dev" device to remote harts sequentially.
> >
> > After the first IPI is sent (let's say from hart 0 to hart 1),
> > hart 0 waits in the "sync" for the remote hart(hart 1) to do "process".
> > Hart 0 waits until the per hart tlb_sync flag is set from the remote hart(hart1) after "process" is done,
> > and then keep on sending the next IPI to hart 2, and then hart 3 ... etc.
> >
> > If I understand it correctly, then would the following scenario have the possibility to occur?
> >
> > In Linux, Hart 0 executes remote I cache flush instruction and sends IPI to hart 1;
> > At the meantime, hart 2 executes the same instruction and tries to send IPI to hart 0.
> >
> > Both hart 0 and hart 2 step into m-mode, their mstatus.mie are disabled,
> > so, when hart 0 tries to send IPI to hart 2 and wait for the sync, deadlock happens.
> > Hart 0 is waiting for hart 2 and hart 2 is also waiting for hart 0 indefinitely because the interrupts being disabled.
> >
> > Is this case likely to happen?
> >
> 
> Nope because each hart uses a busy loop and consume the pending IPIs
> 
> https://github.com/riscv-software-src/opensbi/blob/master/lib/sbi/sbi_tlb.c#L268
>
Hi Atish,

Got it! Thanks for the pointer!

Best regards,
Leo
> > Best regards,
> > Leo
> >
> > --
> > opensbi mailing list
> > opensbi at lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/opensbi
> 
> 
> 
> -- 
> Regards,
> Atish


      reply	other threads:[~2021-10-19  8:55 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-14  9:40 [OpenSBI] rfence related question Leo Liang
2021-10-15 19:45 ` Atish Patra
2021-10-19  8:55   ` Leo Liang [this message]

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=YW6H6BqLTV1UyYto@ubuntu02 \
    --to=ycliang@andestech.com \
    --cc=opensbi@lists.infradead.org \
    /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