From patchwork Fri Nov 25 16:16:26 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Florian Weimer X-Patchwork-Id: 84184 Delivered-To: patch@linaro.org Received: by 10.140.20.101 with SMTP id 92csp179895qgi; Fri, 25 Nov 2016 08:16:52 -0800 (PST) X-Received: by 10.84.214.150 with SMTP id j22mr18851279pli.81.1480090612352; Fri, 25 Nov 2016 08:16:52 -0800 (PST) Return-Path: Received: from sourceware.org (server1.sourceware.org. [209.132.180.131]) by mx.google.com with ESMTPS id 129si45304629pgi.256.2016.11.25.08.16.52 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 25 Nov 2016 08:16:52 -0800 (PST) Received-SPF: pass (google.com: domain of libc-alpha-return-75175-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) client-ip=209.132.180.131; Authentication-Results: mx.google.com; dkim=pass header.i=@sourceware.org; spf=pass (google.com: domain of libc-alpha-return-75175-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) smtp.mailfrom=libc-alpha-return-75175-patch=linaro.org@sourceware.org DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:subject:to:references:from:message-id:date :mime-version:in-reply-to:content-type; q=dns; s=default; b=Wvk4 3LCR8rl2gLyW3s2cFKXXRJi/ObUPPcI1U4cLWsAIWyWRxuaj9IvQ7CFI47d1f8r7 2XZr3BCEJ65s8FOWt2srxGYHaa4ETyTGpXU+vVBoE0Eur3h441E1HQTiBde4Rrn8 OZ/QQIu9Pse4zyf+wlUvZXPpAT89s7z9zkgDN8M= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:subject:to:references:from:message-id:date :mime-version:in-reply-to:content-type; s=default; bh=IB89fefYhV mlBpa1dPMsIhZrZLI=; b=PDklan/udBvHS4YhxohDqKAXcH68g/LRBtdAMQ7Cvt MGqYVg2DPhqRutuYDOSSexKCwQhEvggB2wMH/gthV8HPgwBM2/xdlsT/hhkUnD5a ti+uRkBcoNZtcSyPy+9qIK9+PtDV/60o5eFPtfj8Hiv4273tpzAXrBrRX2w196pa Y= Received: (qmail 27715 invoked by alias); 25 Nov 2016 16:16:41 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 27691 invoked by uid 89); 25 Nov 2016 16:16:40 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-4.9 required=5.0 tests=BAYES_00, RP_MATCHES_RCVD, SPF_HELO_PASS autolearn=ham version=3.3.2 spammy=revealed, Hx-languages-length:1492 X-HELO: mx1.redhat.com Subject: Re: [PATCH] extras: New test/build infrastructure To: GNU C Library References: <9426ee75-3e45-6cde-b659-567398007a32@redhat.com> From: Florian Weimer Message-ID: Date: Fri, 25 Nov 2016 17:16:26 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 MIME-Version: 1.0 In-Reply-To: <9426ee75-3e45-6cde-b659-567398007a32@redhat.com> On 11/25/2016 04:59 PM, Florian Weimer wrote: > I have split up test-skeleton.c into its components. The new test > skeleton should be compatible with compilation in C90 mode. I expect to > use some of these helper functions for future build support on the host > (that is, these routines will have to be compiled twice, once against > the built libc, and once against the host libc). > > I converted a few tests to make sure that this works as expected. > > I do not propose bulk migration at this point. Some obscure use cases > are not supported by the exported hooks. > > If you think this goes in the right direction, I will port over my > container-based nss_files testing, and the resolver test suite to this > infrastructure. It also eliminates the need for the hack in my fix for > bug 16421 I posted earlier. > > Tested on aarch64, ppc64le, s390x, x86_64. Testing on i386 revealed that the following change to the temporary file infrastructure is needed. Florian diff --git a/extras/temp_file.c b/extras/temp_file.c index 32b6560..a409d7a 100644 --- a/extras/temp_file.c +++ b/extras/temp_file.c @@ -16,6 +16,10 @@ License along with the GNU C Library; if not, see . */ +/* This is required to get an mkstemp which can create large files on + some 32-bit platforms. */ +#define _FILE_OFFSET_BITS 64 + #include #include #include