Received: by 2002:a05:6a10:6744:0:0:0:0 with SMTP id w4csp480678pxu; Wed, 14 Oct 2020 06:22:43 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzkvcEvfdpDgPf17ogFK0DNARjXlodthNyOYA1z1wJdp6xy2re6h4FLV6FOxvCQK0cx/8fk X-Received: by 2002:a17:907:2095:: with SMTP id pv21mr5459753ejb.236.1602681762938; Wed, 14 Oct 2020 06:22:42 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1602681762; cv=none; d=google.com; s=arc-20160816; b=bAWmP3UTu+rAe+Tz2S7LxIVR8AayHkM/5Cdaddxp8XdeRtnbD03UnEl5TtbR+vmWjH d7TJri7ac7wXtcte972t56ysL0jvB/P+XgJ0D8Vh9A8KMb2/8hH1yYQRZZFPzjUWZP73 osIcU4onLbq036Jd2l0xao4GJd1bYStceRqqN0DzeIOezpCvEELCJkw/Dm54QTYFF5Ly kURDeWqeblc1j1TSreT1ZjKZpBuWb+7OI3iBa4iNVmlTBjFoLCh6jlNrBjqnPcD678eV 20H7linhi/D5Q8C0P84PTfrKeNU+7wa08Rj2Q56utuDsn5GfnoYzMWeidKhLKcfBtmYa hyMg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date; bh=E5EMAByV9ChdwHo8HkwrHRiHmOLYuWiWA89RvBJyvp4=; b=d8hplsq75O9px6tY+qchTgcsgpypxdYc3nuoLyW6d6Q78XrRPsbKq2OiLvIAiKU22o 7jJ+oQo5ApK6onfBa6WCFRl9w9rn7vrAOzw/UgquOoZdNAv8xBKMQZ6ZvmnHiICAYJMP Ezp0HFFoXNksoMuwyZgtnVHVSuCrY0hz4xK/rMKcs+AJ7f3KrxZ6nCO6rR28v0w2LgRM C5eEGbQ80Whz4D4viueJzgIVC6SlIK9gWMap6tWYe2RRKM7vU/I3PZpVhrr67ZudGQCe /E1HCf1USIYLKIXCUzN9Ke2IOjKsF+bavMA52TlKrqWU9109u9Rnbie42n22kURse/l3 I8nw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id ga21si2218502ejb.100.2020.10.14.06.22.20; Wed, 14 Oct 2020 06:22:42 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727622AbgJNHGd (ORCPT + 99 others); Wed, 14 Oct 2020 03:06:33 -0400 Received: from mx2.suse.de ([195.135.220.15]:35818 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726342AbgJNHGd (ORCPT ); Wed, 14 Oct 2020 03:06:33 -0400 X-Virus-Scanned: by amavisd-new at test-mx.suse.de Received: from relay2.suse.de (unknown [195.135.221.27]) by mx2.suse.de (Postfix) with ESMTP id C33EAB1E9; Wed, 14 Oct 2020 07:06:31 +0000 (UTC) Date: Wed, 14 Oct 2020 09:06:31 +0200 From: Daniel Wagner To: Finn Thain Cc: Nilesh Javali , Arun Easi , linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3] qla2xxx: Return EBUSY on fcport deletion Message-ID: <20201014070631.oe7hbjtfhsohgr54@beryllium.lan> References: <20201012173524.46544-1-dwagner@suse.de> <20201013065232.hdyjdkurkmowkf2f@beryllium.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Finn, On Wed, Oct 14, 2020 at 11:57:04AM +1100, Finn Thain wrote: > > On Tue, 13 Oct 2020, Daniel Wagner wrote: > > How so? I am struggling to see how it could be a change in behavior. But > > then I sometimes fail at simple logic ;) > > > > Me too, so I confirmed the result by executing the code snippets. Thanks for taking the time to explain it to me! > I don't know whether the changes in v3 are desirable or not, I was just > pointing out that the commit log ("valid qpair but the firmware has not > yet started return EBUSY") now seems to disagree with the code. I see where I did my thinko. In v3 we have more ENODEV due to the fact that we test the pointers first (qpair, fpcort). If either of them is invalid we get the ENODEV. Actually, it's makes more sense and is likely to be 'more correct'. Anyway, let me update the commit log. Thanks a lot! Daniel