linux-kernel - Re: [PATCH] Fixup KERN

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]

Message-ID: <Pine.LNX.4.64.0808152202440.7852@axis700.grange>
Date:	Fri, 15 Aug 2008 22:17:57 +0200 (CEST)
From:	Guennadi Liakhovetski <g.liakhovetski@....de>
To:	Nick Warne <nick@...sn.org>
cc:	linux-kernel@...r.kernel.org, linux-scsi@...r.kernel.org
Subject: Re: [PATCH] Fixup KERN_INFO in tmscsim.c

On Fri, 15 Aug 2008, Nick Warne wrote:

> Setting up my new AMD64 I noticed a curious one liner in syslogs:

Wow, are you sure you need the tmscsim driver on your new amd64 system?:-)

> ^Iwith "disable_clustering=1" and report to maintainers

The message priority is indeed missing in the second line, but it has 
nothing to do with the "^I" in your log. I do not know why your syslogd / 
klogd converts a TAB to "^I". But if it is a problem, I think, the correct 
fix would be something like

-		printk(KERN_INFO "DC390: clustering now enabled by default. If you get problems load\n"
-		       "\twith \"disable_clustering=1\" and report to maintainers\n");
+		printk(KERN_INFO "DC390: clustering now enabled by default. If you get problems load\n");
+		printk(KERN_INFO "       with \"disable_clustering=1\" and report to maintainers\n");

Or maybe just remove that printk altogether - it has been there for almost 
2 years now, and I haven't heard a single complaint, but on the other 
hand, maybe you are the first, who at least loaded this driver since 2 
years:-)

Ok, taking into account that this driver is hardly used, I would keep this 
warning in case someone does get a problem.

So, please, redo as above, create with "-p1" format, fix your Sob line

> Signed off by: "Nick Warne" <nick@...sn.org>

which has to be

Signed-off-by: Nick Warne <nick@...sn.org>

and send the patch inline,

Thanks
Guennadi
---
Guennadi Liakhovetski
--
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/