[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 04/11] x86emul: handle AVX512-FP16 move insns


  • To: Jan Beulich <jbeulich@xxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>
  • Date: Wed, 10 Aug 2022 18:04:45 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=xuPvxo01oVHQmAvqMocxKGHGXCnCvTgc2LurSfHXnuk=; b=aI87TLMR5AjdwDgFsjsSUe7XOIfm2mxZ8AUj0Vi2MKZGVKPIhyfObrw7vTnIsOs8cj+jynLOY6H0Y0Qi6AZid/pnoUZsNWqg01ijLLVHk32GPbTInVpFQdaK05sMs2pgwp8mCcZTVreOUncsEoHgEiAno5h70NutABKdEzdIncVGh4xC0w4Jnbjsqpkd1TWB3aJuzX8anlqqmcgGoYfSdJuBFo7KAnHbENRaH+PcS2rnZs0iPDjze3geusMZOQnJeojkeM+UQ7BKHtU8RZ5Fu0SoE4ZIdbUm0TFkac0XnR/Z6HHL1dFsFWaStfbGlHaNHHiqD+0aHpE3MjdDQpPOvw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=LS4LyOaXL6fLvq7b2J5bku4xzbHuP5Wp8mONtcuQdeOi2NnMjy1aH2lD80Av/h1uayVa8LmYSHQ+TgoTgWmCe+7otWZXCjSLyZOhqR692ApgT52ulMlTpIZ87GhLAkyXqYwVe/pCS2YNJEpum/Kl2pAleKaeO3MbprTKFikQFUAAp1VEctyPbkPu5NW2lBA0EsvQX6c5NIA7ARthOEb5aK5DWvo0LlZbGdngo3A+uUQpp4gdmxfNPyuhlylJDoXXkgOuHlEmjPIdmAxi+jCXoY86HSm4K7etyspI15hcegmhr0wb1wDLbXETMNo8xdDj7htWhtAUFdj4c2fNR5BrqA==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com;
  • Cc: Wei Liu <wl@xxxxxxx>, Roger Pau Monne <roger.pau@xxxxxxxxxx>
  • Delivery-date: Wed, 10 Aug 2022 18:05:03 +0000
  • Ironport-data: A9a23:eIE8pKK5AdRiXsi6FE+Rz5QlxSXFcZb7ZxGr2PjKsXjdYENS0jJTz GZJDzzXOauNMDT8c49yb4SzoB5Uv5HdzNZnSAdlqX01Q3x08seUXt7xwmUcns+xwm8vaGo9s q3yv/GZdJhcokf0/0vraP65xZVF/fngbqLmD+LZMTxGSwZhSSMw4TpugOd8iYNz6TSDK1rlV eja/ouOYzdJ5xYuajhOs/La8Es01BjPkGhwUmIWNKgjUGD2zxH5PLpHTYmtIn3xRJVjH+LSb 44vG5ngows1Vz90Yj+Uuu6Tnn8iG9Y+DiDX4pZiYICwgwAqm8AH+v1T2Mzwy6tgo27hc9hZk L2hvHErIOsjFvWkdO81C3G0H8ziVEHvFXCuzXWX6KSuI0P6n3TE4aRJHGc6etIi+sEpXzp+6 tsSIyEvYUXW7w626OrTpuhEoO0GdJWuEKZB/3ZqwHfeEOosRo3FT+PS/9hE0Twsh8dIW/HDe 84ebjkpZxPFC/FNEg5PVNRiw6H33T+jLlW0q3rMzUYzy0HVwBZ8z/7GN93Nd8bRbc5UglyZt iTN+GGR7hQya4zFlWTZqSrEaunnvj2iQqY5LqyC6eNJoUGhy1MSJUUrSg7uyRW+ogvkMz5FE GQW8Cczqak59GSwU8LwGRa/pRasrhMaHtZdDeA+wAWM0bbPpRaUAHAeSTxMY8Bgs9U5LRQ10 neZktWvAiZg2IB5UlqY/7aQ6D+3Zy4cKDdeYTdeFVdYpd7+vIs0kxTDCM55F7K4hcH0Hje2x C2WqC85hPMYistjO7iHwG0rSgmE/vDhJjPZLC2ONo55xmuVvLKYWrE=
  • Ironport-hdrordr: A9a23:Qzk4R6AzceL4Z5flHeiEsseALOsnbusQ8zAXPh9KJCC9I/bzqy nxpp8mPEfP+U0ssHFJo6HiBEEZKUmsuKKdkrNhR4tKOzOW9FdATbsSp7cKpgeNJ8SQzJ876U 4NSclD4ZjLfCBHZKXBkUaF+rQbsb+6GcmT7I+woUuFDzsaEp2IhD0JaDpzZ3cGIDWucqBJca Z0iPAmmxOQPVAsKuirDHgMWObO4/XNiZLdeBYDQzI39QWUijusybjiVzyVxA0XXT9jyaortT GtqX252oyT99WAjjPM3W7a6Jpb3PPn19t4HcSJzuQFNzn2jQ6sRYJ5H5mPpio8ru2D4Esj1P PMvxAjFcJu7G65RBD6nTLdny3blBo+4X7rzlGVxVH5p9bieT48A81dwapEbxrw8SMbzZJB+Z MO+1jcm4tcDBvGkii4zcPPTQtWmk29pmdnufIPjkZYTZAVZNZq3M4iFQJuYdI99RDBmcca+d pVfYfhDTFtAAqnhkXizy1SKRqXLywO91m9MxM/U4euokVrdThCvjclLYok7zc9HdsGOud5D6 6vCNUWqJheCsARdq5zH+EHXI++DXHMWwvFNCaILU3gD7xvAQOFl3fb2sRD2AiRQu1/8LIi3J DaFF9Iv287fEzjTcWIwZ1Q6xjIBGGwRy7kxM1S74Vw/uSUfsuhDQSTDFQ118ewqfQWBcPWH/ 61JZJNGvfmaW/jA5xA0QHyU4RbbXMeTMoWsNAmXE/mmLOCFqT68ujANPrDLrvkFjgpHmv5H3 sYRTD2YN5N60i6M0WI9CQ5m0mdD3AX0agAYJQypdJjubTlHrc8wTQ9mBC++tyBLyFEv+g/YF Z+SYmX4J+GmQ==
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHYgKKvyIyZD1WQk0Gxz6uWvI7+iq2oxhKA
  • Thread-topic: [PATCH 04/11] x86emul: handle AVX512-FP16 move insns

On 15/06/2022 11:28, Jan Beulich wrote:
> --- a/xen/arch/x86/x86_emulate/decode.c
> +++ b/xen/arch/x86/x86_emulate/decode.c
> @@ -585,7 +585,7 @@ static unsigned int decode_disp8scale(en
>          break;
>  
>      case d8s_dq64:
> -        return 2 + (s->op_bytes == 8);
> +        return 1 + !s->fp16 + (s->op_bytes == 8);
>      }

This hunk looks like it wants to be in a previous patch?

~Andrew

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.