All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Thomas Huth <thuth@redhat.com>
Cc: qemu-devel@nongnu.org, Yonggang Luo <luoyonggang@gmail.com>,
	Stefan Weil <sw@weilnetz.de>
Subject: Re: [PATCH] docs/about: Update the support statement for Windows
Date: Thu, 5 May 2022 16:09:56 +0100	[thread overview]
Message-ID: <YnPoxFDyW+J3al8q@redhat.com> (raw)
In-Reply-To: <fb070833-d9d4-4842-8470-979cf7aa859d@redhat.com>

On Thu, May 05, 2022 at 04:12:11PM +0200, Thomas Huth wrote:
> On 05/05/2022 13.55, Daniel P. Berrangé wrote:
> > On Thu, May 05, 2022 at 09:14:57AM +0100, Daniel P. Berrangé wrote:
> > > On Thu, May 05, 2022 at 09:48:41AM +0200, Thomas Huth wrote:
> > > > Our support statement for Windows currently talks about "Vista / Server
> > > > 2008" - which is related to the API of Windows, and this is not easy
> > > > to understand for the non-technical users. It might also not be quite
> > > > true anymore, since we only compile-test QEMU with recent toolchains.
> > > 
> > > We documented Vista / Server 2008 because that is what our code is
> > > declaring it wants in terms of Windows public APIs:
> > > 
> > > In osdep.h we have:
> > > 
> > >    #ifdef _WIN32
> > >    /* as defined in sdkddkver.h */
> > >    #ifndef _WIN32_WINNT
> > >    #define _WIN32_WINNT 0x0600 /* Vista */
> > >    #endif
> > > 
> > > which tells Mingw / MSys not to expose windows header file declarations
> > > that post-date Vista.
> > 
> > Of course we rely on 3rd party libraries and in particular GLib2 is
> > mandatory, and it also set _WIN32_WINNT. So our constraint is the
> > newer of the _WIN32_WINNT constraint set by QEMU and whatever version
> > of GLib2 being compiled against.
> > 
> > QEMU sets a min GLib of 2.56, and that version of GLib sets 0x0601
> > which means Windows >= 7.  So even though QEMU only asks for Vista,
> > in practice our minimum is 7.
> 
> Windows 7 does not receive any security updates since 2020 anymore, so I'd
> rather would bump it to the level of Windows 8.1 directly instead. Or
> directly go for Windows 10 to mimic the behavior that we have with most of
> the Linux distros (max. two major releases at a time).

I like the simplicity of having the same rule everywhere.

I'm a little wary of people being stuck on old versions, when I see
current articles like this showing 20% of people still on Win7, way
more than Win8 surprisingly.

https://www.statista.com/statistics/993868/worldwide-windows-operating-system-market-share/


The thing with bumping the _WIN32_WINNT version higher though is that
we're not really doing it for any compelling technical reasons. Usually
when we bump min glib, it lets us cut out compatibility hacks and/or
get access to new APIs.

Bumping _WIN32_WINNT can do that, but the interesting place where that's
useful is mostly in GLib. I'm not sure what we'd make use of in the QEMU
side from setting a newer _WIN32_WINNT. Hence my suggestion that we just
let min GLib drive our min _WIN32_WINNT setting. If someone can point
to stuff we can improve in QEMU though, thanks to new _WIN32_WINNT that
could top the balance.

With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|



  reply	other threads:[~2022-05-05 15:12 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-05  7:48 [PATCH] docs/about: Update the support statement for Windows Thomas Huth
2022-05-05  8:14 ` Daniel P. Berrangé
2022-05-05 11:55   ` Daniel P. Berrangé
2022-05-05 14:12     ` Thomas Huth
2022-05-05 15:09       ` Daniel P. Berrangé [this message]
2022-05-12  6:59         ` Thomas Huth
2022-05-05  8:27 ` Peter Maydell
2022-05-05  9:14   ` Stefan Weil via

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=YnPoxFDyW+J3al8q@redhat.com \
    --to=berrange@redhat.com \
    --cc=luoyonggang@gmail.com \
    --cc=qemu-devel@nongnu.org \
    --cc=sw@weilnetz.de \
    --cc=thuth@redhat.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 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.