linux-kernel - Re: [PATCH v2 0/4] Extending the first version of the patch to the entire directory

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] [day] [month] [year] [list]

Message-ID: <20140924174201.GB20416@google.com>
Date:	Wed, 24 Sep 2014 11:42:01 -0600
From:	Bjorn Helgaas <bhelgaas@...gle.com>
To:	Quentin Lambert <lambert.quentin@...il.com>
Cc:	"Rafael J. Wysocki" <rjw@...ysocki.net>,
	Len Brown <lenb@...nel.org>, Scott Murray <scott@...teful.org>,
	linux-acpi@...r.kernel.org, linux-pci@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 0/4] Extending the first version of the patch to the
 entire directory

On Sun, Sep 07, 2014 at 08:00:23PM +0200, Quentin Lambert wrote:
> Patch 1 and 2 are straight forward extension from the previous version
> of the patch.
> 
> Patch 3 introduces checkpatch warning due to lines surpassing 80
> characters. Another warning concerning too many preceding tabs
> is not fixed either.
> 
> Patch 4 are less straight forward similar modifications
> the modification made to ibmphp_res.c change the value of len_tmp
> when the first condition in if statement is not met.
> I wasn't quite sure how to solve elegantly the checkpatch error for
> slot.c, I chose to use a label and gotos since it was already used
> and allowed for a slight code factorisation. This being my first patch I
> understand that this may not be the best solution.

Thanks for splitting out patch 4 from 3; it definitely made it easier to
review.

I dropped a couple bits from patch 4 that I'll mention in a reply to that patch.

I applied the rest to pci/cleanup and plan to merge it toward the end of
the v3.18 merge window, thanks!

Bjorn

> Quentin Lambert (4):
>   PCI: fix a missing space coding style issue
>   PCI: fix a simple if coding style issue
>   PCI: remove assignement from if conditions
>   PCI: remove assignement in non straight forward if condition
> 
>  drivers/pci/hotplug/acpi_pcihp.c        |  3 ++-
>  drivers/pci/hotplug/acpiphp_ibm.c       |  2 +-
>  drivers/pci/hotplug/cpci_hotplug_core.c | 13 ++++++----
>  drivers/pci/hotplug/cpcihp_generic.c    | 28 ++++++++++-----------
>  drivers/pci/hotplug/cpcihp_zt5550.c     | 44 ++++++++++++++++-----------------
>  drivers/pci/hotplug/cpqphp.h            |  2 +-
>  drivers/pci/hotplug/cpqphp_core.c       |  3 +--
>  drivers/pci/hotplug/cpqphp_ctrl.c       | 19 ++++++--------
>  drivers/pci/hotplug/cpqphp_nvram.c      | 13 ++++------
>  drivers/pci/hotplug/ibmphp_core.c       | 19 ++++++++------
>  drivers/pci/hotplug/ibmphp_ebda.c       |  3 +--
>  drivers/pci/hotplug/ibmphp_hpc.c        |  3 +--
>  drivers/pci/hotplug/ibmphp_pci.c        |  6 +++--
>  drivers/pci/hotplug/ibmphp_res.c        | 43 ++++++++++++++++++++++----------
>  drivers/pci/hotplug/shpchp_ctrl.c       | 12 ++++++---
>  drivers/pci/hotplug/shpchp_hpc.c        |  3 ++-
>  drivers/pci/pci.c                       | 19 +++++++++++---
>  drivers/pci/slot.c                      | 12 +++++++--
>  18 files changed, 143 insertions(+), 104 deletions(-)
> 
> -- 
> 1.9.1
> 
--
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/