All of lore.kernel.org
 help / color / mirror / Atom feed
From: Julien Grall <julien.grall@linaro.org>
To: Stefano Stabellini <stefano.stabellini@eu.citrix.com>,
	Ian Campbell <Ian.Campbell@citrix.com>
Cc: "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
	Andrii Tseglytskyi <andrii.tseglytskyi@globallogic.com>
Subject: Re: Xen 4.5 random freeze question
Date: Wed, 19 Nov 2014 12:23:53 +0000	[thread overview]
Message-ID: <546C8BD9.8090406@linaro.org> (raw)
In-Reply-To: <alpine.DEB.2.02.1411191216160.27247@kaball.uk.xensource.com>

On 11/19/2014 12:17 PM, Stefano Stabellini wrote:
> On Wed, 19 Nov 2014, Ian Campbell wrote:
>> On Wed, 2014-11-19 at 11:42 +0000, Stefano Stabellini wrote:
>>> So it looks like there is not actually anything wrong, is just that you
>>> have too much inflight irqs? It should cause problems because in that
>>> case GICH_HCR_UIE should be set and you should get a maintenance
>>> interrupt when LRs become available (actually when "none, or only one,
>>> of the List register entries is marked as a valid interrupt").
>>>
>>> Maybe GICH_HCR_UIE is the one that doesn't work properly.
>>
>> How much testing did this aspect get when the no-maint-irq series
>> originally went in? Did you manage to find a workload which filled all
>> the LRs or try artificially limiting the number of LRs somehow in order
>> to provoke it?
>>
>> I ask because my intuition is that this won't happen very much, meaning
>> those code paths may not be as well tested...
> 
> I did test it by artificially limiting the number of LRs to 1.
> However there have been many iterations of that series and I didn't run
> this test at every iteration.

am I the only to think this may not be related to this bug? All the LRs
are full with IRQ of the same priority. So it's valid.

As gic_restore_pending_irqs is called every time that we return to the
guest. It could be anything else.

It would be interesting to see why we are trapping all the time in Xen.

Regards,

-- 
Julien Grall

  reply	other threads:[~2014-11-19 12:23 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-14 14:25 Xen 4.5 random freeze question Andrii Tseglytskyi
2014-11-14 14:35 ` Stefano Stabellini
2014-11-14 14:43   ` Andrii Tseglytskyi
2014-11-14 15:22     ` Stefano Stabellini
2014-11-14 15:39       ` Andrii Tseglytskyi
2014-11-14 15:49         ` Julien Grall
2014-11-14 15:58           ` Andrii Tseglytskyi
2014-11-14 16:15         ` Stefano Stabellini
2014-11-14 16:22           ` Andrii Tseglytskyi
2014-11-14 16:35             ` Julien Grall
2014-11-14 16:40               ` Andrii Tseglytskyi
2014-11-17 15:47                 ` Andrii Tseglytskyi
2014-11-17 16:39                   ` Stefano Stabellini
2014-11-17 17:05                     ` Andrii Tseglytskyi
2014-11-17 18:02                       ` Stefano Stabellini
2014-11-18 10:41                         ` Andrii Tseglytskyi
2014-11-18 11:31                           ` Andrii Tseglytskyi
2014-11-18 12:35                             ` Andrii Tseglytskyi
2014-11-18 15:39                               ` Stefano Stabellini
2014-11-18 16:11                                 ` Andrii Tseglytskyi
2014-11-18 16:14                                   ` Stefano Stabellini
2014-11-18 16:18                                     ` Andrii Tseglytskyi
2014-11-18 16:46                                       ` Andrii Tseglytskyi
2014-11-18 17:51                                         ` Stefano Stabellini
2014-11-19  9:38                                           ` Andrii Tseglytskyi
2014-11-19 11:12                                             ` Stefano Stabellini
2014-11-19 11:16                                               ` Andrii Tseglytskyi
2014-11-19 11:42                                                 ` Stefano Stabellini
2014-11-19 11:57                                                   ` Andrii Tseglytskyi
2014-11-19 11:59                                                     ` Stefano Stabellini
2014-11-19 12:37                                                       ` Andrii Tseglytskyi
2014-11-19 14:52                                                         ` Stefano Stabellini
2014-11-19 15:27                                                           ` Andrii Tseglytskyi
2014-11-19 15:41                                                             ` Stefano Stabellini
2014-11-19 16:01                                                               ` Andrii Tseglytskyi
2014-11-19 16:09                                                                 ` Andrii Tseglytskyi
2014-11-19 16:13                                                                   ` Stefano Stabellini
2014-11-19 16:29                                                                     ` Andrii Tseglytskyi
2014-11-19 16:32                                                                       ` Andrii Tseglytskyi
2014-11-19 16:43                                                                         ` Andrii Tseglytskyi
2014-11-19 16:52                                                                           ` Stefano Stabellini
2014-11-19 16:50                                                                       ` Stefano Stabellini
2014-11-19 17:03                                                                         ` Andrii Tseglytskyi
2014-11-19 17:07                                                                           ` Stefano Stabellini
2014-11-19 17:37                                                                             ` Andrii Tseglytskyi
2014-11-19 17:42                                                                               ` Stefano Stabellini
2014-11-19 17:47                                                                                 ` Andrii Tseglytskyi
2014-11-19 18:06                                                                                   ` Andrii Tseglytskyi
2014-11-19 18:14                                                                                     ` Stefano Stabellini
2014-11-19 18:26                                                                                       ` Julien Grall
2014-11-19 18:31                                                                                         ` Stefano Stabellini
2014-11-19 19:24                                                                                           ` Andrii Tseglytskyi
2014-11-20 10:28                                                                                             ` Stefano Stabellini
2014-11-20 11:15                                                                                               ` Julien Grall
2014-11-20 16:06                                                                                                 ` Andrii Tseglytskyi
2014-11-20 16:15                                                                                                   ` Stefano Stabellini
2014-11-20 16:43                                                                                                     ` Andrii Tseglytskyi
2014-11-19 17:11                                                                           ` Andrii Tseglytskyi
2014-11-19 17:14                                                                             ` Stefano Stabellini
2014-11-19 12:13                                                   ` Ian Campbell
2014-11-19 12:17                                                     ` Stefano Stabellini
2014-11-19 12:23                                                       ` Julien Grall [this message]
2014-11-19 12:40                                                         ` Andrii Tseglytskyi
2014-11-19 13:26                                                           ` Julien Grall
2014-11-19 13:30                                                             ` Andrii Tseglytskyi
2014-11-19 14:05                                                               ` Julien Grall

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=546C8BD9.8090406@linaro.org \
    --to=julien.grall@linaro.org \
    --cc=Ian.Campbell@citrix.com \
    --cc=andrii.tseglytskyi@globallogic.com \
    --cc=stefano.stabellini@eu.citrix.com \
    --cc=xen-devel@lists.xen.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.