drm/nouveau/nvif: use negative oclass identifier for internal classes
authorBen Skeggs <bskeggs@redhat.com>
Thu, 20 Aug 2015 04:54:16 +0000 (14:54 +1000)
committerBen Skeggs <bskeggs@redhat.com>
Fri, 28 Aug 2015 02:40:32 +0000 (12:40 +1000)
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
12 files changed:
drivers/gpu/drm/nouveau/include/nvif/device.h
drivers/gpu/drm/nouveau/include/nvif/ioctl.h
drivers/gpu/drm/nouveau/include/nvif/object.h
drivers/gpu/drm/nouveau/include/nvkm/core/client.h
drivers/gpu/drm/nouveau/include/nvkm/core/object.h
drivers/gpu/drm/nouveau/include/nvkm/core/parent.h
drivers/gpu/drm/nouveau/nouveau_bo.c
drivers/gpu/drm/nouveau/nouveau_drm.c
drivers/gpu/drm/nouveau/nv50_display.c
drivers/gpu/drm/nouveau/nvif/device.c
drivers/gpu/drm/nouveau/nvif/object.c
drivers/gpu/drm/nouveau/nvkm/core/parent.c

index 1973e65f21a682a7092f80e0a9a624e0e939af6d..68b431b24ae6efb4dfca8149f895413a89f76729 100644 (file)
@@ -9,7 +9,7 @@ struct nvif_device {
        struct nv_device_info_v0 info;
 };
 
-int  nvif_device_init(struct nvif_object *, u32 handle, u32 oclass, void *, u32,
+int  nvif_device_init(struct nvif_object *, u32 handle, s32 oclass, void *, u32,
                      struct nvif_device *);
 void nvif_device_fini(struct nvif_device *);
 u64  nvif_device_time(struct nvif_device *);
index 772db0de044aeea32cfa75cf19f408e6b8a353f4..193437c4a1f0627b3a4e5aad20c76118c325cdb2 100644 (file)
@@ -40,7 +40,7 @@ struct nvif_ioctl_sclass_v0 {
        __u8  version;
        __u8  count;
        __u8  pad02[6];
-       __u32 oclass[];
+       __s32 oclass[];
 };
 
 struct nvif_ioctl_new_v0 {
@@ -52,10 +52,10 @@ struct nvif_ioctl_new_v0 {
        __u64 object;
        __u32 handle;
 /* these class numbers are made up by us, and not nvidia-assigned */
-#define NVIF_IOCTL_NEW_V0_PERFMON                                    0x0000ffff
-#define NVIF_IOCTL_NEW_V0_PERFDOM                                    0x0000fffe
-#define NVIF_IOCTL_NEW_V0_CONTROL                                    0x0000fffd
-       __u32 oclass;
+#define NVIF_IOCTL_NEW_V0_CONTROL                                            -1
+#define NVIF_IOCTL_NEW_V0_PERFMON                                            -2
+#define NVIF_IOCTL_NEW_V0_PERFDOM                                            -3
+       __s32 oclass;
        __u8  data[];           /* class data (class.h) */
 };
 
index b46c2f4aa0db29a34912779e4155150fff71654a..66d3425e476449d56e8b76fe40e7f9b63b5d84eb 100644 (file)
@@ -6,7 +6,7 @@
 struct nvif_object {
        struct nvif_client *client;
        u32 handle;
-       u32 oclass;
+       s32 oclass;
        void *priv; /*XXX: hack */
        struct {
                void __iomem *ptr;
@@ -14,11 +14,11 @@ struct nvif_object {
        } map;
 };
 
-int  nvif_object_init(struct nvif_object *, u32 handle, u32 oclass, void *, u32,
+int  nvif_object_init(struct nvif_object *, u32 handle, s32 oclass, void *, u32,
                      struct nvif_object *);
 void nvif_object_fini(struct nvif_object *);
 int  nvif_object_ioctl(struct nvif_object *, void *, u32, void **);
-int  nvif_object_sclass(struct nvif_object *, u32 *, int);
+int  nvif_object_sclass(struct nvif_object *, s32 *, int);
 u32  nvif_object_rd(struct nvif_object *, int, u64);
 void nvif_object_wr(struct nvif_object *, int, u64, u32);
 int  nvif_object_mthd(struct nvif_object *, u32, void *, u32);
index d70d28f90285d2611ae9e7dd2febfe2222c4aa9b..977c8a85e119980b96c80b485d07617663aa9951 100644 (file)
@@ -22,20 +22,11 @@ bool nvkm_client_insert(struct nvkm_client *, struct nvkm_handle *);
 void nvkm_client_remove(struct nvkm_client *, struct nvkm_handle *);
 struct nvkm_handle *nvkm_client_search(struct nvkm_client *, u64 handle);
 
-static inline struct nvkm_client *
-nv_client(void *obj)
-{
-#if CONFIG_NOUVEAU_DEBUG >= NV_DBG_PARANOIA
-       BUG_ON(!nv_iclass(obj, NV_CLIENT_CLASS));
-#endif
-       return obj;
-}
-
 static inline struct nvkm_client *
 nvkm_client(void *obj)
 {
        struct nvkm_object *client = nv_object(obj);
-       while (client && !(nv_iclass(client, NV_CLIENT_CLASS)))
+       while (client && client->parent)
                client = client->parent;
        return (void *)client;
 }
index ef0ff9e1a9480844172be924e6813e41512cd0c4..005cd4a86849109c008be9e24fadd1f8ee30ceb6 100644 (file)
@@ -62,7 +62,7 @@ extern struct nvkm_ofuncs nvkm_object_ofuncs;
 /* Don't allocate dynamically, because lockdep needs lock_class_keys to be in
  * ".data". */
 struct nvkm_oclass {
-       u32 handle;
+       s32 handle;
        struct nvkm_ofuncs * const ofuncs;
        struct nvkm_omthds * const omthds;
        struct lock_class_key lock_class_key;
index 2e5266fa75841b4fb54370f102feb42e5ade3931..bc4dc1f2403fabbde10a6ce9f22b4487f582daad 100644 (file)
@@ -44,8 +44,8 @@ void _nvkm_parent_dtor(struct nvkm_object *);
 #define _nvkm_parent_init nvkm_object_init
 #define _nvkm_parent_fini nvkm_object_fini
 
-int nvkm_parent_sclass(struct nvkm_object *, u16 handle,
+int nvkm_parent_sclass(struct nvkm_object *, s32 handle,
                       struct nvkm_object **pengine,
                       struct nvkm_oclass **poclass);
-int nvkm_parent_lclass(struct nvkm_object *, u32 *, int);
+int nvkm_parent_lclass(struct nvkm_object *, s32 *, int);
 #endif
index 5529fc479d8a34d175a50b9ff9adb5bf23e5e7c4..496c00d585cd2cfc7a74c87db67cd0996cbd8f6b 100644 (file)
@@ -1104,7 +1104,7 @@ nouveau_bo_move_init(struct nouveau_drm *drm)
        static const struct {
                const char *name;
                int engine;
-               u32 oclass;
+               s32 oclass;
                int (*exec)(struct nouveau_channel *,
                            struct ttm_buffer_object *,
                            struct ttm_mem_reg *, struct ttm_mem_reg *);
index 11cf52a7ff75032e3af2ed25cc6308337d34b692..65ceb6fa42094bf6b5a7f3dbcfc4584ee17c3caa 100644 (file)
@@ -153,7 +153,7 @@ nouveau_accel_init(struct nouveau_drm *drm)
 {
        struct nvif_device *device = &drm->device;
        u32 arg0, arg1;
-       u32 sclass[16];
+       s32 sclass[16];
        int ret, i;
 
        if (nouveau_noaccel)
index c8e79740435389858229b4f410ba459d37977916..65e70f085325b8e92d5cc19f551975b9e617fc32 100644 (file)
@@ -65,11 +65,11 @@ struct nv50_chan {
 
 static int
 nv50_chan_create(struct nvif_device *device, struct nvif_object *disp,
-                const u32 *oclass, u8 head, void *data, u32 size,
+                const s32 *oclass, u8 head, void *data, u32 size,
                 struct nv50_chan *chan)
 {
        const u32 handle = (oclass[0] << 16) | head;
-       u32 sclass[8];
+       s32 sclass[8];
        int ret, i;
 
        chan->device = device;
@@ -117,7 +117,7 @@ nv50_pioc_destroy(struct nv50_pioc *pioc)
 
 static int
 nv50_pioc_create(struct nvif_device *device, struct nvif_object *disp,
-                const u32 *oclass, u8 head, void *data, u32 size,
+                const s32 *oclass, u8 head, void *data, u32 size,
                 struct nv50_pioc *pioc)
 {
        return nv50_chan_create(device, disp, oclass, head, data, size,
@@ -139,7 +139,7 @@ nv50_curs_create(struct nvif_device *device, struct nvif_object *disp,
        struct nv50_disp_cursor_v0 args = {
                .head = head,
        };
-       static const u32 oclass[] = {
+       static const s32 oclass[] = {
                GK104_DISP_CURSOR,
                GF110_DISP_CURSOR,
                GT214_DISP_CURSOR,
@@ -167,7 +167,7 @@ nv50_oimm_create(struct nvif_device *device, struct nvif_object *disp,
        struct nv50_disp_cursor_v0 args = {
                .head = head,
        };
-       static const u32 oclass[] = {
+       static const s32 oclass[] = {
                GK104_DISP_OVERLAY,
                GF110_DISP_OVERLAY,
                GT214_DISP_OVERLAY,
@@ -216,7 +216,7 @@ nv50_dmac_destroy(struct nv50_dmac *dmac, struct nvif_object *disp)
 
 static int
 nv50_dmac_create(struct nvif_device *device, struct nvif_object *disp,
-                const u32 *oclass, u8 head, void *data, u32 size, u64 syncbuf,
+                const s32 *oclass, u8 head, void *data, u32 size, u64 syncbuf,
                 struct nv50_dmac *dmac)
 {
        struct nv50_disp_core_channel_dma_v0 *args = data;
@@ -288,7 +288,7 @@ nv50_core_create(struct nvif_device *device, struct nvif_object *disp,
        struct nv50_disp_core_channel_dma_v0 args = {
                .pushbuf = 0xb0007d00,
        };
-       static const u32 oclass[] = {
+       static const s32 oclass[] = {
                GM204_DISP_CORE_CHANNEL_DMA,
                GM107_DISP_CORE_CHANNEL_DMA,
                GK110_DISP_CORE_CHANNEL_DMA,
@@ -324,7 +324,7 @@ nv50_base_create(struct nvif_device *device, struct nvif_object *disp,
                .pushbuf = 0xb0007c00 | head,
                .head = head,
        };
-       static const u32 oclass[] = {
+       static const s32 oclass[] = {
                GK110_DISP_BASE_CHANNEL_DMA,
                GK104_DISP_BASE_CHANNEL_DMA,
                GF110_DISP_BASE_CHANNEL_DMA,
@@ -355,7 +355,7 @@ nv50_ovly_create(struct nvif_device *device, struct nvif_object *disp,
                .pushbuf = 0xb0007e00 | head,
                .head = head,
        };
-       static const u32 oclass[] = {
+       static const s32 oclass[] = {
                GK104_DISP_OVERLAY_CONTROL_DMA,
                GF110_DISP_OVERLAY_CONTROL_DMA,
                GT214_DISP_OVERLAY_CHANNEL_DMA,
index f15d51a69df2b333c8e44f34d922da99e4aafbd5..561fb9d7b15508eefb9b50189a46a196cef1373b 100644 (file)
@@ -37,7 +37,7 @@ nvif_device_fini(struct nvif_device *device)
 }
 
 int
-nvif_device_init(struct nvif_object *parent, u32 handle, u32 oclass,
+nvif_device_init(struct nvif_object *parent, u32 handle, s32 oclass,
                 void *data, u32 size, struct nvif_device *device)
 {
        int ret = nvif_object_init(parent, handle, oclass, data, size,
index b914e34a43c3b72414cdb9143a82a35fb00fd112..0c09e6433fbbb88132ee29cdc77488d8d3403739 100644 (file)
@@ -49,7 +49,7 @@ nvif_object_ioctl(struct nvif_object *object, void *data, u32 size, void **hack)
 }
 
 int
-nvif_object_sclass(struct nvif_object *object, u32 *oclass, int count)
+nvif_object_sclass(struct nvif_object *object, s32 *oclass, int count)
 {
        struct {
                struct nvif_ioctl_v0 ioctl;
@@ -65,7 +65,6 @@ nvif_object_sclass(struct nvif_object *object, u32 *oclass, int count)
        args->sclass.version = 0;
        args->sclass.count = count;
 
-       memcpy(args->sclass.oclass, oclass, size);
        ret = nvif_object_ioctl(object, args, sizeof(*args) + size, NULL);
        ret = ret ? ret : args->sclass.count;
        memcpy(oclass, args->sclass.oclass, size);
@@ -203,7 +202,7 @@ nvif_object_fini(struct nvif_object *object)
 }
 
 int
-nvif_object_init(struct nvif_object *parent, u32 handle, u32 oclass,
+nvif_object_init(struct nvif_object *parent, u32 handle, s32 oclass,
                 void *data, u32 size, struct nvif_object *object)
 {
        struct {
index 5c75c64f227097d5f03c848a2022675783aa40b3..43abd208b4865d517910fd8a8af14495e24e02bb 100644 (file)
@@ -26,7 +26,7 @@
 #include <core/engine.h>
 
 int
-nvkm_parent_sclass(struct nvkm_object *parent, u16 handle,
+nvkm_parent_sclass(struct nvkm_object *parent, s32 handle,
                   struct nvkm_object **pengine,
                   struct nvkm_oclass **poclass)
 {
@@ -66,7 +66,7 @@ nvkm_parent_sclass(struct nvkm_object *parent, u16 handle,
 }
 
 int
-nvkm_parent_lclass(struct nvkm_object *parent, u32 *lclass, int size)
+nvkm_parent_lclass(struct nvkm_object *parent, s32 *lclass, int size)
 {
        struct nvkm_oclass *sclass, *oclass;
        struct nvkm_engine *engine;