This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
RE: [PATCH] Add support for non-contiguous memory regions
- From: Tamar Christina <Tamar dot Christina at arm dot com>
- To: Christophe Lyon <christophe dot lyon at linaro dot org>, binutils <binutils at sourceware dot org>
- Cc: nd <nd at arm dot com>
- Date: Fri, 14 Feb 2020 13:01:45 +0000
- Subject: RE: [PATCH] Add support for non-contiguous memory regions
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=uT6qwnk+qxNuYEXE4LqBMlYj7lDImImcnjcSn/Mw0EQ=; b=gQ6UvYJIwlhY54Xuse82mCB3a0j+u5KIgdJw+dqLE4S4R+C+Z8cc68IH0bzdG7dn7USIHKauUdS6le/7XZZ7qtWLSPZwKYMalgK5UzYWaotyBcWaZEPoAN/ZKmzb3MWG+SkCCy9KKQkMFIKgjOhXLfk+aK9fGu+6tTrFwFD8fHiXyZkwK+PexcFjJmuvbxO59piBVOiqPuXC0GlUfCt/ayDQPV9HFaVaav0K04FTZL40UTy9G+6OoMXaXJ4VE53T3f6O731NJ2YL6yZzMa/IL5HM8VFSLQsJeXV2jvVrX85bMOo5u+r38D2vOhi+2/l9vHLdWotuLTpxCSMbJ45BTw==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=T0nIlxckl8InKRFW5L1mXBH3Pc5m9dHKjeUzO+uCyqhRu1yTKpk0MDUtrojL2chC7kp+QOnB/fVF9hptMpYt8NKCOxJ8sV3qlzzjzoUXuKpaix7Njw+s/+oxrkdtnCT3Wl9bMjovZat/0pyJPjg4zBkS35woslR1U3AWOsU7DfK5w4NITFtoPYJQqKFYd5+uAu7dfpxaafSWgKvSJoOcFgjGREhyrNxzdY4qmaSp9j1v/L4pJ3nVJSVGNdERi66yIz3p+obcGbdYjH1rLB+AlNUdl+WU9HB15qb7KvUUQoqRdbstqcbX9zl296rkU34l2abEcLaix3w/oyvk/pCLHA==
- Original-authentication-results: spf=none (sender IP is ) smtp.mailfrom=Tamar dot Christina at arm dot com;
- References: <CAKdteOZDKNpW2iRWpJ9eykTT+Yb-n9mVDed=YO4+vZg+hKSPOA@mail.gmail.com>
Hi Christophe,
Did you forget to attach the patches perhaps? I don't see any attachments.
Cheers,
Tamar
> -----Original Message-----
> From: binutils-owner@sourceware.org <binutils-owner@sourceware.org>
> On Behalf Of Christophe Lyon
> Sent: Friday, February 14, 2020 12:34 PM
> To: binutils <binutils@sourceware.org>
> Subject: [PATCH] Add support for non-contiguous memory regions
>
> Hi,
>
> This is a follow-up to
> https://sourceware.org/ml/binutils/2019-11/msg00402.html
> https://sourceware.org/ml/binutils/2020-01/msg00064.html
> and
> https://sourceware.org/ml/binutils/2020-02/msg00037.html
>
> Changes between v3 and v4:
> * I ran the tests on more targets (those that have xxx_build_one_stub:
> avr csky hppa mc68ch11 metag nios2 ppc aarch64 m68hc11 m68hc12)
> * this made me update the generic test (match addresses more lazily to cope
> with 64 bits outputs, ignore symbol table whose order depends on the
> target), and skip it for the targets that don't support -mlittle-endian. Is there
> a more generic way of doing this? (for sure, there are other targets that don't
> support that option but that I haven't tested)
> * I updated xxx_build_one_stub similarly to what I needed for arm, although
> I didn't write specific tests
>
> The changes between v2 and v3:
> * added some doc/NEWS
> * fixed bugs detected when running the testsuite with the option activated
> by default. In particular, I've noticed and documented the incompatibility
> with INSERT (or rather, it's likely to cause problems, but depends on the
> actual linker scripts contents), because my new option breaks the
> assumption of the parsing & processing order assumed by INSERT.
> * added new option --enable-non-contiguous-regions-warnings which
> activates some warnings to help understand why the behaviour changes in
> some cases
>
> There are 4 patches, to hopefully make review/comments easier; I can
> squash them at commit time if this is preferable.
> * patch1: is the main (code) patch
> * patch2: generic test
> * patch3: arm tests
> * patch4: powerpc test
>
> OK?
>
> Thanks,
>
> Christophe