qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Peter Maydell <peter.maydell@linaro.org>
To: Danny Canter <danny_canter@apple.com>
Cc: qemu-devel@nongnu.org, qemu-arm@nongnu.org,
	 Cameron Esfahani <dirty@apple.com>,
	rbolshakov@ddn.com, agraf@csgraf.de
Subject: Re: [PATCH] hvf: arm: Allow creating VMs with > 63GB of RAM on macOS 15+
Date: Wed, 31 Jul 2024 09:26:50 +0100	[thread overview]
Message-ID: <CAFEAcA8WN5abC7wARmMd1+ePr2yaZ61wVLpJMcZ-5TUnvxQXFA@mail.gmail.com> (raw)
In-Reply-To: <8CEBAF31-2582-45B5-B391-DE4F392EB932@apple.com>

On Tue, 30 Jul 2024 at 22:53, Danny Canter <danny_canter@apple.com> wrote:
>
> Thanks Peter!
>
> For the macOS 13 comment just so I’m clear, you’re saying the minimum we support is
> 13 now so the conditional compilation for those isn’t required anymore as well? I suppose
> that tracks given the wording that we support the last two macOS releases at any given
> time, that kind of slipped my mind when thinking about what actually needs to be ifdef’d
> here. That certainly cleans things up a bit for some of the functions in this patch.

Yep, that's the idea. We tend to leave the conditionals in a bit beyond
literally just last two releases, so there's a period of "not supported but
it probably still works" for a release that's fallen off the end of our
support list. But in this case since we already have code in QEMU that
requires 13 to even compile, there's no need to add new 13-or-better ifdefs
in this new code.

-- PMM


  reply	other threads:[~2024-07-31  8:28 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-18 23:00 [PATCH] hvf: arm: Allow creating VMs with > 63GB of RAM on macOS 15+ Danny Canter
2024-07-29 16:27 ` Peter Maydell
2024-07-30 21:53   ` Danny Canter
2024-07-31  8:26     ` Peter Maydell [this message]
2024-08-12 14:52 ` Peter Maydell
2024-08-12 22:18   ` Danny Canter
2024-08-13  9:31     ` Peter Maydell
2024-08-17  0:36       ` Danny Canter
2024-08-20 13:42         ` Peter Maydell
2024-09-05 18:45       ` Danny Canter

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=CAFEAcA8WN5abC7wARmMd1+ePr2yaZ61wVLpJMcZ-5TUnvxQXFA@mail.gmail.com \
    --to=peter.maydell@linaro.org \
    --cc=agraf@csgraf.de \
    --cc=danny_canter@apple.com \
    --cc=dirty@apple.com \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=rbolshakov@ddn.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).