-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Am 11.02.2012 15:02, schrieb Jan Kiszka: > On 2012-02-11 14:59, Andreas Färber wrote: >> Am 11.02.2012 14:35, schrieb Jan Kiszka: >>> On 2012-02-11 14:21, Andreas Färber wrote: >>>> CPU base class v3: http://patchwork.ozlabs.org/patch/139284/ >>>> (v4 coming up) >>>> >>>> That doesn't prevent target-specific devices. Although Paolo >>>> does want that to change wrt properties. >> >>> This patch doesn't explain yet what shall happen to >>> cpu_single_env and CPUState accesses from target-specific >>> devices. >> >> True. We can't change them before all targets are converted. So >> far I have 3/14 and still some review comments to work in. >> >> Another patch in that series uses a macro >> s/ENV_GET_OBJECT/ENV_GET_CPU/ to go from CPUState -> CPU while >> we convert targets. >> >> Depending on our taste, cpu_single_env might become >> cpu_single_cpu, single_cpu or cpu_single. >> >>> Do you plan accessors for registers? >> >> No, registers are in target-specific ARMCPU, S390CPU, MIPSCPU, >> etc. and their CPU*State. It would be possible to have static >> inline accessors but so far I've seen no need. > > Then the devices need to have access to a CPUState pointer, just as > so far. Yes and no. They can have any target-specific pointer they want, just as before. But no global first_cpu / cpu_single_env pointer - that's replaced by CPU pointers, through which members of derived classes can be accessed (which did not work for CPUState due to CPU_COMMON members being at target-specific offset in the middle). There's nothing wrong with your patch per se, just that it may need to get refactored some time soonish. We need to be aware of it so that we don't create merge conflicts for Anthony. Andreas - -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.18 (GNU/Linux) iQIcBAEBAgAGBQJPNndNAAoJEPou0S0+fgE/+EoQAJFau/xsn2CYcuKPEsJmAMRk yhOPBT6EJ2Q+34h31uLr3iftxQO9JpnLfEhB7ekTs36i0GklUsCQKgn4rg6vmPKj tLvUk/hF4zuqJzUJOwxnYxYjuzdEGHuEbkCYgclUtNnywHCo3GLXhqP0izSds9mF MhmqD45GblecjUpH7zdM/WTvulQm824hbDFPTCQaH8IQsw0QxT1Y4B71gpQtFJvJ pVk2+qfc488ClhOhPISC5IiQFPnR7DVju82FuDgn6JFq/db9o3KXqIRlQg7pqkPc h4K+Nz/rhzWpR6jtbTKqJV3yWBV9vxs6YDMSICZnGBabTlHh+tKoabg25Aj5zbcM 6Dmw10uFybi+jKlygKiSSxExParaRC9B3EFCk4dUhMC28B+qFSEkRA62Qpjndxwg HCmzg2kSQpufyrWNdWj8W+mNygU/0rm8xcB7fX1vhSOmdu3DNTPIH7P4C9hOfC1g hdIo0DpSd4AFfEIjZ0Loq0XOWKO9V05pOlcVsGmnCmGmfPXFPHWCFq3LGPz9Bj/7 rK1YtReDMXFOhq+QsOuRDuz1pCpPEfT4YhiXRuPsLlIaSszjFx3i6WAxBmh/tTtA oxoGZQPUI3SRZYZPN5W+J5HqRyNkB16ffsrbcHVTmCrUm33yT+7a6S/vPE9NlZpm zy92ShUp7JDvFjtnyOLK =uTCH -----END PGP SIGNATURE----- -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html