| 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
| ||
|
Message-ID: <1760e154-afa9-7013-c756-8e255e89c8d3@synopsys.com>
Date: Wed, 18 Jul 2018 10:09:22 +0100
From: Gustavo Pimentel <Gustavo.Pimentel@...opsys.com>
To: Julia Lawall <julia.lawall@...6.fr>,
Gustavo Pimentel <gustavo.pimentel@...opsys.com>
CC: "bhelgaas@...gle.com" <bhelgaas@...gle.com>,
"lorenzo.pieralisi@....com" <lorenzo.pieralisi@....com>,
"joao.pinto@...opsys.com" <joao.pinto@...opsys.com>,
"jingoohan1@...il.com" <jingoohan1@...il.com>,
"kishon@...com" <kishon@...com>,
"adouglas@...ence.com" <adouglas@...ence.com>,
"jesper.nilsson@...s.com" <jesper.nilsson@...s.com>,
"shawn.lin@...k-chips.com" <shawn.lin@...k-chips.com>,
"linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
"linux-doc@...r.kernel.org" <linux-doc@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"kbuild-all@...org" <kbuild-all@...org>
Subject: Re: [PATCH v12 07/12] pci-epf-test/pci_endpoint_test: Cleanup
PCI_ENDPOINT_TEST memspace (fwd)
Hi Julia,
Thanks for the review.
You're correct, but I'd prefer to change the variable count from u16 to int
type. I know the pci_epc_get_msi() currently doesn't return any negative values,
but since the function signature allows the return of negative values, I think
it'd be good to be prepared for such a future possibility.
Regards,
Gustavo
On 17/07/2018 19:59, Julia Lawall wrote:
> The following code contains the following lines:
>
> 045c759f Gustavo Pimentel 2018-07-17 340 count = pci_epc_get_msi(epc, epf->func_no);
> 045c759f Gustavo Pimentel 2018-07-17 @341 if (reg->irq_number > count || count <= 0)
>
> count has type u16, so it cannot be < 0. Furthermore, pci_epc_get_msi
> never returns a negative value, so the test for < 0 is unnecessary.
>
> julia
>
> ---------- Forwarded message ----------
> Date: Wed, 18 Jul 2018 02:50:45 +0800
> From: kbuild test robot <lkp@...el.com>
> To: kbuild@...org
> Cc: Julia Lawall <julia.lawall@...6.fr>
> Subject: Re: [PATCH v12 07/12] pci-epf-test/pci_endpoint_test: Cleanup
> PCI_ENDPOINT_TEST memspace
>
> CC: kbuild-all@...org
> In-Reply-To: <26fc54d264ca4229360cf632f08e6935b1abb30c.1531818979.git.gustavo.pimentel@...opsys.com>
> References: <26fc54d264ca4229360cf632f08e6935b1abb30c.1531818979.git.gustavo.pimentel@...opsys.com>
> TO: Gustavo Pimentel <gustavo.pimentel@...opsys.com>
> CC: bhelgaas@...gle.com, lorenzo.pieralisi@....com, joao.pinto@...opsys.com, jingoohan1@...il.com, kishon@...com, adouglas@...ence.com, jesper.nilsson@...s.com, shawn.lin@...k-chips.com
> CC: linux-pci@...r.kernel.org, linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org, Gustavo Pimentel <gustavo.pimentel@...opsys.com>
>
> Hi Gustavo,
>
> Thank you for the patch! Perhaps something to improve:
>
> [auto build test WARNING on pci/next]
> [also build test WARNING on v4.18-rc5 next-20180717]
> [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
>
> url: https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_0day-2Dci_linux_commits_Gustavo-2DPimentel_Add-2DMSI-2DX-2Dsupport-2Don-2Dpcitest-2Dtool_20180717-2D233319&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=bkWxpLoW-f-E3EdiDCCa0_h0PicsViasSlvIpzZvPxs&m=ju28bmySEdjoY0RxIN6ae4986tlIzESmT3TcyqLjcSs&s=tJOfNk_YHxRjCj6t3MWt_0PqaBUYpxcqofxQ4TNEKO8&e=
> base: https://urldefense.proofpoint.com/v2/url?u=https-3A__git.kernel.org_pub_scm_linux_kernel_git_helgaas_pci.git&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=bkWxpLoW-f-E3EdiDCCa0_h0PicsViasSlvIpzZvPxs&m=ju28bmySEdjoY0RxIN6ae4986tlIzESmT3TcyqLjcSs&s=iglvG6xPgtxkXhIKK8J49nR3kIwcDHQwvr60gdJjFc4&e= next
> :::::: branch date: 3 hours ago
> :::::: commit date: 3 hours ago
>
>>> drivers/pci/endpoint/functions/pci-epf-test.c:341:33-38: WARNING: Unsigned expression compared with zero: count <= 0
>
> # https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_0day-2Dci_linux_commit_045c759f7ad4eeb208d74b476af11c8bba90e80e&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=bkWxpLoW-f-E3EdiDCCa0_h0PicsViasSlvIpzZvPxs&m=ju28bmySEdjoY0RxIN6ae4986tlIzESmT3TcyqLjcSs&s=mJbi04745s0pc9UI89wYjCXD9nlYdzL4yfClFeU2UgU&e=
> git remote add linux-review https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_0day-2Dci_linux&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=bkWxpLoW-f-E3EdiDCCa0_h0PicsViasSlvIpzZvPxs&m=ju28bmySEdjoY0RxIN6ae4986tlIzESmT3TcyqLjcSs&s=0HOVHucs3OqPvQI__Fs54JiTqpI660FDxgcSmgEv3I0&e=
> git remote update linux-review
> git checkout 045c759f7ad4eeb208d74b476af11c8bba90e80e
> vim +341 drivers/pci/endpoint/functions/pci-epf-test.c
>
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 274
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 275 static void pci_epf_test_cmd_handler(struct work_struct *work)
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 276 {
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 277 int ret;
> 045c759f Gustavo Pimentel 2018-07-17 278 u16 count;
> 3ecf3232 Kishon Vijay Abraham I 2017-08-18 279 u32 command;
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 280 struct pci_epf_test *epf_test = container_of(work, struct pci_epf_test,
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 281 cmd_handler.work);
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 282 struct pci_epf *epf = epf_test->epf;
> 045c759f Gustavo Pimentel 2018-07-17 283 struct device *dev = &epf->dev;
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 284 struct pci_epc *epc = epf->epc;
> 3235b994 Kishon Vijay Abraham I 2017-08-18 285 enum pci_barno test_reg_bar = epf_test->test_reg_bar;
> 3235b994 Kishon Vijay Abraham I 2017-08-18 286 struct pci_epf_test_reg *reg = epf_test->reg[test_reg_bar];
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 287
> 3ecf3232 Kishon Vijay Abraham I 2017-08-18 288 command = reg->command;
> 3ecf3232 Kishon Vijay Abraham I 2017-08-18 289 if (!command)
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 290 goto reset_handler;
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 291
> 3ecf3232 Kishon Vijay Abraham I 2017-08-18 292 reg->command = 0;
> 3235b994 Kishon Vijay Abraham I 2017-08-18 293 reg->status = 0;
> 3ecf3232 Kishon Vijay Abraham I 2017-08-18 294
> 045c759f Gustavo Pimentel 2018-07-17 295 if (reg->irq_type > IRQ_TYPE_MSI) {
> 045c759f Gustavo Pimentel 2018-07-17 296 dev_err(dev, "Failed to detect IRQ type\n");
> 045c759f Gustavo Pimentel 2018-07-17 297 goto reset_handler;
> 045c759f Gustavo Pimentel 2018-07-17 298 }
> 749aaf33 John Keeping 2017-09-20 299
> 3ecf3232 Kishon Vijay Abraham I 2017-08-18 300 if (command & COMMAND_RAISE_LEGACY_IRQ) {
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 301 reg->status = STATUS_IRQ_RAISED;
> 4494738d Cyrille Pitchen 2018-01-30 302 pci_epc_raise_irq(epc, epf->func_no, PCI_EPC_IRQ_LEGACY, 0);
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 303 goto reset_handler;
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 304 }
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 305
> 3ecf3232 Kishon Vijay Abraham I 2017-08-18 306 if (command & COMMAND_WRITE) {
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 307 ret = pci_epf_test_write(epf_test);
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 308 if (ret)
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 309 reg->status |= STATUS_WRITE_FAIL;
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 310 else
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 311 reg->status |= STATUS_WRITE_SUCCESS;
> 045c759f Gustavo Pimentel 2018-07-17 312 pci_epf_test_raise_irq(epf_test, reg->irq_type,
> 045c759f Gustavo Pimentel 2018-07-17 313 reg->irq_number);
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 314 goto reset_handler;
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 315 }
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 316
> 3ecf3232 Kishon Vijay Abraham I 2017-08-18 317 if (command & COMMAND_READ) {
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 318 ret = pci_epf_test_read(epf_test);
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 319 if (!ret)
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 320 reg->status |= STATUS_READ_SUCCESS;
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 321 else
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 322 reg->status |= STATUS_READ_FAIL;
> 045c759f Gustavo Pimentel 2018-07-17 323 pci_epf_test_raise_irq(epf_test, reg->irq_type,
> 045c759f Gustavo Pimentel 2018-07-17 324 reg->irq_number);
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 325 goto reset_handler;
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 326 }
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 327
> 3ecf3232 Kishon Vijay Abraham I 2017-08-18 328 if (command & COMMAND_COPY) {
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 329 ret = pci_epf_test_copy(epf_test);
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 330 if (!ret)
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 331 reg->status |= STATUS_COPY_SUCCESS;
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 332 else
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 333 reg->status |= STATUS_COPY_FAIL;
> 045c759f Gustavo Pimentel 2018-07-17 334 pci_epf_test_raise_irq(epf_test, reg->irq_type,
> 045c759f Gustavo Pimentel 2018-07-17 335 reg->irq_number);
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 336 goto reset_handler;
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 337 }
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 338
> 3ecf3232 Kishon Vijay Abraham I 2017-08-18 339 if (command & COMMAND_RAISE_MSI_IRQ) {
> 045c759f Gustavo Pimentel 2018-07-17 340 count = pci_epc_get_msi(epc, epf->func_no);
> 045c759f Gustavo Pimentel 2018-07-17 @341 if (reg->irq_number > count || count <= 0)
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 342 goto reset_handler;
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 343 reg->status = STATUS_IRQ_RAISED;
> 045c759f Gustavo Pimentel 2018-07-17 344 pci_epc_raise_irq(epc, epf->func_no, PCI_EPC_IRQ_MSI,
> 045c759f Gustavo Pimentel 2018-07-17 345 reg->irq_number);
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 346 goto reset_handler;
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 347 }
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 348
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 349 reset_handler:
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 350 queue_delayed_work(kpcitest_workqueue, &epf_test->cmd_handler,
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 351 msecs_to_jiffies(1));
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 352 }
> 349e7a85 Kishon Vijay Abraham I 2017-03-27 353
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.01.org_pipermail_kbuild-2Dall&d=DwIBAg&c=DPL6_X_6JkXFx7AXWqB0tg&r=bkWxpLoW-f-E3EdiDCCa0_h0PicsViasSlvIpzZvPxs&m=ju28bmySEdjoY0RxIN6ae4986tlIzESmT3TcyqLjcSs&s=6qrmeltWerJZCA5GyZEp0LuxfhZa1jpgfZdQflb0mPE&e= Intel Corporation
>
Powered by blists - more mailing lists