From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 2C7EF45E1E; Tue, 3 Dec 2024 18:00:28 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id E791540270; Tue, 3 Dec 2024 18:00:27 +0100 (CET) Received: from mail-pg1-f170.google.com (mail-pg1-f170.google.com [209.85.215.170]) by mails.dpdk.org (Postfix) with ESMTP id 161D940270 for ; Tue, 3 Dec 2024 18:00:27 +0100 (CET) Received: by mail-pg1-f170.google.com with SMTP id 41be03b00d2f7-7fbd9be84bdso4450554a12.1 for ; Tue, 03 Dec 2024 09:00:27 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1733245226; x=1733850026; darn=dpdk.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=8nsN8yK4NyVZaWMMYUcV0SfwxV25mbEabuR6nD6zzrQ=; b=DXLjqmOHthin4mXAZ5YRp/QhgC4KJJ6YtC+URsQi/JwY5t/mVfoLzxbhvSLLBWs9Sd +yBOEG4dt/6un29VvksabLsUsfy5JbprUYTcC5LWjrmwTd9jKyuMISxEfwFjtOgGxWhZ jKAWPGzWu8vMEceQO34k+GBMMrM76wpawexfAcP6oKIUvN/7gI9SqKGGHnoeXOTjye7y TnEhMkIL6XbvO7jLjQjNFXSKXJKnTd5x6BpBzzrp7ES9Tj3XcO/agUoBcK4lQO4Bhrrz kGgxF2ntfY6HLv81j4W56xxWWiIGHNEoYIbSnrFAkrh61AZHeShOrAqKE8qa/RESb3bo BMQw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1733245226; x=1733850026; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=8nsN8yK4NyVZaWMMYUcV0SfwxV25mbEabuR6nD6zzrQ=; b=mXLvrtBSmIelv3YDoJgUEyl422i/3TvQw1qmUqY2CXO+U8Z1orb5T/3FUyVMXpunbm gFaanhk0x1d9FDO3sCDnJrq5+Oh/JLo//Y66xbgdbmXi0aJ6Jlf3qX2QMbKa8/Q4MVs1 7f5ae/L5D8TcyZj3UiJrTzBg5GKu6fIv2VZW+X1L+Ly5ysDg0j82oX8CCzhMfTmsQoIK JyKWKB7/PirbNMYNzaL7Y0dJiE8OMFaVvvngzUQ/DuRX8tjBNq4HO2sHbS7h8KgXfaNQ 4OwEzGWE7FnlwNwJ1M27tUyAMjkXj0TgAptCoc5I2DDBwa9n3O4iGF6BGpmntfdI8g/g QcAw== X-Gm-Message-State: AOJu0YzfQToLuSdL4eNeXquh9iXMICibYmY/xqk/7xIsVS93RDUfvqno qDC8ygbxZ0B+KTukzxY4WRq3IFoekXU7CZuJ/tI76aO0PsYSzYzp6ffjRgBsbwk= X-Gm-Gg: ASbGncs26XiW+AVujoNCL7ROkJMpmmfufPhIVlb+GjNXSqS61DrtuftBxEpt2d2cIja CLhO3He2cZ1caYTmF/2TrLL3WM6lzkX6/7gbYmVTEiHL/Z12U+m5bzkDTBLmSwjYcmRP3bJd9xv aIPoH3uwGYzGuFElhpXI/j/tYVQz4kP1xEBFAFq0RraKqqSI8rxRd+KHmAHgYpGHp6hrN11NEeh UCtj+CIvy3xxBvbqkkVdFPp72MHXPB81masyPTc82JsSgD+xVuFS9WWT4JmjzqdVubB1q1NKbzu vLlqFhenHCKwgOwfNZqFmRXK2xw= X-Google-Smtp-Source: AGHT+IE1PL0HleXIq2kTqaIPMSllIkqVpAybWfVxLbrhwEiHKfO7wyDPY3yNoqyN/omN+iRMDAA8yQ== X-Received: by 2002:a05:6300:4041:b0:1e1:1659:82a4 with SMTP id adf61e73a8af0-1e16541341fmr5257201637.41.1733245225725; Tue, 03 Dec 2024 09:00:25 -0800 (PST) Received: from hermes.local (204-195-96-226.wavecable.com. [204.195.96.226]) by smtp.gmail.com with ESMTPSA id 41be03b00d2f7-7fc9c385ef8sm9902630a12.56.2024.12.03.09.00.20 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 03 Dec 2024 09:00:20 -0800 (PST) Date: Tue, 3 Dec 2024 09:00:16 -0800 From: Stephen Hemminger To: Viacheslav Ovsiienko Cc: , , , , Subject: Re: [PATCH] net/mlx5: fix hypervisor detection in VLAN workaround Message-ID: <20241203090016.19796e75@hermes.local> In-Reply-To: <20241203162200.2543986-1-viacheslavo@nvidia.com> References: <20241203162200.2543986-1-viacheslavo@nvidia.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org On Tue, 3 Dec 2024 18:22:00 +0200 Viacheslav Ovsiienko wrote: > diff --git a/drivers/net/mlx5/linux/mlx5_vlan_os.c b/drivers/net/mlx5/linux/mlx5_vlan_os.c > index 81611a8d3f..017953d5cc 100644 > --- a/drivers/net/mlx5/linux/mlx5_vlan_os.c > +++ b/drivers/net/mlx5/linux/mlx5_vlan_os.c > @@ -112,7 +112,9 @@ mlx5_vlan_vmwa_init(struct rte_eth_dev *dev, uint32_t ifindex) > /* Check whether there is desired virtual environment */ > hv_type = rte_hypervisor_get(); > switch (hv_type) { > +#if defined(RTE_ARCH_X86) || defined(RTE_ARCH_X86_64) > case RTE_HYPERVISOR_UNKNOWN: > +#endif > case RTE_HYPERVISOR_VMWARE: > /* > * The "white list" of configurations Could you fix that comment as well? We got rid of all use of "whitelist" in DPDK, looks like this one got missed.