From: Julien Grall <julien.grall@arm.com>
To: Dario Faggioli <dario.faggioli@citrix.com>,
Jan Beulich <JBeulich@suse.com>
Cc: Stefano Stabellini <sstabellini@kernel.org>,
Wei Liu <wei.liu2@citrix.com>,
George Dunlap <george.dunlap@eu.citrix.com>,
Andrew Cooper <andrew.cooper3@citrix.com>,
Ian Jackson <Ian.Jackson@eu.citrix.com>, Tim Deegan <tim@xen.org>,
xen-devel <xen-devel@lists.xenproject.org>
Subject: Re: xen/arm: Domain not fully destroyed when using credit2
Date: Tue, 31 Jan 2017 16:30:50 +0000 [thread overview]
Message-ID: <6ea5c07c-aaed-933a-f7fd-dd56d23c0b04@arm.com> (raw)
In-Reply-To: <1485360009.32103.125.camel@citrix.com>
Hi Dario,
On 25/01/17 16:00, Dario Faggioli wrote:
> On Wed, 2017-01-25 at 12:38 +0000, Julien Grall wrote:
>> On 25/01/17 11:10, Dario Faggioli wrote:
> And a good one. I may be wrong (I certainly wasn't around at the time),
> but ISTR out RCU code is imported/inspired by Linux... Looking there
> again may help, but, nowadays, Linux RCU subsystem is a Lernaean Hydra
> monster, with 100 heads and sharpen claws! :-O
>
> And, while, in there, it has to be like that, I don't think we need all
> such complexity, and hence we can't just re-sync. :-/
Yeah, even the tiny RCU code is quite complex :/. I've looked at our RCU
code and noticed there is a link in the header to [1].
It seems to be a documentation about the RCU code we used. From my
understanding of the "RCU Implementations", the authors are expecting a
timer to kick periodically pCPU and check if there is some RCU work pending.
We could add this timer but it would prevent an idle pCPU to stay in low
power mode for a long time. Another solution would be to send an
interrupt to each pCPU when call_rcu is called rather depending on a
mark. Although this would still wake-up the pCPU even it was doing nothing.
Any better ideas?
Cheers,
[1] http://lse.sourceforge.net/locking/rcupdate.html
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
next prev parent reply other threads:[~2017-01-31 16:30 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-23 19:42 xen/arm: Domain not fully destroyed when using credit2 Julien Grall
2017-01-24 0:16 ` Stefano Stabellini
2017-01-24 12:52 ` Julien Grall
2017-01-24 8:20 ` Jan Beulich
2017-01-24 10:50 ` Julien Grall
2017-01-24 11:02 ` Jan Beulich
2017-01-24 12:30 ` Julien Grall
2017-01-24 12:53 ` Dario Faggioli
2017-01-24 13:04 ` Julien Grall
2017-01-24 13:05 ` Julien Grall
2017-01-24 13:19 ` Dario Faggioli
2017-01-24 13:24 ` Julien Grall
2017-01-24 13:40 ` Dario Faggioli
2017-01-24 13:49 ` Julien Grall
2017-01-24 14:16 ` Dario Faggioli
2017-01-24 15:06 ` Julien Grall
2017-01-25 11:10 ` Dario Faggioli
2017-01-25 12:38 ` Julien Grall
2017-01-25 12:40 ` Andrew Cooper
2017-01-25 14:23 ` Julien Grall
2017-01-25 16:00 ` Dario Faggioli
2017-01-31 16:30 ` Julien Grall [this message]
2017-01-31 22:10 ` Stefano Stabellini
2017-02-01 18:21 ` Wei Liu
2017-02-02 11:22 ` Jan Beulich
2017-02-02 11:53 ` Wei Liu
2017-02-02 12:18 ` Julien Grall
2017-02-02 12:51 ` Dario Faggioli
2017-02-02 13:26 ` Julien Grall
2017-02-02 13:32 ` Dario Faggioli
2017-03-28 18:30 ` Julien Grall
2017-03-30 7:38 ` Dario Faggioli
2017-02-02 12:01 ` Dario Faggioli
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=6ea5c07c-aaed-933a-f7fd-dd56d23c0b04@arm.com \
--to=julien.grall@arm.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=JBeulich@suse.com \
--cc=andrew.cooper3@citrix.com \
--cc=dario.faggioli@citrix.com \
--cc=george.dunlap@eu.citrix.com \
--cc=sstabellini@kernel.org \
--cc=tim@xen.org \
--cc=wei.liu2@citrix.com \
--cc=xen-devel@lists.xenproject.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;
as well as URLs for NNTP newsgroup(s).