Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
To:     Kirill Smelkov <kirr@nexedi.com>,
        Julia Lawall <julia.lawall@lip6.fr>,
        Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Cc:     Kurt Schwemmer <kurt.schwemmer@microsemi.com>,
        Bjorn Helgaas <helgaas@kernel.org>, linux-pci@vger.kernel.org,
        linux-kernel@vger.kernel.org, kbuild-all@01.org
References: <alpine.DEB.2.20.1906191227430.3726@hadrien>
 <20190619162713.GA19859@deco.navytux.spb.ru>
From:   Logan Gunthorpe <logang@deltatee.com>
Message-ID: <8ece80ad-d3c9-ac7b-29a5-b8c6f40851af@deltatee.com>
Date:   Wed, 19 Jun 2019 13:47:35 -0600
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
 Thunderbird/60.7.1
MIME-Version: 1.0
In-Reply-To: <20190619162713.GA19859@deco.navytux.spb.ru>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Subject: Re: [PATCH] pci/switchtec: fix stream_open.cocci warnings (fwd)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


On 2019-06-19 10:27 a.m., Kirill Smelkov wrote:
> Hi Julia, everyone.
> 
> On Wed, Jun 19, 2019 at 12:28:47PM +0200, Julia Lawall wrote:
>> Hi,
>>
>> Can you forward this patch to the people below if you think it is
>> appropriate?

>> From: kbuild test robot <lkp@intel.com>
>>
>> drivers/pci/switch/switchtec.c:395:1-17: ERROR: switchtec_fops: .read() can deadlock .write(); change nonseekable_open -> stream_open to fix.
>>
>> Generated by: scripts/coccinelle/api/stream_open.cocci
>>
>> Fixes: a3a1e895d4fa ("pci/switchtec: Don't use completion's wait queue")
>> Signed-off-by: kbuild test robot <lkp@intel.com>
>> ---
>>
>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/rt/linux-rt-devel.git linux-5.0.y-rt-rebase

This is for the RT tree? The patch in the fixes tag isn't in upstream
and I don't understand how this is related to that patch at all. It just
looks like the RT tree hasn't picked up the patch which made this change
in upstream.

I feel like I've seen the change in this patch a bunch of times already
and it appears to be correct in rc5 at least...

Logan


>> head:   31cc76d5590f5e60c2f26f029e40bc7d0441d93f
>> commit: a3a1e895d4fa0508e11ac9107ace883a5b2a4d3b [171/305] pci/switchtec: Don't use completion's wait queue
>> :::::: branch date: 6 days ago
>> :::::: commit date: 6 days ago
>>
>> Please take the patch only if it's a positive warning. Thanks!
>>
>>  switchtec.c |    2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> --- a/drivers/pci/switch/switchtec.c
>> +++ b/drivers/pci/switch/switchtec.c
>> @@ -392,7 +392,7 @@ static int switchtec_dev_open(struct ino
>>  		return PTR_ERR(stuser);
>>
>>  	filp->private_data = stuser;
>> -	nonseekable_open(inode, filp);
>> +	stream_open(inode, filp);
>>
>>  	dev_dbg(&stdev->dev, "%s: %p\n", __func__, stuser);
>>