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: [Xen-devel] [PATCH] pvSCSI: Sanity check for REPORT_LUN emulation

Hi Steven-san,

Sorry for delay.

On Fri, 18 Jul 2008 14:58:41 +0100
Steven Smith <steven.smith@xxxxxxxxxx> wrote:

> > I will attach a patch in order to provide following.
> > 
> > - Sanity check for REPORT_LUN emulation.
> Thank you, that's much better.
> 
> I have one slight issue with the new approach: if a REPORT_LUNS
> command races with a LUN hot-add on the backend, you tell the frontend
> that the command failed.  Might it be better to instead retry the
> command on the backend?
> 
> It's pretty unlikely that anyone would actually hit this race, and the
> current behaviour isn't completely unreasonable, but it seems like
> retrying in the backend would be a bit more robust.  What do you
> think?

Yes, I consider retrying is a right way, because we can expect few 
times retrying will certainly succeed in this case.
I will attach a patch with the number of retry is three. How do you
think three times retry is appropriate?

Best regards,


-----
Jun Kamada

Attachment: linux_pvscsi_add_retry_REPORTLUN.patch
Description: Binary data

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