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

Re: [4.22] Re: [PATCH] x86/domctl: restore all registers in arch_{get,set}_info_guest()


  • To: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
  • From: Frediano Ziglio <freddy77@xxxxxxxxx>
  • Date: Mon, 20 Jul 2026 10:59:00 +0100
  • Arc-authentication-results: i=1; mx.google.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20260327; h=content-transfer-encoding:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature; bh=ZIJPDOGMGXtHFAAoNyKXLbsC7KraedC+8OmoR5/4Gvc=; fh=wk0y1wP5tyDhLPHhZ130vdu8uO7RzrblIn679649v3M=; b=M9/BMeaFoRQ07y02dT6vZfy6w9HA0nizYmykXp0/kVWt1A4WKTby+PO+xGs7Q/y5cq 3C7HEz6GCyPZly6r4iPZ8d+W1TjmANGxxRpE6UHINry3zUuOK21WONXXEGSTCu/wim99 H3+hEO6vy5FA44nHNtTKiLqNrm5phVca2uEyep1nQAOhVlsQPT+JnycWuF2RiPcBpLx1 QqT9VA5NQ99pGwoRuC2ExNBGBDFPjBlv0KvXWc9vCQKIduCJBkxvTU23AMuLdXGrnkSS fcqMnojtDt3RLaRd32d9dbRULu0CtT3TPIOPXkvZ6JXMfbER59c95qVhzzf3oJwBTSmp 4SJw==; darn=lists.xenproject.org
  • Arc-seal: i=1; a=rsa-sha256; t=1784541552; cv=none; d=google.com; s=arc-20260327; b=A+6Wy4nkI9j4KoUf/0drF9wuFvqlT5BfDlo+vomG/yMwGtYRx9zx/gy9icDxN9Ka4u 50M8yiELrwP1V9PBkPRF12Xaimwb8RXHfxmQcIj3hhkGxJ2rv2MVKAqiMWEIUOFeCMb5 svSLzk66/9CPfC01InrMi7FOK4oQpybh2BJVANcmXAwp8fcEnPDtYaLAkKYoIGXSrpMo 7MneMQt8Usgg/Aeq1Ppcq/tVT7xwTr8mdGZ5tsZgjgeMZW3ukPfJPozF5z3pQKtsRHO8 8YOKB78G7f4n2/tHul50Igg/kqtReL1dNzYK8TkgGzdw9BBnsRGXTCILjCrxVKCmYJub QFVw==
  • Authentication-results: eu.smtp.expurgate.cloud; dkim=pass header.s=20251104 header.d=gmail.com header.i="@gmail.com" header.h="Content-Transfer-Encoding:Content-Type:Cc:To:Subject:Message-ID:Date:From:In-Reply-To:References:MIME-Version"
  • Cc: Jan Beulich <jbeulich@xxxxxxxx>, Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>, Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Teddy Astie <teddy.astie@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
  • Delivery-date: Mon, 20 Jul 2026 09:59:22 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Mon, 20 Jul 2026 at 10:09, Andrew Cooper <andrew.cooper3@xxxxxxxxxx> wrote:
>
> On 20/07/2026 8:19 am, Jan Beulich wrote:
> > On 20.07.2026 02:12, Marek Marczykowski-Górecki wrote:
> >> Commit 9f892f84c279 ("x86/domctl: Stop using XLAT_cpu_user_regs()")
> >> converted memcpy() of the cpu_user_regs structure to explicit copy of
> >> its fields. In the compat case, it intentionally missed few of them,
> >> named in the commit message. But the 64bit case missed also r8-r15
> >> registers, which was not intentional. This, at least, caused Linux
> >> 6.18.x crash when resuming PVH domU.
> > Oh, wow, what a bad mistake (including by me as the reviewer).
>
> Yes, I'm very embarrassed by this mistake.  It also highlights a serious
> gap in testing which we need to address.  We've managed a release and a
> half with PV migration plain broken before figuring out why.
>
...

Is migration tested by CI or just for some reason this was not captured?

Frediano



 


Rackspace

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