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  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:   Sun, 12 Feb 2017 11:42:56 +0100 (CET)
From:   Thomas Gleixner <tglx@...utronix.de>
To:     Kishore Karanala <kishore.karanala@...il.com>
cc:     Ingo Molnar <mingo@...nel.org>, "H. Peter Anvin" <hpa@...or.com>,
        x86@...nel.org, LKML <linux-kernel@...r.kernel.org>,
        matt@...eblueprint.co.uk, rafael.j.wysocki@...el.com,
        thgarnie@...gle.com, arbab@...ux.vnet.ibm.com,
        dave.hansen@...ux.intel.com
Subject: Re: [PATCH 2/2] Staging: setup.c: fixed ERROR: trailing whitespace

On Sun, 12 Feb 2017, Kishore Karanala wrote:

You're not listening at all. I told you before:

       The proper subsystem for x86 is x86 and not staging.

You can chose to ignore my review comments, but then you don't have to
wonder that I'm going to ignore you as well.

Aside of that, you got at least TWO messages from your mail server, that
mail to 'glx@...utronix.de' cannot be delivered, but that seems to be
information to ignore as well.

> Fixed ERROR: trailing whitespace issues

What's the point of this?

You used checkpatch.pl on that file. Can you figure out from the name of
that script what's the purpose of that script?

The purpose is obviously to check patches for correctness. The fact that
you can run checkpatch.pl with the -f commandline option on an existing
file does not change that.

And trailing whitespace is the least of our worries, really. 

Thanks,

	tglx

Powered by blists - more mailing lists