From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50836) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XhdCi-0008Hz-He for qemu-devel@nongnu.org; Fri, 24 Oct 2014 07:38:54 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XhdCb-0000Do-UR for qemu-devel@nongnu.org; Fri, 24 Oct 2014 07:38:48 -0400 Received: from mx1.redhat.com ([209.132.183.28]:30127) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XhdCb-0000DW-Mx for qemu-devel@nongnu.org; Fri, 24 Oct 2014 07:38:41 -0400 Message-ID: <544A3A39.8050407@redhat.com> Date: Fri, 24 Oct 2014 13:38:33 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: <1414033363-31032-1-git-send-email-chao.p.peng@linux.intel.com> <20141023194923.GA25413@thinpad.lan.raisama.net> <20141024012716.GB3135@pengc-linux.bj.intel.com> <5449E9BE.9050900@redhat.com> <20141024111238.GG19316@thinpad.lan.raisama.net> In-Reply-To: <20141024111238.GG19316@thinpad.lan.raisama.net> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] target-i386: add Intel AVX-512 support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eduardo Habkost Cc: kvm@vger.kernel.org, "Michael S. Tsirkin" , Marcelo Tosatti , qemu-devel@nongnu.org, Vadim Rozenfeld , Chao Peng , Laszlo Ersek , =?windows-1252?Q?Andreas_F=E4rber?= On 10/24/2014 01:12 PM, Eduardo Habkost wrote: >> > I think we can keep the macros. The actual cleanup would be to have a >> > single member for the 32 512-bit ZMM registers, instead of splitting >> > xmm/ymmh/zmmh/zmm_hi16. This will get rid of the YMM_* and ZMM_* >> > registers. However, we could not use simple memcpy()s to marshal in and >> > out of the XSAVE data. > Agreed. I don't mind keeping those macros in this patch, as this is just > following the existing conventions in the code. Whatever we do to clean > that up, we can do it later. > >> > We can do it in 2.2. > You mean 2.3, or do you want to clean that up in this release? I mean 2.3. Paolo