[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v2] tools/ocaml/xb: Correct calculations of data/space the ring
On Tue, Nov 10, 2015 at 04:06:13PM +0100, Samuel Thibault wrote: > Wei Liu, on Tue 10 Nov 2015 14:59:17 +0000, wrote: > > I think I will port this patch to cxenstored at some point. As far as I > > can tell cxenstored's data / space calculation is bogus in the same way. > > The low-level function return short reads and writes, yes, but that is > handled at a higher level: initialize_fds sets timeout to 0 when there > is still room (domain_can_read() or domain_can_write()). So it will > improve performance a little bit, but not fix actual bugs. > Yeah, that's true. Just IMHO it would be better if we can actually make low level routine correct. That's orthogonal to what the upper layer is doing and should prevent latent bug if upper layer logic changes. Wei. > Samuel -- Wei. _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |