diff options
author | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2009-05-07 04:46:49 -0400 |
---|---|---|
committer | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2009-05-18 05:28:24 -0400 |
commit | f089c0b28cdba1076aa8335dcaaaacc3dafc7d36 (patch) | |
tree | ae9ca55c15a379d20856c56c1ff30ad712c4839e /drivers/mtd/ubi/cdev.c | |
parent | 383d08e045faddd89797959786233d4c0e1ace80 (diff) |
UBI: re-name volumes_mutex to device_mutex
The mutex essencially protects the entire UBI device, so the
old @volumes_mutex name is a little misleading.
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Diffstat (limited to 'drivers/mtd/ubi/cdev.c')
-rw-r--r-- | drivers/mtd/ubi/cdev.c | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/drivers/mtd/ubi/cdev.c b/drivers/mtd/ubi/cdev.c index 8087b0462771..1024c106c899 100644 --- a/drivers/mtd/ubi/cdev.c +++ b/drivers/mtd/ubi/cdev.c | |||
@@ -558,7 +558,7 @@ static long vol_cdev_ioctl(struct file *file, unsigned int cmd, | |||
558 | break; | 558 | break; |
559 | } | 559 | } |
560 | 560 | ||
561 | /* Set volume property command*/ | 561 | /* Set volume property command */ |
562 | case UBI_IOCSETPROP: | 562 | case UBI_IOCSETPROP: |
563 | { | 563 | { |
564 | struct ubi_set_prop_req req; | 564 | struct ubi_set_prop_req req; |
@@ -571,9 +571,9 @@ static long vol_cdev_ioctl(struct file *file, unsigned int cmd, | |||
571 | } | 571 | } |
572 | switch (req.property) { | 572 | switch (req.property) { |
573 | case UBI_PROP_DIRECT_WRITE: | 573 | case UBI_PROP_DIRECT_WRITE: |
574 | mutex_lock(&ubi->volumes_mutex); | 574 | mutex_lock(&ubi->device_mutex); |
575 | desc->vol->direct_writes = !!req.value; | 575 | desc->vol->direct_writes = !!req.value; |
576 | mutex_unlock(&ubi->volumes_mutex); | 576 | mutex_unlock(&ubi->device_mutex); |
577 | break; | 577 | break; |
578 | default: | 578 | default: |
579 | err = -EINVAL; | 579 | err = -EINVAL; |
@@ -810,7 +810,9 @@ static int rename_volumes(struct ubi_device *ubi, | |||
810 | re->desc->vol->vol_id, re->desc->vol->name); | 810 | re->desc->vol->vol_id, re->desc->vol->name); |
811 | } | 811 | } |
812 | 812 | ||
813 | mutex_lock(&ubi->device_mutex); | ||
813 | err = ubi_rename_volumes(ubi, &rename_list); | 814 | err = ubi_rename_volumes(ubi, &rename_list); |
815 | mutex_unlock(&ubi->device_mutex); | ||
814 | 816 | ||
815 | out_free: | 817 | out_free: |
816 | list_for_each_entry_safe(re, re1, &rename_list, list) { | 818 | list_for_each_entry_safe(re, re1, &rename_list, list) { |
@@ -854,9 +856,9 @@ static long ubi_cdev_ioctl(struct file *file, unsigned int cmd, | |||
854 | if (err) | 856 | if (err) |
855 | break; | 857 | break; |
856 | 858 | ||
857 | mutex_lock(&ubi->volumes_mutex); | 859 | mutex_lock(&ubi->device_mutex); |
858 | err = ubi_create_volume(ubi, &req); | 860 | err = ubi_create_volume(ubi, &req); |
859 | mutex_unlock(&ubi->volumes_mutex); | 861 | mutex_unlock(&ubi->device_mutex); |
860 | if (err) | 862 | if (err) |
861 | break; | 863 | break; |
862 | 864 | ||
@@ -885,9 +887,9 @@ static long ubi_cdev_ioctl(struct file *file, unsigned int cmd, | |||
885 | break; | 887 | break; |
886 | } | 888 | } |
887 | 889 | ||
888 | mutex_lock(&ubi->volumes_mutex); | 890 | mutex_lock(&ubi->device_mutex); |
889 | err = ubi_remove_volume(desc, 0); | 891 | err = ubi_remove_volume(desc, 0); |
890 | mutex_unlock(&ubi->volumes_mutex); | 892 | mutex_unlock(&ubi->device_mutex); |
891 | 893 | ||
892 | /* | 894 | /* |
893 | * The volume is deleted (unless an error occurred), and the | 895 | * The volume is deleted (unless an error occurred), and the |
@@ -924,9 +926,9 @@ static long ubi_cdev_ioctl(struct file *file, unsigned int cmd, | |||
924 | pebs = div_u64(req.bytes + desc->vol->usable_leb_size - 1, | 926 | pebs = div_u64(req.bytes + desc->vol->usable_leb_size - 1, |
925 | desc->vol->usable_leb_size); | 927 | desc->vol->usable_leb_size); |
926 | 928 | ||
927 | mutex_lock(&ubi->volumes_mutex); | 929 | mutex_lock(&ubi->device_mutex); |
928 | err = ubi_resize_volume(desc, pebs); | 930 | err = ubi_resize_volume(desc, pebs); |
929 | mutex_unlock(&ubi->volumes_mutex); | 931 | mutex_unlock(&ubi->device_mutex); |
930 | ubi_close_volume(desc); | 932 | ubi_close_volume(desc); |
931 | break; | 933 | break; |
932 | } | 934 | } |
@@ -950,9 +952,7 @@ static long ubi_cdev_ioctl(struct file *file, unsigned int cmd, | |||
950 | break; | 952 | break; |
951 | } | 953 | } |
952 | 954 | ||
953 | mutex_lock(&ubi->volumes_mutex); | ||
954 | err = rename_volumes(ubi, req); | 955 | err = rename_volumes(ubi, req); |
955 | mutex_unlock(&ubi->volumes_mutex); | ||
956 | kfree(req); | 956 | kfree(req); |
957 | break; | 957 | break; |
958 | } | 958 | } |