X-Recipient: archive-cygwin AT delorie DOT com X-Original-To: cygwin AT cygwin DOT com Delivered-To: cygwin AT cygwin DOT com DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 9E7733857C46 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=SystematicSw.ab.ca Authentication-Results: sourceware.org; spf=none smtp.mailfrom=brian DOT inglis AT systematicsw DOT ab DOT ca X-Authority-Analysis: v=2.4 cv=bZHV7MDB c=1 sm=1 tr=0 ts=5f6b67f2 a=kiZT5GMN3KAWqtYcXc+/4Q==:117 a=kiZT5GMN3KAWqtYcXc+/4Q==:17 a=IkcTkHD0fZMA:10 a=TImcKGuyeGIbufSLrCcA:9 a=QEXdDO2ut3YA:10 Subject: Re: Updated: w32api-{headers,runtime}-8.0.0-1 (x86/x86_64) To: cygwin AT cygwin DOT com References: <9a0a7e24-45fb-55d8-67f6-377b153810ab AT SystematicSw DOT ab DOT ca> <6d714acb-1df5-0c53-0409-65b4c43e8b36 AT SystematicSw DOT ab DOT ca> <0c884817-574a-e171-9bb5-b8d313a5f6ee AT SystematicSw DOT ab DOT ca> <6ac829c6-5281-ce39-d22b-d57b7d476c1c AT cornell DOT edu> From: Brian Inglis Autocrypt: addr=Brian DOT Inglis AT SystematicSw DOT ab DOT ca; prefer-encrypt=mutual; keydata= mDMEXopx8xYJKwYBBAHaRw8BAQdAnCK0qv/xwUCCZQoA9BHRYpstERrspfT0NkUWQVuoePa0 LkJyaWFuIEluZ2xpcyA8QnJpYW4uSW5nbGlzQFN5c3RlbWF0aWNTdy5hYi5jYT6IlgQTFggA PhYhBMM5/lbU970GBS2bZB62lxu92I8YBQJeinHzAhsDBQkJZgGABQsJCAcCBhUKCQgLAgQW AgMBAh4BAheAAAoJEB62lxu92I8Y0ioBAI8xrggNxziAVmr+Xm6nnyjoujMqWcq3oEhlYGAO WacZAQDFtdDx2koSVSoOmfaOyRTbIWSf9/Cjai29060fsmdsDLg4BF6KcfMSCisGAQQBl1UB BQEBB0Awv8kHI2PaEgViDqzbnoe8B9KMHoBZLS92HdC7ZPh8HQMBCAeIfgQYFggAJhYhBMM5 /lbU970GBS2bZB62lxu92I8YBQJeinHzAhsMBQkJZgGAAAoJEB62lxu92I8YZwUBAJw/74rF IyaSsGI7ewCdCy88Lce/kdwX7zGwid+f8NZ3AQC/ezTFFi5obXnyMxZJN464nPXiggtT9gN5 RSyTY8X+AQ== Organization: Systematic Software Message-ID: <06124a28-ecf3-fed1-7df3-947b91641f26@SystematicSw.ab.ca> Date: Wed, 23 Sep 2020 09:21:20 -0600 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.12.0 MIME-Version: 1.0 In-Reply-To: <6ac829c6-5281-ce39-d22b-d57b7d476c1c@cornell.edu> Content-Language: en-CA X-CMAE-Envelope: MS4xfAOZdB8EEXTCZyLxY+fszVOpMgUr/wiDT2+SlMY0BWElM5tQJ65+Aq0/9oS1Fkua7ZIk+xvT07Z8Ws0fRVpmIXSc/f9e2Q+toP3cNHwzkbI7BHvRNt4P AOnlNKQz0xUOfQX8baixz20QUMGwx1mI1a7kvjz+aEiqsJK9h5mhdhYVH95p9Nft4WLv20DJyEYTGPtJteBYqUmYeTZZ1oAtoJU= X-Spam-Status: No, score=-5.3 required=5.0 tests=BAYES_00, BODY_8BITS, KAM_DMARC_STATUS, KAM_LAZY_DOMAIN_SECURITY, NICE_REPLY_A, RCVD_IN_DNSWL_LOW, SPF_HELO_NONE, SPF_NONE, TXREP autolearn=no autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: cygwin AT cygwin DOT com X-Mailman-Version: 2.1.29 Precedence: list List-Id: General Cygwin discussions and problem reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: cygwin AT cygwin DOT com Content-Type: text/plain; charset="utf-8" Errors-To: cygwin-bounces AT cygwin DOT com Sender: "Cygwin" Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by delorie.com id 08NFLphq009944 On 2020-09-23 08:49, Ken Brown via Cygwin wrote: > On 9/23/2020 12:44 AM, Brian Inglis wrote: >> On 2020-09-22 22:17, Brian Inglis wrote: >>> On 2020-09-22 16:10, Brian Inglis wrote: >>>> On 2020-09-18 23:19, JonY via Cygwin-announce wrote: >>>>> Now released for both 32bit and 64bit Cygwin: >>>>> >>>>> w32api-headers-8.0.0-1 >>>>> w32api-runtime-8.0.0-1 >>>>> >>>>> Based on mingw-w64-v8.0.0. >>> >>> Sorry, ambigous - to clarify: may be a definition conflict between updated >>> headers rebuilding cygwin both 32 and 64 just after latest updates: see >>> attached log. >>> Cygwin both 32 and 64 builds worked without any problems just before the latest >>> updates. >> >> Tad confusing log messages, but it appears the actual issue may not be a >> conflict to do with PVOID, but with PMEM_EXTENDED_PARAMETER between the below: >> >> $ fgrep -wB12 PMEM_EXTENDED_PARAMETER /usr/include/w32api/winnt.h >>    typedef struct DECLSPEC_ALIGN(8) MEM_EXTENDED_PARAMETER { >>      __C89_NAMELESS struct { >>          DWORD64 Type : MEM_EXTENDED_PARAMETER_TYPE_BITS; >>          DWORD64 Reserved : 64 - MEM_EXTENDED_PARAMETER_TYPE_BITS; >>      }; >>      __C89_NAMELESS union { >>          DWORD64 ULong64; >>          PVOID Pointer; >>          SIZE_T Size; >>          HANDLE Handle; >>          DWORD ULong; >>      }; >>    } MEM_EXTENDED_PARAMETER, *PMEM_EXTENDED_PARAMETER; >> $ fgrep -wA8 PMEM_EXTENDED_PARAMETER winsup/utils/cygpath.cc >> #define PMEM_EXTENDED_PARAMETER PVOID >> #include >> #include >> #include >> #include >> #include >> >> #include "wide_path.h" >> #include "loadlib.h" >> >> It seems likely that we could perhaps just remove the conflicting #define, but >> without some idea of the original intent and its impact, and a good cygpath >> regression test suite, that could perhaps be a rash decision. > > This is related to the extended memory API that Corinna started using last > April.  At the time the relevant declarations and definitions weren't in the > mingw-w64 headers yet, so Corinna had to do some ad hoc stuff.  Now that they've > been added, a lot of that stuff should be removed.  I'll do it later today. Thanks Ken, Please let me know so I can pull and rebuild cygwin 32 and 64 to test both with another cpuinfo update. I'm going to try to wait until 5.9 release before committing and submitting another cpuinfo patch, as feature updates are starting to be made visible in cpuinfo. I thought there were going to be none, then they added some feature updates, a couple of which were visible, and I thought that might be it, so sent a patch with those, but more are coming with RCs now. -- Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada This email may be disturbing to some readers as it contains too much technical detail. Reader discretion is advised. [Data in binary units and prefixes, physical quantities in SI.] -- Problem reports: https://cygwin.com/problems.html FAQ: https://cygwin.com/faq/ Documentation: https://cygwin.com/docs.html Unsubscribe info: https://cygwin.com/ml/#unsubscribe-simple