WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

RE: [GIT/PATCH 0/5] Re: [Xen-devel] unfair servicing of DomU vbd request

Thanks James.
Patch applied and crash log attached.
 
I've been able to merge the code accrording to Ian post on
http://lists.colo.xensource.com/archives/html/xen-devel/2011-03/msg00225.html
 
But unfortunately, the starting of 24HVMS become rather slow, and
after some period of time, whole Host stop responding.
It takes some more time to find what is going on.

 
> Subject: RE: [GIT/PATCH 0/5] Re: [Xen-devel] unfair servicing of DomU vbd requests
> Date: Tue, 8 Mar 2011 22:04:42 +1100
> From: james.harper@xxxxxxxxxxxxxxxx
> To: tinnycloud@xxxxxxxxxxx
>
> >
> > Hi James.
> >
> > I go through the patch, since all its changes happens inside
> > __xen_evtchn_do_upcall(),
> > so I apply the patch into my 2.6.31.kernel by manual. (I was failed to
> merge a
> > usable,
> > 2.6.38dom0, since quite a lot branch needed, such as blkback, netback)
> >
> > But still, the blue screen shows up. Log attached.
> > Is the timeout better than before?
> >
> > Beside, I think I need find some to examine my harddisk.
> >
>
> Not really important, but the ASSERT you are seeing in dump mode happens
> when xenvbd finally does complete the request, but obviously dump mode
> knows nothing about it. The following patch initialises the reset flag
> on all requests in dump mode so that when they do come back they are
> just discarded. Crash dump should now complete (although slowly in your
> case!):
>
> diff -r 4722ecc909c3 xenvbd/xenvbd_scsiport.c
> --- a/xenvbd/xenvbd_scsiport.c Thu Mar 03 23:21:10 2011 +1100
> +++ b/xenvbd/xenvbd_scsiport.c Tue Mar 08 22:01:57 2011 +1100
> @@ -278,6 +278,12 @@
> {
> xvdd->shadows[i].req.id = i;
> put_shadow_on_freelist(xvdd, &xvdd->shadows[i]);
> + if (dump_mode)
> + {
> + /* set reset = TRUE to pick up in-flight requests when dump
> mode kicked in */
> + /* do it after put_shadow_on_freelist because that resets the
> reset flag */
> + xvdd->shadows[i].reset = TRUE;
> + }
> }
> }
>
> James

Attachment: qemu-dm-w3.MR_cp10.vhd.log
Description: Text document

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel