All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Jan Beulich" <JBeulich@novell.com>
To: Keir Fraser <Keir.Fraser@cl.cam.ac.uk>
Cc: xen-devel@lists.xensource.com
Subject: Re: noirqdebug_setup() in arch/i386/kernel/setup-xen.c
Date: Fri, 07 Apr 2006 12:15:31 +0200	[thread overview]
Message-ID: <443657E3.76F0.0078.0@novell.com> (raw)
In-Reply-To: <f56e35a5fe84d18859164a4d46bb022a@cl.cam.ac.uk>

>>> Keir Fraser <Keir.Fraser@cl.cam.ac.uk> 07.04.06 11:59:51 >>>
>
>On 7 Apr 2006, at 10:00, Jan Beulich wrote:
>
>> Can anyone explain why i386 calls noirqdebug_setup() unconditionally, 
>> while x86-64 doesn't? I believe this is the reason
>> why we have been seeing spurious interrupt reports only on 64-bits so 
>> far.
>>
>> What was this supposed to suppress originally? Is this still 
>> applicable? It would seem to me that hiding problems in
>> this area is rather undesirable...
>
>It's disabled to handle the case where a shared IRQ line is shared 
>across multiple driver domains. In that case, if one domain's device is 
>busy then both domains will get interrupt notifications, but the 
>non-busy driver domain will think that it's receiving spurious 
>interrupts (since it doesn't see IRQ_HANDLED from the busy device 
>driver ISR, since it's running in a different driver domain).

Then, once we are able to figure the interrupt problems we currently have, this should probably be re-enabled
temporarily and for *all* architectures, but be replaced by proper logic (i.e. Xen informing the domain e.g. by means of
a flag passed with the injected IRQ that it should expect the interrupt to not getting handled; Xen should then also
expect some notification back from the domain that the IRQ was handled, so it can combine this from all involved domains
and still report and act on problems similar to the way Linux does).

Jan

  reply	other threads:[~2006-04-07 10:15 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-04-07  9:00 noirqdebug_setup() in arch/i386/kernel/setup-xen.c Jan Beulich
2006-04-07  9:59 ` Keir Fraser
2006-04-07 10:15   ` Jan Beulich [this message]
2006-04-07 11:16     ` Keir Fraser

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=443657E3.76F0.0078.0@novell.com \
    --to=jbeulich@novell.com \
    --cc=Keir.Fraser@cl.cam.ac.uk \
    --cc=xen-devel@lists.xensource.com \
    /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.