public inbox for cygwin-cvs@sourceware.org help / color / mirror / Atom feed
From: Corinna Vinschen <corinna@sourceware.org> To: cygwin-cvs@sourceware.org Subject: [newlib-cygwin] Cygwin: drop wincap::needs_query_information Date: Sun, 4 Dec 2022 13:27:17 +0000 (GMT) [thread overview] Message-ID: <20221204132717.52AEB38518B5@sourceware.org> (raw) https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=48953788b377c5e4e789f21106c47b645e1366de commit 48953788b377c5e4e789f21106c47b645e1366de Author: Corinna Vinschen <corinna@vinschen.de> AuthorDate: Tue Nov 15 17:10:57 2022 +0100 Commit: Corinna Vinschen <corinna@vinschen.de> CommitDate: Sun Dec 4 14:01:41 2022 +0100 Cygwin: drop wincap::needs_query_information Only required for Windows 7 and 8. Signed-off-by: Corinna Vinschen <corinna@vinschen.de> Diff: --- winsup/cygwin/local_includes/wincap.h | 2 -- winsup/cygwin/sigproc.cc | 9 +-------- winsup/cygwin/wincap.cc | 12 ------------ 3 files changed, 1 insertion(+), 22 deletions(-) diff --git a/winsup/cygwin/local_includes/wincap.h b/winsup/cygwin/local_includes/wincap.h index 9fff129097ae..1075f3bea85e 100644 --- a/winsup/cygwin/local_includes/wincap.h +++ b/winsup/cygwin/local_includes/wincap.h @@ -17,7 +17,6 @@ struct wincaps ops generated by gcc are off by 4 bytes. */ struct __attribute__ ((aligned (8))) { unsigned is_server : 1; - unsigned needs_query_information : 1; unsigned has_precise_system_time : 1; unsigned has_microsoft_accounts : 1; unsigned has_new_pebteb_region : 1; @@ -76,7 +75,6 @@ public: } intptr_t IMPLEMENT (mmap_storage_high) bool IMPLEMENT (is_server) - bool IMPLEMENT (needs_query_information) bool IMPLEMENT (has_precise_system_time) bool IMPLEMENT (has_microsoft_accounts) bool IMPLEMENT (has_new_pebteb_region) diff --git a/winsup/cygwin/sigproc.cc b/winsup/cygwin/sigproc.cc index a99876dc9fb9..ce36c8be37fb 100644 --- a/winsup/cygwin/sigproc.cc +++ b/winsup/cygwin/sigproc.cc @@ -846,14 +846,7 @@ child_info::child_info (unsigned in_cb, child_info_types chtype, DWORD perms = PROCESS_QUERY_LIMITED_INFORMATION | PROCESS_VM_READ | PROCESS_VM_OPERATION | SYNCHRONIZE; if (type == _CH_FORK) - { - perms |= PROCESS_DUP_HANDLE; - /* VirtualQueryEx is documented to require PROCESS_QUERY_INFORMATION. - That's true for Windows 7, but PROCESS_QUERY_LIMITED_INFORMATION - appears to be sufficient on Windows 8 and later. */ - if (wincap.needs_query_information ()) - perms |= PROCESS_QUERY_INFORMATION; - } + perms |= PROCESS_DUP_HANDLE; if (!DuplicateHandle (GetCurrentProcess (), GetCurrentProcess (), GetCurrentProcess (), &parent, perms, TRUE, 0)) diff --git a/winsup/cygwin/wincap.cc b/winsup/cygwin/wincap.cc index 7309516bf311..15400d8d0f57 100644 --- a/winsup/cygwin/wincap.cc +++ b/winsup/cygwin/wincap.cc @@ -24,7 +24,6 @@ wincaps wincap_7 __attribute__((section (".cygwin_dll_common"), shared)) = { mmap_storage_high:__MMAP_STORAGE_HIGH_LEGACY, { is_server:false, - needs_query_information:true, has_precise_system_time:false, has_microsoft_accounts:false, has_new_pebteb_region:false, @@ -55,7 +54,6 @@ wincaps wincap_8 __attribute__((section (".cygwin_dll_common"), shared)) = { mmap_storage_high:__MMAP_STORAGE_HIGH_LEGACY, { is_server:false, - needs_query_information:true, has_precise_system_time:true, has_microsoft_accounts:true, has_new_pebteb_region:false, @@ -86,7 +84,6 @@ wincaps wincap_8_1 __attribute__((section (".cygwin_dll_common"), shared)) = { mmap_storage_high:__MMAP_STORAGE_HIGH, { is_server:false, - needs_query_information:false, has_precise_system_time:true, has_microsoft_accounts:true, has_new_pebteb_region:false, @@ -117,7 +114,6 @@ wincaps wincap_10_1507 __attribute__((section (".cygwin_dll_common"), shared)) mmap_storage_high:__MMAP_STORAGE_HIGH, { is_server:false, - needs_query_information:false, has_precise_system_time:true, has_microsoft_accounts:true, has_new_pebteb_region:false, @@ -148,7 +144,6 @@ wincaps wincap_10_1607 __attribute__((section (".cygwin_dll_common"), shared)) mmap_storage_high:__MMAP_STORAGE_HIGH, { is_server:false, - needs_query_information:false, has_precise_system_time:true, has_microsoft_accounts:true, has_new_pebteb_region:false, @@ -179,7 +174,6 @@ wincaps wincap_10_1703 __attribute__((section (".cygwin_dll_common"), shared)) = mmap_storage_high:__MMAP_STORAGE_HIGH, { is_server:false, - needs_query_information:false, has_precise_system_time:true, has_microsoft_accounts:true, has_new_pebteb_region:true, @@ -210,7 +204,6 @@ wincaps wincap_10_1709 __attribute__((section (".cygwin_dll_common"), shared)) = mmap_storage_high:__MMAP_STORAGE_HIGH, { is_server:false, - needs_query_information:false, has_precise_system_time:true, has_microsoft_accounts:true, has_new_pebteb_region:true, @@ -241,7 +234,6 @@ wincaps wincap_10_1803 __attribute__((section (".cygwin_dll_common"), shared)) = mmap_storage_high:__MMAP_STORAGE_HIGH, { is_server:false, - needs_query_information:false, has_precise_system_time:true, has_microsoft_accounts:true, has_new_pebteb_region:true, @@ -272,7 +264,6 @@ wincaps wincap_10_1809 __attribute__((section (".cygwin_dll_common"), shared)) = mmap_storage_high:__MMAP_STORAGE_HIGH, { is_server:false, - needs_query_information:false, has_precise_system_time:true, has_microsoft_accounts:true, has_new_pebteb_region:true, @@ -303,7 +294,6 @@ wincaps wincap_10_1903 __attribute__((section (".cygwin_dll_common"), shared)) = mmap_storage_high:__MMAP_STORAGE_HIGH, { is_server:false, - needs_query_information:false, has_precise_system_time:true, has_microsoft_accounts:true, has_new_pebteb_region:true, @@ -334,7 +324,6 @@ wincaps wincap_10_2004 __attribute__((section (".cygwin_dll_common"), shared)) = mmap_storage_high:__MMAP_STORAGE_HIGH, { is_server:false, - needs_query_information:false, has_precise_system_time:true, has_microsoft_accounts:true, has_new_pebteb_region:true, @@ -365,7 +354,6 @@ wincaps wincap_11 __attribute__((section (".cygwin_dll_common"), shared)) = { mmap_storage_high:__MMAP_STORAGE_HIGH, { is_server:false, - needs_query_information:false, has_precise_system_time:true, has_microsoft_accounts:true, has_new_pebteb_region:true,
reply other threads:[~2022-12-04 13:27 UTC|newest] Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=20221204132717.52AEB38518B5@sourceware.org \ --to=corinna@sourceware.org \ --cc=cygwin-cvs@sourceware.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: linkBe 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 read-only IMAP folder(s) and NNTP newsgroup(s).