From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f202.google.com (mail-pf1-f202.google.com [209.85.210.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 2D7D11A285 for ; Tue, 26 May 2026 16:55:34 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.202 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779814535; cv=none; b=lalJeR9Ma2EhYwgbDzFRoAqY/5ZhkANTs5RjOiYiqmy7d9m3R2KN59oVmKn2SLhzeVE7OtBab+ApEls3osoBbsowWJA4IwBwR9Vl9XmEjU2yEsRynVpOqJUfV7LhYR9N/nqfoslOAKmOT1IGjJAPLav4a2Fp7PZjwiBRbI6HMHg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779814535; c=relaxed/simple; bh=lS88pgO4vl0WQoEIrTfiXdwjUHeGJQ0+p5JkXjqIuL0=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=PnvAFp9xHcON2Ik2KG8JDgXCZOmPWaUlcFiSMP9pqZqhsd+DiiIAcW/VwROnCGhWHac95Jj6J1OzZKIv7QkSq54J5rGdpyAnFNj38WW39Nks6qLn2NnE00tKJm4VeIZHnlsBCB9yV9/AvlgCU4f3I0+5pPYEx83qZMmLUOCRQU4= 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=j37vd0hP; arc=none smtp.client-ip=209.85.210.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="j37vd0hP" Received: by mail-pf1-f202.google.com with SMTP id d2e1a72fcca58-835423c69ffso5603668b3a.1 for ; Tue, 26 May 2026 09:55:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20251104; t=1779814533; x=1780419333; 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=LZT31wOgcdRUmGTT5DQeYgf6s1URS8IbeXG7+gb+Vm8=; b=j37vd0hPHYczoNM5Gcznb5RSBUDWvyG3nNAnMaq8ZKy/cPbiFfgG60jYs5dJJgFwPC XTnv6SXr6o7QSvLnGGwxB+A58zuKGIWVRBThB1tpGqAeKceULrRcVhaeiea+UoCOHksV QdG6x8G6UVmzAnT9+vSeLcdCX8YRfmRDI8wKrp8OSRnZL5bK0KWQjyv9pzo7bYGEf+6i NzK4qIUc9qcEzNODJs1oTw7+HkJdiAF9LeOSIPygMkTqfs6Hq7MpZi7KEPP5HQYsHwWt o8ANiYhz4Aibu5fCXJkplfW/NfMED4ei2WKBDiGUTTDURsbEgnBIOWp889kFItbhlpaO MAdA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779814533; x=1780419333; 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=LZT31wOgcdRUmGTT5DQeYgf6s1URS8IbeXG7+gb+Vm8=; b=qtUEfcRv9XqrvRiN+zJVc9PldbqV6n31XLtEJN6yxnbtdb3l0/qoWbWAnwZ5MfU4W/ xwoLTXHWs3Chi5s9hOtJBf3U6EdvIO+1DFYZMu1o2U+2QaKSLNfgQgTRctE1U/WtVKM+ klHBXGAsti4bST2TvKWZzTNlRk6FWtLBt3Z560PfwsxiAHN+1U6/DFEMP1THFTwCtMMo jjy2SJmGG6J8DdLeZcPI8drqycF9fOpHLjrsDEMbiJ81Cq5zaOQ+mncHrpSvjtBCFhoy XJg7yPUeoY3D/vOMbe+smEj6lYfnylJMW5/y+aTnvcYS9/qgzk5/ac7v7A8klTYTA9V4 SdUQ== X-Forwarded-Encrypted: i=1; AFNElJ891pCmboGfU7U4ooAA7oHCTObZbj9DDXfDCELya+KaTijMY1G0N6HAbcsYef9PjaeAxnfMrxht8Z37ZwY=@vger.kernel.org X-Gm-Message-State: AOJu0YybIgw+pOUM0l5/+2feik5yoKRXuXjj4eEmu0oWSDRKUXspYag8 YYSSsLjT5SXUCLUTNT4ZY8LIQkTTsw4rJcvdAvJnjOFuhlNqwM17uN/4hBENxqurKSEWTnIQvSf A67t0OA== X-Received: from pfbfx7.prod.google.com ([2002:a05:6a00:8207:b0:82f:6245:a6ec]) (user=seanjc job=prod-delivery.src-stubby-dispatcher) by 2002:a05:6a00:298f:b0:835:3d5e:1dd3 with SMTP id d2e1a72fcca58-8415f586b5bmr17521220b3a.42.1779814532656; Tue, 26 May 2026 09:55:32 -0700 (PDT) Date: Tue, 26 May 2026 09:55:32 -0700 In-Reply-To: <20260522-fix-sev-gmem-post-populate-v2-0-3f196bfad5a1@google.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20260522-fix-sev-gmem-post-populate-v2-0-3f196bfad5a1@google.com> Message-ID: Subject: Re: [PATCH v2 0/5] guest_memfd fixes for bind and populate From: Sean Christopherson To: Ackerley Tng Cc: Paolo Bonzini , Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , x86@kernel.org, "H. Peter Anvin" , Kiryl Shutsemau , Rick Edgecombe , Vishal Annapurve , Yan Zhao , Michael Roth , Isaku Yamahata , Chao Peng , Xiaoyao Li , Zongyao Chen , kvm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-coco@lists.linux.dev, Yu Zhang , Fuad Tabba Content-Type: text/plain; charset="us-ascii" On Fri, May 22, 2026, Ackerley Tng wrote: > This series is a group of fixes for the bind and populate flows for > guest_memfd, and fixes some issues reported by Sashiko after reviewing the > guest_memfd in-place conversions series [1] and another fixup series Sean > posted [3]. In the future, please don't bundle unrelated changes. The SNP specific changes are related and should be a series, but the signed integer thing and the lack of error handling on xa_store_range() are completely unrelated, because the fact that Sashiko kept complaining about pre-existing issues. I totally understand why you bundled these together, but that obviously didn't stop Sashiko from complaining about pre-existing issues, over and over. Unnecessarily bundling can lead to exactly what's happening here: the three SNP changes are ready to go, but the two unrelated guest_memfd changes need new versions. Which isn't hard to deal with, but it's extra friction that is easily avoided. I'll apply the SNP changes, and send a new version of the signed vs. unsigned issue. Please send a new version of the xa_store_range() error handling (or prove that I'm wrong). Thanks!