Thread (29 messages) 29 messages, 6 authors, 2016-02-08
STALE3821d
Revisions (10)
  1. v1 [diff vs current]
  2. v1 [diff vs current]
  3. v1 [diff vs current]
  4. v1 [diff vs current]
  5. v2 [diff vs current]
  6. v1 [diff vs current]
  7. v1 [diff vs current]
  8. v1 current
  9. v3 [diff vs current]
  10. v4 [diff vs current]

[PATCH 0/3] kallsyms: remove special handling for CONFIG_ARM

From: Ard Biesheuvel <hidden>
Date: 2016-02-03 20:41:21

On 3 February 2016 at 21:30, Ard Biesheuvel [off-list ref] wrote:
On 3 February 2016 at 21:23, Nicolas Pitre [off-list ref] wrote:
quoted
On Wed, 3 Feb 2016, Ard Biesheuvel wrote:
quoted
On 3 February 2016 at 21:01, Russell King - ARM Linux
[off-list ref] wrote:
quoted
On Wed, Feb 03, 2016 at 02:41:29PM +0100, Ard Biesheuvel wrote:
quoted
quoted
bf353edc T __stubs_start
This is a global symbol
quoted
bf353edc T _etext
bf353ee0 t __stubs_start
This is a local symbol
Stop this madness.  This gets you an immediate NAK on the patch set.
What madness? This does not change at all with my patches, these
duplicates already exist in the current code.
That doesn't make it any better, and was cause for issue as Chris
demonstrated.  Can we get rid of that duplication please?
4/4 in my v2 removes the redundant symbols.
... actually, that only removes the redundant __stub_start, and the
redundant __vectors_start is already removed in patch #1. Patch #4
(v2) may be shot down for other reasons, in that case I can simply
move the removal of __stubs_start to patch #1 instead.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help