From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 2155) id 52AEB38518B5; Sun, 4 Dec 2022 13:27:17 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 52AEB38518B5 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sourceware.org; s=default; t=1670160437; bh=vxWsA+JyN9iqHlRAvOMlAufsLgiuGi8xjEaxAdP9GKI=; h=From:To:Subject:Date:From; b=ZOfu7xyfkW5hui4nIVlOrNL2RKCoCjpCQ91CK0DqDCwHNsirxxxbuIt8Y5/nVJhO+ PQTmIJMH2D+KdCxxDrSs1mHdNZdn4p1vj0yQ8xvSeGfollrcnRDqr6YqfLSccc0Ck0 aze81wHDLdMogQPWVE+IoMXB+vNe4mTTTbAEU/Qs= Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable From: Corinna Vinschen To: cygwin-cvs@sourceware.org Subject: [newlib-cygwin] Cygwin: drop wincap::needs_query_information X-Act-Checkin: newlib-cygwin X-Git-Author: Corinna Vinschen X-Git-Refname: refs/heads/master X-Git-Oldrev: b541558b7a59e92b8b2518c714e38430466ce541 X-Git-Newrev: 48953788b377c5e4e789f21106c47b645e1366de Message-Id: <20221204132717.52AEB38518B5@sourceware.org> Date: Sun, 4 Dec 2022 13:27:17 +0000 (GMT) List-Id: https://sourceware.org/git/gitweb.cgi?p=3Dnewlib-cygwin.git;h=3D48953788b37= 7c5e4e789f21106c47b645e1366de commit 48953788b377c5e4e789f21106c47b645e1366de Author: Corinna Vinschen AuthorDate: Tue Nov 15 17:10:57 2022 +0100 Commit: Corinna Vinschen CommitDate: Sun Dec 4 14:01:41 2022 +0100 Cygwin: drop wincap::needs_query_information =20 Only required for Windows 7 and 8. =20 Signed-off-by: Corinna Vinschen 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_in= cludes/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_typ= es chtype, DWORD perms =3D PROCESS_QUERY_LIMITED_INFORMATION | PROCESS_VM_READ | PROCESS_VM_OPERATION | SYNCHRONIZE; if (type =3D=3D _CH_FORK) - { - perms |=3D 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 |=3D PROCESS_QUERY_INFORMATION; - } + perms |=3D PROCESS_DUP_HANDLE; =20 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_com= mon"), shared)) =3D { 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_com= mon"), shared)) =3D { 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_c= ommon"), shared)) =3D { 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 (".cygwi= n_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 (".cygwi= n_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)) =3D 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)) =3D 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)) =3D 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)) =3D 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)) =3D 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)) =3D 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)) =3D { 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,