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 29905 invoked from network); 17 Apr 2020 17:15:46 -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; 17 Apr 2020 17:15:46 -0000 Received: (qmail 1634 invoked by uid 550); 17 Apr 2020 17:15:44 -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 1616 invoked from network); 17 Apr 2020 17:15:41 -0000 DKIM-Signature: a=rsa-sha256; v=1; c=relaxed/relaxed; d=mg.codeaurora.org; q=dns/txt; s=smtp; t=1587143744; h=Content-Transfer-Encoding: Content-Type: MIME-Version: Message-ID: Date: Subject: In-Reply-To: References: To: From: Sender; bh=IbfmYrQsp+n/otGGYR+tTlBVDru59/qDM6o2jcUFGSk=; b=WuEX9U3Q1XQeShk9VZeKyuHcLHTAuTfZcNG4SnOwY0DknvM2p6U506O1yF7F8jiZbq6aDgya tFEvgSrViqwEyYiKUjtJ9EOXnVo9/NTjJvXXqyPfwFveJhRQkgXRkOqkWlkA53Ge6heVhKEC avF11RQ7ble1LRPmi19H8A5FPVE= 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 29EB9C432C2 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: References: <093e01d6139c$90bd9da0$b238d8e0$@codeaurora.org> <20200416031601.GR11469@brightrain.aerifal.cx> <0cf901d61408$deb5f950$9c21ebf0$@codeaurora.org> <20200416161746.GW11469@brightrain.aerifal.cx> In-Reply-To: <20200416161746.GW11469@brightrain.aerifal.cx> Date: Fri, 17 Apr 2020 12:15:13 -0500 Message-ID: <173801d614db$c2cd8310$48688930$@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+PAczPG+ECBluJkaansU7A Content-Language: en-us Subject: RE: [musl][hexagon] testing updates > -----Original Message----- > From: 'Rich Felker' > Sent: Thursday, April 16, 2020 11:18 AM > To: sidneym@codeaurora.org > Cc: musl@lists.openwall.com > Subject: Re: [musl][hexagon] testing updates > > On Thu, Apr 16, 2020 at 11:05:36AM -0500, sidneym@codeaurora.org wrote: > > > > > > > -----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. > > Remove the time_t and suseconds_t TYPEDEFs from alltypes.h.in. They are > no longer allowed to vary per-arch. Then make sure you rebuild > *everything* (libc, the test suite, any other code you're linking against musl > and trying to use). I'm pretty sure you have a mix of files that have been > build with different things you've tried and that is the source of your > problems. > > > 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. > > These files are only used on archs that had an old ABI with 32-bit time_t, > which I asked about before and you seemed to say you don't have. If you > *do* actually have an existing ecosystem of stuff that possibly needs to keep > working, we need to figure out what that is and whether it's even possible to > support (it might not be if it's a mess/mix of different things you've tried). If > not then these files will not even be built and are not needed. > This isn't something I had understood very well. Since Hexagon's unistd.h defines __ARCH_WANT_TIME32_SYSCALLS then we are using the old ABI I believe. At the moment I don't feel very strongly about maintaining this, that might change the more I learn. I'm looking at what it would take to migrate away from this. Thanks, > Rich