All of lore.kernel.org
 help / color / mirror / Atom feed
From: Frediano Ziglio <frediano.ziglio@citrix.com>
To: Konrad Rzeszutek Wilk <konrad@kernel.org>,
	"Keir (Xen.org)" <keir@xen.org>,
	Ian Campbell <Ian.Campbell@citrix.com>,
	Jan Beulich <JBeulich@suse.com>,
	George Dunlap <George.Dunlap@eu.citrix.com>
Cc: Frediano Ziglio <frediano.ziglio@citrix.com>, xen-devel@lists.xen.org
Subject: [PATCH 4/5] Check no constructor section
Date: Wed, 6 Feb 2013 14:32:59 +0000	[thread overview]
Message-ID: <1360161180-3448-5-git-send-email-frediano.ziglio@citrix.com> (raw)
In-Reply-To: <1360161180-3448-1-git-send-email-frediano.ziglio@citrix.com>

Safety check for initialization objects.
Constructors are not used so check some module does not use them.

Signed-off-by: Frediano Ziglio <frediano.ziglio@citrix.com>
---
 xen/Rules.mk |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/Rules.mk b/xen/Rules.mk
index 3f0b262..ae35a08 100644
--- a/xen/Rules.mk
+++ b/xen/Rules.mk
@@ -166,7 +166,7 @@ SPECIAL_DATA_SECTIONS := rodata $(foreach n,1 2 4 8,rodata.str1.$(n)) \
 $(filter %.init.o,$(obj-y) $(obj-bin-y) $(extra-y)): %.init.o: %.o Makefile
 	$(OBJDUMP) -h $< | sed -n '/[0-9]/{s,00*,0,g;p}' | while read idx name sz rest; do \
 		case "$$name" in \
-		.text|.text.*|.data|.data.*|.bss) \
+		.text|.text.*|.data|.data.*|.bss|.ctors) \
 			test $$sz != 0 || continue; \
 			echo "Error: size of $<:$$name is 0x$$sz" >&2; \
 			exit $$(expr $$idx + 1);; \
-- 
1.7.9.5

  parent reply	other threads:[~2013-02-06 14:32 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-06 14:32 [PATCH v5] Coverage support Frediano Ziglio
2013-02-06 14:32 ` [PATCH 1/5] Call constructors during initialization Frediano Ziglio
2013-02-06 14:59   ` Jan Beulich
2013-02-06 14:32 ` [PATCH 2/5] Adding support for coverage information Frediano Ziglio
2013-02-06 14:32 ` [PATCH 3/5] Implement code to read coverage informations Frediano Ziglio
2013-02-06 17:12   ` Ian Campbell
2013-02-06 18:57     ` Frediano Ziglio
2013-02-06 14:32 ` Frediano Ziglio [this message]
2013-02-06 15:01   ` [PATCH 4/5] Check no constructor section Jan Beulich
2013-02-06 16:10     ` Frediano Ziglio
2013-02-06 16:04       ` George Dunlap
2013-02-06 14:33 ` [PATCH 5/5] Add small utility to deal with test coverage information from Xen Frediano Ziglio

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1360161180-3448-5-git-send-email-frediano.ziglio@citrix.com \
    --to=frediano.ziglio@citrix.com \
    --cc=George.Dunlap@eu.citrix.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=JBeulich@suse.com \
    --cc=keir@xen.org \
    --cc=konrad@kernel.org \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.