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 E3F4A45A97; Thu, 3 Oct 2024 21:15:02 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8FB9D40289; Thu, 3 Oct 2024 21:15:02 +0200 (CEST) Received: from mail-yw1-f182.google.com (mail-yw1-f182.google.com [209.85.128.182]) by mails.dpdk.org (Postfix) with ESMTP id 0A4D240289 for ; Thu, 3 Oct 2024 21:15:01 +0200 (CEST) Received: by mail-yw1-f182.google.com with SMTP id 00721157ae682-6c3f1939d12so10741697b3.2 for ; Thu, 03 Oct 2024 12:15:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20230601.gappssmtp.com; s=20230601; t=1727982900; x=1728587700; 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=h8dorIdrbdsDXvnwTp/xWktFMKZ+/T0BwIXwAgwlxFA=; b=T5VikpzufDjIREVr3z1CVeQ6MW7GG0FiOcMTbHb8fUT0co4OS4ckN46QD/GITg1bfv I6Vb4uj20jsgchwsgQrYbpeZJwxvkShYQ/bFWGQY0gajFRdWb4zItL8unVnKBqhLclnE fOmrOu+DBKmtUb36XK4ShixcizIIsV1rYcQ3b9HftgQVTNv2c8f7kGU6qiMXppIR/mXv BnDsY0Uxisrx7M+N6SE4lcsIbR91swTSWgyGDvvEkRcoe61VmKNR+HRK8UJusTqCnRB5 UafPZnxYrgxvgkCWR6wVdF+ogD9Sw/HE7+L/9go4ccWHC40PlWaAMoJ+U4BtClJFPh6d LdCg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1727982900; x=1728587700; 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=h8dorIdrbdsDXvnwTp/xWktFMKZ+/T0BwIXwAgwlxFA=; b=M6RAi3md2EXwq6uZlNby6qxukojcfnnUeav5/0P3VHMVdE9gVw7O43k++y71Vnf3ky T3/sz1TrNodDOd71qtLSh0Vbbe+AQnSBLxDBAf1nwOG3X1fUKlsoBZtKbNuRbpqZvxMk uVdalqRRHeUejbwm2qP2W3D0ijpfrwrEC2UCwYIMZFsuOp8KE6MXpiWlBB/zCvGFGJtH 4Icp8/J3yqsjTkslR40zJDW4J1LN/+yvPWINmBco33u649zPcqZyn1PCoBMF5OrrskJL 34lCGMpojdhQ0+bzWsHAhEGYCjVn1CLEGAH9fGLI3FawnMTBVTLIc0VYepITOF681qE2 I7bw== X-Gm-Message-State: AOJu0YxkkcJhsYDMVrabDgG8EL3TTYJrh41WEbqjJ/HQCIXJyZgCDTYr BNjTwlqH/5qyLQUC70d57lrWJkfKOy7DEJnhIDNcCEX3paWytcp8miEnRkgQo8A= X-Google-Smtp-Source: AGHT+IH9jY0OWvZotmCu+ygKp2hlN5SQh5092oJ6yBAjG1FxQVtGvr0bLmbm9vwM5l2ycK7SUBL3KA== X-Received: by 2002:a05:690c:dd1:b0:6dd:c768:e58c with SMTP id 00721157ae682-6e2c6e8c605mr5412757b3.0.1727982900354; Thu, 03 Oct 2024 12:15:00 -0700 (PDT) Received: from hermes.local (204-195-96-226.wavecable.com. [204.195.96.226]) by smtp.gmail.com with ESMTPSA id 00721157ae682-6e2bbbb7b0fsm3475247b3.6.2024.10.03.12.14.59 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 03 Oct 2024 12:15:00 -0700 (PDT) Date: Thu, 3 Oct 2024 12:14:57 -0700 From: Stephen Hemminger To: Alexander Kozyrev Cc: , , , , Subject: Re: [PATCH] drivers: fix error CQE handling Message-ID: <20241003121457.37537b40@hermes.local> In-Reply-To: <20230502184922.3249201-1-akozyrev@nvidia.com> References: <20230502184922.3249201-1-akozyrev@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, 2 May 2023 21:49:22 +0300 Alexander Kozyrev wrote: > The completion queue element size can be independently configured > to report either 64 or 128 bytes CQEs by programming cqe_sz parameter > at CQ creation. This parameter depends on the cache line size and > affects both regular CQEs and error CQEs. But the error handling > assumes that an error CQE is 64 bytes and doesn't take the padding > into consideration on platforms with 128-byte cache lines. > Fix the error CQE size in all error handling routines in mlx5. > > Fixes: 957e45fb7b ("net/mlx5: handle Tx completion with error") > Cc: stable@dpdk.org > > Signed-off-by: Alexander Kozyrev I have no idea about internals of mlx5 but the concept seems fine. Patch needs to be rebased. Please title it with net/mlx5 not drivers.