From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by sourceware.org (Postfix) with ESMTPS id 608803856DC0 for ; Fri, 29 Apr 2022 14:03:17 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 608803856DC0 Received: from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-573-Yo5hHXgaMIWPGd9n8q-Utg-1; Fri, 29 Apr 2022 10:03:06 -0400 X-MC-Unique: Yo5hHXgaMIWPGd9n8q-Utg-1 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.rdu2.redhat.com [10.11.54.4]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 391EA3C0E1A5; Fri, 29 Apr 2022 14:03:05 +0000 (UTC) Received: from oldenburg.str.redhat.com (unknown [10.39.193.187]) by smtp.corp.redhat.com (Postfix) with ESMTPS id AD12B2026D2F; Fri, 29 Apr 2022 14:03:04 +0000 (UTC) From: Florian Weimer To: Adhemerval Zanella Cc: libc-alpha@sourceware.org Subject: Re: [PATCH v3 2/2] Default to --with-default-link=no (bug 25812) References: <15c7f6e9afe2c2b9c51ebc6372682a39b0932712.1649691083.git.fweimer@redhat.com> <138c11d6-0def-2ff4-73da-c4346a855043@linaro.org> <87a6c458r1.fsf@oldenburg.str.redhat.com> Date: Fri, 29 Apr 2022 16:03:03 +0200 In-Reply-To: (Adhemerval Zanella's message of "Fri, 29 Apr 2022 10:55:47 -0300") Message-ID: <87ilqs0yy0.fsf@oldenburg.str.redhat.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.78 on 10.11.54.4 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain X-Spam-Status: No, score=-5.7 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, RCVD_IN_DNSWL_LOW, SPF_HELO_NONE, SPF_NONE, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: libc-alpha@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libc-alpha mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 29 Apr 2022 14:03:20 -0000 * Adhemerval Zanella: > On 29/04/2022 10:17, Florian Weimer wrote: >> * Adhemerval Zanella: >> >>> On 11/04/2022 12:32, Florian Weimer via Libc-alpha wrote: >>>> This is necessary to place the libio vtables into the RELRO segment. >>>> New tests elf/tst-relro-ldso and elf/tst-relro-libc are added to >>>> verify that this is what actually happens. >>>> >>>> The new tests fail on ia64 due to lack of (default) RELRO support >>>> inbutils, so they are XFAILed there. >>> >>> I always frown upon new configure options, since they tend to degrade over >>> time (as --disable-shared for instance [1]) and require additional >>> maintaining costs. Why can't we make the required semantic the default >>> (since usually relro has minimal costs it a net gain in hardening)? >> >> This is what the patch does, I think. > > I meant to not add --with-default-link= and just not use the extra linker > script. That would be nice. I want to get rid of the pointer arrays, and after the vfprintf changes (and similar vscanf changes) are merged, we won't need the vtable array anymore, either. That should eliminate all special section placement requirements. Thanks, Florian