On 10/03/2021 18:44, Laurent Dufour wrote:
It is better to rely on the API provided by the MM layer instead of
directly manipulating the mm_users field.
Signed-off-by: Laurent Dufour
---
Thanks!
Acked-by: Frederic Barrat
drivers/misc/cxl/fault.c | 2 +-
1 file changed, 1
On 24/12/2020 14:24, Zheng Yongjun wrote:
mutex lock can be initialized automatically with DEFINE_MUTEX()
rather than explicitly calling mutex_init().
Signed-off-by: Zheng Yongjun
---
Thanks!
Acked-by: Frederic Barrat
drivers/misc/ocxl/file.c | 3 +--
1 file changed, 1 insertion
On 14/11/2020 04:37, Alexey Kardashevskiy wrote:
I'll try to go through this patch over the week-end (or more probably
early next week), and try to understand where our understandings
differ.
Great, thanks! Fred spotted a problem with irq_free_descs() not doing
kobject_put() anymore and this
on parameter 'device'
description in 'get_function_0'
Cc: Frederic Barrat
Cc: Andrew Donnellan
Cc: Arnd Bergmann
Cc: Greg Kroah-Hartman
Cc: linuxppc-...@lists.ozlabs.org
Signed-off-by: Lee Jones
---
Thanks!
Acked-by: Frederic Barrat
drivers/misc/ocxl/config.c | 2 +-
1
sting/sysfs-class-ocxl| 3 +
Patches 20, 28 and 31 look good for cxl and ocxl.
Acked-by: Frederic Barrat
Fred
Le 21/09/2020 à 15:10, Qinglang Miao a écrit :
Simplify the return expression.
Signed-off-by: Qinglang Miao
---
Thanks!
Acked-by: Frederic Barrat
drivers/misc/ocxl/core.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/misc/ocxl/core.c b/drivers
Le 19/09/2020 à 20:10, Sasha Levin a écrit :
On Fri, Sep 18, 2020 at 08:35:06AM +0200, Frederic Barrat wrote:
Le 18/09/2020 à 03:57, Sasha Levin a écrit :
From: Frederic Barrat
[ Upstream commit 05dd7da76986937fb288b4213b1fa10dbe0d1b33 ]
This patch is not desirable for stable, for
zil Yildiran
---
OK, that makes sense, thanks!
Acked-by: Frederic Barrat
drivers/misc/ocxl/Kconfig | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/misc/ocxl/Kconfig b/drivers/misc/ocxl/Kconfig
index 6551007a066c..947294f6d7f4 100644
--- a/drivers/misc
Le 18/09/2020 à 03:57, Sasha Levin a écrit :
From: Frederic Barrat
[ Upstream commit 05dd7da76986937fb288b4213b1fa10dbe0d1b33 ]
This patch is not desirable for stable, for 5.4 and 4.19 (it was already
flagged by autosel back in April. Not sure why it's showing again now)
Fred
Le 18/06/2020 à 17:37, Fenghua Yu a écrit :
The first 3 patches clean up pasid and flag defitions to prepare for
following patches.
If you think this patch can be dropped, we will drop it.
Yes, I think that's the case.
Thanks,
Fred
Le 13/06/2020 à 02:41, Fenghua Yu a écrit :
PASID is defined as "int" although it's a 20-bit value and shouldn't be
negative int. To be consistent with type defined in iommu, define PASID
as "unsigned int".
It looks like this patch was considered because of the use of 'pasid' in
variable o
: b087e6190ddc ("cxl: Export optional AFU configuration record in sysfs")
Reported-by: Hulk Robot
Signed-off-by: Wang Hai
Indeed, a call to kobject_put() is needed when the init fails.
Thanks!
Acked-by: Frederic Barrat
---
drivers/misc/cxl/sysfs.c | 2 +-
1 file changed, 1 insertion(+),
diff --git a/drivers/misc/ocxl/link.c b/drivers/misc/ocxl/link.c
index 2874811a4398..9e303a5f4d85 100644
--- a/drivers/misc/ocxl/link.c
+++ b/drivers/misc/ocxl/link.c
@@ -738,7 +738,7 @@ int ocxl_link_add_lpc_mem(void *link_handle, u64 size)
}
EXPORT_SYMBOL_GPL(ocxl_link_add_lpc_mem);
Le 19/09/2019 à 06:55, Alastair D'Silva a écrit :
On Wed, 2019-09-18 at 16:02 +0200, Frederic Barrat wrote:
Le 17/09/2019 à 03:42, Alastair D'Silva a écrit :
From: Alastair D'Silva
Tally up the LPC memory on an OpenCAPI link & allow it to be mapped
Signed-off-b
Le 19/09/2019 à 02:58, Alastair D'Silva a écrit :
On Wed, 2019-09-18 at 16:03 +0200, Frederic Barrat wrote:
Le 17/09/2019 à 03:42, Alastair D'Silva a écrit :
From: Alastair D'Silva
Map & release OpenCAPI LPC memory.
Signed-off-by: Alastair D'Silva
---
arch
Le 17/09/2019 à 03:43, Alastair D'Silva a écrit :
From: Alastair D'Silva
Add functions to map/unmap LPC memory
Signed-off-by: Alastair D'Silva
---
drivers/misc/ocxl/config.c| 4 +++
drivers/misc/ocxl/core.c | 50 +++
drivers/misc/ocxl/link.
Le 17/09/2019 à 03:42, Alastair D'Silva a écrit :
From: Alastair D'Silva
Tally up the LPC memory on an OpenCAPI link & allow it to be mapped
Signed-off-by: Alastair D'Silva
---
drivers/misc/ocxl/core.c | 9 +
drivers/misc/ocxl/link.c | 61 +
Le 17/09/2019 à 03:42, Alastair D'Silva a écrit :
From: Alastair D'Silva
Map & release OpenCAPI LPC memory.
Signed-off-by: Alastair D'Silva
---
arch/powerpc/include/asm/pnv-ocxl.h | 2 ++
arch/powerpc/platforms/powernv/ocxl.c | 42 +++
2 files changed, 44 ins
Le 21/06/2019 à 03:47, Oliver O'Halloran a écrit :
On Thu, May 23, 2019 at 5:51 PM Christoph Hellwig wrote:
These have been unused ever since they've been added to the kernel.
Signed-off-by: Christoph Hellwig
---
arch/powerpc/include/asm/pnv-pci.h| 4 --
arch/powerpc/platforms
a
---
Thanks for the update.
Acked-by: Frederic Barrat
arch/powerpc/mm/book3s64/radix_tlb.c | 5 +
drivers/misc/ocxl/context.c | 9 ++---
drivers/misc/ocxl/link.c | 28
3 files changed, 35 insertions(+), 7 deletions(-)
di
Le 18/06/2019 à 03:50, Andrew Donnellan a écrit :
On 17/6/19 2:41 pm, Alastair D'Silva wrote:
From: Alastair D'Silva
If an OpenCAPI context is to be used directly by a kernel driver, there
may not be a suitable mm to use.
The patch makes the mm parameter to ocxl_context_attach optional.
S
Yamada
---
Thanks!
Acked-by: Frederic Barrat
include/uapi/misc/ocxl.h | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/include/uapi/misc/ocxl.h b/include/uapi/misc/ocxl.h
index 97937cfa3baa..6d29a60a896a 100644
--- a/include/uapi/misc/ocxl.h
+++ b/include
lt n' is
redundant.
...
Signed-off-by: Bartlomiej Zolnierkiewicz
---
for cxl and ocxl:
Acked-by: Frederic Barrat
drivers/misc/Kconfig | 10 --
drivers/misc/altera-stapl/Kconfig |1 -
drivers/misc/c2port/Kconfig |2 --
drivers/misc/cb710/K
Le 25/03/2019 à 17:49, Greg Kurz a écrit :
Hi Alastair,
I forgot to mention it during v3 but please don't link new version
of a patchset to the previous one with --in-reply-to. This is to
ensure I can see them in my email client without having to scroll
back many days in the past (which likel
Le 25/03/2019 à 06:44, Alastair D'Silva a écrit :
From: Alastair D'Silva
External drivers that communicate via OpenCAPI will need to make
MMIO calls to interact with the devices.
Signed-off-by: Alastair D'Silva
Reviewed-by: Greg Kurz
---
Acked-by: Frederic Barrat
Le 25/03/2019 à 06:44, Alastair D'Silva a écrit :
From: Alastair D'Silva
Most OpenCAPI operations require a valid context, so
exposing these functions to external drivers is necessary.
Signed-off-by: Alastair D'Silva
Reviewed-by: Greg Kurz
---
See comment on previous patch regarding mer
Le 27/02/2019 à 09:18, Andrew Donnellan a écrit :
On 27/2/19 7:04 pm, Alastair D'Silva wrote:
-Original Message-
From: Andrew Donnellan
Sent: Wednesday, 27 February 2019 6:55 PM
To: Alastair D'Silva ; 'Alastair D'Silva'
Cc: 'Greg Kurz' ; 'Fr
Le 27/02/2019 à 05:57, Alastair D'Silva a écrit :
From: Alastair D'Silva
Use %# instead of using a literal '0x'
Signed-off-by: Alastair D'Silva
---
I don't really care either way, but it looks ok.
Acked-by: Frederic Barrat
drivers/misc/ocxl/config
any case:
Acked-by: Frederic Barrat
drivers/misc/ocxl/config.c| 2 --
drivers/misc/ocxl/ocxl_internal.h | 26 +-
include/misc/ocxl.h | 23 ---
3 files changed, 25 insertions(+), 26 deletions(-)
diff --git a/drivers/misc/
Le 27/02/2019 à 05:57, Alastair D'Silva a écrit :
From: Alastair D'Silva
The 'extern' keyword adds no value here.
Signed-off-by: Alastair D'Silva
---
Acked-by: Frederic Barrat
drivers/misc/ocxl/ocxl_internal.h | 54 +++-
Le 27/02/2019 à 05:57, Alastair D'Silva a écrit :
From: Alastair D'Silva
No need for a return value in read_pasid as it only returns 0.
Signed-off-by: Alastair D'Silva
Reviewed-by: Greg Kurz
---
Thanks!
Acked-by: Frederic Barrat
drivers/misc/ocxl/config.c | 9 ++-
Le 27/01/2019 à 05:31, Andrew Donnellan a écrit :
[+ linuxppc-dev, because cxl/ocxl are handled through powerpc - please
cc on future versions of this series]
On 26/1/19 8:13 am, Olof Johansson wrote:
We're starting to see more of these kind of devices, the current
upcoming wave will likely
Le 05/12/2018 à 20:16, Rob Herring a écrit :
Remove directly accessing device_type property and use the
of_node_is_type accessor instead. While not using it here, this is
part of eventually removing the struct device_node.type pointer.
Cc: Frederic Barrat
Cc: Arnd Bergmann
Cc: Greg Kroah
Le 04/10/2018 à 07:02, zhong jiang a écrit :
It is not safe to dereference an object before a null test. It is
not needed and just remove them. Ftrace can be used instead.
Signed-off-by: zhong jiang
---
Acked-by: Frederic Barrat
drivers/misc/cxl/guest.c | 2 --
1 file changed, 2
nlock(&ctx->status_mutex);
return -ENOENT;
+ }
O_o Thanks for fixing it
Acked-by: Frederic Barrat
ctx->tidr = current->thread.tidr;
}
Le 11/05/2018 à 12:06, Alastair D'Silva a écrit :
-Original Message-
From: Frederic Barrat
Sent: Friday, 11 May 2018 7:25 PM
To: Alastair D'Silva ; linuxppc-...@lists.ozlabs.org
Cc: linux-kernel@vger.kernel.org; linux-...@vger.kernel.org;
mi...@neuling
der
---
It looks ok, and it passed some basic testing.
Acked-by: Frederic Barrat
Fred
drivers/misc/cxl/context.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/drivers/misc/cxl/context.c b/drivers/misc/cxl/context.c
index 7ff315a..c6ec872 100644
--- a/dri
Le 11/05/2018 à 08:13, Alastair D'Silva a écrit :
From: Alastair D'Silva
Signed-off-by: Alastair D'Silva
---
Acked-by: Frederic Barrat
Documentation/accelerators/ocxl.rst | 11 +++
1 file changed, 11 insertions(+)
diff --git a/Documentation/accelerat
Le 11/05/2018 à 08:13, Alastair D'Silva a écrit :
From: Alastair D'Silva
In order for a userspace AFU driver to call the POWER9 specific
OCXL_IOCTL_ENABLE_P9_WAIT, it needs to verify that it can actually
make that call.
Signed-off-by: Alastair D'Silva
---
Acked-by:
Le 11/05/2018 à 08:13, Alastair D'Silva a écrit :
From: Alastair D'Silva
In order to successfully issue as_notify, an AFU needs to know the TID
to notify, which in turn means that this information should be
available in userspace so it can be communicated to the AFU.
Signed-off-by: Alastair
Le 11/05/2018 à 08:13, Alastair D'Silva a écrit :
From: Alastair D'Silva
The function removes the process element from NPU cache.
Signed-off-by: Alastair D'Silva
---
Acked-by: Frederic Barrat
arch/powerpc/include/asm/pnv-ocxl.h | 2 +-
arch/powerpc/platforms/pow
va
---
Thanks for adding the comment. It assumes the reader is aware that the
TIDR value is only used for the notification using the 'wait'
instruction, but that's likely to be the case.
Reviewed-by: Frederic Barrat
arch/powerpc/include/asm/switch_to.h | 1 -
arch/po
Le 11/05/2018 à 08:12, Alastair D'Silva a écrit :
From: Alastair D'Silva
Switch the use of TIDR on it's CPU feature, rather than assuming it
is available based on architecture.
Signed-off-by: Alastair D'Silva
---
Reviewed-by: Frederic Barrat
arch/powerpc/
Le 11/05/2018 à 08:12, Alastair D'Silva a écrit :
From: Alastair D'Silva
This patch adds a CPU feature bit to show whether the CPU has
the TIDR register available, enabling as_notify/wait in userspace.
Signed-off-by: Alastair D'Silva
---
Reviewed-by: Frederic Barrat
Le 18/04/2018 à 03:08, Alastair D'Silva a écrit :
From: Alastair D'Silva
Signed-off-by: Alastair D'Silva
---
Acked-by: Frederic Barrat
Fred
Documentation/accelerators/ocxl.rst | 11 +++
1 file changed, 11 insertions(+)
diff --git a/Documentation/accelerat
-
Intended?
Other than that,
Acked-by: Frederic Barrat
mmap
diff --git a/drivers/misc/ocxl/file.c b/drivers/misc/ocxl/file.c
index eb409a469f21..33ae46ce0a8a 100644
--- a/drivers/misc/ocxl/file.c
+++ b/drivers/misc/ocxl/file.c
@@ -168,12 +168,32 @@ static long afu_ioctl_enable
Le 18/04/2018 à 03:08, Alastair D'Silva a écrit :
From: Alastair D'Silva
In order to successfully issue as_notify, an AFU needs to know the TID
to notify, which in turn means that this information should be
available in userspace so it can be communicated to the AFU.
Signed-off-by: Alastair
Le 18/04/2018 à 03:08, Alastair D'Silva a écrit :
From: Alastair D'Silva
The function removes the process element from NPU cache.
Signed-off-by: Alastair D'Silva
---
Acked-by: Frederic Barrat
arch/powerpc/include/asm/pnv-ocxl.h | 2 +-
arch/powerpc/platforms/pow
ent with the
above (at the minimum) :-)
With a comment:
Reviewed-by: Frederic Barrat
Fred
Signed-off-by: Alastair D'Silva
---
arch/powerpc/include/asm/switch_to.h | 1 -
arch/powerpc/kernel/process.c| 97 +---
2 files changed, 1 insertion(+),
Le 18/04/2018 à 03:08, Alastair D'Silva a écrit :
From: Alastair D'Silva
Switch the use of TIDR on it's CPU feature, rather than assuming it
is available based on architecture.
Signed-off-by: Alastair D'Silva
---
Reviewed-by: Frederic Barrat
arch/powerpc/
Le 18/04/2018 à 03:08, Alastair D'Silva a écrit :
From: Alastair D'Silva
This patch adds a CPU feature bit to show whether the CPU has
the TIDR register available, enabling as_notify/wait in userspace.
Signed-off-by: Alastair D'Silva
---
arch/powerpc/include/asm/cputable.h | 3 ++-
arch/
: Frederic Barrat
drivers/misc/ocxl/pci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/misc/ocxl/pci.c b/drivers/misc/ocxl/pci.c
index 0051d9e..21f4254 100644
--- a/drivers/misc/ocxl/pci.c
+++ b/drivers/misc/ocxl/pci.c
@@ -519,7 +519,7 @@ static struct ocxl_fn
Le 22/02/2018 à 05:17, Alastair D'Silva a écrit :
From: Alastair D'Silva
Signed-off-by: Alastair D'Silva
---
Acked-by: Frederic Barrat
Documentation/accelerators/ocxl.rst | 5 +
1 file changed, 5 insertions(+)
diff --git a/Documentation/accelerators/ocxl.rst
effort in
userspace.
Signed-off-by: Alastair D'Silva
---
Thanks!
Acked-by: Frederic Barrat
drivers/misc/ocxl/file.c | 27 +++
include/uapi/misc/ocxl.h | 17 +
2 files changed, 44 insertions(+)
diff --git a/drivers/misc/ocxl/file.c b/drivers/m
Le 21/02/2018 à 07:43, Balbir Singh a écrit :
On Wed, Feb 21, 2018 at 3:57 PM, Alastair D'Silva wrote:
From: Alastair D'Silva
Some required information is not exposed to userspace currently (eg. the
PASID), pass this information back, along with other information which
is currently communic
Le 12/02/2018 à 09:58, David Binderman a écrit :
Hello there,
linux-4.16-rc1/drivers/misc/ocxl/file.c:320]: (style) Checking if
unsigned variable 'used' is less than zero.
Source code is
used = append_xsl_error(ctx, &header, buf + sizeof(header));
if (used < 0)
ero: used < 0
Fixes: 5ef3166e8a32 ("ocxl: Driver code for 'generic' opencapi devices")
Signed-off-by: Colin Ian King
---
Thanks!
Acked-by: Frederic Barrat
drivers/misc/ocxl/file.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/misc/ocxl/
Le 25/01/2018 à 14:17, Greg KH a écrit :
On Tue, Jan 23, 2018 at 12:31:47PM +0100, Frederic Barrat wrote:
ocxl.rst gives a quick, high-level view of opencapi.
Update ioctl-number.txt to reflect ioctl numbers being used by the
ocxl driver
Signed-off-by: Frederic Barrat
---
Documentation
host and
device must negotiate what templates are supported on both sides and
at what rates those messages can be sent.
Signed-off-by: Frederic Barrat
Acked-by: Andrew Donnellan
---
arch/powerpc/include/asm/opal-api.h| 5 -
arch/powerpc/include/asm/opal.h| 6
e.
Signed-off-by: Andrew Donnellan
Signed-off-by: Frederic Barrat
---
arch/powerpc/platforms/powernv/pci-ioda.c | 13 +
1 file changed, 13 insertions(+)
diff --git a/arch/powerpc/platforms/powernv/pci-ioda.c
b/arch/powerpc/platforms/powernv/pci-ioda.c
index e780263a14ee..d5af70082
tion being quite general. The number
of available actags on POWER9 makes it more likely to be hit. This is
somewhat mitigated by the fact that existing AFUs are coded by
requesting a reasonable count of actags and existing devices carry
only one AFU.
Signed-off-by: Frederic Barrat
---
arch/powerpc/i
OCXL_BASE triggers the platform support needed by the driver.
Signed-off-by: Frederic Barrat
Signed-off-by: Andrew Donnellan
---
drivers/misc/Kconfig | 1 +
drivers/misc/Makefile | 1 +
drivers/misc/ocxl/Kconfig | 31 +++
drivers/misc/ocxl/Makefile
by
the NPU
- map/unmap some NPU mmio registers to get the fault context when the
NPU raises an address translation fault
The rest are wrappers around the previously-introduced opal calls.
Signed-off-by: Frederic Barrat
---
arch/powerpc/include/asm/pnv-ocxl.h | 29 +
arch/powerpc
hed to the
opencapi device.
Signed-off-by: Frederic Barrat
Signed-off-by: Andrew Donnellan
Signed-off-by: Alastair D'Silva
---
drivers/misc/ocxl/config.c| 712 ++
drivers/misc/ocxl/context.c | 230
drivers/misc/ocxl/file.c
and a write to that page will trigger an interrupt.
Signed-off-by: Frederic Barrat
---
arch/powerpc/include/asm/pnv-ocxl.h | 3 +
arch/powerpc/platforms/powernv/ocxl.c | 30 ++
drivers/misc/ocxl/afu_irq.c | 197 ++
drivers/misc/ocxl/context.c
ocxl.rst gives a quick, high-level view of opencapi.
Update ioctl-number.txt to reflect ioctl numbers being used by the
ocxl driver
Signed-off-by: Frederic Barrat
---
Documentation/ABI/testing/sysfs-class-ocxl | 35 +++
Documentation/accelerators/ocxl.rst| 160
Define a few trace points so that we can use the standard tracing
mechanism for debug and/or monitoring.
Signed-off-by: Frederic Barrat
---
drivers/misc/ocxl/afu_irq.c | 5 ++
drivers/misc/ocxl/context.c | 2 +
drivers/misc/ocxl/link.c| 11 ++-
drivers/misc/ocxl/trace.c | 6
Signed-off-by: Frederic Barrat
Signed-off-by: Andrew Donnellan
---
MAINTAINERS | 12
1 file changed, 12 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index a6e86e20761e..8a0357f3b7bc 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -9820,6 +9820,18 @@ M: Josh Poimboeuf
calling a in-kernel library.
It is still a bit theoretical, for lack of real hardware, and will
likely need adjustements down the road. But we used the cxlflash
driver as a guinea pig.
Signed-off-by: Frederic Barrat
---
drivers/misc/ocxl/config.c| 13 ++-
drivers/misc/ocxl/link.c
unnoticed.
Signed-off-by: Frederic Barrat
Acked-by: Andrew Donnellan
---
drivers/misc/cxl/pci.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/misc/cxl/pci.c b/drivers/misc/cxl/pci.c
index 19969ee86d6f..758842f65a1b 100644
--- a/drivers/misc/cxl/pci.c
+++ b/drivers/misc/cxl/pci.c
12: add documentation for new sysfs files
patch 13: follow alphabetical order for new entry in MAINTAINERS
Andrew Donnellan (1):
powerpc/powernv: Set correct configuration space size for opencapi
devices
Frederic Barrat (12):
powerpc/powernv: Introduce new PHB type for opencapi links
p
e existing PNV_PHB_NPU type to PNV_PHB_NPU_NVLINK and add a
new type PNV_PHB_NPU_OCAPI.
Signed-off-by: Frederic Barrat
Signed-off-by: Andrew Donnellan
---
arch/powerpc/platforms/powernv/npu-dma.c | 2 +-
arch/powerpc/platforms/powernv/pci-ioda.c | 41 +--
ar
Hi,
diff --git a/arch/powerpc/kernel/process.c b/arch/powerpc/kernel/process.c
index 2010e4c..f20c1ad 100644
--- a/arch/powerpc/kernel/process.c
+++ b/arch/powerpc/kernel/process.c
@@ -1560,6 +1560,7 @@ void clear_thread_tidr(struct task_struct *t)
free_thread_tidr(t->thread.tidr);
Le 10/01/2018 à 00:21, Michael Ellerman a écrit :
Would also be great to add something describing the relationship to CAPI and
cxl. Otherwise people will be confused about whether they need this one or the
other one or both.
OK, I'll add something.
Fred
Le 03/01/2018 à 06:48, Andrew Donnellan a écrit :
On 19/12/17 02:21, Frederic Barrat wrote:
OCXL_BASE triggers the platform support needed by the driver.
Signed-off-by: Frederic Barrat
---
drivers/misc/Kconfig | 1 +
drivers/misc/Makefile | 1 +
drivers/misc/ocxl/Kconfig
Le 18/12/2017 à 17:48, Philippe Ombredanne a écrit :
--- /dev/null
+++ b/drivers/misc/ocxl/trace.h
@@ -0,0 +1,189 @@
+/*
+ * Copyright 2017 IBM Corp.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License
+ * as publ
Le 18/12/2017 à 17:04, Joe Perches a écrit :
+OCXL (Open Coherent Accelerator Processor Interface OpenCAPI) DRIVER
+M: Frederic Barrat
+M: Andrew Donnellan
+L: linuxppc-...@lists.ozlabs.org
+S: Supported
+F: arch/powerpc/platforms/powernv/ocxl.c
+F: arch/powerpc/include
ize.
Signed-off-by: Andrew Donnellan
Signed-off-by: Frederic Barrat
---
arch/powerpc/platforms/powernv/pci-ioda.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/arch/powerpc/platforms/powernv/pci-ioda.c
b/arch/powerpc/platforms/powernv/pci-ioda.c
index c37b5d288f9c..b8ec76aa266f
e existing PNV_PHB_NPU type to PNV_PHB_NPU_NVLINK and add a
new type PNV_PHB_NPU_OCAPI.
Signed-off-by: Frederic Barrat
Signed-off-by: Andrew Donnellan
---
arch/powerpc/platforms/powernv/npu-dma.c | 2 +-
arch/powerpc/platforms/powernv/pci-ioda.c | 46 +++
ar
tion being quite general. The number
of available actags on POWER9 makes it more likely to be hit. This is
somewhat mitigated by the fact that existing AFUs are coded by
requesting a reasonable count of actags and existing devices carry
only one AFU.
Signed-off-by: Frederic Barrat
---
arch/powerpc/i
calling a in-kernel library.
It is still a bit theoretical, for lack of real hardware, and will
likely need adjustements down the road. But we used the cxlflash
driver as a guinea pig.
Signed-off-by: Frederic Barrat
---
drivers/misc/ocxl/config.c| 13 ++-
drivers/misc/ocxl/link.c
and a write to that page will trigger an interrupt.
Signed-off-by: Frederic Barrat
---
arch/powerpc/include/asm/pnv-ocxl.h | 3 +
arch/powerpc/platforms/powernv/ocxl.c | 30 +
drivers/misc/ocxl/afu_irq.c | 204 ++
drivers/misc/ocxl/context.c
hed to the
opencapi device.
Signed-off-by: Frederic Barrat
Signed-off-by: Andrew Donnellan
Signed-off-by: Alastair D'Silva
---
drivers/misc/ocxl/config.c| 718 ++
drivers/misc/ocxl/context.c | 237 +
drivers/misc/ocxl/file.c
OCXL_BASE triggers the platform support needed by the driver.
Signed-off-by: Frederic Barrat
---
drivers/misc/Kconfig | 1 +
drivers/misc/Makefile | 1 +
drivers/misc/ocxl/Kconfig | 25 +
drivers/misc/ocxl/Makefile | 10 ++
4 files changed, 37
ocxl.rst gives a quick, high-level view of opencapi.
Update ioctl-number.txt to reflect ioctl numbers being used by the
ocxl driver
Signed-off-by: Frederic Barrat
---
Documentation/accelerators/ocxl.rst | 151 +++
Documentation/ioctl/ioctl-number.txt | 1 +
2
Define a few trace points so that we can use the standard tracing
mechanism for debug and/or monitoring.
Signed-off-by: Frederic Barrat
---
drivers/misc/ocxl/afu_irq.c | 5 ++
drivers/misc/ocxl/context.c | 2 +
drivers/misc/ocxl/link.c| 11 ++-
drivers/misc/ocxl/trace.c | 13
Signed-off-by: Frederic Barrat
Signed-off-by: Andrew Donnellan
---
MAINTAINERS | 12
1 file changed, 12 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index a6e86e20761e..edc9e1db352b 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3919,6 +3919,18 @@ F: drivers/scsi
unnoticed.
Signed-off-by: Frederic Barrat
---
drivers/misc/cxl/pci.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/misc/cxl/pci.c b/drivers/misc/cxl/pci.c
index 19969ee86d6f..758842f65a1b 100644
--- a/drivers/misc/cxl/pci.c
+++ b/drivers/misc/cxl/pci.c
@@ -125,8 +125,6 @@ static co
host and
device must negotiate what templates are supported on both sides and
at what rates those messages can be sent.
Signed-off-by: Frederic Barrat
---
arch/powerpc/include/asm/opal-api.h| 5 -
arch/powerpc/include/asm/opal.h| 6 ++
arch/powerpc/platforms
he adapter-local memory
Many people contributed directly or indirectly, from the software,
hardware and bringup teams. In particular Andrew Donnellan and
Alastair D'Silva, who are developing the related firmware and library.
Feedback welcome!
Frederic Barrat (13):
powerpc/powernv: Introdu
by
the NPU
- map/unmap some NPU mmio registers to get the fault context when the
NPU raises an address translation fault
The rest are wrappers around the previously-introduced opal calls.
Signed-off-by: Frederic Barrat
---
arch/powerpc/include/asm/pnv-ocxl.h | 36 ++
arch/powerpc
Le 19/01/2017 à 04:50, Greg Kurz a écrit :
The kernel API does not use anything from this header file.
Signed-off-by: Greg Kurz
---
Acked-by: Frederic Barrat
Le 30/10/2016 à 22:40, Christophe JAILLET a écrit :
'cxl_dev_context_init()' returns an error pointer in case of error, not
NULL. So test it with IS_ERR.
Signed-off-by: Christophe JAILLET
---
Thanks for the 3 patches!
Acked-by: Frederic Barrat
hile fixing it, simplify a bit the code.
Signed-off-by: Christophe JAILLET
---
Acked-by: Frederic Barrat
Le 29/07/2016 à 13:38, Michael Ellerman a écrit :
But who does keep a reference on the device_node? I can't see it anywhere. Which
means in theory the device_node can be freed out from under you.
You have a reference for afu_np as part of for_each_child_of_node(), but it's
dropped as soon as you
y: SF Markus Elfring
Reported-by: Julia Lawall
Signed-off-by: Andrew Donnellan
Thanks!
Reviewed-by: Frederic Barrat
Hi Vaibhav,
Le 09/03/2016 15:37, Vaibhav Jain a écrit :
I would propose these two apis.
/*
* fetches an event from the driver event queue. NULL means that queue
* is empty. Can sleep if needed. The memory for cxl_event is allocated
* by module being called. Hence it can be potentially be la
Hi Ian,
Le 08/03/2016 02:48, Ian Munsie a écrit :
diff --git a/drivers/misc/cxl/file.c b/drivers/misc/cxl/file.c
...
+static inline bool ctx_event_pending(struct cxl_context *ctx)
+{
+ if (ctx->pending_irq || ctx->pending_fault || ctx->pending_afu_err)
+ return true;
+
+
98 matches
Mail list logo
