linux-kernel - Re: [PATCH] Generic WorkQueue Engine (GenWQE) device driver v2

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:	Thu, 24 Oct 2013 11:09:42 +0100
From:	Greg KH <gregkh@...uxfoundation.org>
To:	Frank Haverkamp <haver@...ux.vnet.ibm.com>
Cc:	linux-kernel@...r.kernel.org, arnd@...db.de,
	cody@...ux.vnet.ibm.com, schwidefsky@...ibm.com,
	utz.bacher@...ibm.com, jsvogt@...ibm.com, MIJUNG@...ibm.com,
	cascardo@...ux.vnet.ibm.com, michael@...ra.de
Subject: Re: [PATCH] Generic WorkQueue Engine (GenWQE) device driver v2

On Wed, Oct 23, 2013 at 06:15:21PM +0200, Frank Haverkamp wrote:
> Rework comments:
>  o Removed __DATE__ macros as suggested by Michal Marek
>  o Removed startup pr_info() as suggested by Greg KH
> 
> Plus some little things we found during test and review:
>  o Removed some obsolete comments
>  o Fixed a bug in the initialization path when
>    genwqe_set_interrupt_capability() fails
> 
> Greg, I tried my luck now with git send-email. Please let
> me know if that works better.

That's one of the the worst commit message that I've ever seen, you
really want this as the thing that shows up in the git commit log?

I have no idea what this code is, what it does, nor what it should do.

Also, you are using sysfs, but have no Documetation/ABI/ modifications,
which isn't allowed.

I need a whole more information before I can even start to review the
code to try to determine if it's ok or not.

greg k-h
--
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/