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

Re: [Xen-devel] Oxenstored short-write issue



On Tue, Oct 27, 2015 at 03:56:07PM +0000, David Scott wrote:
> Hi,
> 
> > On 27 Oct 2015, at 15:37, Wei Liu <wei.liu2@xxxxxxxxxx> wrote:
> > 
> > Hi Dave
> > 
> > We discovered an issue in oxenstored: when it writes to the ring, it writes
> > a chunk of contiguous data. When it tries to write across ring boundary,
> > it returns a short-write. That leads to stalling mini-os's xenbus ring.
> > 
> > Samuel provided a diff which solves the issue:
> > http://www.gossamer-threads.com/lists/xen/devel/403800#403800
> > 
> > You can find our discussions in that thread. Note that thread was mainly
> > for a patch to fix an issue in mini-os but we discovered that oxenstored
> > issue along the way.
> 
> Interesting! I always thought it was a bit odd that oxenstored could write a 
> partial packet but I thought it would retry higher up and complete the 
> transmit before sleeping. Itâs certainly clearer to complete the write at the 
> lower-level though.
> 
> So I think the diff looks ok.
> 

OK, if you think that hunk is good. I will submit a proper patch for it.

Samuel, can I have your S-o-B on that patch?

Wei.

> Cheers,
> Dave

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

 


Rackspace

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