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

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> 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.

Jan

> 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
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel