From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pj1-f73.google.com (mail-pj1-f73.google.com [209.85.216.73]) (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 B454626F2A7 for ; Wed, 18 Feb 2026 23:12:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.216.73 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771456361; cv=none; b=urHgwWasRWOPNZEbOU2Nve0cvowK9PgjqgJGDfkKhehnbHfTZpfKLGvjtwHVzfZSu2R50HZ/KOmCs8/5/QspnOOU/mzledQKzWrm5nPMJ/cGKOMy6zttJn552EYNd9pgJa+R4g/Wy2svfwtiEpXP9dGM+wWVcm3quoRiFLQquEQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771456361; c=relaxed/simple; bh=J0OEKES3G7uLLP0gC9Q6SUYN7A1Uk5HZtzFIVnAXE8o=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=dLEYQNcxnRzWHlQXM/A2zsV2z/TwT11Kakfu9euIQRjeRzTVhuCWQF8rhNactbBejXkkQccWTJFUfH4mLaBPl5pzIO1774Ww/YosyIJ0U+mu1yyNzopo2bhZHRLwAfCisHYU1fFJQA2dYcuu94UwwFkLW2cD9tKXmTRL7+dCZtk= 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=zq463mtm; arc=none smtp.client-ip=209.85.216.73 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="zq463mtm" Received: by mail-pj1-f73.google.com with SMTP id 98e67ed59e1d1-3569ceeb335so269152a91.2 for ; Wed, 18 Feb 2026 15:12:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1771456360; x=1772061160; 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=wDTU/+n7WxF2jTJpTD2DZkMo6ds7ZDdb6/ZoYXHP1ZI=; b=zq463mtmVNFvEj8x8e6x49W+PN7xoLXvyxQ5Tsds/o6oR5F72UIpO3mJlnkeKjovcA Q3d4/9RzWMOMsvNn8IThTeLdlG8AkNgZ7BPKYAgMJoTmNE6LUica762LWc/kRtvRtp8A +eaEZ02uFUKkwThGf7p2MRokWuK3/SAI7Io/sY1Tuk2IP+5d9qV2eDcGgH3Z5LLaVn8K OHVn9ndHKxv7wHzy1zLF8o2J1HqRUtTKTSCXOGbiovhxR6GUup3E3x8mJ52gNYy4lTO8 kzQIkpqAHeH+fDAlooZokVDxvpAEUM8kCcOeMimGuigUAukN8R8XvZIchEluUFPsstrv oSZA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1771456360; x=1772061160; 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=wDTU/+n7WxF2jTJpTD2DZkMo6ds7ZDdb6/ZoYXHP1ZI=; b=DQQs0Lpkuw31V9p8TxqB82G+Kn/87IGCc41c2dALrHne4ByOkQvNty0IgRar5ZZET3 oUVPMV4vSEN/7yLRPzv6snrgesPQqkqkVRbodZURKHloGuEUweGBAYU37A6vqbK7jWZk APvuPEF3LwZO7qCryRDFlaimeSVuvbSIilVDsjiAlbjmpWBbp1hHhmriSqdBkDRtsf0r 5pI/uzlKJCl2XvD3xbSVUIo44x+3NySi3W+PMpi3GV9FtG0k2IpbklF+iAHrBC71CxAH qIP5Ucx7JnSV1LGLIPtvevOYEznlNfqTEsDzcXsbVK7ATubgtxT7FoR5ZNHSTL1UO4tg 8cEA== X-Forwarded-Encrypted: i=1; AJvYcCUsJgCU4KliXWmrgYzAO54RvEPoeWUuhOYEAbd9JDPdtJC+gJE1LQ4r9VqowC7pX8Q9ds+FzSfXrqjyb3Q=@vger.kernel.org X-Gm-Message-State: AOJu0YzcaNFKlFRAMLXlMgG9mbz8t0OfbPndcfgg66V1pn7TME7FfQqM iRAbchGKPFL1wQyh7yYoKl/d9TdpK6I5TQE03IL8Ngf5bu574m5BUh0mI9p+zGj/PSe95R0jiNK fs2bXyw== X-Received: from pjzb12.prod.google.com ([2002:a17:90a:e38c:b0:353:454:939c]) (user=seanjc job=prod-delivery.src-stubby-dispatcher) by 2002:a17:90b:3b92:b0:356:2fc5:30f5 with SMTP id 98e67ed59e1d1-35889055dfbmr2539808a91.13.1771456359932; Wed, 18 Feb 2026 15:12:39 -0800 (PST) Date: Wed, 18 Feb 2026 15:12:38 -0800 In-Reply-To: <20260211162842.454151-4-yosry.ahmed@linux.dev> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20260211162842.454151-1-yosry.ahmed@linux.dev> <20260211162842.454151-4-yosry.ahmed@linux.dev> Message-ID: Subject: Re: [PATCH v2 3/5] KVM: nSVM: Move sync'ing to vmcb12 cache after completing interrupts From: Sean Christopherson To: Yosry Ahmed Cc: Paolo Bonzini , kvm@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset="us-ascii" On Wed, Feb 11, 2026, Yosry Ahmed wrote: > nested_sync_control_from_vmcb02() sync's some fields from vmcb02 to the > cached vmcb12 after a VMRUN of L2, mainly to keep the cache up-to-date > for save/restore. However, NextRIP is sync'd separately after > completing interrupts, as svm_complete_soft_interrupt() may update it > (e.g. for soft IRQ re-injection). > > Move the call to nested_sync_control_from_vmcb02() after completing > interrupts, moving the NextRIP sync (and the FIXME) inside it. This > keeps the sync code together, and puts the FIXME in a more adequate > location, as it applies to most/all fields sync'd by > nested_sync_control_from_vmcb02(). > > Moving the call is safe, as nothing in-between accesses any of the VMCB > fields sync'd by nested_sync_control_from_vmcb02(), except NextRIP. > > Opportunistically make some whitespace fixes. No functional change > intended. > > Signed-off-by: Yosry Ahmed > --- As discussed off-list, I think I'll skip this patch, I'd prefer to go straight to addressing the FIXME. For me, the ugliness of the FIXME is a good thing: if we make the code awful enough, we'll hopefully be more motivated to fix it :-)