| Message ID | 20180417051102.7577-1-vaibhav@linux.vnet.ibm.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 9a6d2022bacd8fca0be6297459a02dfd28dad6ba |
| Headers | show |
| Series | cxl: Configure PSL to not use APC virtual machines | expand |
On 17/04/18 15:11, Vaibhav Jain wrote: > APC virtual machines arent used on POWER-9 chips and are already > disabled in on-chip CAPP. They also need to be disabled on the PSL via > 'PSL Data Send Control Register' by setting bit(47). This forces the > PSL to send commands to CAPP with queue.id == 0. > > Signed-off-by: Vaibhav Jain <vaibhav@linux.vnet.ibm.com> LGTM. Does this need to be sent to stable? Acked-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com>
On Tue, 2018-04-17 at 10:41 +0530, Vaibhav Jain wrote: > APC virtual machines arent used on POWER-9 chips and are already > disabled in on-chip CAPP. They also need to be disabled on the PSL > via > 'PSL Data Send Control Register' by setting bit(47). This forces the > PSL to send commands to CAPP with queue.id == 0. > > Signed-off-by: Vaibhav Jain <vaibhav@linux.vnet.ibm.com> > --- > drivers/misc/cxl/pci.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/misc/cxl/pci.c b/drivers/misc/cxl/pci.c > index c32432168e6b..af30ee848d35 100644 > --- a/drivers/misc/cxl/pci.c > +++ b/drivers/misc/cxl/pci.c > @@ -516,9 +516,9 @@ static int > init_implementation_adapter_regs_psl9(struct cxl *adapter, > cxl_p1_write(adapter, CXL_PSL9_FIR_CNTL, psl_fircntl); > > /* Setup the PSL to transmit packets on the PCIe before the > - * CAPP is enabled > + * CAPP is enabled. Make sure that CAPP virtual machines are > disabled > */ > - cxl_p1_write(adapter, CXL_PSL9_DSNDCTL, > 0x0001001000002A10ULL); > + cxl_p1_write(adapter, CXL_PSL9_DSNDCTL, > 0x0001001000012A10ULL); > > /* > * A response to an ASB_Notify request is returned by the > Reviewed-by: Alastair D'Silva <alastair@d-silva.org>
Le 17/04/2018 à 07:11, Vaibhav Jain a écrit : > APC virtual machines arent used on POWER-9 chips and are already > disabled in on-chip CAPP. They also need to be disabled on the PSL via > 'PSL Data Send Control Register' by setting bit(47). This forces the > PSL to send commands to CAPP with queue.id == 0. > > Signed-off-by: Vaibhav Jain <vaibhav@linux.vnet.ibm.com> > --- > drivers/misc/cxl/pci.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/misc/cxl/pci.c b/drivers/misc/cxl/pci.c > index c32432168e6b..af30ee848d35 100644 > --- a/drivers/misc/cxl/pci.c > +++ b/drivers/misc/cxl/pci.c > @@ -516,9 +516,9 @@ static int init_implementation_adapter_regs_psl9(struct cxl *adapter, > cxl_p1_write(adapter, CXL_PSL9_FIR_CNTL, psl_fircntl); > > /* Setup the PSL to transmit packets on the PCIe before the > - * CAPP is enabled > + * CAPP is enabled. Make sure that CAPP virtual machines are disabled > */ > - cxl_p1_write(adapter, CXL_PSL9_DSNDCTL, 0x0001001000002A10ULL); > + cxl_p1_write(adapter, CXL_PSL9_DSNDCTL, 0x0001001000012A10ULL); > > /* > * A response to an ASB_Notify request is returned by the > Thanks Reviewed-by: Christophe Lombard <clombard@linux.vnet.ibm.com>
On Tue, 2018-04-17 at 05:11:02 UTC, Vaibhav Jain wrote: > APC virtual machines arent used on POWER-9 chips and are already > disabled in on-chip CAPP. They also need to be disabled on the PSL via > 'PSL Data Send Control Register' by setting bit(47). This forces the > PSL to send commands to CAPP with queue.id == 0. > > Signed-off-by: Vaibhav Jain <vaibhav@linux.vnet.ibm.com> > Acked-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com> > Reviewed-by: Alastair D'Silva <alastair@d-silva.org> > Reviewed-by: Christophe Lombard <clombard@linux.vnet.ibm.com> Applied to powerpc next, thanks. https://git.kernel.org/powerpc/c/9a6d2022bacd8fca0be6297459a02d cheers
diff --git a/drivers/misc/cxl/pci.c b/drivers/misc/cxl/pci.c index c32432168e6b..af30ee848d35 100644 --- a/drivers/misc/cxl/pci.c +++ b/drivers/misc/cxl/pci.c @@ -516,9 +516,9 @@ static int init_implementation_adapter_regs_psl9(struct cxl *adapter, cxl_p1_write(adapter, CXL_PSL9_FIR_CNTL, psl_fircntl); /* Setup the PSL to transmit packets on the PCIe before the - * CAPP is enabled + * CAPP is enabled. Make sure that CAPP virtual machines are disabled */ - cxl_p1_write(adapter, CXL_PSL9_DSNDCTL, 0x0001001000002A10ULL); + cxl_p1_write(adapter, CXL_PSL9_DSNDCTL, 0x0001001000012A10ULL); /* * A response to an ASB_Notify request is returned by the
APC virtual machines arent used on POWER-9 chips and are already disabled in on-chip CAPP. They also need to be disabled on the PSL via 'PSL Data Send Control Register' by setting bit(47). This forces the PSL to send commands to CAPP with queue.id == 0. Signed-off-by: Vaibhav Jain <vaibhav@linux.vnet.ibm.com> --- drivers/misc/cxl/pci.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)