pg.ddx.io  pgsql-bugs@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Julien Rouhaud <rjuju123@gmail.com>
To: Michael Paquier <michael@paquier.xyz>
Cc: Thomas Munro <thomas.munro@gmail.com>
Cc: okano.naoki@jp.fujitsu.com, PostgreSQL mailing lists <pgsql-bugs@lists.postgresql.org>
Subject: Re: BUG #17448: In Windows 10, version 1703 and later, huge_pages doesn't work.
Date: Fri, 1 Apr 2022 14:19:46 +0800
Message-ID: <YkaZgufzxtyTeJv+@jrouhaud> (raw)
In-Reply-To: <YkaUuhwppy2hiBi9@paquier.xyz>
References: <Yj6oxYsBE5LXA8ZP@paquier.xyz>
	<20220326082408.moh55zuecxjmewnm@jrouhaud>
	<CA+hUKGLwOSf-QwVtCcKx9xvugN2BSzEBBeD5DNyTXgcSRp5sTw@mail.gmail.com>
	<YkQMyhsP7xgLDDT/@paquier.xyz>
	<20220331045908.bm5edcacvymgwwv2@jrouhaud>
	<YkVbbZBiC4yxXPuc@paquier.xyz>
	<20220331080055.hjrrsrd4lxwlt7f5@jrouhaud>
	<20220331104659.ol6pqe7s3tpkymuk@jrouhaud>
	<20220331110309.ws25d6dl5yyzprye@jrouhaud>
	<YkaUuhwppy2hiBi9@paquier.xyz>

On Fri, Apr 01, 2022 at 02:59:22PM +0900, Michael Paquier wrote:
> On Thu, Mar 31, 2022 at 07:03:09PM +0800, Julien Rouhaud wrote:
> > There's no such indication on IsWindowsVersionOrGreater(), but after seeing
> > various comments on forums from angry people, it may be a hint that it behaves
> > similarly.  I'm not sure what to do at this point, maybe just always use the
> > flag (the PG_ version which may be 0), hoping that hopefully windows won't
> > define it if it can't handle it?
>
> Looking at the internals of versionhelpers.h, would it work to use as
> arguments for IsWindowsVersionOrGreater() the following, in this
> order?  As of:
> - HIBYTE(_WIN32_WINNT_WINTHRESHOLD)
> - LOBYTE(_WIN32_WINNT_WINTHRESHOLD)
> - 1703
>
> Just to drop an idea.

I will test that in a bit.  I still think that at least some API exists to give
the real answer since winver and similar report correct info, I just don't know
what those are.

Note that if you want to test yourself you could use this script [1] using the
evaluation virtual machine [2] to automatically setup a windows 11 environment
with everything needed to compile postgres with a extra dependencies.  The
whole process is a bit long though, so I can also give you access to my vm if
you prefer, probably the latency shouldn't be too bad :)

[1] https://github.com/rjuju/pg_msvc_generator/blob/master/bootstrap.ps1
[2] https://developer.microsoft.com/en-us/windows/downloads/virtual-machines/





view thread (25+ messages)  latest in thread

Message-ID: <YkaZgufzxtyTeJv+@jrouhaud>
Permalink:  ../YkaZgufzxtyTeJv+@jrouhaud/
Also on:    postgresql.org/message-id/YkaZgufzxtyTeJv+@jrouhaud

 · 

reply

Reply instructions:

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

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-bugs@postgresql.org
  Cc: rjuju123@gmail.com, michael@paquier.xyz, thomas.munro@gmail.com, pgsql-bugs@lists.postgresql.org
  Subject: Re: BUG #17448: In Windows 10, version 1703 and later, huge_pages doesn't work.
  In-Reply-To: <YkaZgufzxtyTeJv+@jrouhaud>

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

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox