Re: [Qemu-devel] [PATCH v5 0/7] tcg: allocate TB structs preceding translate

2017-06-09 Thread Emilio G. Cota
On Fri, Jun 09, 2017 at 12:58:17 -0700, Richard Henderson wrote: > >>=== OUTPUT BEGIN === > >>Checking PATCH 1/7: util: add cacheinfo... > >>ERROR: do not initialise globals to 0 or NULL > >>#149: FILE: util/cacheinfo.c:11: > >>+int qemu_icache_linesize = 0; > >> > >>ERROR: do not initialise global

Re: [Qemu-devel] [PATCH v5 0/7] tcg: allocate TB structs preceding translate

2017-06-09 Thread Richard Henderson
On 06/09/2017 12:52 PM, Emilio G. Cota wrote: On Thu, Jun 08, 2017 at 22:37:12 -0700, Richard Henderson wrote: This is a follow-up to Emilio's patch set. My primary changes to Emilio's patches are to the first patch, in merging the existing implementations from tcg/ppc/tcg-target.inc.c into uti

Re: [Qemu-devel] [PATCH v5 0/7] tcg: allocate TB structs preceding translate

2017-06-09 Thread Emilio G. Cota
On Thu, Jun 08, 2017 at 22:37:12 -0700, Richard Henderson wrote: > This is a follow-up to Emilio's patch set. > > My primary changes to Emilio's patches are to the first patch, in > merging the existing implementations from tcg/ppc/tcg-target.inc.c > into util/cacheinfo.c. > > Then I've a few fol

Re: [Qemu-devel] [PATCH v5 0/7] tcg: allocate TB structs preceding translate

2017-06-09 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Subject: [Qemu-devel] [PATCH v5 0/7] tcg: allocate TB structs preceding translate Message-id: 20170609053719.26251-1-...@twiddle.net === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n

[Qemu-devel] [PATCH v5 0/7] tcg: allocate TB structs preceding translate

2017-06-08 Thread Richard Henderson
This is a follow-up to Emilio's patch set. My primary changes to Emilio's patches are to the first patch, in merging the existing implementations from tcg/ppc/tcg-target.inc.c into util/cacheinfo.c. Then I've a few follow-up patches to take advantage of the new TB placement for arm platforms. I'

[Qemu-devel] [PATCH v5 0/7] tcg: allocate TB structs preceding translate

2017-06-08 Thread Richard Henderson
This is a follow-up to Emilio's patch set. My primary changes to Emilio's patches are to the first patch, in merging the existing implementations from tcg/ppc/tcg-target.inc.c into util/cacheinfo.c. Then I've a few follow-up patches to take advantage of the new TB placement for arm platforms. I'

[Qemu-devel] [PATCH v5 0/7] tcg: allocate TB structs preceding translate

2017-06-08 Thread Richard Henderson
This is a follow-up to Emilio's patch set. My primary changes to Emilio's patches are to the first patch, in merging the existing implementations from tcg/ppc/tcg-target.inc.c into util/cacheinfo.c. Then I've a few follow-up patches to take advantage of the new TB placement for arm platforms. I'