Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "G, GurucharanX" <gurucharanx.g@intel.com>
To:     Jonathan Toppins <jtoppins@redhat.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>
CC:     "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "Jakub Kicinski" <kuba@kernel.org>,
        "intel-wired-lan@lists.osuosl.org" <intel-wired-lan@lists.osuosl.org>,
        "David S. Miller" <davem@davemloft.net>
Subject: RE: [Intel-wired-lan] [PATCH v2] ice: change "can't set link" message
 to dbg level
Thread-Topic: [Intel-wired-lan] [PATCH v2] ice: change "can't set link"
 message to dbg level
Thread-Index: AQHYGSX4f1yBEjaVTU2GO55MmC32ZqyMTBHA
Date:   Thu, 10 Feb 2022 05:29:53 +0000
Message-ID: <BYAPR11MB336795B622887929554E63B0FC2F9@BYAPR11MB3367.namprd11.prod.outlook.com>
References: <b25f9e524c404820b310c73012507c8e65a2ef97.1643834329.git.jtoppins@redhat.com>
 <cfb30f5c84364c8eff96c0a3ea0231e5dfda17e4.1643910316.git.jtoppins@redhat.com>
In-Reply-To: <cfb30f5c84364c8eff96c0a3ea0231e5dfda17e4.1643910316.git.jtoppins@redhat.com>
Accept-Language: en-GB, en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?u+qP2373O3c1eF55FOIcXBu6XjTUex+/PLxv4QLSwoWvW6QNAvGOZmVcFmvu?=
 =?us-ascii?Q?KbEvPQHIbiz2AxTjIVwQ24UYEcRKiZ//H2iW5nCnC4hu/ErUEJ4qTYoQNpRj?=
 =?us-ascii?Q?l/j16odQgC22ecN/SnZKBqLN6edBOA4yXyy+SddA+//Pij/gscZjh93aFkqq?=
 =?us-ascii?Q?GB1vFUBuWhpubZJqyExcr5M726En3d5Ve20voAskVWhFu59W4kmrdka13xrn?=
 =?us-ascii?Q?hPj4xLBDbBKme2LkrYrInK9jEiKNxq+Z3C00Km3PlP2MMHotwHhZnrx3VVlN?=
 =?us-ascii?Q?Krm9Uo2cZQG0wgJNlMstH+6GL5vRjivtZOAuMVf4MBHGTXokEF4Wg4ywMan7?=
 =?us-ascii?Q?KCcXAol3nKE7nowBneIbjeUpkvtahg+0NRqzfb8Y92QYsrMv9KcqrW3gVx2F?=
 =?us-ascii?Q?/IJyow5gnnGWNotHZRDqsj7x5lPW4tTcbpWPWaTaqii6A8frc+5PCkBC+yey?=
 =?us-ascii?Q?z1WfYBRKRBsb375IDRO5+q4wb01w8UmeFnPZ76ZtJ7QNkeR8+UxIJLRb0Q/J?=
 =?us-ascii?Q?uG5Xwm8LJjOF/m6bUQBiU7jOs9r/16uH/Btbnx7PbyAcaNWs8lG7M8/6D5XX?=
 =?us-ascii?Q?UI+dR3F4Mzoh5wQXGI4jb7Sxnig9eXzA9xWjXtiuUtu0rbXH2T1hLAcSjb4c?=
 =?us-ascii?Q?LM9CnVVe4H3azpl5ggrhnQj3Y2i3RjBtb5SKusDM/du2aYAboGcbKVECz/m8?=
 =?us-ascii?Q?4YewVGLc4t3HcV/e7PKKiVd69Klt25Nw62wB0+E5SCZnDHfg7ggCm+ax7YcA?=
 =?us-ascii?Q?16VnV0zUvS1xim2QNY+E1kx/fJxQwCGuNqUd5KyTG+2QbRVCzUWEUhR4UoJu?=
 =?us-ascii?Q?PbuFCRekR6L1ZWgL1bk+Izy6aifHt5uby3zcsylbQrsrfyFfpkeXpFl5crH+?=
 =?us-ascii?Q?Zda3bTUT5b2VEKHADAry0dxzC+PM5KBR5VZTMlHUiino7ML9wFd6B+Xvngzx?=
 =?us-ascii?Q?lzszPhGuQfjYEjGw+VlIeOCD94MvAugvlPNxa6WEC2TyGZP9gnn/qSrNyomH?=
 =?us-ascii?Q?TXj8tg7qteT6lcbsTAh9cc4l7qqjOXKHqyz8cu+vaHH7K1DqjZ5pJRm6wVNI?=
 =?us-ascii?Q?vVOJsskWdWyjWvoOa0bYmm7CFMoY442GUHq7JoDTt+/htiBl6KYxcZSxa5sH?=
 =?us-ascii?Q?KqZNIhcibw8kuzs9JJOSCMFAc58BKm2jHdOW/jppeEDq+Gx4+LHZtdSdD5xh?=
 =?us-ascii?Q?5DrWGkwM3tqvja9dxy2T7eJVgM0cJFpwAeb6SwllDaLLbwjKnTxbThFdow0P?=
 =?us-ascii?Q?ijL89Oryhu/0Zpy94UoduWQDj8WFwxmzWgGEoOpS9PFFuGrxMLi9FeKP3av2?=
 =?us-ascii?Q?Ic83eN62Xf2C/zYtaesnWzvBgkXLufd+n8RQ+slh8Q8trJBYJHyz1q19AcM2?=
 =?us-ascii?Q?GusDgJyjwQljDVbIvTZRAU9Nb+qUaVpVUUmu9U5PGZDIGt7+z72Yh1/8ep9j?=
 =?us-ascii?Q?5HyLYiTf036OvTeAEsPUiakILNgD/ybr5yfoJ1+m7IrfasbioNFfUSRJBGg7?=
 =?us-ascii?Q?QJtkN119/6Mw4/aDduI3eGbAkgJzqHNlGoT5/1lazbuiJzBgpsguP8LUoBKM?=
 =?us-ascii?Q?OE956jiRe/5yQS8yyyQ6mTugK/aSsvSAYQpfW7EiaSbA3EhiBEBYiJjJj1AW?=
 =?us-ascii?Q?Sw=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BYAPR11MB3367.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 222ed90d-2662-4320-4ee0-08d9ec565e34
X-MS-Exchange-CrossTenant-originalarrivaltime: 10 Feb 2022 05:29:54.0683
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: Eg+TPVRO1vr4/T9fwduzumhZqm6B/37O+9zeqc+NSAScunP/ott0rQ0qap9Pl2Co3ZhDy9VFHmHcqAAE8986+A==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR11MB1870
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-7.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI,
        SPF_HELO_NONE,SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> -----Original Message-----
> From: Intel-wired-lan <intel-wired-lan-bounces@osuosl.org> On Behalf Of
> Jonathan Toppins
> Sent: Thursday, February 3, 2022 11:15 PM
> To: netdev@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org; Jakub Kicinski <kuba@kernel.org>; intel=
-
> wired-lan@lists.osuosl.org; David S. Miller <davem@davemloft.net>
> Subject: [Intel-wired-lan] [PATCH v2] ice: change "can't set link" messag=
e to
> dbg level
>=20
> In the case where the link is owned by manageability, the firmware is not
> allowed to set the link state, so an error code is returned.
> This however is non-fatal and there is nothing the operator can do, so
> instead of confusing the operator with messages they can do nothing about
> hide this message behind the debug log level.
>=20
> Signed-off-by: Jonathan Toppins <jtoppins@redhat.com>
> ---
>  drivers/net/ethernet/intel/ice/ice_lib.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>=20

Tested-by: Gurucharan G <gurucharanx.g@intel.com> (A Contingent worker at I=
ntel)