qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Fam Zheng <famz@redhat.com>
To: Eric Blake <eblake@redhat.com>
Cc: qemu-devel@nongnu.org, no-reply@patchew.org, berto@igalia.com,
	kwolf@redhat.com, qemu-block@nongnu.org, den@openvz.org,
	mreitz@redhat.com
Subject: Re: [Qemu-devel] [PATCH 00/31] Allow configuring the qcow2 L2 cache entry size
Date: Fri, 13 Oct 2017 00:44:25 +0800	[thread overview]
Message-ID: <20171012164425.GA16319@lemon> (raw)
In-Reply-To: <f8bb407a-9f1b-4679-e630-764b96b79edf@redhat.com>

On Thu, 10/12 09:11, Eric Blake wrote:
> Fam,
> 
> On 10/12/2017 09:02 AM, no-reply@patchew.org wrote:
> > Hi,
> > 
> > This series failed automatic build test. Please find the testing commands and
> > their output below. If you have docker installed, you can probably reproduce it
> > locally.
> > 
> 
> >   CC      block/qed.o
> >   CC      block/qed-l2-cache.o
> > ^[[01m^[[K/tmp/qemu-test/src/block/qcow2.c:^[[m^[[K In function '^[[01m^[[Kqcow2_update_options_prepare^[[m^[[K':
> > ^[[01m^[[K/tmp/qemu-test/src/block/qcow2.c:899:25:^[[m^[[K ^[[01;31m^[[Kerror: ^[[m^[[K'^[[01m^[[Kl2_cache_entry_size^[[m^[[K' may be used uninitialized in this function [^[[01;31m^[[K-Werror=maybe-uninitialized^[[m^[[K]
> >      r->l2_table_cache = ^[[01;31m^[[Kqcow2_cache_create(bs, l2_cache_size,^[[m^[[K
> >                          ^[[01;31m^[[K^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^[[m^[[K
> >  ^[[01;31m^[[K                                           l2_cache_entry_size)^[[m^[[K;
> >                                             ^[[01;31m^[[K~~~~~~~~~~~~~~~~~~~~^[[m^[[K
> 
> Any chance we can tweak patchew to tell gcc to NOT use color escape
> sequences on errors?  While useful in an ANSI terminal, it tends to come
> across awkwardly in email clients that don't render ANSI escapes, and
> just adds bulk to the transcript.
> 

It's odd that three is "TERM=xterm" in this env. I'll check where it is from.

Fam

      reply	other threads:[~2017-10-12 16:44 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-12 13:05 [Qemu-devel] [PATCH 00/31] Allow configuring the qcow2 L2 cache entry size Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 01/31] qcow2: Fix documentation of get_cluster_table() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 02/31] qcow2: Add table size field to Qcow2Cache Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 03/31] qcow2: Remove BDS parameter from qcow2_cache_get_table_addr() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 04/31] qcow2: Remove BDS parameter from qcow2_cache_get_table_idx() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 05/31] qcow2: Remove BDS parameter from qcow2_cache_table_release() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 06/31] qcow2: Remove BDS parameter from qcow2_cache_entry_mark_dirty() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 07/31] qcow2: Remove BDS parameter from qcow2_cache_put() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 08/31] qcow2: Remove BDS parameter from qcow2_cache_destroy() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 09/31] qcow2: Remove BDS parameter from qcow2_cache_clean_unused() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 10/31] qcow2: Remove BDS parameter from qcow2_cache_discard() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 11/31] qcow2: Remove BDS parameter from qcow2_cache_is_table_offset() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 12/31] qcow2: Add offset_to_l1_index() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 13/31] qcow2: Add l2_slice_size field to BDRVQcow2State Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 14/31] qcow2: Add offset_to_l2_slice_index() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 15/31] qcow2: Update l2_load() to support L2 slices Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 16/31] qcow2: Update l2_allocate() " Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 17/31] qcow2: Update get_cluster_table() " Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 18/31] qcow2: Update qcow2_get_cluster_offset() " Alberto Garcia
2017-10-13 12:52   ` Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 19/31] qcow2: Update qcow2_alloc_cluster_link_l2() " Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 20/31] qcow2: Update handle_copied() " Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 21/31] qcow2: Update handle_alloc() " Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 22/31] qcow2: Update discard_single_l2() " Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 23/31] qcow2: Update zero_single_l2() " Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 24/31] qcow2: Update qcow2_update_snapshot_refcount() " Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 25/31] qcow2: Update expand_zero_clusters_in_l1() " Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 26/31] qcow2: Update qcow2_truncate() " Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 27/31] qcow2: Rename l2_table in qcow2_alloc_compressed_cluster_offset() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 28/31] qcow2: Rename l2_table in count_contiguous_clusters() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 29/31] qcow2: Rename l2_table in count_contiguous_clusters_unallocated() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 30/31] qcow2: Rename l2_table in count_cow_clusters() Alberto Garcia
2017-10-12 13:05 ` [Qemu-devel] [PATCH 31/31] qcow2: Allow configuring the L2 slice size Alberto Garcia
2017-10-12 13:42 ` [Qemu-devel] [PATCH 00/31] Allow configuring the qcow2 L2 cache entry size no-reply
2017-10-12 14:02 ` no-reply
2017-10-12 14:11   ` Eric Blake
2017-10-12 16:44     ` Fam Zheng [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171012164425.GA16319@lemon \
    --to=famz@redhat.com \
    --cc=berto@igalia.com \
    --cc=den@openvz.org \
    --cc=eblake@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=mreitz@redhat.com \
    --cc=no-reply@patchew.org \
    --cc=qemu-block@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).