Message ID | 20140917122939.GT6586@spoyarek.pnq.redhat.com |
---|---|
State | Committed |
Headers |
Received: (qmail 2272 invoked by alias); 17 Sep 2014 12:29:52 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: <libc-alpha.sourceware.org> List-Unsubscribe: <mailto:libc-alpha-unsubscribe-##L=##H@sourceware.org> List-Subscribe: <mailto:libc-alpha-subscribe@sourceware.org> List-Archive: <http://sourceware.org/ml/libc-alpha/> List-Post: <mailto:libc-alpha@sourceware.org> List-Help: <mailto:libc-alpha-help@sourceware.org>, <http://sourceware.org/ml/#faqs> Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 2262 invoked by uid 89); 17 Sep 2014 12:29:51 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.7 required=5.0 tests=AWL, BAYES_00, RP_MATCHES_RCVD, SPF_HELO_PASS autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Date: Wed, 17 Sep 2014 17:59:40 +0530 From: Siddhesh Poyarekar <siddhesh@redhat.com> To: libc-alpha@sourceware.org Cc: Roland McGrath <roland@hack.frob.com> Subject: [PATCH v2 06/19] Define IN_MODULE for translation units that define NOT_IN_libc Message-ID: <20140917122939.GT6586@spoyarek.pnq.redhat.com> References: <1408618663-2281-1-git-send-email-siddhesh@redhat.com> <1408618663-2281-7-git-send-email-siddhesh@redhat.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="VjP/dwTbBl6I9PQk" Content-Disposition: inline In-Reply-To: <1408618663-2281-7-git-send-email-siddhesh@redhat.com> User-Agent: Mutt/1.5.22.1-rc1 (2013-10-16) |
Commit Message
Siddhesh Poyarekar
Sept. 17, 2014, 12:29 p.m. UTC
Hi, I have now rebased this patch against the current master. interp.c does not need the CPPFLAGS anymore, so sotruss-lib is the only code to be built as extramodules to eventually replace the NOT_IN_libc macro. Build tested on x86_64 to ensure that the generated code is unchanged. Siddhesh * elf/Makefile (libof-sotruss-lib): Set as extramodules.
Comments
Ping! On Wed, Sep 17, 2014 at 05:59:40PM +0530, Siddhesh Poyarekar wrote: > Hi, > > I have now rebased this patch against the current master. interp.c > does not need the CPPFLAGS anymore, so sotruss-lib is the only code to > be built as extramodules to eventually replace the NOT_IN_libc macro. > > Build tested on x86_64 to ensure that the generated code is unchanged. > > Siddhesh > > * elf/Makefile (libof-sotruss-lib): Set as extramodules. > > diff --git a/elf/Makefile b/elf/Makefile > index 677635b..7a5a9aa 100644 > --- a/elf/Makefile > +++ b/elf/Makefile > @@ -96,6 +96,7 @@ extra-objs += sotruss-lib.os sotruss-lib.so > install-others += $(inst_auditdir)/sotruss-lib.so > install-bin-script += sotruss > generated += sotruss > +libof-sotruss-lib = extramodules > CPPFLAGS-sotruss-lib = -DNOT_IN_libc > $(objpfx)sotruss-lib.so: $(objpfx)sotruss-lib.os > $(build-module-asneeded)
Rebased against current upstream and patch is unchanged. Ping! Siddhesh On Wed, Sep 24, 2014 at 02:37:31PM +0530, Siddhesh Poyarekar wrote: > Ping! > > On Wed, Sep 17, 2014 at 05:59:40PM +0530, Siddhesh Poyarekar wrote: > > Hi, > > > > I have now rebased this patch against the current master. interp.c > > does not need the CPPFLAGS anymore, so sotruss-lib is the only code to > > be built as extramodules to eventually replace the NOT_IN_libc macro. > > > > Build tested on x86_64 to ensure that the generated code is unchanged. > > > > Siddhesh > > > > * elf/Makefile (libof-sotruss-lib): Set as extramodules. > > > > diff --git a/elf/Makefile b/elf/Makefile > > index 677635b..7a5a9aa 100644 > > --- a/elf/Makefile > > +++ b/elf/Makefile > > @@ -96,6 +96,7 @@ extra-objs += sotruss-lib.os sotruss-lib.so > > install-others += $(inst_auditdir)/sotruss-lib.so > > install-bin-script += sotruss > > generated += sotruss > > +libof-sotruss-lib = extramodules > > CPPFLAGS-sotruss-lib = -DNOT_IN_libc > > $(objpfx)sotruss-lib.so: $(objpfx)sotruss-lib.os > > $(build-module-asneeded) > >
Ping! On Wed, Oct 01, 2014 at 01:52:54PM +0530, Siddhesh Poyarekar wrote: > Rebased against current upstream and patch is unchanged. > > Ping! > > Siddhesh > > On Wed, Sep 24, 2014 at 02:37:31PM +0530, Siddhesh Poyarekar wrote: > > Ping! > > > > On Wed, Sep 17, 2014 at 05:59:40PM +0530, Siddhesh Poyarekar wrote: > > > Hi, > > > > > > I have now rebased this patch against the current master. interp.c > > > does not need the CPPFLAGS anymore, so sotruss-lib is the only code to > > > be built as extramodules to eventually replace the NOT_IN_libc macro. > > > > > > Build tested on x86_64 to ensure that the generated code is unchanged. > > > > > > Siddhesh > > > > > > * elf/Makefile (libof-sotruss-lib): Set as extramodules. > > > > > > diff --git a/elf/Makefile b/elf/Makefile > > > index 677635b..7a5a9aa 100644 > > > --- a/elf/Makefile > > > +++ b/elf/Makefile > > > @@ -96,6 +96,7 @@ extra-objs += sotruss-lib.os sotruss-lib.so > > > install-others += $(inst_auditdir)/sotruss-lib.so > > > install-bin-script += sotruss > > > generated += sotruss > > > +libof-sotruss-lib = extramodules > > > CPPFLAGS-sotruss-lib = -DNOT_IN_libc > > > $(objpfx)sotruss-lib.so: $(objpfx)sotruss-lib.os > > > $(build-module-asneeded) > > > > > >
OK
diff --git a/elf/Makefile b/elf/Makefile index 677635b..7a5a9aa 100644 --- a/elf/Makefile +++ b/elf/Makefile @@ -96,6 +96,7 @@ extra-objs += sotruss-lib.os sotruss-lib.so install-others += $(inst_auditdir)/sotruss-lib.so install-bin-script += sotruss generated += sotruss +libof-sotruss-lib = extramodules CPPFLAGS-sotruss-lib = -DNOT_IN_libc $(objpfx)sotruss-lib.so: $(objpfx)sotruss-lib.os $(build-module-asneeded)