From: Eric Blake <eblake@redhat.com>
To: qemu-devel@nongnu.org, no-reply@patchew.org, berto@igalia.com
Cc: kwolf@redhat.com, famz@redhat.com, qemu-block@nongnu.org,
den@openvz.org, mreitz@redhat.comFam Zheng <famz@redhat.com>
Subject: Re: [Qemu-devel] [PATCH 00/31] Allow configuring the qcow2 L2 cache entry size
Date: Thu, 12 Oct 2017 09:11:41 -0500 [thread overview]
Message-ID: <f8bb407a-9f1b-4679-e630-764b96b79edf@redhat.com> (raw)
In-Reply-To: <150781692103.288.8938679290451561790@b58463cdfd5f>
[-- Attachment #1: Type: text/plain, Size: 1350 bytes --]
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.
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 619 bytes --]
next prev parent reply other threads:[~2017-10-12 14:12 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 [this message]
2017-10-12 16:44 ` Fam Zheng
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=f8bb407a-9f1b-4679-e630-764b96b79edf@redhat.com \
--to=eblake@redhat.com \
--cc=berto@igalia.com \
--cc=den@openvz.org \
--cc=famz@redhat.com \
--cc=kwolf@redhat.com \
--cc=mreitz@redhat.comFam \
--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).