|
From: | Richard Henderson |
Subject: | Re: [Qemu-devel] [PATCH 2/4] Octeon cpu definitions in target-mips and Octeon specific changes in set_thread_area syscall |
Date: | Mon, 15 Aug 2011 08:43:43 -0700 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:5.0) Gecko/20110707 Thunderbird/5.0 |
On 08/15/2011 04:25 AM, address@hidden wrote: > ((CPUMIPSState *) cpu_env)->tls_value = arg1; > + if (((CPUMIPSState *) cpu_env)->insn_flags & CPU_OCTEON) { > + /* tls entry is moved to k0 so that this can be used later > + currently this thing is tested only for Octeon */ > + ((CPUMIPSState *) cpu_env)->active_tc.gpr[26] = arg1; > + } You wanted INSN_OCTEON, not CPU_OCTEON, which includes CPU_MIPS64R2. That said, this is *not* in the current linux kernel. And I question the wisdom of changing the user-space ABI for TLS for a single CPU. I think you'd better leave this out until it's actually accepted upstream. r~
[Prev in Thread] | Current Thread | [Next in Thread] |