From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1C0FCC433EF for ; Mon, 28 Feb 2022 11:10:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233902AbiB1LLS (ORCPT ); Mon, 28 Feb 2022 06:11:18 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57456 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235596AbiB1LLA (ORCPT ); Mon, 28 Feb 2022 06:11:00 -0500 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 9B69A7085C for ; Mon, 28 Feb 2022 03:09:22 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1646046561; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=mk8A/cRIjUQBRWWsNYUbvFqswRRH6zL512hJ5ISpuac=; b=g7fHDQjLO4kMnSZTGIJyZPFuVdlFcVDLnGrD/n26W+m+T5iF7omRkpjau+I95UovaZYAW6 yX9BOewRutl6EwKn27RF+gq/mP16oKcJUZri7pneB25L0jTQTP5t+ayQKV4YNnLbNty+Rd OCWpEHXQdjjXqT+L0yUjtqV51KEci1s= Received: from mail-qt1-f199.google.com (mail-qt1-f199.google.com [209.85.160.199]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-225-luONIc8SPrGh_djl1X6MIg-1; Mon, 28 Feb 2022 06:09:20 -0500 X-MC-Unique: luONIc8SPrGh_djl1X6MIg-1 Received: by mail-qt1-f199.google.com with SMTP id x26-20020ac84a1a000000b002ddcbd9e933so6066583qtq.11 for ; Mon, 28 Feb 2022 03:09:20 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:in-reply-to:references:date :message-id:mime-version; bh=mk8A/cRIjUQBRWWsNYUbvFqswRRH6zL512hJ5ISpuac=; b=1fButGjckQjGlo1P6pRt70E2s6Pap0xXOLPPIliLxMsSC241Z/OcCRbxVqpLSl6BaD zBF96sIfAKr7Q/aUDbVWx7ET4WAklSEt8u127NRmUzNfoxg2j/JBO0u5STZp5vB2obSj HpI4ARBOaq/v4pid9XyRgC1I5qrVPXY/idL5CnJtXoUZWg9z1oGa+To1ONsuVvwfn/xN 978LqTFEjZRIofF0kt8XzIF8VxLx0/UR6CAbVIYg9qnx7d1Ke/oLc4/5gKYArqgBO8Mb UfshXcszw8/8lcAYUbXOaxULT5gmVM2rKJ9k+kno4XWb6uVwWRQzLGjtCV5wIQODBZ7/ XeFQ== X-Gm-Message-State: AOAM532dix2Dj8c9PLzPG+GP1AcU6+zXLSsJf01Y17N7/SFrH/d0BXOn B4kcnjABQHSef4Op/EmMd48uNGeqstqPa+6nbdk9Y6nKoGejR8SmC/pvEiGDJ9qS1mgANGqjTiE V000IVxY5zhOuL8Vg/eRXR55Y X-Received: by 2002:ac8:590c:0:b0:2de:4c71:354e with SMTP id 12-20020ac8590c000000b002de4c71354emr16194170qty.315.1646046559789; Mon, 28 Feb 2022 03:09:19 -0800 (PST) X-Google-Smtp-Source: ABdhPJyF44gxifV5SBNXIq21oM3Wnww4M1ERoBoKJLH46fR/MI/Pe/poA+ls11pAGSqMnwJ88bUw7A== X-Received: by 2002:ac8:590c:0:b0:2de:4c71:354e with SMTP id 12-20020ac8590c000000b002de4c71354emr16194153qty.315.1646046559535; Mon, 28 Feb 2022 03:09:19 -0800 (PST) Received: from fedora (ec2-3-80-233-239.compute-1.amazonaws.com. [3.80.233.239]) by smtp.gmail.com with ESMTPSA id t22-20020a05622a181600b002dd4f62308dsm6667218qtc.57.2022.02.28.03.09.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 28 Feb 2022 03:09:18 -0800 (PST) From: Vitaly Kuznetsov To: Siddharth Chandrasekaran Cc: kvm@vger.kernel.org, Sean Christopherson , Wanpeng Li , Jim Mattson , linux-kernel@vger.kernel.org, Paolo Bonzini Subject: Re: [PATCH 0/4] KVM: x86: hyper-v: XMM fast hypercalls fixes In-Reply-To: References: <20220222154642.684285-1-vkuznets@redhat.com> <871qzrdr6x.fsf@redhat.com> Date: Mon, 28 Feb 2022 12:09:14 +0100 Message-ID: <87sfs3cko5.fsf@redhat.com> MIME-Version: 1.0 Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Siddharth Chandrasekaran writes: > On Fri, Feb 25, 2022 at 02:17:04PM +0100, Paolo Bonzini wrote: >> On 2/25/22 14:13, Vitaly Kuznetsov wrote: >> > Let's say we have 1 half of XMM0 consumed. Now: >> > >> > i = 0; >> > j = 1; >> > if (1) >> > sparse_banks[0] = sse128_lo(hc->xmm[0]); >> > >> > This doesn't look right as we need to get the upper half of XMM0. >> > >> > I guess it should be reversed, >> > >> > if (j % 2) >> > sparse_banks[i] = sse128_hi(hc->xmm[j / 2]); >> > else >> > sparse_banks[i] = sse128_lo(hc->xmm[j / 2]); > > Maybe I am missing parts of this series.. I dont see this change in any > of the 4 patches Vitaly sent. Yes, they look swapped to me too. > There was a conflict with a patch series from Sean: https://lore.kernel.org/kvm/20211207220926.718794-1-seanjc@google.com/ and this is a part of the resolution: commit c0f1eaeb9e628bf86bf50f11cb4a2b671528391e Merge: 4dfc4ec2b7f5 47d3e5cdfe60 Author: Paolo Bonzini Date: Fri Feb 25 06:28:10 2022 -0500 Merge branch 'kvm-hv-xmm-hypercall-fixes' into HEAD -- Vitaly