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 C431E43C03; Tue, 27 Feb 2024 07:59:02 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id B2BD24027D; Tue, 27 Feb 2024 07:59:02 +0100 (CET) Received: from mail-oo1-f43.google.com (mail-oo1-f43.google.com [209.85.161.43]) by mails.dpdk.org (Postfix) with ESMTP id D3A7B40150 for ; Tue, 27 Feb 2024 07:58:46 +0100 (CET) Received: by mail-oo1-f43.google.com with SMTP id 006d021491bc7-5a03f90d1c1so1835045eaf.1 for ; Mon, 26 Feb 2024 22:58:46 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; t=1709017126; x=1709621926; darn=dpdk.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=XEAjFhJ0OxXMLgQS01UtqpGn4d4zt7LffIt8ieBjpmo=; b=UkNrMfn0yJDRLZvGq1qWVPoI2YRfYrDur9xjP+0EK1MN8OQ3uKvc2Bw43CdOzQ0m04 tcHQ5N2fLOIR4zZq7xrQ/UoDRjnO94Y4NmFbxfkUt9GrrIikIim3ryyjNB+TsDbsInS8 aZsArD3pcaT18AZDADv8fob5BfjH4Ry31UdPQ= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1709017126; x=1709621926; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=XEAjFhJ0OxXMLgQS01UtqpGn4d4zt7LffIt8ieBjpmo=; b=MWTvqz8KdesMhEg9O6by4VfCDRF1S8VryWp1FWc+3dHl9liW+inew/NqIoX7NL/EEv FWZthTYFtb1kwfhsbhFdU8I1SloIJvI3i79+vokycrkM0cMuLjpbPT4xUm6Q6L3Na49a hsIboh+KXcl10ama9l55ySWapaohSVP8JkEelT5H4bIisGnPpgPsYGTKo9/D1VCrm1OI WyBvHC+J/9X4Pvaz5Y4QVnMeKqcptL5m2xPsKAETNXQK8VTdNXumMKoG/HZM8kcrHXsR +RtlE7npSfLOmxJfXf+IZMIvAO68koFmbiyTuVV3dkdUBWLwnORgL2I6mpb+OD5NvGlS AwGg== X-Forwarded-Encrypted: i=1; AJvYcCV5S1HniA/co58B6NKLsFpSPzDY2inRGYnQ5X7z6O5xUJ3slYrXG8PDtRAvo4eleOT6CTm7l9xIc1u+HQ== X-Gm-Message-State: AOJu0Yzx8jpAoJaTTUtXwZIUp6v+iN292yeQJVmlYRwAAsF8AkRlF3Md GvTc0xbKRzT0FDKM9hmbr2KnWJ7H4Y21wMaivz1we3t4UZ8mhBNTHggZdZ5HVPTpQPe/zSzteRT fxxCIkcUU+JcKfpOSHgtW6DUH1+12aU/NaLGzEQ== X-Google-Smtp-Source: AGHT+IGOQzxGJyZ/gUmD7xIMfE6Kl8sBIxBpEm7PhccSOXC2pLgkR8yRUxZg9ijwXJEhxu36DLCXtYr2pCCONn3dqos= X-Received: by 2002:a4a:6f4d:0:b0:5a0:402c:f1b7 with SMTP id i13-20020a4a6f4d000000b005a0402cf1b7mr8125449oof.1.1709017126033; Mon, 26 Feb 2024 22:58:46 -0800 (PST) MIME-Version: 1.0 References: In-Reply-To: From: Patrick Robb Date: Tue, 27 Feb 2024 01:58:35 -0500 Message-ID: Subject: Re: Intel QAT 8970 accel card on ARM Ampere Server To: Ruifeng Wang Cc: =?UTF-8?Q?Juraj_Linke=C5=A1?= , Dharmik Jayesh Thakkar , David Marchand , Honnappa Nagarahalli , "ci@dpdk.org" , nd Content-Type: multipart/alternative; boundary="000000000000b673db0612578e4e" X-BeenThere: ci@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK CI discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: ci-bounces@dpdk.org --000000000000b673db0612578e4e Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Tue, Nov 14, 2023 at 2:35=E2=80=AFAM Ruifeng Wang = wrote: > Hi Patrick, > > > > It seems kernel v5.15 has a defect on this. A similar issue was fixed by > commit: > > 40da865381ad ("crypto: qat - remove unneeded packed attribute") > > > > Could you patch the kernel and try again? > > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit= /?id=3D40da865381ad061ab75a7a9da469ed4e623bdfeb > > > > Thanks, > > Ruifeng > > > Hi Ruifeng, Sorry for the delay on this - there has been a work item backlog at the Community Lab we've been working through. I did rebuild the patch today with these changes from the commit (or similar, as the commit above was for the qat_common file in a different state, but I tried to remain as true to the commit as possible). And that does seem to have resolved the seg fault problem! Thank you so much for picking this commit out of obscurity and sending it our way! root@arm-ampere-dut:~# echo 16 > /sys/bus/pci/drivers/c6xx/0000:03:00.0/sriov_numvfs root@arm-ampere-dut:~# cat /sys/bus/pci/drivers/c6xx/0000:03:00.0/sriov_numvfs 16 Wunderbar! The only other thing I changed (just because I was floating the idea with Dharmik before) was in the kernel .config I changed the qat_c62x and qat_c62xvf modules from statically built in (=3Dy) to loadable (=3Dm). Of course, this should not matter, and I presume the change in behavior relates to those brought in from the commit above. I just want to present fully all changes made so there is a complete picture. I will continue on this tomorrow according to where this conversation left off, and try to move this quickly. If indeed there are no more blockers I think we are very close. As a reminder, when standing up a new testing plan, we want to make sure at least 1 rep from each vendor has SSH access and can remotely login to help with system tuning, troubleshooting, etc. for the testbed and test plan. Who would be the best person from ARM for this at this time, given the context on QAT testing? Ruifeng? Dharmik? Someone else? Thanks, I'll keep yall apprised of the situation. --000000000000b673db0612578e4e Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable


=
On Tue, Nov 14, 2023 at 2:35=E2=80=AF= AM Ruifeng Wang <Ruifeng.Wang@ar= m.com> wrote:

Hi Pat= rick,

=C2=A0

It see= ms kernel v5.15 has a defect on this. A similar issue was fixed by commit:<= u>

40da86= 5381ad ("crypto: qat - remove unneeded packed attribute")<= u>

=C2=A0

Could = you patch the kernel and try again?

https= ://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=3D= 40da865381ad061ab75a7a9da469ed4e623bdfeb

=C2=A0

Thanks= ,

Ruifen= g

=C2=A0

Hi Ruifeng,

Sorry for the delay= on this=C2=A0- there has been a work item backlog at the Community Lab we&= #39;ve been working through.=C2=A0

I did rebuild t= he patch today with these changes from the commit (or similar, as the commi= t above was for the qat_common file in a different state, but I tried to re= main as true to the commit as possible).=C2=A0

And= that does seem to have resolved the seg fault problem! Thank you so much f= or picking this commit out of obscurity and sending it our way!=C2=A0
=

root@arm-ampere-dut:~# echo 16 > /sys/bus/pci/driver= s/c6xx/0000:03:00.0/sriov_numvfs
root@arm-ampere-dut:~# cat /= sys/bus/pci/drivers/c6xx/0000:03:00.0/sriov_numvfs
16

=
Wunderbar!

The only other thing I chang= ed (just because I was floating the idea with Dharmik before) was in the ke= rnel .config I changed the qat_c62x and qat_c62xvf modules from statically = built in (=3Dy) to loadable (=3Dm). Of course, this should not matter, and = I presume the change in behavior relates to those brought in from the commi= t above. I just want to present fully all changes made so there is a comple= te picture.=C2=A0

I will continue on this tomorrow= according to where this conversation left off, and try to move this quickl= y. If indeed there are no more blockers I think we are very close. As a rem= inder, when standing up a new testing plan, we want to make sure at least 1= rep from each vendor has SSH access and can remotely login to help with sy= stem tuning, troubleshooting, etc. for the testbed and test plan. Who would= be the best person from ARM for this at this time, given the context on QA= T testing? Ruifeng? Dharmik? Someone else?=C2=A0

T= hanks, I'll keep yall apprised of the situation.

--000000000000b673db0612578e4e--