From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-yw1-f202.google.com (mail-yw1-f202.google.com [209.85.128.202]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3768D763FF for ; Tue, 20 Feb 2024 16:15:09 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.202 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708445717; cv=none; b=OHoscpW/hnzh5fJAt1Sz1N0LoStZ1UqPAJLCqLYK7ajvVb8ZyDgD+3s1z/kzF1aoSfwJ+H/b1HwSx6gKpNUZ6EiGM6N0MHrZIKgPS/97D+zufzD+4suv5ouUXMKl8oi6nq4OMUS+DOAUE8sZlQAQgdz7vVuJwEweesfuB+TkAPI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708445717; c=relaxed/simple; bh=Ey6ErHBVbqQQFaARRSSxWyFZtNcBtj/qBDxHXsIykls=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=hT8l6qekJM6KOp5RVkQr+rVHrJJRl/86j3CjfMx1K4vIC12I3vFR1v3VXuZ7iDVqapA/ooRj9x/y+NE2ja6jSfySo+h4CZqOZI6nZ9U9oOVZseKQlZf7VXyt1e9O1tFjvKQXIf2dTPMx4luaIAfIU7p5vvCHI3Zgl9ArNNopRUQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--seanjc.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=dTxSX285; arc=none smtp.client-ip=209.85.128.202 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--seanjc.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="dTxSX285" Received: by mail-yw1-f202.google.com with SMTP id 00721157ae682-5ee22efe5eeso74171597b3.3 for ; Tue, 20 Feb 2024 08:15:09 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1708445708; x=1709050508; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=gGVaEpoRy83RsqKWj1BDNY8ZLAKIFU3H8chjcBdzZQ4=; b=dTxSX285wINLcioxkombr9MEQCvXPobFahkDQ2lvGX6ul7kYwAp1vItnlUC1ZVavMz BnG+UjuDNyIFZcTkCBpgeCa0S8Hipa7Q4//YuGIybW2fTdvlG4r6KM6uhcdqha+tTif6 wNJhmMjpQ8Spt7jw1F+GRSAvETTnQHP7aFJmsxOZ07nCTm8Yh8ZgmqUiownXRHzPtiCL 5p4X5vH0Y6va2UZEmkuXGAmiou5GMjN18X5MWNW1wjJLMKzvqqzfJqBPdV3kESleQ0vK C+Xj2iOzvjcmrqTMDb/TVaTMdk5nLYvQLMcnLl4AoesGBfvCy8J21iqCCXEO7SpipZ+8 1KiQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1708445708; x=1709050508; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=gGVaEpoRy83RsqKWj1BDNY8ZLAKIFU3H8chjcBdzZQ4=; b=Nf1Ng7ym1PllTVdRMhcFiImUxucnA9x3O87uqMIRmvlQ86CZXpsXrRAB8nsFEMQmQ/ WliYG2C9XJTcDIs+sWc1RGrLLznPPDeTRRfb02dpGr6dCMRRe+MSQ0mbrEeYEiy1N67q +MWa6HZa/MbXwcOR5Qfbmd1gqP4YPhMIA8hp0Df5SFO941EaX4fK4NWnJuz4oaX390wd yLgBuvKWshI1rKgy4kssqVwdVUeWuWp4pCjDKCZiZ58aUpOjZQq2Tp6NCYCQqhjDeiVr bAeHOpQuGfV9751BOEiGXRJqnh9dJ64kGQ4QEhNfHRbGG3P9KV1SWzBefgHwC42LZ253 R/MA== X-Forwarded-Encrypted: i=1; AJvYcCWVbfPwZZyW+50bo6i2S5PN3bZUK1arlpw4RPS2Fri8GIU+Ta5U2z3mYA0mWnWHAutpzzyLXlcMB17jMDsAGQn0Et/pVOc157DawaLY0V48 X-Gm-Message-State: AOJu0YwSQ3U2QdvAK/LtYNKxpZOxawWxz0pD0qMCfw/0+Y4u3DcxzYgb LMaE/q5yog1tDNmCAH163uJHZOy6F5QBqwvvNF7CV/PIPWvh2LGiHy6rUHI1hxcJXZ+1+kocIWF jOA== X-Google-Smtp-Source: AGHT+IHOl1ZsAPNJTwpqdmUVaVuTbX0oezG+zWKz4lEs+PmYyLIBMZWVYtpKpRe+ZKQU8wz9tv5wxLQvY2M= X-Received: from zagreus.c.googlers.com ([fda3:e722:ac3:cc00:7f:e700:c0a8:5c37]) (user=seanjc job=sendgmr) by 2002:a05:6902:1547:b0:dc7:3189:4e75 with SMTP id r7-20020a056902154700b00dc731894e75mr587165ybu.3.1708445707962; Tue, 20 Feb 2024 08:15:07 -0800 (PST) Date: Tue, 20 Feb 2024 08:15:06 -0800 In-Reply-To: <05973da0-f68c-4c84-8806-bdba92f2ed6e@xen.org> Precedence: bulk X-Mailing-List: linux-kselftest@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20240215152916.1158-1-paul@xen.org> <170838297541.2281798.7838961694439257911.b4-ty@google.com> <05973da0-f68c-4c84-8806-bdba92f2ed6e@xen.org> Message-ID: Subject: Re: [PATCH v13 00/21] KVM: xen: update shared_info and vcpu_info handling From: Sean Christopherson To: paul@xen.org Cc: Paolo Bonzini , Jonathan Corbet , Christian Borntraeger , Janosch Frank , Claudio Imbrenda , David Hildenbrand , Heiko Carstens , Vasily Gorbik , Alexander Gordeev , Sven Schnelle , Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , x86@kernel.org, "H. Peter Anvin" , David Woodhouse , Shuah Khan , kvm@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-s390@vger.kernel.org, linux-kselftest@vger.kernel.org Content-Type: text/plain; charset="us-ascii" On Tue, Feb 20, 2024, Paul Durrant wrote: > On 20/02/2024 15:55, Sean Christopherson wrote: > > On Thu, 15 Feb 2024 15:28:55 +0000, Paul Durrant wrote: > > > From: Paul Durrant > > > > > > This series contains a new patch from Sean added since v12 [1]: > > > > > > * KVM: s390: Refactor kvm_is_error_gpa() into kvm_is_gpa_in_memslot() > > > > > > This frees up the function name kvm_is_error_gpa() such that it can then be > > > re-defined in: > > > > > > [...] > > > > *sigh* > > > > I forgot to hit "send" on this yesterday. But lucky for me, that worked out in > > my favor as I needed to rebase on top of kvm/kvm-uapi to avoid pointless conflicts > > in the uapi headeres. > > > > So.... > > > > Applied to kvm-x86 xen, minus 18 and 19 (trylock stuff) and 21 (locking cleanup > > that we're doing elsewhere). > > > > Looks like you meant 17 & 18? Doh, yes. > > Paul and David, please take (another) look at the end result to make sure you don't > > object to any of my tweaks and that I didn't botch anything. > > > > What was the issue with 17? It was reasonable clean-up and I'd like to keep > it even without 18 being applied (and I totally understand your reasons for > that). I omitted it purely to avoid creating an unnecessary dependency for the trylock patch. That way the trylock patch (or whatever it morphs into) can be applied on any branch (along with the cleanup), i.e. doesn't need to be taken through kvm-x86/xen.