Re: [Xen-devel] [PATCH 0/11 OPW] All outstanding and corrected patches

[Matt Wilson <msw@xxxxxxxxx>]

On Mon, Nov 11, 2013 at 03:23:49PM -0800, Kelley Nielsen wrote:
> These patches update libxl_qmp.c and libxl_utils.c to use the new
> convenience macros defined in libxl_internal.h. These have all been
> submitted before, and some are acked; but since some changes and
> corrections have been made, they're collected here together to keep 
> them organized.

Hi Kelley,

Two minor points of feedback on process, rather than these actual
changes:

1) Patch 10 is missing below.
2) There's not a summary of what's changed since the last time you
   posted the series. It'd be nice to have a summary of which patches
   have been reviewed / Ack'ed in the cover letter. For a more complex
   example, see other cover letters like:
     http://lists.xen.org/archives/html/xen-devel/2013-11/msg01374.html

--msw

> [PATCH OPW 01/11] libxl: change most remaining LIBXL_LOG to LOG in
> [PATCH OPW 02/11] libxl: Use new macro LOGE() in libxl_qmp.c
> [PATCH OPW 03/11] libxl: use GCSPRINTF in place of libxl_sprintf() in
> [PATCH OPW 04/11] libxl: use GCSPRINTF instead of libxl__sprintf
> [PATCH OPW 05/11] libxl: use LOGE instead of LIBXL__LOG_ERRNO in
> [PATCH OPW 06/11] libxl: use LOG and LOGE instead of LIBXL__LOG* in
> [PATCH OPW 07/11] libxl: use LOG instead of LIBXL__LOG in
> [PATCH OPW 09/11] opw: libxl: use macro CTX in libxl_qmp.c
> [PATCH OPW 11/11] opw: libxl: use CTX macro in libxl_utils.c

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