On 14.10.2013 23:52, Wenchao Xia wrote: > Some code in qcow2-snapshot.c directly access bs->file, so in those
s/access/accesses/ > points error can't be injected by other events. Since the code in Perhaps "places" instead of "points"? (And s/error/errors/) > qcow2-snapshot.c is qcow2's internal detail similar as L1 table, Maybe something like "qcow2-snapshot.c is similar to the other qcow2 internal code (in regards to e.g. the L1 table)" would be easier to parse, although I'm not sure whether it has the same meaning you wanted to convey. > so add some debug events. Maybe remove the "so", because you already had a "since" in the previous subordinate clause? > Signed-off-by: Wenchao Xia <xiaw...@linux.vnet.ibm.com> > --- Aside from that: Reviewed-by: Max Reitz <mre...@redhat.com>
