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
| ||
|
Date: Tue, 26 Jun 2007 14:16:39 +0200 From: Thomas Gleixner <tglx@...utronix.de> To: Andrew Morton <akpm@...ux-foundation.org> Cc: Andi Kleen <ak@...e.de>, Ingo Molnar <mingo@...e.hu>, Arjan van de Ven <arjan@...radead.org>, Venkatesh Pallipadi <venkatesh.pallipadi@...el.com>, John Stultz <johnstul@...ibm.com>, Chris Wright <chrisw@...s-sol.org>, LKML <linux-kernel@...r.kernel.org> Subject: Re: [patch -mm 10/28] highres: Improve debug output On Mon, 2007-06-25 at 17:14 -0700, Andrew Morton wrote: > On Sat, 23 Jun 2007 13:32:35 -0000 > Thomas Gleixner <tglx@...utronix.de> wrote: > > > if (!dev || !(dev->features & CLOCK_EVT_FEAT_ONESHOT) || > > - !tick_device_is_functional(dev)) > > + !tick_device_is_functional(dev)) { > > + > > + printk(KERN_INFO "Clockevents: " > > + "could not switch to one-shot mode:"); > > + if (!dev) { > > + printk(" no tick device\n"); > > + } else { > > + if (!tick_device_is_functional(dev)) > > + printk(" %s is not functional.\n", dev->name); > > + else if (!(dev->features & CLOCK_EVT_FEAT_ONESHOT)) > > + printk(" %s does not support one-shot mode.\n", > > + dev->name); > > + } > > There is a logic path through here where the printk doesn't get its \n > termination? And it will fail to print the reason for the failure, too. > > Maybe that's a can't-happen, in which case the CLOCK_EVT_FEAT_ONESHOT test > is superfluous? Right, the "else if (...)" test is bogus. A simple "else" is sufficient. tglx - 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/
Powered by blists - more mailing lists