From nobody Thu Mar 28 21:04:43 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org ARC-Seal: i=1; a=rsa-sha256; t=1616670811; cv=none; d=zohomail.com; s=zohoarc; b=GMPB6UBXeaO51kttGoSx023HXT0a+HmGSuMF/uwQy/QingP1WgtUyhAK9d0NL/k4QRF9/X5fY91JnYp6k8JS6oOSSS0gJzKjKJSceobHQ6wWk7Hny8La2EYhAHyQeX9Mim6LvIXupqpVbfm7yLW1f01Km22Ov7HkR5maUwkEJr0= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1616670811; h=Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:Message-ID:Sender:Subject:To; bh=aOQI9Mr0Yo2TD1tNatsazTuCAXLpcS3ebRcXECoMhhw=; b=F77hDceetbO81MvhZlx9IWvow0X00c76NmAu0EI8eQEg7aiDVFwSnTzQspdewlmiRIC6qNr5gLPNuVhMrk15/fkwanZkAFJC9FC0lytVCf8IiqjMVzAir+AXBrBL8KGu8tJQzj2mxVRFDD+aAtn6Z3obC4kXLvIIQKgOH7zGUjM= ARC-Authentication-Results: i=1; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 161667081113715.866897580597765; Thu, 25 Mar 2021 04:13:31 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.101376.193857 (Exim 4.92) (envelope-from ) id 1lPNvO-0003vg-KP; Thu, 25 Mar 2021 11:13:14 +0000 Received: by outflank-mailman (output) from mailman id 101376.193857; Thu, 25 Mar 2021 11:13:14 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1lPNvO-0003vZ-HF; Thu, 25 Mar 2021 11:13:14 +0000 Received: by outflank-mailman (input) for mailman id 101376; Thu, 25 Mar 2021 11:13:12 +0000 Received: from mail.xenproject.org ([104.130.215.37]) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1lPNvM-0003vL-OL for xen-devel@lists.xenproject.org; Thu, 25 Mar 2021 11:13:12 +0000 Received: from xenbits.xenproject.org ([104.239.192.120]) by mail.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1lPNvL-0002Mn-CZ; Thu, 25 Mar 2021 11:13:11 +0000 Received: from 54-240-197-235.amazon.com ([54.240.197.235] helo=ufe34d9ed68d054.ant.amazon.com) by xenbits.xenproject.org with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1lPNvL-0002fE-0C; Thu, 25 Mar 2021 11:13:11 +0000 X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=xen.org; s=20200302mail; h=Message-Id:Date:Subject:Cc:To:From; bh=aOQI9Mr0Yo2TD1tNatsazTuCAXLpcS3ebRcXECoMhhw=; b=QG5KoAnmaAdnEe3OVNDsQPEB4g rEKBuSu/My5VuL6dh8yDf5xTv0L4Ghp0dyk4nvUHuKPHWWIOY5NW3Ud0L5m9hCp1/e4tVVtYaGG5b 3O0X/YuM25ucfX993UFIV9PvD3MUmohDmRuxwdlZkZRqC1QS9wd9z0Y9TKpm9CthZo8I=; From: Julien Grall To: xen-devel@lists.xenproject.org Cc: julien@xen.org, Julien Grall , Andrew Cooper , George Dunlap , Ian Jackson , Jan Beulich , Stefano Stabellini , Wei Liu , raphning@gmail.com, edvin.torok@citrix.com, jgross@suse.com Subject: [PATCH for-4.15?] docs/design: Update xenstore-migration.md Date: Thu, 25 Mar 2021 11:12:55 +0000 Message-Id: <20210325111255.16330-1-julien@xen.org> X-Mailer: git-send-email 2.17.1 X-ZohoMail-DKIM: pass (identity @xen.org) Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" From: Julien Grall It is not very clear the shared page adddress is not contained in the connection record. Additionally, it is misleading to say the grant will always point to the share paged as a domain is free to revoke the permission. The restore code would need to make sure it doesn't fail/crash if this is happening. The sentence is now replaced with a paragraph explaining why the GFN is not preserved and that the grant is not guarantee to exist during restore. Take the opportunity to replace "code" with "node" when description the permission. Reported-by: Raphael Ning Signed-off-by: Julien Grall Reviewed-by: Juergen Gross --- Cc: raphning@gmail.com Cc: edvin.torok@citrix.com Cc: jgross@suse.com Ian, would you still consider documentation update for 4.15? --- docs/designs/xenstore-migration.md | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/docs/designs/xenstore-migration.md b/docs/designs/xenstore-mig= ration.md index 1a5b94b31da8..5f1155273ec3 100644 --- a/docs/designs/xenstore-migration.md +++ b/docs/designs/xenstore-migration.md @@ -216,8 +216,10 @@ For `shared ring` connections it is as follows: | | by xenstored to communicate with `domid` | | | | =20 -Since the ABI guarantees that entry 1 in `domid`'s grant table will always -contain the GFN of the shared page. +The GFN of the shared page is not preserved because the ABI reserves +entry 1 in `domid`'s grant table to point to the xenstore shared page. +Note there is no guarantee the page will still be valid at the time of +the restore because a domain can revoke the permission. =20 For `socket` connections it is as follows: =20 @@ -398,7 +400,7 @@ A node permission specifier has the following format: | | | | `domid` | The domain-id to which the permission relates | =20 -Note that perm1 defines the domain owning the code. See [4] for more +Note that perm1 defines the domain owning the node. See [4] for more explanation of node permissions. =20 * * * --=20 2.17.1