Remove unused dev->map_count. We always iterate the maplist with
list_for_each() and the count is not updated or used for stats.main
parent
2dc672a790
commit
d6a82ff9c1
|
@ -554,7 +554,6 @@ typedef struct drm_device {
|
|||
|
||||
/* Memory management */
|
||||
drm_map_list_t *maplist; /* Linked list of regions */
|
||||
int map_count; /* Number of mappable regions */
|
||||
|
||||
drm_map_t **context_sareas;
|
||||
int max_context;
|
||||
|
|
|
@ -316,7 +316,6 @@ static int DRM(setup)( drm_device_t *dev )
|
|||
if(dev->maplist == NULL) return -ENOMEM;
|
||||
memset(dev->maplist, 0, sizeof(*dev->maplist));
|
||||
INIT_LIST_HEAD(&dev->maplist->head);
|
||||
dev->map_count = 0;
|
||||
|
||||
dev->vmalist = NULL;
|
||||
dev->sigdata.lock = dev->lock.hw_lock = NULL;
|
||||
|
|
|
@ -197,7 +197,7 @@ int DRM(getmap)( struct inode *inode, struct file *filp,
|
|||
idx = map.offset;
|
||||
|
||||
down(&dev->struct_sem);
|
||||
if (idx < 0 || idx >= dev->map_count) {
|
||||
if (idx < 0) {
|
||||
up(&dev->struct_sem);
|
||||
return -EINVAL;
|
||||
}
|
||||
|
|
|
@ -554,7 +554,6 @@ typedef struct drm_device {
|
|||
|
||||
/* Memory management */
|
||||
drm_map_list_t *maplist; /* Linked list of regions */
|
||||
int map_count; /* Number of mappable regions */
|
||||
|
||||
drm_map_t **context_sareas;
|
||||
int max_context;
|
||||
|
|
|
@ -316,7 +316,6 @@ static int DRM(setup)( drm_device_t *dev )
|
|||
if(dev->maplist == NULL) return -ENOMEM;
|
||||
memset(dev->maplist, 0, sizeof(*dev->maplist));
|
||||
INIT_LIST_HEAD(&dev->maplist->head);
|
||||
dev->map_count = 0;
|
||||
|
||||
dev->vmalist = NULL;
|
||||
dev->sigdata.lock = dev->lock.hw_lock = NULL;
|
||||
|
|
|
@ -197,7 +197,7 @@ int DRM(getmap)( struct inode *inode, struct file *filp,
|
|||
idx = map.offset;
|
||||
|
||||
down(&dev->struct_sem);
|
||||
if (idx < 0 || idx >= dev->map_count) {
|
||||
if (idx < 0) {
|
||||
up(&dev->struct_sem);
|
||||
return -EINVAL;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue