Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Blue Swirl
On Sun, May 15, 2011 at 5:06 PM, Aurelien Jarno wrote: > On Sun, May 15, 2011 at 05:02:37PM +0300, Blue Swirl wrote: >> On Sun, May 15, 2011 at 4:43 PM, Aurelien Jarno wrote: >> > On Sun, May 15, 2011 at 04:16:20PM +0300, Blue Swirl wrote: >> >> On Sun, May 15, 2011 at 3:49 PM, Aurelien Jarno >

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Blue Swirl
On Sun, May 15, 2011 at 5:03 PM, Aurelien Jarno wrote: > On Sun, May 15, 2011 at 04:42:05PM +0300, Blue Swirl wrote: >> On Sun, May 15, 2011 at 4:02 PM, Aurelien Jarno wrote: >> > On Sun, May 15, 2011 at 03:37:00PM +0300, Blue Swirl wrote: >> >> On Sun, May 15, 2011 at 3:14 PM, Laurent Desnogues

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Aurelien Jarno
On Sun, May 15, 2011 at 05:02:37PM +0300, Blue Swirl wrote: > On Sun, May 15, 2011 at 4:43 PM, Aurelien Jarno wrote: > > On Sun, May 15, 2011 at 04:16:20PM +0300, Blue Swirl wrote: > >> On Sun, May 15, 2011 at 3:49 PM, Aurelien Jarno > >> wrote: > >> > On Sun, May 15, 2011 at 03:30:17PM +0300, B

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Aurelien Jarno
On Sun, May 15, 2011 at 04:42:05PM +0300, Blue Swirl wrote: > On Sun, May 15, 2011 at 4:02 PM, Aurelien Jarno wrote: > > On Sun, May 15, 2011 at 03:37:00PM +0300, Blue Swirl wrote: > >> On Sun, May 15, 2011 at 3:14 PM, Laurent Desnogues > >> wrote: > >> > On Sun, May 15, 2011 at 1:33 PM, Blue Swi

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Blue Swirl
On Sun, May 15, 2011 at 4:43 PM, Aurelien Jarno wrote: > On Sun, May 15, 2011 at 04:16:20PM +0300, Blue Swirl wrote: >> On Sun, May 15, 2011 at 3:49 PM, Aurelien Jarno wrote: >> > On Sun, May 15, 2011 at 03:30:17PM +0300, Blue Swirl wrote: >> >> On Sun, May 15, 2011 at 2:36 PM, Aurelien Jarno >

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Aurelien Jarno
On Sun, May 15, 2011 at 04:16:20PM +0300, Blue Swirl wrote: > On Sun, May 15, 2011 at 3:49 PM, Aurelien Jarno wrote: > > On Sun, May 15, 2011 at 03:30:17PM +0300, Blue Swirl wrote: > >> On Sun, May 15, 2011 at 2:36 PM, Aurelien Jarno > >> wrote: > >> > On Sun, May 15, 2011 at 02:12:27PM +0300, B

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Blue Swirl
On Sun, May 15, 2011 at 4:02 PM, Aurelien Jarno wrote: > On Sun, May 15, 2011 at 03:37:00PM +0300, Blue Swirl wrote: >> On Sun, May 15, 2011 at 3:14 PM, Laurent Desnogues >> wrote: >> > On Sun, May 15, 2011 at 1:33 PM, Blue Swirl wrote: >> > [...] >> >>> x86_64 uses r14 as TCG_AREG0. Despite the

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Blue Swirl
On Sun, May 15, 2011 at 3:49 PM, Aurelien Jarno wrote: > On Sun, May 15, 2011 at 03:30:17PM +0300, Blue Swirl wrote: >> On Sun, May 15, 2011 at 2:36 PM, Aurelien Jarno wrote: >> > On Sun, May 15, 2011 at 02:12:27PM +0300, Blue Swirl wrote: >> >> On Sun, May 15, 2011 at 1:19 PM, Aurelien Jarno >

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Aurelien Jarno
On Sun, May 15, 2011 at 03:37:00PM +0300, Blue Swirl wrote: > On Sun, May 15, 2011 at 3:14 PM, Laurent Desnogues > wrote: > > On Sun, May 15, 2011 at 1:33 PM, Blue Swirl wrote: > > [...] > >>> x86_64 uses r14 as TCG_AREG0. Despite the instructions being quite > >>> simple (only 2 movi_i32), the r

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Aurelien Jarno
On Sun, May 15, 2011 at 03:30:17PM +0300, Blue Swirl wrote: > On Sun, May 15, 2011 at 2:36 PM, Aurelien Jarno wrote: > > On Sun, May 15, 2011 at 02:12:27PM +0300, Blue Swirl wrote: > >> On Sun, May 15, 2011 at 1:19 PM, Aurelien Jarno > >> wrote: > >> > I still don't get where are this list of po

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Blue Swirl
On Sun, May 15, 2011 at 3:14 PM, Laurent Desnogues wrote: > On Sun, May 15, 2011 at 1:33 PM, Blue Swirl wrote: > [...] >>> x86_64 uses r14 as TCG_AREG0. Despite the instructions being quite >>> simple (only 2 movi_i32), the resulting code makes 2 access to env to >>> save the two registers. Havin

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Blue Swirl
On Sun, May 15, 2011 at 2:37 PM, Aurelien Jarno wrote: > On Sun, May 15, 2011 at 02:33:55PM +0300, Blue Swirl wrote: >> On Sun, May 15, 2011 at 2:14 PM, Aurelien Jarno wrote: >> > On Sun, May 15, 2011 at 02:03:40PM +0300, Blue Swirl wrote: >> >> On Sun, May 15, 2011 at 12:27 PM, Laurent Desnogues

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Blue Swirl
On Sun, May 15, 2011 at 2:36 PM, Aurelien Jarno wrote: > On Sun, May 15, 2011 at 02:12:27PM +0300, Blue Swirl wrote: >> On Sun, May 15, 2011 at 1:19 PM, Aurelien Jarno wrote: >> > I still don't get where are this list of possible changes? Did I miss >> > something in another thread? >> >> I'm ref

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Laurent Desnogues
On Sun, May 15, 2011 at 1:33 PM, Blue Swirl wrote: [...] >> x86_64 uses r14 as TCG_AREG0. Despite the instructions being quite >> simple (only 2 movi_i32), the resulting code makes 2 access to env to >> save the two registers. Having to reload the env pointer each time to a >> register would clear

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Aurelien Jarno
On Sun, May 15, 2011 at 02:33:55PM +0300, Blue Swirl wrote: > On Sun, May 15, 2011 at 2:14 PM, Aurelien Jarno wrote: > > On Sun, May 15, 2011 at 02:03:40PM +0300, Blue Swirl wrote: > >> On Sun, May 15, 2011 at 12:27 PM, Laurent Desnogues > >> wrote: > >> > On Sun, May 15, 2011 at 9:15 AM, Blue Sw

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Aurelien Jarno
On Sun, May 15, 2011 at 02:12:27PM +0300, Blue Swirl wrote: > On Sun, May 15, 2011 at 1:19 PM, Aurelien Jarno wrote: > > I still don't get where are this list of possible changes? Did I miss > > something in another thread? > > I'm referring to the patches I sent. Ok, patches 1, 2 and 4 to 7 loo

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Blue Swirl
On Sun, May 15, 2011 at 2:14 PM, Aurelien Jarno wrote: > On Sun, May 15, 2011 at 02:03:40PM +0300, Blue Swirl wrote: >> On Sun, May 15, 2011 at 12:27 PM, Laurent Desnogues >> wrote: >> > On Sun, May 15, 2011 at 9:15 AM, Blue Swirl wrote: >> >> On Sun, May 15, 2011 at 1:04 AM, Aurelien Jarno >>

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Aurelien Jarno
On Sun, May 15, 2011 at 02:03:40PM +0300, Blue Swirl wrote: > On Sun, May 15, 2011 at 12:27 PM, Laurent Desnogues > wrote: > > On Sun, May 15, 2011 at 9:15 AM, Blue Swirl wrote: > >> On Sun, May 15, 2011 at 1:04 AM, Aurelien Jarno > >> wrote: > >>> On Sun, May 15, 2011 at 12:52:35AM +0300, Blue

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Blue Swirl
On Sun, May 15, 2011 at 1:19 PM, Aurelien Jarno wrote: > On Sun, May 15, 2011 at 12:58:49PM +0300, Blue Swirl wrote: >> On Sun, May 15, 2011 at 12:24 PM, Aurelien Jarno >> wrote: >> > On Sun, May 15, 2011 at 10:15:52AM +0300, Blue Swirl wrote: >> >> On Sun, May 15, 2011 at 1:04 AM, Aurelien Jarn

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Blue Swirl
On Sun, May 15, 2011 at 12:27 PM, Laurent Desnogues wrote: > On Sun, May 15, 2011 at 9:15 AM, Blue Swirl wrote: >> On Sun, May 15, 2011 at 1:04 AM, Aurelien Jarno wrote: >>> On Sun, May 15, 2011 at 12:52:35AM +0300, Blue Swirl wrote: On Sun, May 15, 2011 at 12:16 AM, Aurelien Jarno w

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Aurelien Jarno
On Sun, May 15, 2011 at 12:58:49PM +0300, Blue Swirl wrote: > On Sun, May 15, 2011 at 12:24 PM, Aurelien Jarno wrote: > > On Sun, May 15, 2011 at 10:15:52AM +0300, Blue Swirl wrote: > >> On Sun, May 15, 2011 at 1:04 AM, Aurelien Jarno > >> wrote: > >> > On Sun, May 15, 2011 at 12:52:35AM +0300,

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Blue Swirl
On Sun, May 15, 2011 at 12:24 PM, Aurelien Jarno wrote: > On Sun, May 15, 2011 at 10:15:52AM +0300, Blue Swirl wrote: >> On Sun, May 15, 2011 at 1:04 AM, Aurelien Jarno wrote: >> > On Sun, May 15, 2011 at 12:52:35AM +0300, Blue Swirl wrote: >> >> On Sun, May 15, 2011 at 12:16 AM, Aurelien Jarno

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Aurelien Jarno
On Sun, May 15, 2011 at 11:27:17AM +0200, Laurent Desnogues wrote: > On Sun, May 15, 2011 at 9:15 AM, Blue Swirl wrote: > > On Sun, May 15, 2011 at 1:04 AM, Aurelien Jarno > > wrote: > >> On Sun, May 15, 2011 at 12:52:35AM +0300, Blue Swirl wrote: > >>> On Sun, May 15, 2011 at 12:16 AM, Aurelien

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Laurent Desnogues
On Sun, May 15, 2011 at 9:15 AM, Blue Swirl wrote: > On Sun, May 15, 2011 at 1:04 AM, Aurelien Jarno wrote: >> On Sun, May 15, 2011 at 12:52:35AM +0300, Blue Swirl wrote: >>> On Sun, May 15, 2011 at 12:16 AM, Aurelien Jarno >>> wrote: >>> > On Sat, May 14, 2011 at 10:35:20PM +0300, Blue Swirl w

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Aurelien Jarno
On Sun, May 15, 2011 at 10:15:52AM +0300, Blue Swirl wrote: > On Sun, May 15, 2011 at 1:04 AM, Aurelien Jarno wrote: > > On Sun, May 15, 2011 at 12:52:35AM +0300, Blue Swirl wrote: > >> On Sun, May 15, 2011 at 12:16 AM, Aurelien Jarno > >> wrote: > >> > On Sat, May 14, 2011 at 10:35:20PM +0300,

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-15 Thread Blue Swirl
On Sun, May 15, 2011 at 1:04 AM, Aurelien Jarno wrote: > On Sun, May 15, 2011 at 12:52:35AM +0300, Blue Swirl wrote: >> On Sun, May 15, 2011 at 12:16 AM, Aurelien Jarno >> wrote: >> > On Sat, May 14, 2011 at 10:35:20PM +0300, Blue Swirl wrote: >> >> Here's a RFC series for eliminating AREG0. >>

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-14 Thread Aurelien Jarno
On Sat, May 14, 2011 at 11:16:16PM +0200, Aurelien Jarno wrote: > On Sat, May 14, 2011 at 10:35:20PM +0300, Blue Swirl wrote: > > Here's a RFC series for eliminating AREG0. > > > > Blue Swirl (11): > > Move user emulator stuff from cpu-exec.c to user-exec.c > > Delete unused tb_invalidate_page

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-14 Thread Aurelien Jarno
On Sun, May 15, 2011 at 12:52:35AM +0300, Blue Swirl wrote: > On Sun, May 15, 2011 at 12:16 AM, Aurelien Jarno wrote: > > On Sat, May 14, 2011 at 10:35:20PM +0300, Blue Swirl wrote: > >> Here's a RFC series for eliminating AREG0. > >> > >> Blue Swirl (11): > >>   Move user emulator stuff from cpu-

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-14 Thread Blue Swirl
On Sun, May 15, 2011 at 12:16 AM, Aurelien Jarno wrote: > On Sat, May 14, 2011 at 10:35:20PM +0300, Blue Swirl wrote: >> Here's a RFC series for eliminating AREG0. >> >> Blue Swirl (11): >>   Move user emulator stuff from cpu-exec.c to user-exec.c >>   Delete unused tb_invalidate_page_range >> >>

Re: [Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-14 Thread Aurelien Jarno
On Sat, May 14, 2011 at 10:35:20PM +0300, Blue Swirl wrote: > Here's a RFC series for eliminating AREG0. > > Blue Swirl (11): > Move user emulator stuff from cpu-exec.c to user-exec.c > Delete unused tb_invalidate_page_range > > The above should be OK to commit. > > cpu_loop_exit: avoid us

[Qemu-devel] [PATCH RFC 00/11] AREG0 elimination

2011-05-14 Thread Blue Swirl
Here's a RFC series for eliminating AREG0. Blue Swirl (11): Move user emulator stuff from cpu-exec.c to user-exec.c Delete unused tb_invalidate_page_range The above should be OK to commit. cpu_loop_exit: avoid using AREG0 Delegate setup of TCG temporaries to targets These two are not, u