From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qv1-f45.google.com (mail-qv1-f45.google.com [209.85.219.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 13DDB6125 for ; Sun, 23 Mar 2025 13:13:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.219.45 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742735626; cv=none; b=Nlm7eMTn2HmTs6fwZt2nIRj57hh8XR9UyPwTEt+16x2Y4R6O2VbcZgL4nT5GVtoEL1ik6jfqNSOEhRxf2qR18EM0etJnum5Foh6n06A3xdaFN8K8PzlbKtxJA6X+Do8XcnOlGaz1JUsUqhFCFpfvOp1+4bASNhLD4EcCpuiM2Ew= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742735626; c=relaxed/simple; bh=LKr4Tq6eNnj7Xe9zMSlrQR5ZDQsgfe5FGBhc/jpYjx4=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=N3jtiLgWhQzvGypAHR/arnwl49RL4wpgIbF1+itp8ZJ6t40xxHOjFQSmT7H2kLDZzX2PZ4IIbzMg6OGgxvYv3J9XmyKpzR5pxvr58q4uAvWwXqIjjFGcKGc8Wy5V4q0kqr0jx0kOLWzHLvVNEouZAniEFnvc/t6sKMsH3Y+jjBo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ziepe.ca; spf=pass smtp.mailfrom=ziepe.ca; dkim=pass (2048-bit key) header.d=ziepe.ca header.i=@ziepe.ca header.b=h31+9vo4; arc=none smtp.client-ip=209.85.219.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ziepe.ca Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ziepe.ca Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=ziepe.ca header.i=@ziepe.ca header.b="h31+9vo4" Received: by mail-qv1-f45.google.com with SMTP id 6a1803df08f44-6e8fce04655so33510676d6.3 for ; Sun, 23 Mar 2025 06:13:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ziepe.ca; s=google; t=1742735623; x=1743340423; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=p+wF4IQZggkSDSdyJgdkeavpzbtB2emtmQgy0osOszU=; b=h31+9vo43BhA93H9pHpR+HEhQ5yPlKSVbK8yGik1pIJ4Phz65z92yUh0cL+eefK/jH WMod7R5tqqH/pWN8UChIr7h5wPC1zu8SS46nWVR65WNlKk7gXGN7XcRv9+Iq6pLycHi+ n3amt9DQmhwiuu7GVqCQhJXBOZx+uq3CFx6hE8wzgc3xjXxt5H2SNTlH95ybS35fBDQd 4b5wPsIuyTHdYkPirmyLg4d7hYw68sdEAVzIvVp8gbQNnrDT1VXzBcmpU6TNWaaR2iyV 0EGwHJtxODz3m5YQwZjI9PDAoT/q7c7MfISyUbRUShGXqi3WbhhEXExs4KWePfLasDV8 bBuw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1742735623; x=1743340423; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=p+wF4IQZggkSDSdyJgdkeavpzbtB2emtmQgy0osOszU=; b=NPJhZLD6jzi1VikK+yhIR1jqxDHIFIhZVSboXy+IauykS3jBBA5evkkcPpIdM0Nq61 szsqOGZNyvCpgD5+4BNf/bGQoenFQY7UeOQiXbEQrw3zhiGNbl2mwY6L2jb7/3t/dmmG O71Pf1Tq7/BFma61lZjBx526lQtPkQDO9M2NZ1cin2pEP0n9iSA5L84xKnW3akX5Ex4l 5SGFLQTSaomkCqaGGGMbRXItb6VtnuOq6inFDYB2tiwwaKQLGlpp53Grt2CMXMQwiiuE uia/3PnAai912qC6JN1UUaRDggm/F/uqDk7CDnbdfgmcxygsuC9jKfFHW6wGXAsCMapB wCfw== X-Forwarded-Encrypted: i=1; AJvYcCU7snyTrdfxGAAYonIPeH8bDOSyqDEZbyBFuaANqnsX61qML8dD0oKmTqcG86NIcEIPlzjsFVAd0Fdw@vger.kernel.org X-Gm-Message-State: AOJu0Ywiu28hUdFvKpygZ3YQIKKEanP0jC1ROHomeCW32wZb7lDEMzyb bcaespdMVcm09ze/Qo+PN6Nwe7k2WVTludm1jvnAFJqzDSVj2E7g2zspYr47UBc= X-Gm-Gg: ASbGnctvX9ErayB7P1NYgsqRL28jYzbObBenh5gVBTdnXp9yRa4i5uxN2tOlhukcaP/ LBbhgaOMQXRka5tujuCbq5u1XmACMRzsB74fRQ9IP/rsiPJAn8mwYgt5ztr8pzXy/4inT1oEel2 trp6IivqUM9Kwl7bkhHuJ9FWryyH9u/Vqou7NHq3dP/FNYMBdfA2MvgyPzp+dVU8MUrgQclK2gi 9IEKYYbSikLVtU77JW6/7FSUCvY796YZ8MoC5ckgphXAjx2i1kKvJXWz9C5mILyQOi+S6L5tepI 48BuAjU/K/Z2VzAXBMvwwiX+Pv7mxOp5Eg2QNPA5uTRYJ4jIkr9Rl4AH2srQ32fGXWk0XbNM9ZZ f5ZqvRFObwNfg9eb1USfzL2g= X-Google-Smtp-Source: AGHT+IF4UIJE9UDyWQ9LTQ9Krxp2Ixpd1OB6m6mGqwsy9YA+QPXyhTykLF+PW3HLeymx/bkZOK2ZKQ== X-Received: by 2002:ad4:5d41:0:b0:6eb:2fd4:30a7 with SMTP id 6a1803df08f44-6eb3f32b63fmr177643166d6.33.1742735622728; Sun, 23 Mar 2025 06:13:42 -0700 (PDT) Received: from ziepe.ca (hlfxns017vw-142-167-219-86.dhcp-dynamic.fibreop.ns.bellaliant.net. [142.167.219.86]) by smtp.gmail.com with ESMTPSA id 6a1803df08f44-6eb3efc5aa4sm31520026d6.71.2025.03.23.06.13.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 23 Mar 2025 06:13:41 -0700 (PDT) Received: from jgg by wakko with local (Exim 4.97) (envelope-from ) id 1twL92-00000001R6u-3RMd; Sun, 23 Mar 2025 10:13:40 -0300 Date: Sun, 23 Mar 2025 10:13:40 -0300 From: Jason Gunthorpe To: Yue Haibing Cc: benve@cisco.com, neescoba@cisco.com, leon@kernel.org, liyuyu6@huawei.com, umalhi@cisco.com, roland@purestorage.com, linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH -next] RDMA/usnic: Fix passing zero to PTR_ERR in usnic_ib_pci_probe() Message-ID: <20250323131340.GA305901@ziepe.ca> References: <20250323033414.1716788-1-yuehaibing@huawei.com> Precedence: bulk X-Mailing-List: linux-rdma@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250323033414.1716788-1-yuehaibing@huawei.com> On Sun, Mar 23, 2025 at 11:34:14AM +0800, Yue Haibing wrote: > drivers/infiniband/hw/usnic/usnic_ib_main.c:590 > usnic_ib_pci_probe() warn: passing zero to 'PTR_ERR' > > Use err code in usnic_err() to fix this. > > Fixes: e3cf00d0a87f ("IB/usnic: Add Cisco VIC low-level hardware driver") > Signed-off-by: Yue Haibing > --- > drivers/infiniband/hw/usnic/usnic_ib_main.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/infiniband/hw/usnic/usnic_ib_main.c b/drivers/infiniband/hw/usnic/usnic_ib_main.c > index 4ddcd5860e0f..e40370f9ff25 100644 > --- a/drivers/infiniband/hw/usnic/usnic_ib_main.c > +++ b/drivers/infiniband/hw/usnic/usnic_ib_main.c > @@ -587,9 +587,9 @@ static int usnic_ib_pci_probe(struct pci_dev *pdev, > > pf = usnic_ib_discover_pf(vf->vnic); > if (IS_ERR_OR_NULL(pf)) { usnic_ib_discover_pf() doesn't return NULL, just remove this test instead. You could also fix this: us_ibdev = usnic_ib_device_add(parent_pci); if (IS_ERR_OR_NULL(us_ibdev)) { us_ibdev = us_ibdev ? us_ibdev : ERR_PTR(-EFAULT); So that device_add doesn't weirdly return NULL and EFAULT, probably just return NULL on all failures. Jason