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

2017-06-08 Thread Richard Henderson
On 06/08/2017 10:44 AM, Emilio G. Cota wrote: On Thu, Jun 08, 2017 at 09:44:11 -0700, Richard Henderson wrote: On 06/07/2017 04:30 PM, Emilio G. Cota wrote: On Wed, Jun 07, 2017 at 08:55:29 -0700, Richard Henderson wrote: Then I've a few follow-up patches to take advantage of the new TB

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

2017-06-08 Thread Emilio G. Cota
On Thu, Jun 08, 2017 at 09:44:11 -0700, Richard Henderson wrote: > On 06/07/2017 04:30 PM, Emilio G. Cota wrote: > >On Wed, Jun 07, 2017 at 08:55:29 -0700, Richard Henderson wrote: > >>Then I've a few follow-up patches to take advantage of the new TB > >>placement for arm platforms. I've had a

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

2017-06-08 Thread Richard Henderson
On 06/07/2017 04:30 PM, Emilio G. Cota wrote: On Wed, Jun 07, 2017 at 08:55:29 -0700, Richard Henderson wrote: Then I've a few follow-up patches to take advantage of the new TB placement for arm platforms. I've had a look at the asm output for ppc64 and s390x, and don't see anything obvious

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

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

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

2017-06-07 Thread Emilio G. Cota
On Wed, Jun 07, 2017 at 08:55:29 -0700, Richard Henderson wrote: > Then I've a few follow-up patches to take advantage of the new TB > placement for arm platforms. I've had a look at the asm output for > ppc64 and s390x, and don't see anything obvious that can be improved. Nice! Just tested

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

2017-06-07 Thread no-reply
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. Subject: [Qemu-devel] [PATCH v4 0/7] tcg: allocate TB structs preceding translate Type: series Message-id

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

2017-06-07 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.