linux-kernel - Re: Fwd: Commit 76a2a6ee8a0660a29127f05989ac59ae1ce865fa breaks PXA270 (at least)?

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 29 Jul 2008 15:46:05 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	Russell King - ARM Linux <linux@....linux.org.uk>
Cc:	a.p.zijlstra@...llo.nl, vatsa@...ux.vnet.ibm.com, efault@....de,
	mingo@...e.hu, linux-kernel@...r.kernel.org
Subject: Re: Fwd: Commit 76a2a6ee8a0660a29127f05989ac59ae1ce865fa breaks
 PXA270 (at least)?

On Tue, 29 Jul 2008 23:31:05 +0100
Russell King - ARM Linux <linux@....linux.org.uk> wrote:

> But then some bright spark thought it would be a good idea to get
> rid of printk_clock().

<does git-log, searches for printk_clock>

  commit 86faf39d0fc04272b05fab1db6d683f3ac7199d1
  Author: Ingo Molnar <mingo@...e.hu>
  Date:   Fri Jan 25 21:07:59 2008 +0100
  
      sched: remove printk_clock references from ia64
      
      remove remaining printk_clock references from ia64.
      
      Signed-off-by: Ingo Molnar <mingo@...e.hu>
  
  commit b842271fbb9c8b5fd0e1c3e1895a3b67ba5bcc54
  Author: Ingo Molnar <mingo@...e.hu>
  Date:   Fri Jan 25 21:07:59 2008 +0100
  
      sched: remove printk_clock()
      
      printk_clock() is obsolete - it has been replaced with cpu_clock().
      
      Signed-off-by: Ingo Molnar <mingo@...e.hu>
  

That's it?  It rates a 0.5/10 for changelogging :(

Looking further on...

  commit e97126cd9056b3b42cdc862ace2ed66f8026f55b
  Author: Russell King <rmk@...-67.arm.linux.org.uk>
  Date:   Mon Jan 8 19:49:12 2007 +0000
  
      [ARM] Provide basic printk_clock() implementation
      
      Current sched_clock() implementations on ARM cause unbootable kernels
      with PRINTK_TIME support enabled.  To avoid this, provide a basic
      printk_clock() implementation which avoids sched_clock() being called
      before the page tables have been set up.
  

which I assume is why arm is crashing again?


Really, I think arch-overrideable printk_clock() was a good idea. 
printk is just _special_.  It's called wildly early and it is called in
all conceivable contexts and it just must work no matter what.  It's
totally understandable that an architecture would need to override
printk's timestamp generator.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/