Re: [Qemu-devel] [PATCH 00/31] Allow configuring the qcow2 L2 cache entry size

[Fam Zheng]

On Thu, 10/12 09:11, Eric Blake wrote:
> Fam,
> 
> On 10/12/2017 09:02 AM, address@hidden 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
> > /tmp/qemu-test/src/block/qcow2.c: In function 
> > 'qcow2_update_options_prepare':
> > /tmp/qemu-test/src/block/qcow2.c:899:25: error: 
> > 'l2_cache_entry_size' may be used uninitialized in this 
> > function [-Werror=maybe-uninitialized]
> >      r->l2_table_cache = qcow2_cache_create(bs, 
> > l2_cache_size,
> >                          
> > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >                                             
> > l2_cache_entry_size);
> >                                             
> > ~~~~~~~~~~~~~~~~~~~~
> 
> 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