nvrm: move stale wait checking into the kernel tegra-11.2.6
authorChris Johnson <cwj@nvidia.com>
Sat, 30 Apr 2011 21:24:44 +0000 (14:24 -0700)
committerNiket Sirsi <nsirsi@nvidia.com>
Mon, 16 May 2011 22:15:20 +0000 (15:15 -0700)
The kernel now receives wait tracking data (similar to gathers and
relocs) and compares the current syncpt with the threshold value.

If it's old, it gets a kernel mapping and rewrites the method data
to use a kernel reserved syncpt that is always 0 (so trivially pops
when seen by the HW).

Bug 519650
Bug 785525
Bug 803452

Note: reset the author in this commit to fix a email addr problem
and since from the latest/last cherry pick there was a reworking
of the code to be compatible with different user space versions
it also seemed reasonable.

(cherry picked from commit 4069d8e67665624ad3dceb628e572980dd57acd0)
(cherry picked from commit 6e4336408588e348804a62e53386acc9abc06823)
(cherry picked from commit 87a9efe751716ca741caac72b9061fdfdcec540a)

Change-Id: I9c6076da2384f373d5f402bee4406b09b4ebc4ff
Reviewed-on: http://git-master/r/23159
Reviewed-on: http://git-master/r/30281
Tested-by: Chris Johnson <cwj@nvidia.com>
Reviewed-by: Ken Adams <kadams@nvidia.com>
Reviewed-by: Prajakta Gudadhe <pgudadhe@nvidia.com>
arch/arm/mach-tegra/include/mach/nvhost.h
arch/arm/mach-tegra/include/mach/nvmap.h
drivers/video/tegra/host/dev.c
drivers/video/tegra/host/nvhost_channel.h
drivers/video/tegra/host/nvhost_hardware.h
drivers/video/tegra/host/nvhost_syncpt.c
drivers/video/tegra/host/nvhost_syncpt.h
drivers/video/tegra/nvmap/nvmap.c

index fc0336a42b5058b3c0e9e20125cbc1b53a855f00..fc18f2e6292eb07aa6730100835948bc8e16fb21 100644 (file)
@@ -76,6 +76,7 @@ int nvhost_bus_register(struct nvhost_master *host);
 #define NVHOST_NO_TIMEOUT (-1)
 #define NVHOST_IOCTL_MAGIC 'H'
 
+/* version 0 header */
 struct nvhost_submit_hdr {
        __u32 syncpt_id;
        __u32 syncpt_incrs;
@@ -83,6 +84,22 @@ struct nvhost_submit_hdr {
        __u32 num_relocs;
 };
 
+#define NVHOST_SUBMIT_VERSION_V0                               0x0
+#define NVHOST_SUBMIT_VERSION_V1                               0x1
+#define NVHOST_SUBMIT_VERSION_MAX_SUPPORTED            NVHOST_SUBMIT_VERSION_V1
+
+/* version 1 header (same as version 0, w/ addt'l fields) */
+struct nvhost_submit_hdr_ext {
+       __u32 syncpt_id;                /* version 0 fields */
+       __u32 syncpt_incrs;
+       __u32 num_cmdbufs;
+       __u32 num_relocs;
+       __u32 submit_version;   /* version 1 fields */
+       __u32 num_waitchks;
+       __u32 waitchk_mask;
+       __u32 pad[5];                   /* future expansion */
+};
+
 struct nvhost_cmdbuf {
        __u32 mem;
        __u32 offset;
@@ -96,6 +113,13 @@ struct nvhost_reloc {
        __u32 target_offset;
 };
 
+struct nvhost_waitchk {
+       __u32 mem;
+       __u32 offset;
+       __u32 syncpt_id;
+       __u32 thresh;
+};
+
 struct nvhost_get_param_args {
        __u32 value;
 };
@@ -116,9 +140,11 @@ struct nvhost_set_nvmap_fd_args {
        _IOW(NVHOST_IOCTL_MAGIC, 5, struct nvhost_set_nvmap_fd_args)
 #define NVHOST_IOCTL_CHANNEL_NULL_KICKOFF      \
        _IOR(NVHOST_IOCTL_MAGIC, 6, struct nvhost_get_param_args)
+#define NVHOST_IOCTL_CHANNEL_SUBMIT_EXT        \
+       _IOW(NVHOST_IOCTL_MAGIC, 7, struct nvhost_submit_hdr_ext)
 #define NVHOST_IOCTL_CHANNEL_LAST              \
-       _IOC_NR(NVHOST_IOCTL_CHANNEL_NULL_KICKOFF)
-#define NVHOST_IOCTL_CHANNEL_MAX_ARG_SIZE sizeof(struct nvhost_get_param_args)
+       _IOC_NR(NVHOST_IOCTL_CHANNEL_SUBMIT_EXT)
+#define NVHOST_IOCTL_CHANNEL_MAX_ARG_SIZE sizeof(struct nvhost_submit_hdr_ext)
 
 struct nvhost_ctrl_syncpt_read_args {
        __u32 id;
index 7422d1a44d8a09a8d941bf7836d1e80092882820..7a79748e54333fc61a1b723609e2b70c5f618289 100644 (file)
@@ -97,6 +97,10 @@ int nvmap_pin_array(struct nvmap_client *client, struct nvmap_handle *gather,
 void nvmap_unpin_handles(struct nvmap_client *client,
                         struct nvmap_handle **h, int nr);
 
+int nvmap_patch_wait(struct nvmap_client *client,
+                    struct nvmap_handle *patch,
+                    u32 patch_offset, u32 patch_value);
+
 struct nvmap_platform_carveout {
        const char *name;
        unsigned int usage_mask;
index 8b577cc9045fbd351a2c4c203941e8443697f1ba..fdbf5cc8efc163893da842eb81b78ca2141132f8 100644 (file)
@@ -46,10 +46,7 @@ static int nvhost_minor = NVHOST_CHANNEL_BASE;
 struct nvhost_channel_userctx {
        struct nvhost_channel *ch;
        struct nvhost_hwctx *hwctx;
-       u32 syncpt_id;
-       u32 syncpt_incrs;
-       u32 cmdbufs_pending;
-       u32 relocs_pending;
+       struct nvhost_submit_hdr_ext hdr;
        struct nvmap_handle_ref *gather_mem;
        struct nvhost_op_pair *gathers;
        int num_gathers;
@@ -57,6 +54,8 @@ struct nvhost_channel_userctx {
        struct nvmap_pinarray_elem pinarray[NVHOST_MAX_HANDLES];
        struct nvmap_handle *unpinarray[NVHOST_MAX_HANDLES];
        struct nvmap_client *nvmap;
+       struct nvhost_waitchk waitchks[NVHOST_MAX_WAIT_CHECKS];
+       u32 num_waitchks;
 };
 
 struct nvhost_ctrl_userctx {
@@ -137,10 +136,25 @@ static void add_gather(struct nvhost_channel_userctx *ctx, int idx,
        ctx->gathers[idx].op1 = nvhost_opcode_gather(0, words);
 }
 
+static int set_submit(struct nvhost_channel_userctx *ctx)
+{
+       int err = 0;
+
+       /* submit should have at least 1 cmdbuf */
+       if (!ctx->hdr.num_cmdbufs)
+               return -EFAULT;
+
+       /* leave room for ctx switch */
+       ctx->num_gathers = 2;
+       ctx->pinarray_size = 0;
+       return err;
+}
+
 static void reset_submit(struct nvhost_channel_userctx *ctx)
 {
-       ctx->cmdbufs_pending = 0;
-       ctx->relocs_pending = 0;
+       ctx->hdr.num_cmdbufs = 0;
+       ctx->hdr.num_relocs = 0;
+       ctx->hdr.num_waitchks = 0;
 }
 
 static ssize_t nvhost_channelwrite(struct file *filp, const char __user *buf,
@@ -152,22 +166,19 @@ static ssize_t nvhost_channelwrite(struct file *filp, const char __user *buf,
 
        while (remaining) {
                size_t consumed;
-               if (!priv->relocs_pending && !priv->cmdbufs_pending) {
+               if (!priv->hdr.num_relocs && !priv->hdr.num_cmdbufs && !priv->hdr.num_waitchks) {
                        consumed = sizeof(struct nvhost_submit_hdr);
                        if (remaining < consumed)
                                break;
-                       if (copy_from_user(&priv->syncpt_id, buf, consumed)) {
+                       if (copy_from_user(&priv->hdr, buf, consumed)) {
                                err = -EFAULT;
                                break;
                        }
-                       if (!priv->cmdbufs_pending) {
-                               err = -EFAULT;
+                       BUG_ON(priv->hdr.submit_version != NVHOST_SUBMIT_VERSION_V0);
+                       err = set_submit(priv);
+                       if (err)
                                break;
-                       }
-                       /* leave room for ctx switch */
-                       priv->num_gathers = 2;
-                       priv->pinarray_size = 0;
-               } else if (priv->cmdbufs_pending) {
+               } else if (priv->hdr.num_cmdbufs) {
                        struct nvhost_cmdbuf cmdbuf;
                        consumed = sizeof(cmdbuf);
                        if (remaining < consumed)
@@ -178,12 +189,12 @@ static ssize_t nvhost_channelwrite(struct file *filp, const char __user *buf,
                        }
                        add_gather(priv, priv->num_gathers++,
                                   cmdbuf.mem, cmdbuf.words, cmdbuf.offset);
-                       priv->cmdbufs_pending--;
-               } else if (priv->relocs_pending) {
+                       priv->hdr.num_cmdbufs--;
+               } else if (priv->hdr.num_relocs) {
                        int numrelocs = remaining / sizeof(struct nvhost_reloc);
                        if (!numrelocs)
                                break;
-                       numrelocs = min_t(int, numrelocs, priv->relocs_pending);
+                       numrelocs = min_t(int, numrelocs, priv->hdr.num_relocs);
                        consumed = numrelocs * sizeof(struct nvhost_reloc);
                        if (copy_from_user(&priv->pinarray[priv->pinarray_size],
                                                buf, consumed)) {
@@ -191,7 +202,19 @@ static ssize_t nvhost_channelwrite(struct file *filp, const char __user *buf,
                                break;
                        }
                        priv->pinarray_size += numrelocs;
-                       priv->relocs_pending -= numrelocs;
+                       priv->hdr.num_relocs -= numrelocs;
+               } else if (priv->hdr.num_waitchks) {
+                       struct nvhost_waitchk *waitp;
+                       consumed = sizeof(struct nvhost_waitchk);
+                       if (remaining < consumed)
+                               break;
+                       waitp = &priv->waitchks[priv->num_waitchks];
+                       if (copy_from_user(waitp, buf, consumed)) {
+                               err = -EFAULT;
+                               break;
+                       }
+                       priv->num_waitchks++;
+                       priv->hdr.num_waitchks--;
                } else {
                        err = -EFAULT;
                        break;
@@ -219,9 +242,9 @@ static int nvhost_ioctl_channel_flush(struct nvhost_channel_userctx *ctx,
        u32 syncval;
        int num_unpin;
        int err;
-       int nulled_incrs = null_kickoff ? ctx->syncpt_incrs : 0;
+       int nulled_incrs = null_kickoff ? ctx->hdr.syncpt_incrs : 0;
 
-       if (ctx->relocs_pending || ctx->cmdbufs_pending) {
+       if (ctx->hdr.num_relocs || ctx->hdr.num_cmdbufs || ctx->hdr.num_waitchks) {
                reset_submit(ctx);
                dev_err(&ctx->ch->dev->pdev->dev, "channel submit out of sync\n");
                return -EFAULT;
@@ -256,6 +279,22 @@ static int nvhost_ioctl_channel_flush(struct nvhost_channel_userctx *ctx,
                return err;
        }
 
+       /* remove stale waits */
+       if (ctx->num_waitchks) {
+               err = nvhost_syncpt_wait_check(ctx->nvmap,
+                               &ctx->ch->dev->syncpt, ctx->hdr.waitchk_mask,
+                               ctx->waitchks, ctx->num_waitchks);
+               if (err) {
+                       dev_warn(&ctx->ch->dev->pdev->dev,
+                               "nvhost_syncpt_wait_check failed: %d\n", err);
+                       mutex_unlock(&ctx->ch->submitlock);
+                       nvmap_unpin_handles(ctx->nvmap, ctx->unpinarray, num_unpin);
+                       nvhost_module_idle(&ctx->ch->mod);
+                       return err;
+               }
+               ctx->num_waitchks = 0;
+       }
+
        /* context switch */
        if (ctx->ch->cur_ctx != ctx->hwctx) {
                struct nvhost_hwctx *hw = ctx->hwctx;
@@ -264,7 +303,7 @@ static int nvhost_ioctl_channel_flush(struct nvhost_channel_userctx *ctx,
                        ctx->gathers[gather_idx].op1 =
                                nvhost_opcode_gather(0, hw->restore_size);
                        ctx->gathers[gather_idx].op2 = hw->restore_phys;
-                       ctx->syncpt_incrs += hw->restore_incrs;
+                       ctx->hdr.syncpt_incrs += hw->restore_incrs;
                }
                hw = ctx->ch->cur_ctx;
                if (hw) {
@@ -272,7 +311,7 @@ static int nvhost_ioctl_channel_flush(struct nvhost_channel_userctx *ctx,
                        ctx->gathers[gather_idx].op1 =
                                nvhost_opcode_gather(0, hw->save_size);
                        ctx->gathers[gather_idx].op2 = hw->save_phys;
-                       ctx->syncpt_incrs += hw->save_incrs;
+                       ctx->hdr.syncpt_incrs += hw->save_incrs;
                        num_intrs = 1;
                        ctxsw.syncpt_val = hw->save_incrs - 1;
                        ctxsw.intr_data = hw;
@@ -291,24 +330,24 @@ static int nvhost_ioctl_channel_flush(struct nvhost_channel_userctx *ctx,
        }
 
        /* get absolute sync value */
-       if (BIT(ctx->syncpt_id) & NVSYNCPTS_CLIENT_MANAGED)
+       if (BIT(ctx->hdr.syncpt_id) & NVSYNCPTS_CLIENT_MANAGED)
                syncval = nvhost_syncpt_set_max(&ctx->ch->dev->syncpt,
-                                               ctx->syncpt_id, ctx->syncpt_incrs);
+                                               ctx->hdr.syncpt_id, ctx->hdr.syncpt_incrs);
        else
                syncval = nvhost_syncpt_incr_max(&ctx->ch->dev->syncpt,
-                                               ctx->syncpt_id, ctx->syncpt_incrs);
+                                               ctx->hdr.syncpt_id, ctx->hdr.syncpt_incrs);
 
        /* patch absolute syncpt value into interrupt triggers */
-       ctxsw.syncpt_val += syncval - ctx->syncpt_incrs;
+       ctxsw.syncpt_val += syncval - ctx->hdr.syncpt_incrs;
 
        nvhost_channel_submit(ctx->ch, ctx->nvmap, &ctx->gathers[gather_idx],
                              (null_kickoff ? 2 : ctx->num_gathers) - gather_idx, &ctxsw, num_intrs,
                              ctx->unpinarray, num_unpin,
-                             ctx->syncpt_id, syncval,
+                             ctx->hdr.syncpt_id, syncval,
                              nulled_incrs);
 
        /* schedule a submit complete interrupt */
-       nvhost_intr_add_action(&ctx->ch->dev->intr, ctx->syncpt_id, syncval,
+       nvhost_intr_add_action(&ctx->ch->dev->intr, ctx->hdr.syncpt_id, syncval,
                        NVHOST_INTR_ACTION_SUBMIT_COMPLETE, ctx->ch, NULL);
 
        mutex_unlock(&ctx->ch->submitlock);
@@ -342,7 +381,29 @@ static long nvhost_channelctl(struct file *filp,
        case NVHOST_IOCTL_CHANNEL_NULL_KICKOFF:
                err = nvhost_ioctl_channel_flush(priv, (void *)buf, 1);
                break;
+       case NVHOST_IOCTL_CHANNEL_SUBMIT_EXT:
+       {
+               struct nvhost_submit_hdr_ext *hdr = (struct nvhost_submit_hdr_ext *)buf;
+
+               if (priv->hdr.num_relocs || priv->hdr.num_cmdbufs || priv->hdr.num_waitchks) {
+                       reset_submit(priv);
+                       dev_err(&priv->ch->dev->pdev->dev, "channel submit out of sync\n");
+                       err = -EFAULT;
+                       break;
+               }
+               if (hdr->submit_version > NVHOST_SUBMIT_VERSION_MAX_SUPPORTED) {
+                       dev_err(&priv->ch->dev->pdev->dev, "submit version %d > max supported %d\n",
+                               hdr->submit_version, NVHOST_SUBMIT_VERSION_MAX_SUPPORTED);
+                       err = -EINVAL;
+                       break;
+               }
+               memcpy(&priv->hdr, hdr, sizeof(struct nvhost_submit_hdr_ext));
+               err = set_submit(priv);
+               break;
+       }
        case NVHOST_IOCTL_CHANNEL_GET_SYNCPOINTS:
+               /* host syncpt ID is used by the RM (and never be given out) */
+               BUG_ON(priv->ch->desc->syncpts & (1 << NVSYNCPT_GRAPHICS_HOST));
                ((struct nvhost_get_param_args *)buf)->value =
                        priv->ch->desc->syncpts;
                break;
index 59ba06543a48fb35f843a2835517bb7d37aa63d3..37ac769d4a612149e841ce5247014b68fc28bf07 100644 (file)
@@ -32,6 +32,7 @@
 
 #define NVHOST_CHANNEL_BASE 0
 #define NVHOST_NUMCHANNELS (NV_HOST1X_CHANNELS - 1)
+#define NVHOST_MAX_WAIT_CHECKS 256
 #define NVHOST_MAX_GATHERS 512
 #define NVHOST_MAX_HANDLES 1280
 
index a7663489727e754bcaf5733815ace44fa4be99f0..d1411b515ea5364841addc30ac763f1902cf83ce 100644 (file)
@@ -141,6 +141,12 @@ enum {
        NV_CLASS_HOST_INDDATA = 0x2e
 };
 
+static inline u32 nvhost_class_host_wait_syncpt(
+       unsigned indx, unsigned threshold)
+{
+       return (indx << 24) | (threshold & 0xffffff);
+}
+
 static inline u32 nvhost_class_host_wait_syncpt_base(
        unsigned indx, unsigned base_indx, unsigned offset)
 {
index dd2ab0d379e04b48a1a715034489248950f8c37d..1881716ed4288a37f5c38a6c812b4b1ff589424f 100644 (file)
@@ -226,7 +226,7 @@ done:
 }
 
 static const char *s_syncpt_names[32] = {
-       "", "", "", "", "", "", "", "", "", "", "", "",
+       "gfx_host", "", "", "", "", "", "", "", "", "", "", "",
        "vi_isp_0", "vi_isp_1", "vi_isp_2", "vi_isp_3", "vi_isp_4", "vi_isp_5",
        "2d_0", "2d_1",
        "", "",
@@ -254,3 +254,63 @@ void nvhost_syncpt_debug(struct nvhost_syncpt *sp)
 
        }
 }
+
+/* returns true, if a <= b < c using wrapping comparison */
+static inline bool nvhost_syncpt_is_between(u32 a, u32 b, u32 c)
+{
+       return b-a < c-a;
+}
+
+/* returns true, if x >= y (mod 1 << 32) */
+static bool nvhost_syncpt_wrapping_comparison(u32 x, u32 y)
+{
+       return nvhost_syncpt_is_between(y, x, (1UL<<31UL)+y);
+}
+
+/* check for old WAITs to be removed (avoiding a wrap) */
+int nvhost_syncpt_wait_check(struct nvmap_client *nvmap,
+                       struct nvhost_syncpt *sp, u32 waitchk_mask,
+                       struct nvhost_waitchk *waitp, u32 waitchks)
+{
+       u32 idx;
+       int err = 0;
+
+       /* get current syncpt values */
+       for (idx = 0; idx < NV_HOST1X_SYNCPT_NB_PTS; idx++) {
+               if (BIT(idx) & waitchk_mask) {
+                       nvhost_syncpt_update_min(sp, idx);
+               }
+       }
+
+       BUG_ON(!waitp);
+
+       /* compare syncpt vs wait threshold */
+       while (waitchks) {
+               u32 syncpt, override;
+
+               BUG_ON(waitp->syncpt_id > NV_HOST1X_SYNCPT_NB_PTS);
+
+               syncpt = atomic_read(&sp->min_val[waitp->syncpt_id]);
+               if (nvhost_syncpt_wrapping_comparison(syncpt, waitp->thresh)) {
+
+                       /* wait has completed already, so can be removed */
+                       dev_dbg(&syncpt_to_dev(sp)->pdev->dev,
+                                       "drop WAIT id %d (%s) thresh 0x%x, syncpt 0x%x\n",
+                                       waitp->syncpt_id,  nvhost_syncpt_name(waitp->syncpt_id),
+                                       waitp->thresh, syncpt);
+
+                       /* move wait to a kernel reserved syncpt (that's always 0) */
+                       override = nvhost_class_host_wait_syncpt(NVSYNCPT_GRAPHICS_HOST, 0);
+
+                       /* patch the wait */
+                       err = nvmap_patch_wait(nvmap,
+                                               (struct nvmap_handle *)waitp->mem,
+                                               waitp->offset, override);
+                       if (err)
+                               break;
+               }
+               waitchks--;
+               waitp++;
+       }
+       return err;
+}
index f161f205140676d7afb8a5fc4f60d885196156de..b4ce3c6ee6d4df76e813c18d510ca7a226b00d87 100644 (file)
 
 #include <linux/kernel.h>
 #include <linux/sched.h>
+#include <mach/nvhost.h>
+#include <mach/nvmap.h>
 #include <asm/atomic.h>
 
 #include "nvhost_hardware.h"
 
+#define NVSYNCPT_GRAPHICS_HOST              (0)
 #define NVSYNCPT_VI_ISP_0                   (12)
 #define NVSYNCPT_VI_ISP_1                   (13)
 #define NVSYNCPT_VI_ISP_2                   (14)
@@ -142,6 +145,9 @@ static inline int nvhost_syncpt_wait(struct nvhost_syncpt *sp, u32 id, u32 thres
        return nvhost_syncpt_wait_timeout(sp, id, thresh, MAX_SCHEDULE_TIMEOUT);
 }
 
+int nvhost_syncpt_wait_check(struct nvmap_client *nvmap,
+                       struct nvhost_syncpt *sp, u32 mask,
+                       struct nvhost_waitchk *waitp, u32 num_waits);
 
 const char *nvhost_syncpt_name(u32 id);
 
index 00b9a5adb49fb102415f9741fbd379873eff76c9..e8d795006082916fc12bba028327e0944908d4bc 100644 (file)
@@ -730,3 +730,53 @@ void nvmap_free(struct nvmap_client *client, struct nvmap_handle_ref *r)
 {
        nvmap_free_handle_id(client, nvmap_ref_to_id(r));
 }
+
+/*
+ * create a mapping to the user's buffer and write it
+ * (uses similar logic from nvmap_reloc_pin_array to map the cmdbuf)
+ */
+int nvmap_patch_wait(struct nvmap_client *client,
+                               struct nvmap_handle *patch,
+                               u32 patch_offset, u32 patch_value)
+{
+       unsigned long phys;
+       unsigned int pfn, last_pfn = 0;
+       void *addr;
+       pte_t **pte;
+
+       if (patch_offset >= patch->size) {
+               nvmap_warn(client, "read/write outside of handle\n");
+               return -EFAULT;
+       }
+
+       pte = nvmap_alloc_pte(client->dev, &addr);
+       if (IS_ERR(pte))
+               return PTR_ERR(pte);
+
+       /* derive physaddr of cmdbuf WAIT to patch */
+       if (patch->heap_pgalloc) {
+               unsigned int page = patch_offset >> PAGE_SHIFT;
+               phys = page_to_phys(patch->pgalloc.pages[page]);
+               phys += (patch_offset & ~PAGE_MASK);
+       } else {
+               phys = patch->carveout->base + patch_offset;
+       }
+
+       pfn = __phys_to_pfn(phys);
+
+       /* write PTE, so addr points to cmdbuf PFN */
+       if (pfn != last_pfn) {
+               pgprot_t prot = nvmap_pgprot(patch, pgprot_kernel);
+               unsigned long kaddr = (unsigned long)addr;
+               set_pte_at(&init_mm, kaddr, *pte, pfn_pte(pfn, prot));
+               flush_tlb_kernel_page(kaddr);
+               last_pfn = pfn;
+       }
+
+       /* write patch_value to addr + page offset */
+       __raw_writel(patch_value, addr + (phys & ~PAGE_MASK));
+
+       nvmap_free_pte(client->dev, pte);
+       wmb();
+       return 0;
+}