Commit b008c0fc authored by David Herrmann's avatar David Herrmann Committed by Dave Airlie

drm: remove unused "struct drm_freelist"

This object is not used except for static fields in drm_bufs *cough*.
Inline the watermark fields and drop the unused structure definition.
Signed-off-by: default avatarDavid Herrmann <dh.herrmann@gmail.com>
Reviewed-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: default avatarDave Airlie <airlied@redhat.com>
parent 8bed5cc7
...@@ -1217,7 +1217,6 @@ int drm_infobufs(struct drm_device *dev, void *data, ...@@ -1217,7 +1217,6 @@ int drm_infobufs(struct drm_device *dev, void *data,
struct drm_buf_desc __user *to = struct drm_buf_desc __user *to =
&request->list[count]; &request->list[count];
struct drm_buf_entry *from = &dma->bufs[i]; struct drm_buf_entry *from = &dma->bufs[i];
struct drm_freelist *list = &dma->bufs[i].freelist;
if (copy_to_user(&to->count, if (copy_to_user(&to->count,
&from->buf_count, &from->buf_count,
sizeof(from->buf_count)) || sizeof(from->buf_count)) ||
...@@ -1225,19 +1224,19 @@ int drm_infobufs(struct drm_device *dev, void *data, ...@@ -1225,19 +1224,19 @@ int drm_infobufs(struct drm_device *dev, void *data,
&from->buf_size, &from->buf_size,
sizeof(from->buf_size)) || sizeof(from->buf_size)) ||
copy_to_user(&to->low_mark, copy_to_user(&to->low_mark,
&list->low_mark, &from->low_mark,
sizeof(list->low_mark)) || sizeof(from->low_mark)) ||
copy_to_user(&to->high_mark, copy_to_user(&to->high_mark,
&list->high_mark, &from->high_mark,
sizeof(list->high_mark))) sizeof(from->high_mark)))
return -EFAULT; return -EFAULT;
DRM_DEBUG("%d %d %d %d %d\n", DRM_DEBUG("%d %d %d %d %d\n",
i, i,
dma->bufs[i].buf_count, dma->bufs[i].buf_count,
dma->bufs[i].buf_size, dma->bufs[i].buf_size,
dma->bufs[i].freelist.low_mark, dma->bufs[i].low_mark,
dma->bufs[i].freelist.high_mark); dma->bufs[i].high_mark);
++count; ++count;
} }
} }
...@@ -1290,8 +1289,8 @@ int drm_markbufs(struct drm_device *dev, void *data, ...@@ -1290,8 +1289,8 @@ int drm_markbufs(struct drm_device *dev, void *data,
if (request->high_mark < 0 || request->high_mark > entry->buf_count) if (request->high_mark < 0 || request->high_mark > entry->buf_count)
return -EINVAL; return -EINVAL;
entry->freelist.low_mark = request->low_mark; entry->low_mark = request->low_mark;
entry->freelist.high_mark = request->high_mark; entry->high_mark = request->high_mark;
return 0; return 0;
} }
......
...@@ -132,7 +132,7 @@ int drm_bufs_info(struct seq_file *m, void *data) ...@@ -132,7 +132,7 @@ int drm_bufs_info(struct seq_file *m, void *data)
i, i,
dma->bufs[i].buf_size, dma->bufs[i].buf_size,
dma->bufs[i].buf_count, dma->bufs[i].buf_count,
atomic_read(&dma->bufs[i].freelist.count), 0,
dma->bufs[i].seg_count, dma->bufs[i].seg_count,
seg_pages, seg_pages,
seg_pages * PAGE_SIZE / 1024); seg_pages * PAGE_SIZE / 1024);
......
...@@ -346,18 +346,6 @@ struct drm_waitlist { ...@@ -346,18 +346,6 @@ struct drm_waitlist {
spinlock_t write_lock; spinlock_t write_lock;
}; };
struct drm_freelist {
int initialized; /**< Freelist in use */
atomic_t count; /**< Number of free buffers */
struct drm_buf *next; /**< End pointer */
wait_queue_head_t waiting; /**< Processes waiting on free bufs */
int low_mark; /**< Low water mark */
int high_mark; /**< High water mark */
atomic_t wfh; /**< If waiting for high mark */
spinlock_t lock;
};
typedef struct drm_dma_handle { typedef struct drm_dma_handle {
dma_addr_t busaddr; dma_addr_t busaddr;
void *vaddr; void *vaddr;
...@@ -375,7 +363,8 @@ struct drm_buf_entry { ...@@ -375,7 +363,8 @@ struct drm_buf_entry {
int page_order; int page_order;
struct drm_dma_handle **seglist; struct drm_dma_handle **seglist;
struct drm_freelist freelist; int low_mark; /**< Low water mark */
int high_mark; /**< High water mark */
}; };
/* Event queued up for userspace to read */ /* Event queued up for userspace to read */
......
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