[prev in list] [next in list] [prev in thread] [next in thread]
List: linux-pci
Subject: Re: [PATCH v4 15/17] misc: pci_endpoint_test: Re-init completion for every test
From: Manivannan Sadhasivam <mani () kernel ! org>
Date: 2023-03-31 5:41:54
Message-ID: 20230331052954.GD4973 () thinkpad
[Download RAW message or body]
On Thu, Mar 30, 2023 at 05:53:55PM +0900, Damien Le Moal wrote:
> The irq_raised completion used to detect the end of a test case is
> initialized when the test device is probed, but never reinitialized
> again before a test case. As a result, the irq_raised completion
> synchronization is effective only for the first ioctl test case
> executed. Any subsequent call to wait_for_completion() by another
> ioctl() call will immediately return, potentially too early, leading to
> false positive failures.
>
> Fix this by reinitializing the irq_raised completion before starting a
> new ioctl() test command.
>
> Fixes: 2c156ac71c6b ("misc: Add host side PCI driver for PCI test function device")
> Cc: stable@vger.kernel.org
> Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
Reviewed-by: Manivannan Sadhasivam <mani@kernel.org>
- Mani
> ---
> drivers/misc/pci_endpoint_test.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/drivers/misc/pci_endpoint_test.c b/drivers/misc/pci_endpoint_test.c
> index 01235236e9bc..24efe3b88a1f 100644
> --- a/drivers/misc/pci_endpoint_test.c
> +++ b/drivers/misc/pci_endpoint_test.c
> @@ -729,6 +729,10 @@ static long pci_endpoint_test_ioctl(struct file *file, unsigned int cmd,
> struct pci_dev *pdev = test->pdev;
>
> mutex_lock(&test->mutex);
> +
> + reinit_completion(&test->irq_raised);
> + test->last_irq = -ENODATA;
> +
> switch (cmd) {
> case PCITEST_BAR:
> bar = arg;
> --
> 2.39.2
>
--
மணிவண்ணன் சதாசிவம்
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic