Re: [Qemu-devel] [PATCH 1/6] libqblock APIs

[Wenchao Xia]

于 2012-9-3 21:18, Paolo Bonzini 写道:
> Il 03/09/2012 11:18, Wenchao Xia ha scritto:
> >    1 QBroker. These structure was used to retrieve errors, every thread must
> > create one first, Later maybe thread related staff could be added into it.
>
> Can you use GError instead?
  read through the GError doc, GError is defined as following:
struct GError {
  GQuark       domain;
  gint         code;
  gchar       *message;
};
  I am worried about the message member, I guess program would be
aborted if OOM, which I was tring to avoid, so I used char err_msg[1024]
in my code, and make things simpler.

> >    3 QBlockInfoImageStatic. Now it is not folded with location and format.
>
> What does "Static" mean?
 It is about sorting the information into following kinds:
1) static. It is values that defined at creating time/modifying time,
mostly some settings, and it would not be automatically changed in I/O.
2) dynamic. Some information that would changes in I/O and other
operations, such as allocated_size, snapshots.
3) statistics.
  Now only static one is provided, so I added _static suffix.

> Paolo


--
Best Regards

Wenchao Xia