All of lore.kernel.org
 help / color / mirror / Atom feed
From: George Dunlap <George.Dunlap@citrix.com>
To: Jan Beulich <jbeulich@suse.com>
Cc: George Dunlap <dunlapg@umich.edu>,
	Andrew Cooper <Andrew.Cooper3@citrix.com>,
	Roger Pau Monne <roger.pau@citrix.com>, Wei Liu <wl@xen.org>,
	Stefano Stabellini <sstabellini@kernel.org>,
	Julien Grall <julien@xen.org>,
	Volodymyr Babchuk <Volodymyr_Babchuk@epam.com>,
	Bertrand Marquis <bertrand.marquis@arm.com>,
	Henry Wang <Henry.Wang@arm.com>,
	Anthony Perard <anthony.perard@citrix.com>,
	Xen-devel <xen-devel@lists.xenproject.org>
Subject: Re: [PATCH 1/4] xen: Introduce non-broken hypercalls for the p2m pool size
Date: Mon, 31 Oct 2022 10:12:41 +0000	[thread overview]
Message-ID: <5EC7E359-3583-4846-8AF8-B84197B83BEF@citrix.com> (raw)
In-Reply-To: <2a8a490b-fd5c-5fe5-64ec-975c9c4784d0@suse.com>


[-- Attachment #1.1: Type: text/plain, Size: 2252 bytes --]



> On 31 Oct 2022, at 09:26, Jan Beulich <jbeulich@suse.com> wrote:
> 
> On 28.10.2022 17:27, George Dunlap wrote:
>> On Thu, Oct 27, 2022 at 8:12 AM Jan Beulich <jbeulich@suse.com> wrote:
>> 
>>> On 26.10.2022 21:22, Andrew Cooper wrote:
>>>> On 26/10/2022 14:42, Jan Beulich wrote:
>>> 
>> 
>> 
>>>> paging isn't a great name. While it's what we call the infrastructure
>>>> in x86, it has nothing to do with paging things out to disk (the thing
>>>> everyone associates the name with), nor the xenpaging infrastructure
>>>> (Xen's version of what OS paging supposedly means).
>>> 
>>> Okay, "paging" can be somewhat misleading. But "p2m" also doesn't fit
>>> the use(s) on x86. Yet we'd like to use a name clearly better than the
>>> previous (and yet more wrong/misleading) "shadow". I have to admit that
>>> I can't think of any other sensible name, and among the ones discussed
>>> I still think "paging" is the one coming closest despite the
>>> generally different meaning of the word elsewhere.
>>> 
>> 
>> Inside the world of operating systems / hypervisors, "paging" has always
>> meant "things related to a pagetable"; this includes "paging out to disk".
>> In fact, the latter already has a perfectly good name -- "swap" (e.g., swap
>> file, swappiness, hypervisor swap).
>> 
>> Grep for "paging" inside of Xen. We have the paging lock, paging modes,
>> nested paging, and so on. There's absolutely no reason to start thinking
>> of "paging" as exclusively meaning "hypervisor swap".
> 
> Just to clarify: You actually support my thinking that "paging" is an okay
> term to use here? I ask because, perhaps merely because of not being a
> native speaker, to me content and wording suggest different things: The
> former appears to support my response to Andrew, while the latter reads to
> me as if you were objecting.

Sorry, the tone was “objecting” because it was directed mainly at Andrew’s arguments.  I thought about replying only to his mail, but it seemed like since I was clearly “joining the discussion”, it would make more sense to quote you too.  I could probably have made it more clear by leading with something like, “I tend to agree with Jan here. …”

 -George


[-- Attachment #1.2: Type: text/html, Size: 8407 bytes --]

[-- Attachment #2: Message signed with OpenPGP --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

  reply	other threads:[~2022-10-31 10:13 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-26 10:20 [PATCH for-4.17 0/4] XSA-409 fixes Andrew Cooper
2022-10-26 10:20 ` [PATCH 1/4] xen: Introduce non-broken hypercalls for the p2m pool size Andrew Cooper
2022-10-26 13:42   ` Jan Beulich
2022-10-26 19:22     ` Andrew Cooper
2022-10-26 21:24       ` Julien Grall
2022-10-27  6:56         ` Jan Beulich
2022-10-27  9:27           ` Julien Grall
2022-10-27  7:11       ` Jan Beulich
2022-10-28 15:27         ` George Dunlap
2022-10-31  9:26           ` Jan Beulich
2022-10-31 10:12             ` George Dunlap [this message]
2022-11-16  1:19           ` Stefano Stabellini
2022-11-16  8:26             ` Jan Beulich
2022-10-27  7:42   ` Jan Beulich
2022-10-26 10:20 ` [PATCH 2/4] tools/tests: Unit test for " Andrew Cooper
2022-10-26 14:24   ` Jan Beulich
2022-10-26 14:35     ` Andrew Cooper
2022-10-26 10:20 ` [PATCH 3/4] xen/arm, libxl: Revert XEN_DOMCTL_shadow_op; use p2m mempool hypercalls Andrew Cooper
2022-10-26 13:22   ` Jason Andryuk
2022-10-26 13:25     ` Andrew Cooper
2022-11-16  1:37   ` Stefano Stabellini
2022-11-16  1:48     ` Andrew Cooper
2022-11-16  2:00       ` Stefano Stabellini
2022-11-16  2:39         ` Henry Wang
2022-11-16  8:30         ` Jan Beulich
2022-11-16 23:41           ` Andrew Cooper
2022-11-16 23:44             ` Stefano Stabellini
2022-11-16 23:51               ` Julien Grall
2022-11-16 23:56                 ` Stefano Stabellini
2022-11-17  8:18             ` Jan Beulich
2022-10-26 10:20 ` [PATCH 4/4] xen/arm: Correct the p2m pool size calculations Andrew Cooper
2022-11-11 10:11   ` Henry Wang
2022-11-11 10:54     ` Julien Grall

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5EC7E359-3583-4846-8AF8-B84197B83BEF@citrix.com \
    --to=george.dunlap@citrix.com \
    --cc=Andrew.Cooper3@citrix.com \
    --cc=Henry.Wang@arm.com \
    --cc=Volodymyr_Babchuk@epam.com \
    --cc=anthony.perard@citrix.com \
    --cc=bertrand.marquis@arm.com \
    --cc=dunlapg@umich.edu \
    --cc=jbeulich@suse.com \
    --cc=julien@xen.org \
    --cc=roger.pau@citrix.com \
    --cc=sstabellini@kernel.org \
    --cc=wl@xen.org \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.