Commit 66ae644b authored by Uma Krishnan's avatar Uma Krishnan Committed by Martin K. Petersen

scsi: cxlflash: Register for translation errors

While enabling a context on the link, a predefined callback can be registered
with the OCXL provider services to be notified on translation errors. These
errors can in turn be passed back to the user on a read operation.
Signed-off-by: default avatarUma Krishnan <ukrishn@linux.vnet.ibm.com>
Acked-by: default avatarMatthew R. Ochs <mrochs@linux.vnet.ibm.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent f81face7
...@@ -334,6 +334,25 @@ static u64 ocxlflash_get_irq_objhndl(void *ctx_cookie, int irq) ...@@ -334,6 +334,25 @@ static u64 ocxlflash_get_irq_objhndl(void *ctx_cookie, int irq)
return (__force u64)ctx->irqs[irq].vtrig; return (__force u64)ctx->irqs[irq].vtrig;
} }
/**
* ocxlflash_xsl_fault() - callback when translation error is triggered
* @data: Private data provided at callback registration, the context.
* @addr: Address that triggered the error.
* @dsisr: Value of dsisr register.
*/
static void ocxlflash_xsl_fault(void *data, u64 addr, u64 dsisr)
{
struct ocxlflash_context *ctx = data;
spin_lock(&ctx->slock);
ctx->fault_addr = addr;
ctx->fault_dsisr = dsisr;
ctx->pending_fault = true;
spin_unlock(&ctx->slock);
wake_up_all(&ctx->wq);
}
/** /**
* start_context() - local routine to start a context * start_context() - local routine to start a context
* @ctx: Adapter context to be started. * @ctx: Adapter context to be started.
...@@ -378,7 +397,8 @@ static int start_context(struct ocxlflash_context *ctx) ...@@ -378,7 +397,8 @@ static int start_context(struct ocxlflash_context *ctx)
mm = current->mm; mm = current->mm;
} }
rc = ocxl_link_add_pe(link_token, ctx->pe, pid, 0, 0, mm, NULL, NULL); rc = ocxl_link_add_pe(link_token, ctx->pe, pid, 0, 0, mm,
ocxlflash_xsl_fault, ctx);
if (unlikely(rc)) { if (unlikely(rc)) {
dev_err(dev, "%s: ocxl_link_add_pe failed rc=%d\n", dev_err(dev, "%s: ocxl_link_add_pe failed rc=%d\n",
__func__, rc); __func__, rc);
...@@ -512,6 +532,7 @@ static void *ocxlflash_dev_context_init(struct pci_dev *pdev, void *afu_cookie) ...@@ -512,6 +532,7 @@ static void *ocxlflash_dev_context_init(struct pci_dev *pdev, void *afu_cookie)
ctx->hw_afu = afu; ctx->hw_afu = afu;
ctx->irq_bitmap = 0; ctx->irq_bitmap = 0;
ctx->pending_irq = false; ctx->pending_irq = false;
ctx->pending_fault = false;
out: out:
return ctx; return ctx;
err2: err2:
...@@ -965,7 +986,7 @@ static void *ocxlflash_create_afu(struct pci_dev *pdev) ...@@ -965,7 +986,7 @@ static void *ocxlflash_create_afu(struct pci_dev *pdev)
*/ */
static inline bool ctx_event_pending(struct ocxlflash_context *ctx) static inline bool ctx_event_pending(struct ocxlflash_context *ctx)
{ {
if (ctx->pending_irq) if (ctx->pending_irq || ctx->pending_fault)
return true; return true;
return false; return false;
...@@ -1070,6 +1091,12 @@ static ssize_t afu_read(struct file *file, char __user *buf, size_t count, ...@@ -1070,6 +1091,12 @@ static ssize_t afu_read(struct file *file, char __user *buf, size_t count,
event.irq.irq = bit + 1; event.irq.irq = bit + 1;
if (bitmap_empty(&ctx->irq_bitmap, ctx->num_irqs)) if (bitmap_empty(&ctx->irq_bitmap, ctx->num_irqs))
ctx->pending_irq = false; ctx->pending_irq = false;
} else if (ctx->pending_fault) {
event.header.size += sizeof(struct cxl_event_data_storage);
event.header.type = CXL_EVENT_DATA_STORAGE;
event.fault.addr = ctx->fault_addr;
event.fault.dsisr = ctx->fault_dsisr;
ctx->pending_fault = false;
} }
spin_unlock_irqrestore(&ctx->slock, lock_flags); spin_unlock_irqrestore(&ctx->slock, lock_flags);
......
...@@ -70,4 +70,8 @@ struct ocxlflash_context { ...@@ -70,4 +70,8 @@ struct ocxlflash_context {
int num_irqs; /* Number of interrupts */ int num_irqs; /* Number of interrupts */
bool pending_irq; /* Pending interrupt on the context */ bool pending_irq; /* Pending interrupt on the context */
ulong irq_bitmap; /* Bits indicating pending irq num */ ulong irq_bitmap; /* Bits indicating pending irq num */
u64 fault_addr; /* Address that triggered the fault */
u64 fault_dsisr; /* Value of dsisr register at fault */
bool pending_fault; /* Pending translation fault */
}; };
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment