From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 3A504C433FE for ; Tue, 19 Apr 2022 07:51:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S243341AbiDSHx7 (ORCPT ); Tue, 19 Apr 2022 03:53:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44092 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234750AbiDSHx6 (ORCPT ); Tue, 19 Apr 2022 03:53:58 -0400 Received: from mx0a-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com [148.163.158.5]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B821E617C; Tue, 19 Apr 2022 00:51:16 -0700 (PDT) Received: from pps.filterd (m0098416.ppops.net [127.0.0.1]) by mx0b-001b2d01.pphosted.com (8.16.1.2/8.16.1.2) with SMTP id 23J7ctNv006757; Tue, 19 Apr 2022 07:51:14 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=message-id : date : mime-version : subject : to : cc : references : from : in-reply-to : content-type : content-transfer-encoding; s=pp1; bh=A1a7xWbcOcUfmV0lXpHiy7uYOtzD1vJDZCycSndM4FU=; b=hlHTQSn9/ujPkz4Q5Vv4tDHzZmbYUIY2XElFQ38fv2OXEdtvKxCYY9JH9Rql7jV/KZom jAe8I+cOWW4uaDFhLRjVdIOWDt/3SQTRKCAAWTPZWGoTasuCOE4bsff8VYwQ/8vdBWpb VKJEL/BF1OeA5+B8pnMTC1VaAEaEUYvHlJXrbZxiFgnoRgiNSTRcNVyAOHPZjNkY+dDU 5qkaLTf7UPsXvGmX/L6NFCWl57Z73Wt4xs0VdDNMfT65Bq2OW7TobarpJYz78CMJ393l o/0osxUV0Qxa7mCfl4CFW+6qWu+7s7c1x4K1S2qKOy6T9A2KEnWD0F/gjZtypeWmOf6n lQ== Received: from pps.reinject (localhost [127.0.0.1]) by mx0b-001b2d01.pphosted.com with ESMTP id 3fg77jryxq-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 19 Apr 2022 07:51:14 +0000 Received: from m0098416.ppops.net (m0098416.ppops.net [127.0.0.1]) by pps.reinject (8.16.0.43/8.16.0.43) with SMTP id 23J7c3ap016080; Tue, 19 Apr 2022 07:51:14 GMT Received: from ppma06ams.nl.ibm.com (66.31.33a9.ip4.static.sl-reverse.com [169.51.49.102]) by mx0b-001b2d01.pphosted.com with ESMTP id 3fg77jryx7-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 19 Apr 2022 07:51:14 +0000 Received: from pps.filterd (ppma06ams.nl.ibm.com [127.0.0.1]) by ppma06ams.nl.ibm.com (8.16.1.2/8.16.1.2) with SMTP id 23J7lcAL029247; Tue, 19 Apr 2022 07:51:12 GMT Received: from b06cxnps4074.portsmouth.uk.ibm.com (d06relay11.portsmouth.uk.ibm.com [9.149.109.196]) by ppma06ams.nl.ibm.com with ESMTP id 3ffn2huw7y-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 19 Apr 2022 07:51:11 +0000 Received: from d06av24.portsmouth.uk.ibm.com (mk.ibm.com [9.149.105.60]) by b06cxnps4074.portsmouth.uk.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 23J7p82U47907228 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Tue, 19 Apr 2022 07:51:08 GMT Received: from d06av24.portsmouth.uk.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id BAA0342045; Tue, 19 Apr 2022 07:51:08 +0000 (GMT) Received: from d06av24.portsmouth.uk.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 599A24203F; Tue, 19 Apr 2022 07:51:07 +0000 (GMT) Received: from [9.171.88.57] (unknown [9.171.88.57]) by d06av24.portsmouth.uk.ibm.com (Postfix) with ESMTP; Tue, 19 Apr 2022 07:51:07 +0000 (GMT) Message-ID: <06585446-b28c-a15b-6267-ace24cbc6885@linux.ibm.com> Date: Tue, 19 Apr 2022 09:54:24 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.3.0 Subject: Re: [PATCH v5 08/21] s390/pci: stash associated GISA designation Content-Language: en-US To: Matthew Rosato , linux-s390@vger.kernel.org Cc: alex.williamson@redhat.com, cohuck@redhat.com, schnelle@linux.ibm.com, farman@linux.ibm.com, borntraeger@linux.ibm.com, hca@linux.ibm.com, gor@linux.ibm.com, gerald.schaefer@linux.ibm.com, agordeev@linux.ibm.com, svens@linux.ibm.com, frankja@linux.ibm.com, david@redhat.com, imbrenda@linux.ibm.com, vneethv@linux.ibm.com, oberpar@linux.ibm.com, freude@linux.ibm.com, thuth@redhat.com, pasic@linux.ibm.com, pbonzini@redhat.com, corbet@lwn.net, kvm@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org References: <20220404174349.58530-1-mjrosato@linux.ibm.com> <20220404174349.58530-9-mjrosato@linux.ibm.com> From: Pierre Morel In-Reply-To: <20220404174349.58530-9-mjrosato@linux.ibm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-TM-AS-GCONF: 00 X-Proofpoint-GUID: eLGopDV-HRdi8V-tf2fkNCd8wGRVFc3s X-Proofpoint-ORIG-GUID: tuIdsk7RDlmz7OByL1oplM80XoINRn0q X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.205,Aquarius:18.0.858,Hydra:6.0.486,FMLib:17.11.64.514 definitions=2022-04-19_02,2022-04-15_01,2022-02-23_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 malwarescore=0 priorityscore=1501 mlxscore=0 mlxlogscore=999 spamscore=0 impostorscore=0 bulkscore=0 adultscore=0 phishscore=0 lowpriorityscore=0 clxscore=1015 suspectscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2202240000 definitions=main-2204190040 Precedence: bulk List-ID: X-Mailing-List: linux-doc@vger.kernel.org On 4/4/22 19:43, Matthew Rosato wrote: > For passthrough devices, we will need to know the GISA designation of the > guest if interpretation facilities are to be used. Setup to stash this in > the zdev and set a default of 0 (no GISA designation) for now; a subsequent > patch will set a valid GISA designation for passthrough devices. > Also, extend mpcific routines to specify this stashed designation as part > of the mpcific command. > > Signed-off-by: Matthew Rosato Reviewed-by: Pierre Morel > --- > arch/s390/include/asm/pci.h | 1 + > arch/s390/include/asm/pci_clp.h | 3 ++- > arch/s390/pci/pci.c | 6 ++++++ > arch/s390/pci/pci_clp.c | 5 +++++ > arch/s390/pci/pci_irq.c | 5 +++++ > 5 files changed, 19 insertions(+), 1 deletion(-) > > diff --git a/arch/s390/include/asm/pci.h b/arch/s390/include/asm/pci.h > index fdb9745ee998..42a4a312a6dd 100644 > --- a/arch/s390/include/asm/pci.h > +++ b/arch/s390/include/asm/pci.h > @@ -123,6 +123,7 @@ struct zpci_dev { > enum zpci_state state; > u32 fid; /* function ID, used by sclp */ > u32 fh; /* function handle, used by insn's */ > + u32 gisa; /* GISA designation for passthrough */ > u16 vfn; /* virtual function number */ > u16 pchid; /* physical channel ID */ > u8 pfgid; /* function group ID */ > diff --git a/arch/s390/include/asm/pci_clp.h b/arch/s390/include/asm/pci_clp.h > index 1f4b666e85ee..f3286bc5ba6e 100644 > --- a/arch/s390/include/asm/pci_clp.h > +++ b/arch/s390/include/asm/pci_clp.h > @@ -173,7 +173,8 @@ struct clp_req_set_pci { > u16 reserved2; > u8 oc; /* operation controls */ > u8 ndas; /* number of dma spaces */ > - u64 reserved3; > + u32 reserved3; > + u32 gisa; /* GISA designation */ > } __packed; > > /* Set PCI function response */ > diff --git a/arch/s390/pci/pci.c b/arch/s390/pci/pci.c > index e563cb65c0c4..a86cd1cbb80e 100644 > --- a/arch/s390/pci/pci.c > +++ b/arch/s390/pci/pci.c > @@ -120,6 +120,7 @@ int zpci_register_ioat(struct zpci_dev *zdev, u8 dmaas, > fib.pba = base; > fib.pal = limit; > fib.iota = iota | ZPCI_IOTA_RTTO_FLAG; > + fib.gd = zdev->gisa; > cc = zpci_mod_fc(req, &fib, &status); > if (cc) > zpci_dbg(3, "reg ioat fid:%x, cc:%d, status:%d\n", zdev->fid, cc, status); > @@ -133,6 +134,8 @@ int zpci_unregister_ioat(struct zpci_dev *zdev, u8 dmaas) > struct zpci_fib fib = {0}; > u8 cc, status; > > + fib.gd = zdev->gisa; > + > cc = zpci_mod_fc(req, &fib, &status); > if (cc) > zpci_dbg(3, "unreg ioat fid:%x, cc:%d, status:%d\n", zdev->fid, cc, status); > @@ -160,6 +163,7 @@ int zpci_fmb_enable_device(struct zpci_dev *zdev) > atomic64_set(&zdev->unmapped_pages, 0); > > fib.fmb_addr = virt_to_phys(zdev->fmb); > + fib.gd = zdev->gisa; > cc = zpci_mod_fc(req, &fib, &status); > if (cc) { > kmem_cache_free(zdev_fmb_cache, zdev->fmb); > @@ -178,6 +182,8 @@ int zpci_fmb_disable_device(struct zpci_dev *zdev) > if (!zdev->fmb) > return -EINVAL; > > + fib.gd = zdev->gisa; > + > /* Function measurement is disabled if fmb address is zero */ > cc = zpci_mod_fc(req, &fib, &status); > if (cc == 3) /* Function already gone. */ > diff --git a/arch/s390/pci/pci_clp.c b/arch/s390/pci/pci_clp.c > index 1057d7af4a55..deed35edea14 100644 > --- a/arch/s390/pci/pci_clp.c > +++ b/arch/s390/pci/pci_clp.c > @@ -229,12 +229,16 @@ static int clp_set_pci_fn(struct zpci_dev *zdev, u32 *fh, u8 nr_dma_as, u8 comma > { > struct clp_req_rsp_set_pci *rrb; > int rc, retries = 100; > + u32 gisa = 0; > > *fh = 0; > rrb = clp_alloc_block(GFP_KERNEL); > if (!rrb) > return -ENOMEM; > > + if (command != CLP_SET_DISABLE_PCI_FN) > + gisa = zdev->gisa; > + > do { > memset(rrb, 0, sizeof(*rrb)); > rrb->request.hdr.len = sizeof(rrb->request); > @@ -243,6 +247,7 @@ static int clp_set_pci_fn(struct zpci_dev *zdev, u32 *fh, u8 nr_dma_as, u8 comma > rrb->request.fh = zdev->fh; > rrb->request.oc = command; > rrb->request.ndas = nr_dma_as; > + rrb->request.gisa = gisa; > > rc = clp_req(rrb, CLP_LPS_PCI); > if (rrb->response.hdr.rsp == CLP_RC_SETPCIFN_BUSY) { > diff --git a/arch/s390/pci/pci_irq.c b/arch/s390/pci/pci_irq.c > index f2b3145b6697..a2b42a63a53b 100644 > --- a/arch/s390/pci/pci_irq.c > +++ b/arch/s390/pci/pci_irq.c > @@ -43,6 +43,7 @@ static int zpci_set_airq(struct zpci_dev *zdev) > fib.fmt0.aibvo = 0; /* each zdev has its own interrupt vector */ > fib.fmt0.aisb = virt_to_phys(zpci_sbv->vector) + (zdev->aisb / 64) * 8; > fib.fmt0.aisbo = zdev->aisb & 63; > + fib.gd = zdev->gisa; > > return zpci_mod_fc(req, &fib, &status) ? -EIO : 0; > } > @@ -54,6 +55,8 @@ static int zpci_clear_airq(struct zpci_dev *zdev) > struct zpci_fib fib = {0}; > u8 cc, status; > > + fib.gd = zdev->gisa; > + > cc = zpci_mod_fc(req, &fib, &status); > if (cc == 3 || (cc == 1 && status == 24)) > /* Function already gone or IRQs already deregistered. */ > @@ -72,6 +75,7 @@ static int zpci_set_directed_irq(struct zpci_dev *zdev) > fib.fmt = 1; > fib.fmt1.noi = zdev->msi_nr_irqs; > fib.fmt1.dibvo = zdev->msi_first_bit; > + fib.gd = zdev->gisa; > > return zpci_mod_fc(req, &fib, &status) ? -EIO : 0; > } > @@ -84,6 +88,7 @@ static int zpci_clear_directed_irq(struct zpci_dev *zdev) > u8 cc, status; > > fib.fmt = 1; > + fib.gd = zdev->gisa; > cc = zpci_mod_fc(req, &fib, &status); > if (cc == 3 || (cc == 1 && status == 24)) > /* Function already gone or IRQs already deregistered. */ > -- Pierre Morel IBM Lab Boeblingen