Commit a7e93dcd authored by Roman Tereshonkov's avatar Roman Tereshonkov Committed by David Woodhouse

mtd: fix master device identification for mtd repartition

Function mtd_has_master renamed as mtd_is_partition to follow the function logic.
The patch fixes the problem of checking the right mtd device for partition creation.
To delete partition checking is not needed here so as it is done in mtd_del_partition.
By master we consider the mtd device which does not belong to any partition.
Signed-off-by: default avatarRoman Tereshonkov <roman.tereshonkov@nokia.com>
Signed-off-by: default avatarArtem Bityutskiy <Artem.Bityutskiy@nokia.com>
Signed-off-by: default avatarDavid Woodhouse <David.Woodhouse@intel.com>
parent e14feafb
...@@ -522,10 +522,6 @@ static int mtd_blkpg_ioctl(struct mtd_info *mtd, ...@@ -522,10 +522,6 @@ static int mtd_blkpg_ioctl(struct mtd_info *mtd,
if (!capable(CAP_SYS_ADMIN)) if (!capable(CAP_SYS_ADMIN))
return -EPERM; return -EPERM;
/* Only master mtd device must be used to control partitions */
if (!mtd_is_master(mtd))
return -EINVAL;
if (copy_from_user(&a, arg, sizeof(struct blkpg_ioctl_arg))) if (copy_from_user(&a, arg, sizeof(struct blkpg_ioctl_arg)))
return -EFAULT; return -EFAULT;
...@@ -535,6 +531,10 @@ static int mtd_blkpg_ioctl(struct mtd_info *mtd, ...@@ -535,6 +531,10 @@ static int mtd_blkpg_ioctl(struct mtd_info *mtd,
switch (a.op) { switch (a.op) {
case BLKPG_ADD_PARTITION: case BLKPG_ADD_PARTITION:
/* Only master mtd device must be used to add partitions */
if (mtd_is_partition(mtd))
return -EINVAL;
return mtd_add_partition(mtd, p.devname, p.start, p.length); return mtd_add_partition(mtd, p.devname, p.start, p.length);
case BLKPG_DEL_PARTITION: case BLKPG_DEL_PARTITION:
......
...@@ -720,19 +720,19 @@ int parse_mtd_partitions(struct mtd_info *master, const char **types, ...@@ -720,19 +720,19 @@ int parse_mtd_partitions(struct mtd_info *master, const char **types,
} }
EXPORT_SYMBOL_GPL(parse_mtd_partitions); EXPORT_SYMBOL_GPL(parse_mtd_partitions);
int mtd_is_master(struct mtd_info *mtd) int mtd_is_partition(struct mtd_info *mtd)
{ {
struct mtd_part *part; struct mtd_part *part;
int nopart = 0; int ispart = 0;
mutex_lock(&mtd_partitions_mutex); mutex_lock(&mtd_partitions_mutex);
list_for_each_entry(part, &mtd_partitions, list) list_for_each_entry(part, &mtd_partitions, list)
if (&part->mtd == mtd) { if (&part->mtd == mtd) {
nopart = 1; ispart = 1;
break; break;
} }
mutex_unlock(&mtd_partitions_mutex); mutex_unlock(&mtd_partitions_mutex);
return nopart; return ispart;
} }
EXPORT_SYMBOL_GPL(mtd_is_master); EXPORT_SYMBOL_GPL(mtd_is_partition);
...@@ -89,7 +89,7 @@ static inline int mtd_has_cmdlinepart(void) { return 1; } ...@@ -89,7 +89,7 @@ static inline int mtd_has_cmdlinepart(void) { return 1; }
static inline int mtd_has_cmdlinepart(void) { return 0; } static inline int mtd_has_cmdlinepart(void) { return 0; }
#endif #endif
int mtd_is_master(struct mtd_info *mtd); int mtd_is_partition(struct mtd_info *mtd);
int mtd_add_partition(struct mtd_info *master, char *name, int mtd_add_partition(struct mtd_info *master, char *name,
long long offset, long long length); long long offset, long long length);
int mtd_del_partition(struct mtd_info *master, int partno); int mtd_del_partition(struct mtd_info *master, int partno);
......
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