[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] dump with xen-unstable & linux 3.2.17

>>> On 22.05.12 at 23:00, Ben Guthro <ben@xxxxxxxxxx> wrote:
> I've bisected this to the following commit in the xen-unstable git tree.
> I'll be able to dive in a little deeper tomorrow.
> If you see anything here that looks suspicious to the crash
> referenced... let me know.

As the change was really a re-write of a submission by JÃrgen,
I'm adding him to Cc.

Unless he has an immediate idea, we definitely want to
understand why "cpus" is empty - hence we'd want to see
*online, *vc->cpu_affinity, vc->cpu_id, and maybe
vc->processor. (Printing them is probably not a good idea
here, so I'd instead suggest just copying them to [additional]
on-stack variables, making sure the compiler doesn't optimize
them away.)

Probably it would be good to also know what each active
vCPU's ->cpu_affinity was right before suspend and/or right
after resume (perhaps in freeze_domains() and/or
thaw_domains(). That way we'd at least know whether the
affinity - despite the offending changeset's inverse intention -
did get changed during the resume process.


> commit e4a31205766d518276d0fbc1780bcd63db1a502d
> Author: Keir Fraser <keir@xxxxxxx>
> Date:   Thu Mar 22 12:20:13 2012 +0000
>     Introduce system_state variable.
>     Use it to replace x86-specific early_boot boolean variable.
>     Also use it to detect suspend/resume case during cpu offline/online
>     to avoid unnecessarily breaking vcpu and cpupool affinities.
>     Signed-off-by: Keir Fraser <keir@xxxxxxx>
>     Acked-by: Juergen Gross <juergen.gross@xxxxxxxxxxxxxx>
> http://xenbits.xen.org/gitweb/?p=xen-unstable.git;a=commit;h=e4a31205766d5182 
> 76d0fbc1780bcd63db1a502d
> or, if you prefer mercurial:
> http://xenbits.xen.org/hg/staging/xen-unstable.hg/rev/d5ccb2d1dbd1 
> On Tue, May 22, 2012 at 2:26 PM, Ben Guthro <ben@xxxxxxxxxx> wrote:
>> OK, I'll compare what got committed with our old patches against 4.0 &
>> see if there are any obvious differences.
>> Tom Goetz wrote those, originally - and he's on vacation - so
>> specifics about them may have to wait until he returns.
>> On Tue, May 22, 2012 at 2:00 PM, Konrad Rzeszutek Wilk
>> <konrad.wilk@xxxxxxxxxx> wrote:
>>> On Tue, May 22, 2012 at 01:55:25PM -0400, Ben Guthro wrote:
>>>> Are you referring to kernel, or Xen patches?
>>> Xen.
>>>> I'm currently seeing the issue in question with the xen-unstable tip
>>>> none of our patches pushed...
>>> I pushed the serial ones.
>>>> ...or are you suggesting that the patch, as committed is incorrect?
>>> I just tried to suspend and resume Xen 4.1 with and without serial console
>>> and it only works when there is no serial console.
>>> I think something is buggy. Not sure what (this was with a normal
>>> built-in motherboard serial port).

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.