Date: Fri, 1 Apr 2022 14:19:46 +0800
From: Julien Rouhaud <rjuju123@gmail.com>
To: Michael Paquier <michael@paquier.xyz>
Cc: Thomas Munro <thomas.munro@gmail.com>, 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.
Message-ID: <YkaZgufzxtyTeJv+@jrouhaud>
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>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <YkaUuhwppy2hiBi9@paquier.xyz>
Archived-At: 
 <https://www.postgresql.org/message-id/YkaZgufzxtyTeJv%2B%40jrouhaud>
Precedence: bulk

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/