* [PATCH V2] x86/PCI: MMCONFIG: cleanup and add address warning to pci_mmconfig_insert
@ 2013-07-27 19:32 ethan.kernel
2013-07-28 3:09 ` Mark D Rustad
0 siblings, 1 reply; 3+ messages in thread
From: ethan.kernel @ 2013-07-27 19:32 UTC (permalink / raw)
To: bhelgaas, yinghai, jbarnes
Cc: linux-kernel, jiang.liu, linux-pci, joe.jin, ethan.zhao
Cleanup the -EINVAL return value handling and add warning message for invalid
start,end,addr parameters.
V2: Corrected code style and tested for Linux v 3.11-rc2
Signed-off-by: ethan.zhao <ethan.zhao@oracle.com>
---
arch/x86/pci/mmconfig-shared.c | 12 ++++++------
1 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/arch/x86/pci/mmconfig-shared.c b/arch/x86/pci/mmconfig-shared.c
index 082e881..37f6c7f 100644
--- a/arch/x86/pci/mmconfig-shared.c
+++ b/arch/x86/pci/mmconfig-shared.c
@@ -700,8 +700,13 @@ int pci_mmconfig_insert(struct device *dev, u16 seg, u8 start, u8 end,
if (!(pci_probe & PCI_PROBE_MMCONF) || pci_mmcfg_arch_init_failed)
return -ENODEV;
- if (start > end)
+ if (start > end || !addr) {
+ dev_warn(dev, FW_INFO
+ "Invalid address to add MMCONFIG"
+ "start %02x end %02x addr %pR\n",
+ start, end, addr);
return -EINVAL;
+ }
mutex_lock(&pci_mmcfg_lock);
cfg = pci_mmconfig_lookup(seg, start);
@@ -716,11 +721,6 @@ int pci_mmconfig_insert(struct device *dev, u16 seg, u8 start, u8 end,
return -EEXIST;
}
- if (!addr) {
- mutex_unlock(&pci_mmcfg_lock);
- return -EINVAL;
- }
-
rc = -EBUSY;
cfg = pci_mmconfig_alloc(seg, start, end, addr);
if (cfg == NULL) {
--
1.7.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH V2] x86/PCI: MMCONFIG: cleanup and add address warning to pci_mmconfig_insert
2013-07-27 19:32 [PATCH V2] x86/PCI: MMCONFIG: cleanup and add address warning to pci_mmconfig_insert ethan.kernel
@ 2013-07-28 3:09 ` Mark D Rustad
2013-07-28 3:29 ` ethan
0 siblings, 1 reply; 3+ messages in thread
From: Mark D Rustad @ 2013-07-28 3:09 UTC (permalink / raw)
To: ethan.kernel
Cc: bhelgaas, yinghai, jbarnes, linux-kernel, jiang.liu, linux-pci,
joe.jin, ethan.zhao
Another nit below:
On Jul 27, 2013, at 12:32 PM, ethan.kernel@gmail.com wrote:
> Cleanup the -EINVAL return value handling and add warning message for invalid
> start,end,addr parameters.
>
> V2: Corrected code style and tested for Linux v 3.11-rc2
>
> Signed-off-by: ethan.zhao <ethan.zhao@oracle.com>
> ---
> arch/x86/pci/mmconfig-shared.c | 12 ++++++------
> 1 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/arch/x86/pci/mmconfig-shared.c b/arch/x86/pci/mmconfig-shared.c
> index 082e881..37f6c7f 100644
> --- a/arch/x86/pci/mmconfig-shared.c
> +++ b/arch/x86/pci/mmconfig-shared.c
> @@ -700,8 +700,13 @@ int pci_mmconfig_insert(struct device *dev, u16 seg, u8 start, u8 end,
> if (!(pci_probe & PCI_PROBE_MMCONF) || pci_mmcfg_arch_init_failed)
> return -ENODEV;
>
> - if (start > end)
> + if (start > end || !addr) {
> + dev_warn(dev, FW_INFO
> + "Invalid address to add MMCONFIG"
> + "start %02x end %02x addr %pR\n",
As it is here, there will be no space between MMCONFIG and the word start. Better still, put the entire string on one line to aid those searching for the message. There is a general exception to the 80-column rule for user-visible messages such as this.
<snip>
--
Mark Rustad, MRustad@gmail.com
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH V2] x86/PCI: MMCONFIG: cleanup and add address warning to pci_mmconfig_insert
2013-07-28 3:09 ` Mark D Rustad
@ 2013-07-28 3:29 ` ethan
0 siblings, 0 replies; 3+ messages in thread
From: ethan @ 2013-07-28 3:29 UTC (permalink / raw)
To: Mark D Rustad
Cc: bhelgaas@google.com, yinghai@kernel.org, jbarnes@virtuousgeek.org,
linux-kernel@vger.kernel.org, jiang.liu@huawei.com,
linux-pci@vger.kernel.org, joe.jin@oracle.com, ethan.zhao
thanks，got it ,you are right.
在 2013-7-28，11:09，Mark D Rustad <mrustad@gmail.com> 写道：
> Another nit below:
>
> On Jul 27, 2013, at 12:32 PM, ethan.kernel@gmail.com wrote:
>
>> Cleanup the -EINVAL return value handling and add warning message for invalid
>> start,end,addr parameters.
>>
>> V2: Corrected code style and tested for Linux v 3.11-rc2
>>
>> Signed-off-by: ethan.zhao <ethan.zhao@oracle.com>
>> ---
>> arch/x86/pci/mmconfig-shared.c | 12 ++++++------
>> 1 files changed, 6 insertions(+), 6 deletions(-)
>>
>> diff --git a/arch/x86/pci/mmconfig-shared.c b/arch/x86/pci/mmconfig-shared.c
>> index 082e881..37f6c7f 100644
>> --- a/arch/x86/pci/mmconfig-shared.c
>> +++ b/arch/x86/pci/mmconfig-shared.c
>> @@ -700,8 +700,13 @@ int pci_mmconfig_insert(struct device *dev, u16 seg, u8 start, u8 end,
>> if (!(pci_probe & PCI_PROBE_MMCONF) || pci_mmcfg_arch_init_failed)
>> return -ENODEV;
>>
>> - if (start > end)
>> + if (start > end || !addr) {
>> + dev_warn(dev, FW_INFO
>> + "Invalid address to add MMCONFIG"
>> + "start %02x end %02x addr %pR\n",
>
> As it is here, there will be no space between MMCONFIG and the word start. Better still, put the entire string on one line to aid those searching for the message. There is a general exception to the 80-column rule for user-visible messages such as this.
>
> <snip>
>
> --
> Mark Rustad, MRustad@gmail.com
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-07-28 3:29 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-07-27 19:32 [PATCH V2] x86/PCI: MMCONFIG: cleanup and add address warning to pci_mmconfig_insert ethan.kernel
2013-07-28 3:09 ` Mark D Rustad
2013-07-28 3:29 ` ethan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox