On 07/30/2014 02:53 AM, Stefan Hajnoczi wrote: > When request A is a strict superset of request B: > > AAAAAAAA > BBBB > > multiwrite_merge() merges them as follows: > > AABBBB > > The tail of request A should have been included: > > AABBBBAA > > This patch fixes data loss but this code path is probably rare. Since > guests cannot assume ordering between in-flight requests, few > applications submit overlapping write requests. > > Reported-by: Slava Pestov <sviatoslav.pes...@gmail.com> > Signed-off-by: Stefan Hajnoczi <stefa...@redhat.com> > --- > block.c | 6 ++++++ > 1 file changed, 6 insertions(+) >
Reviewed-by: Eric Blake <ebl...@redhat.com> Technically a bug fix, so okay for 2.1 if desired; but it is awfully late and as you said it is fairly unlikely for guests to be relying on overlapping multiwrite, so I'm okay with the decision to delay it to 2.2 and just have stable 2.1.1/downstream backports. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature