From mboxrd@z Thu Jan 1 00:00:00 1970 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on inbox.vuxu.org X-Spam-Level: X-Spam-Status: No, score=-3.3 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, MAILING_LIST_MULTI,RCVD_IN_DNSWL_MED,RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL,SPF_PASS autolearn=ham autolearn_force=no version=3.4.2 Received: (qmail 3594 invoked from network); 16 Apr 2020 16:06:18 -0000 Received-SPF: pass (mother.openwall.net: domain of lists.openwall.com designates 195.42.179.200 as permitted sender) receiver=inbox.vuxu.org; client-ip=195.42.179.200 envelope-from= Received: from mother.openwall.net (195.42.179.200) by inbox.vuxu.org with UTF8ESMTPZ; 16 Apr 2020 16:06:18 -0000 Received: (qmail 27985 invoked by uid 550); 16 Apr 2020 16:06:15 -0000 Mailing-List: contact musl-help@lists.openwall.com; run by ezmlm Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-ID: Reply-To: musl@lists.openwall.com Received: (qmail 27945 invoked from network); 16 Apr 2020 16:06:13 -0000 DKIM-Signature: a=rsa-sha256; v=1; c=relaxed/relaxed; d=mg.codeaurora.org; q=dns/txt; s=smtp; t=1587053175; h=Content-Transfer-Encoding: Content-Type: MIME-Version: Message-ID: Date: Subject: In-Reply-To: References: Cc: To: From: Sender; bh=K/95RoUd+qp6y2afEACj4EpO/fX3pccZ38Du3PFVNYE=; b=H0tgqxalOitiTnNILvOtUXRrImYw1OYAZ5k7gHmLcaeZ8NF+iZbP79Jx5LXvb07HoHV6xTV2 wfutjXn6L1el7w3Jb4H+UYzszlFzM429O5pTKQke/oul0jYagTqP1UN+NOJTxpL4pNB75XuF oUpQOuCGvaDyBdDQ4FqjbFm7hYY= X-Mailgun-Sending-Ip: 104.130.122.27 X-Mailgun-Sid: WyI1MGQzMyIsICJtdXNsQGxpc3RzLm9wZW53YWxsLmNvbSIsICJiZTllNGEiXQ== Sender: sidneym=codeaurora.org@mg.codeaurora.org DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 09239C433CB Authentication-Results: aws-us-west-2-caf-mail-1.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: aws-us-west-2-caf-mail-1.web.codeaurora.org; spf=none smtp.mailfrom=sidneym@codeaurora.org From: To: "'Rich Felker'" Cc: References: <093e01d6139c$90bd9da0$b238d8e0$@codeaurora.org> <20200416031601.GR11469@brightrain.aerifal.cx> In-Reply-To: <20200416031601.GR11469@brightrain.aerifal.cx> Date: Thu, 16 Apr 2020 11:05:36 -0500 Message-ID: <0cf901d61408$deb5f950$9c21ebf0$@codeaurora.org> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQKpvFrF4hnSMamzdQO/1o68IggQlgH84T+PpsSm7NA= Content-Language: en-us Subject: RE: [musl][hexagon] testing updates > -----Original Message----- > From: Rich Felker > Sent: Wednesday, April 15, 2020 10:16 PM > To: sidneym@codeaurora.org > Cc: musl@lists.openwall.com > Subject: Re: [musl][hexagon] testing updates > > On Wed, Apr 15, 2020 at 10:10:20PM -0500, sidneym@codeaurora.org wrote: > > Updated alltypes.h.in and added sem.h. This change cleared the > > following > > errors: > > > > src/functional/pthread_mutex-static.exe > > > > src/functional/pthread_mutex.exe > > > > src/functional/pthread_mutex_pi-static.exe > > > > src/functional/pthread_mutex_pi.exe > > src/functional/sem_init-static.exe > > > > src/functional/sem_init.exe > > I'm confused how these changed at all from the changes you made. > sem_init is for POSIX semaphores not sysv ipc ones. The bits/sem.h things > don't have anything to do with it. The sem.h change shouldn't have been included in the patch. The change of time_t from a 64 to 32 bit value changed the size of timespec used in the pthread_cond and sem_timedwait I pruned our original port, possibly too much in some cases, but in this case I'd like some guidance since no other arch needed time_t as a 32-bit type. There is a large chunk of code compat/time32 which I have not tried to use yet but I have a feeling I might need to. > > > src/regression/pthread_cond-smasher-static.exe > > > > src/regression/pthread_cond-smasher.exe > > > > src/regression/pthread_cond_wait-cancel_ignored-static.exe > > > > src/regression/pthread_cond_wait-cancel_ignored.exe > > > > src/regression/pthread_once-deadlock-static.exe > > Likewise these shouldn't have changed either. If they did it's probably some > other hidden state in your test environment. > > > The patch is here: > > > https://github.com/quic/musl/commit/ca20acd472a8e9e58e584d51c4cd00ce > d6 > > f37087 > > The change to alltypes.h.in is wrong. It changes time_t to 32-bit, which is no > longer a supported configuration and not the intent. With that part reverted > and the sysvipc bits headers fixed as described before, I think you might > have it mostly working. > > Rich