Re: [Xen-ia64-devel] [PATCH]Fix infinite loop in fault handler if debugger is not connected - Alex Williamson - com.xensource.lists.xen-ia64-devel

4 messages in com.xensource.lists.xen-ia64-develRe: [Xen-ia64-devel] [PATCH]Fix infin...

From	Sent On	Attachments
SUZUKI Kazuhiro	23 Jan 2008 00:38	.patch
Alex Williamson	28 Jan 2008 07:35
SUZUKI Kazuhiro	30 Jan 2008 01:39	.patch
Alex Williamson	30 Jan 2008 10:49

Subject:	Re: [Xen-ia64-devel] [PATCH]Fix infinite loop in fault handler if debugger is not connected
From:	Alex Williamson (alex...@hp.com)
Date:	01/30/2008 10:49:04 AM
List:	com.xensource.lists.xen-ia64-devel

On Wed, 2008-01-30 at 18:40 +0900, SUZUKI Kazuhiro wrote:

Hi Alex,

Thank you for your comment.

vcpu_increment_iip() takes a pointer of struct vcpu and calculate regs from vcpu. If current->arch._thread.on_ustack == 1 then regs == vcpu_regs(current), otherwise regs is created in the stack. vcpu_regs(current) indicates guest's registers. So we cannot use vcpu_increment_iip() in this part.

How about this patch.

Thanks, this looks better. I made one minor change to make use of the ia64_psr() macro instead of manually extracting cr_ipsr. Thanks,

Alex