Received: by 2002:ab2:3141:0:b0:1ed:23cc:44d1 with SMTP id i1csp751161lqg; Fri, 1 Mar 2024 23:49:03 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCVA+w7Cw/MlVpZR+0TUz3qTdq9y2FtQalOZPUVfRWt7r5/8gz5yQnfrwfyxRyOhMW0MxRUnG4lTvJlG531DXlfo7qCoIyKgpHaXfdvEkA== X-Google-Smtp-Source: AGHT+IGwc3dScF554m0Ab9cmLFRPSkaa52t+HwB6Y+p+8penHl/QwwqbM5B31JD9LhTju6tntoo2 X-Received: by 2002:a05:6a00:27a0:b0:6e5:5f6a:4966 with SMTP id bd32-20020a056a0027a000b006e55f6a4966mr4964732pfb.6.1709365742757; Fri, 01 Mar 2024 23:49:02 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1709365742; cv=pass; d=google.com; s=arc-20160816; b=gD+1pKnvzKJJuA3v0ZiXAMGykvm3nLp01R79ca+TXPsznHDePttK5z3qj4CpAFnFBB 5ofGOK62qSfadNconHEhqgMzOpjTqN+mY5z+Q2JOCtVFVLFI1w/QI0RJRHMqZjS2zDJo 6cboB++B3t1oyY3jfJ3yxcCc0vb+RBWULlhNXbDTlVTlp/+KLSdl7lhJaC7lF4Rgap99 gspwWcn60A3J4rTgLTBtSLt3+muVGrG1/NiYq4KPlDtMflmVGu/HLAg/ly6+eM3dWDLp UqDkHMfaqqDBxvpVzsOVV1kgoibBMOlvYo0LejNIC4+dMJCZQwF6ZydSBXIMwX+/2zLV SGSw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=in-reply-to:content-disposition:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:message-id:subject:cc :to:from:date:dkim-signature; bh=HJOnO548gm2BJBKrCeJqyCM669o9B9+MpiayAoE0cHs=; fh=LvMadvKUweSoOk4yGwZjfWTCt7260Xo1sqkQYuWIFWk=; b=JV9lGaWQBNtqzizBHL4fHZWdcmWl1Cmgkuh1jz1ghc1RMtAjpZSNe4p8fPL51boG3E sdex/qAx6ygJkSs57EdXYBEfw+1WYtTPhUkxEuC5ce6w7noBOYy2Sm2k4c4lsoQYsJ7r +glNv2qc/6cIsZ1ShMk+84aw02kIZhp+/PtFE0y3I6x1z/x9athVSlJBvPni+ztnQwi8 SdN3G44QzNurwEjlOprwclL+G1phF9R9DY2d4KM+jW5HfK6qLTZkHTIuHZD9QcNbu0/7 7a7NXuIcPUokIkOvulIZIBqJF9iH0CSpwJkc0GQosyqwEfIR99Q4eYVIBPPu+BGuJz2p kzgQ==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=vDUBl1M5; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-89355-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-89355-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [139.178.88.99]) by mx.google.com with ESMTPS id j2-20020a056a00234200b006e46627907asi5106257pfj.4.2024.03.01.23.49.02 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 01 Mar 2024 23:49:02 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-89355-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=vDUBl1M5; arc=pass (i=1 dkim=pass dkdomain=kernel.org); spf=pass (google.com: domain of linux-kernel+bounces-89355-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) smtp.mailfrom="linux-kernel+bounces-89355-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 5FE8D28417F for ; Sat, 2 Mar 2024 07:49:02 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 15C1B125DE; Sat, 2 Mar 2024 07:48:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="vDUBl1M5" Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 3FB3D11724 for ; Sat, 2 Mar 2024 07:48:55 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709365736; cv=none; b=iuiH4dIpxSy7XEvHulhQTK4g1MU4zx9ItZocessl6cI4GYttRNHALao0YpjU4C/5TmjVcs1lQRTBdYZ23puRC0BhkuiQZYhHrFLxlp9To6gIhOli0nT+gPTgSvo9IMprAaEWGzL1jyWAn9LXKZMAVpKL/ZCuKeGmeF8AILk+5MI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709365736; c=relaxed/simple; bh=TOmAj42S1SC0gXibQSRZTpZCMLTUiIkWf0MokO/SOLU=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=Ym1zG/lUlrraYYpJuMylrNChrg2GWVNkTQ2NIGPVFLdh4L8sp7+Bjmfu1H5EWYT0iclbnSUCg7V2Y7zKV+YyabsGKiLzjKz8DN0xVqN7pxcFLvIIYbaG4rkYX/w4ku1nktf3yBxJJx0TTI315Vs2TZ21ZJgWIx37ZLXNfWKSJ5k= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=vDUBl1M5; arc=none smtp.client-ip=10.30.226.201 Received: by smtp.kernel.org (Postfix) with ESMTPSA id AFA88C433C7; Sat, 2 Mar 2024 07:48:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1709365735; bh=TOmAj42S1SC0gXibQSRZTpZCMLTUiIkWf0MokO/SOLU=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=vDUBl1M5CS5nP6L+5fgC6r9oiINchHNLMCKTAG9h8Bmu17F8dHpflXVX5uBfV1iJg /UXkuxx2xY23grUcJdGbcEHFtbkgGtr5if1J8i8RG+G7DnABSkVJhBW5JXm86N8PyM qcLLOZ0eCnQgL4A1hsRCGKbW7vo67qUo7ut2TVViO9fyxGyeEH8cncABXj3yFV/HTj Hl+jCO5pRnLmJnPvNxXccynxJzLrdw17RzksqgsZKx/w/2uQBSrJdz50JN/Y1zzp8G Rexj5zBVCBYolDxDibuuE5grFXXZGC7lN7OjmCYBAmUWtvz5NcLuucCubd4H5uqolG vyA1dsMIaFdlQ== Date: Fri, 1 Mar 2024 23:48:53 -0800 From: Saeed Mahameed To: Vegard Nossum Cc: Arnd Bergmann , Greg Kroah-Hartman , Leon Romanovsky , Jason Gunthorpe , Jiri Pirko , Leonid Bloch , Itay Avraham , Jakub Kicinski , Saeed Mahameed , David Ahern , Aron Silverton , Christoph Hellwig , andrew.gospodarek@broadcom.com, linux-kernel@vger.kernel.org Subject: Re: [PATCH V4 4/5] misc: mlx5ctl: Add command rpc ioctl Message-ID: References: <20240207072435.14182-1-saeed@kernel.org> <20240207072435.14182-5-saeed@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii; format=flowed Content-Disposition: inline In-Reply-To: On 29 Feb 12:49, Vegard Nossum wrote: > >On 07/02/2024 08:24, Saeed Mahameed wrote: >>@@ -328,6 +420,11 @@ static int mlx5ctl_probe(struct auxiliary_device *adev, >> goto abort; >> } >>+ err = sysfs_create_link_nowarn(&mcdev->miscdev.this_device->kobj, >>+ &mdev->device->kobj, "mdev"); >>+ if (err) >>+ mlx5ctl_dbg(mcdev, "mlx5ctl: failed to create sysfs link err %d\n", err); >>+ > >Should this propagate the error to the caller? > this link is informational only and not necessary for the driver function, it meant to help user-space apps to associate mlx5ctl driver with it parent mlx5_core device. >What happens if/when mlx5ctl_remove()/sysfs_remove_link() gets called >later for this kobj? > sysfs_remove_link() will eventually call kernfs_remove_by_name_ns() and it will return -ENOENT; if not found, and it will be silently dropped.