All of lore.kernel.org
 help / color / mirror / Atom feed
From: tip-bot for Catalin Marinas <catalin.marinas@arm.com>
To: linux-tip-commits@vger.kernel.org
Cc: linux-kernel@vger.kernel.org, hpa@zytor.com, mingo@redhat.com,
	bug-track@fisher-privat.net, penberg@cs.helsinki.fi,
	sam@ravnborg.org, catalin.marinas@arm.com, tglx@linutronix.de,
	mingo@elte.hu
Subject: [tip:x86/urgent] x86: Include all of .data.* sections in _edata on 64-bit
Date: Sat, 18 Jul 2009 12:06:11 GMT	[thread overview]
Message-ID: <tip-8bcdbe427924a1e4b4e4cf68020e92e9f93fe011@git.kernel.org> (raw)
In-Reply-To: <1247565175.28240.37.camel@pc1117.cambridge.arm.com>

Commit-ID:  8bcdbe427924a1e4b4e4cf68020e92e9f93fe011
Gitweb:     http://git.kernel.org/tip/8bcdbe427924a1e4b4e4cf68020e92e9f93fe011
Author:     Catalin Marinas <catalin.marinas@arm.com>
AuthorDate: Tue, 14 Jul 2009 10:52:55 +0100
Committer:  Ingo Molnar <mingo@elte.hu>
CommitDate: Sat, 18 Jul 2009 13:59:20 +0200

x86: Include all of .data.* sections in _edata on 64-bit

The .data.read_mostly and .data.cacheline_aligned sections
aren't covered by the _sdata .. _edata range on x86-64. This
affects kmemleak reporting leading to possible false
positives by not scanning the whole data section.

Signed-off-by: Catalin Marinas <catalin.marinas@arm.com>
Tested-by: Alexey Fisher <bug-track@fisher-privat.net>
Acked-by: Sam Ravnborg <sam@ravnborg.org>
Cc: Pekka Enberg <penberg@cs.helsinki.fi>
LKML-Reference: <1247565175.28240.37.camel@pc1117.cambridge.arm.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Cc: Sam Ravnborg <sam@ravnborg.org>


---
 arch/x86/kernel/vmlinux.lds.S |    7 -------
 1 files changed, 0 insertions(+), 7 deletions(-)

diff --git a/arch/x86/kernel/vmlinux.lds.S b/arch/x86/kernel/vmlinux.lds.S
index 367e878..59f31d2 100644
--- a/arch/x86/kernel/vmlinux.lds.S
+++ b/arch/x86/kernel/vmlinux.lds.S
@@ -112,11 +112,6 @@ SECTIONS
 		_sdata = .;
 		DATA_DATA
 		CONSTRUCTORS
-
-#ifdef CONFIG_X86_64
-		/* End of data section */
-		_edata = .;
-#endif
 	} :data
 
 #ifdef CONFIG_X86_32
@@ -156,10 +151,8 @@ SECTIONS
 	.data.read_mostly : AT(ADDR(.data.read_mostly) - LOAD_OFFSET) {
 		*(.data.read_mostly)
 
-#ifdef CONFIG_X86_32
 		/* End of data section */
 		_edata = .;
-#endif
 	}
 
 #ifdef CONFIG_X86_64

  parent reply	other threads:[~2009-07-18 12:07 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-14  6:08 60 memory leaks.. or is it some thing wrong with kmemleak? Alexey Fisher
2009-07-14  6:08 ` Alexey Fisher
     [not found] ` <4A5C20E5.6010203-M18mAb7Tlt0yCq4wW13eYl6hYfS7NtTn@public.gmane.org>
2009-07-14  7:19   ` Pekka Enberg
2009-07-14  7:19     ` Pekka Enberg
     [not found]     ` <84144f020907140019g511723dctb541f6333d1a082b-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2009-07-14  8:28       ` Alexey Fisher
2009-07-14  8:28         ` Alexey Fisher
     [not found]         ` <4A5C41C8.7040904-M18mAb7Tlt0yCq4wW13eYl6hYfS7NtTn@public.gmane.org>
2009-07-14  9:39           ` Catalin Marinas
2009-07-14  9:39             ` Catalin Marinas
     [not found]             ` <1247564356.28240.30.camel-hhZApKj8DF/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2009-07-14  9:52               ` [PATCH] x86: _edata should include all .data.* sections on X86_64 Catalin Marinas
2009-07-14  9:52                 ` Catalin Marinas
     [not found]                 ` <1247565175.28240.37.camel-hhZApKj8DF/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2009-07-14 10:13                   ` Alexey Fisher
2009-07-14 10:13                     ` Alexey Fisher
     [not found]                     ` <4A5C5A59.5080304-M18mAb7Tlt0yCq4wW13eYl6hYfS7NtTn@public.gmane.org>
2009-07-14 10:31                       ` Catalin Marinas
2009-07-14 10:31                         ` Catalin Marinas
     [not found]                         ` <1247567499.28240.48.camel-hhZApKj8DF/YkXV2EHHjLW3o5bpOHsLO@public.gmane.org>
2009-07-14 10:37                           ` Alexey Fisher
2009-07-14 10:37                             ` Alexey Fisher
     [not found]                             ` <4A5C5FD0.3020204-M18mAb7Tlt0yCq4wW13eYl6hYfS7NtTn@public.gmane.org>
2009-07-14 12:26                               ` ext4 memory leak (was Re: [PATCH] x86: _edata should include all .data.* sections on X86_64) Catalin Marinas
2009-07-14 12:26                                 ` Catalin Marinas
2009-07-15  8:03                                 ` Aneesh Kumar K.V
2009-07-15  8:03                                   ` Aneesh Kumar K.V
2009-07-15  8:54                                   ` Alexey Fisher
2009-07-15  8:54                                     ` Alexey Fisher
     [not found]                                     ` <4A5D9939.3000500-M18mAb7Tlt0yCq4wW13eYl6hYfS7NtTn@public.gmane.org>
2009-07-18 11:55                                       ` Ingo Molnar
2009-07-18 11:55                                         ` Ingo Molnar
     [not found]                                         ` <20090718115556.GA31007-X9Un+BFzKDI@public.gmane.org>
2009-07-18 13:30                                           ` Alexey Fisher
2009-07-18 13:30                                             ` Alexey Fisher
     [not found]                                             ` <4A61CE59.3030905-M18mAb7Tlt0yCq4wW13eYl6hYfS7NtTn@public.gmane.org>
2009-07-18 22:44                                               ` Catalin Marinas
2009-07-18 22:44                                                 ` Catalin Marinas
2009-07-18 22:33                                         ` Catalin Marinas
2009-07-15 10:33                                   ` Catalin Marinas
2009-07-14 12:42                               ` [PATCH] x86: _edata should include all .data.* sections on X86_64 Catalin Marinas
2009-07-14 12:42                                 ` Catalin Marinas
2009-07-16 20:23                   ` Sam Ravnborg
2009-07-16 20:23                     ` Sam Ravnborg
2009-07-18 12:06                 ` tip-bot for Catalin Marinas [this message]
2009-07-18 20:29               ` [kmemleak] 60 wornings on sysfs_new_dirent+0x10c Alexey Fisher
2009-07-18 20:29                 ` Alexey Fisher

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=tip-8bcdbe427924a1e4b4e4cf68020e92e9f93fe011@git.kernel.org \
    --to=catalin.marinas@arm.com \
    --cc=bug-track@fisher-privat.net \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-tip-commits@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=mingo@redhat.com \
    --cc=penberg@cs.helsinki.fi \
    --cc=sam@ravnborg.org \
    --cc=tglx@linutronix.de \
    /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.