All of lore.kernel.org
 help / color / mirror / Atom feed
From: George Dunlap <george.dunlap@eu.citrix.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: Eddie Dong <eddie.dong@intel.com>,
	"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>,
	"Tim (Xen.org)" <tim@xen.org>,
	Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>,
	Roger Pau Monne <roger.pau@citrix.com>
Subject: Re: Bug: Windows 2003 fails to install on xen-unstable tip
Date: Tue, 23 Apr 2013 11:38:54 +0100	[thread overview]
Message-ID: <517664BE.401@eu.citrix.com> (raw)
In-Reply-To: <51767DC402000078000CFE48@nat28.tlf.novell.com>

On 23/04/13 11:25, Jan Beulich wrote:
>>>> On 22.04.13 at 17:32, George Dunlap <george.dunlap@eu.citrix.com> wrote:
>> (XEN) pt on: 10 @14fd4b61ef^M
>> (XEN) B=42 [A:2a B:02 C:50 pt:10/0] @14fd6f73d9^M
>> (XEN) C=50 pt=10/0 @14fdb032f8^M
>> (XEN) C=c0 pt=10/6 @14fddabc31^M
>> (XEN) C=00 pt=10/0 @14fe04d583^M
>> (XEN) C=c0 pt=10/0 @14feaa9978^M
>> (XEN) C=00 pt=10/0 @14fed2ae4b^M
>> (XEN) C=c0 pt=10/0 @14ff98fe90^M
>> (XEN) C=00 pt=10/0 @14ffc0fbc1^M
>> (XEN) irq.c:270: Dom1 PCI link 0 changed 5 -> 0^M
>> (XEN) irq.c:270: Dom1 PCI link 1 changed 10 -> 0^M
>> (XEN) irq.c:270: Dom1 PCI link 2 changed 11 -> 0^M
>> (XEN) irq.c:270: Dom1 PCI link 3 changed 5 -> 0^M
>> (XEN) pt off #105 @15e95c8ff8^M
>> (XEN) pt irq @15da75fb9c (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15db644b09 (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15dc53074a (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15dd41309b (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15de2f93db (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15df1e6232 (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15e00c3c2f (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15e0faca4c (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15e1e94f41 (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15e2d7b303 (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15e3c61780 (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15e4b4cd99 (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15e5a33c65 (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15e690a53a (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15e77f1d0f (rtc_periodic_interrupt+0x81/0x93)^M
>> (XEN) pt irq @15e86d876a (rtc_periodic_interrupt+0x81/0x93)^M
> So we send IRQs as regularly as we're expected to, but Windows
> doesn't even look at REG_C. To me it seems perfectly valid to stop
> sending further IRQs in that case.

Perhaps something else is going on -- e.g., Windows is *reading* 
something different from the RTC, and acting differently in response to 
that?

It's obviously a quirk in Windows 2003, as XP and Win7 seem to deal with 
it just fine.  Nonetheless, we can't very well just say, "Well, it's a 
bug in Windows, so go talk to Microsoft". :-)

>
> In any event the time stamps appear to confirm that the respective
> second REG_C reads are likely checks at the end of the interrupt
> handler in Windows, and that the increased deferral of turning off
> the periodic timer didn't make a difference.
>
> Just to double check - could you comment out entirely the first
> (normal code, i.e. not the one marked //todo?) "else if" in
> rtc_periodic_interrupt() (including its body of course)? I would
> expect this to not make a difference, and if so I don't see how
> Windows expects to be woken up again (I would guess that
> they internally have some gating logic preventing the normal IRQ8
> handling to happen, yet of course we don't know what would
> reset that state).

In fact, when I comment out that region, then it hangs in the guest BIOS 
before even attempting to boot the CD.

I took a look at the offending changeset, and unfortunately it seems to 
have a lot of different changes, so it's not 100% clear which ones are 
related or not.   I'll give it a try anyway. Do you not have boot images 
of Windows 2003 that you can test yourself?

  -George

  reply	other threads:[~2013-04-23 10:38 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-16 14:22 Bug: Windows 2003 fails to install on xen-unstable tip George Dunlap
2013-04-16 14:34 ` Jan Beulich
2013-04-16 14:35   ` George Dunlap
2013-04-17  8:27     ` Jan Beulich
2013-04-17 18:13       ` George Dunlap
2013-04-18  8:13         ` Jan Beulich
2013-04-18  9:03           ` George Dunlap
2013-04-19 10:05         ` Jan Beulich
2013-04-19 10:44           ` Roger Pau Monné
2013-04-19 12:47             ` Jan Beulich
2013-04-19 13:28               ` Roger Pau Monné
2013-04-19 14:33                 ` Jan Beulich
2013-04-19 14:41                   ` Roger Pau Monné
2013-04-19 13:51               ` George Dunlap
2013-04-22 11:05                 ` Jan Beulich
2013-04-22 15:32                   ` George Dunlap
2013-04-23 10:25                     ` Jan Beulich
2013-04-23 10:38                       ` George Dunlap [this message]
2013-04-23 11:33                         ` Jan Beulich
2013-04-23 11:52                           ` George Dunlap
2013-04-23 11:57                             ` George Dunlap
2013-04-23 12:45                               ` Jan Beulich
2013-04-23 12:25                             ` Jan Beulich
2013-04-23 13:00                             ` Jan Beulich
2013-04-23 14:21                               ` George Dunlap
2013-04-23 14:43                                 ` jacek burghardt
2013-04-23 14:47                                   ` George Dunlap
2013-04-23 14:55                                 ` Jan Beulich
2013-04-23 14:57                                   ` George Dunlap
2013-04-23 15:02                                     ` Jan Beulich
2013-04-23 15:46                                       ` George Dunlap
2013-04-23 16:04                                         ` Jan Beulich
2013-04-23 16:12                                           ` George Dunlap
2013-04-23 16:30                                             ` Tim Deegan
2013-04-24 16:09                                               ` Jan Beulich
2013-04-25 16:02                         ` Jan Beulich
2013-04-25 16:34                           ` Tim Deegan
2013-04-25 16:40                             ` George Dunlap
2013-04-26  6:43                             ` Jan Beulich
2013-04-26 16:10                             ` Jan Beulich
2013-04-26 16:56                               ` Ian Campbell
2013-04-29  6:53                                 ` Jan Beulich
2013-04-29  8:20                                   ` Ian Campbell
2013-04-29  8:37                                     ` Jan Beulich
2013-04-29  8:40                                       ` Paul Durrant
2013-04-29  8:47                                         ` Jan Beulich
2013-04-29  9:15                                           ` Paul Durrant
2013-04-29  9:53                                             ` Paul Durrant
2013-04-29 10:43                                               ` Jan Beulich
2013-05-01 14:49                                           ` Paul Durrant
2013-05-02  6:32                                             ` Jan Beulich
2013-05-02  8:23                                               ` Paul Durrant
2013-04-29  8:37                                     ` Paul Durrant
2013-04-29 10:29                                     ` Tim Deegan
2013-04-29 11:07                                       ` Ian Campbell
2013-04-29 11:27                                         ` Jan Beulich
2013-04-29 11:40                                           ` Ian Campbell
2013-04-29 11:35                                         ` Tim Deegan
2013-04-29 11:42                                           ` Jan Beulich
2013-04-29 11:44                                           ` Ian Campbell
2013-04-29 11:49                                             ` Tim Deegan
2013-04-29 11:57                                               ` Ian Campbell
2013-04-29 12:01                                               ` Jan Beulich
2013-04-29  8:35                                 ` Paul Durrant
2013-04-23 10:55                       ` Tim Deegan
2013-04-16 15:24 ` Ren, Yongjie
2013-04-16 15:32   ` George Dunlap
2013-04-16 15:37     ` Tim Deegan
2013-04-16 15:56       ` George Dunlap
2013-04-18  2:21     ` Ren, Yongjie

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=517664BE.401@eu.citrix.com \
    --to=george.dunlap@eu.citrix.com \
    --cc=JBeulich@suse.com \
    --cc=eddie.dong@intel.com \
    --cc=roger.pau@citrix.com \
    --cc=suravee.suthikulpanit@amd.com \
    --cc=tim@xen.org \
    --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.