Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 740e6233 authored by Linux Build Service Account's avatar Linux Build Service Account Committed by Gerrit - the friendly Code Review server
Browse files

Merge "soc: qcom: socinfo: Protect current_image using semaphore lock."

parents 724abfa0 950a0410

drivers/soc/qcom/socinfo.c

+34 −4
Original line number Diff line number Diff line
@@ -47,6 +47,7 @@ 
#define SMEM_IMAGE_VERSION_OEM_OFFSET 96

#define SMEM_IMAGE_VERSION_PARTITION_APPS 10



static DECLARE_RWSEM(current_image_rwsem);

enum {

	HW_PLATFORM_UNKNOWN = 0,

	HW_PLATFORM_SURF    = 1,
@@ -1047,7 +1048,9 @@ msm_get_image_version(struct device *dev, 
		pr_err("Failed to get image version base address");

		return snprintf(buf, SMEM_IMAGE_VERSION_NAME_SIZE, "Unknown");

	}

	down_read(&current_image_rwsem);

	string_address += current_image * SMEM_IMAGE_VERSION_SINGLE_BLOCK_SIZE;

	up_read(&current_image_rwsem);

	return snprintf(buf, SMEM_IMAGE_VERSION_NAME_SIZE, "%-.75s\n",

			string_address);

}
@@ -1060,14 +1063,19 @@ msm_set_image_version(struct device *dev, 
{

	char *store_address;



	if (current_image != SMEM_IMAGE_VERSION_PARTITION_APPS)

	down_read(&current_image_rwsem);

	if (current_image != SMEM_IMAGE_VERSION_PARTITION_APPS) {

		up_read(&current_image_rwsem);

		return count;

	}

	store_address = socinfo_get_image_version_base_address();

	if (IS_ERR_OR_NULL(store_address)) {

		pr_err("Failed to get image version base address");

		up_read(&current_image_rwsem);

		return count;

	}

	store_address += current_image * SMEM_IMAGE_VERSION_SINGLE_BLOCK_SIZE;

	up_read(&current_image_rwsem);

	snprintf(store_address, SMEM_IMAGE_VERSION_NAME_SIZE, "%-.75s", buf);

	return count;

}
@@ -1085,7 +1093,9 @@ msm_get_image_variant(struct device *dev, 
		return snprintf(buf, SMEM_IMAGE_VERSION_VARIANT_SIZE,

		"Unknown");

	}

	down_read(&current_image_rwsem);

	string_address += current_image * SMEM_IMAGE_VERSION_SINGLE_BLOCK_SIZE;

	up_read(&current_image_rwsem);

	string_address += SMEM_IMAGE_VERSION_VARIANT_OFFSET;

	return snprintf(buf, SMEM_IMAGE_VERSION_VARIANT_SIZE, "%-.20s\n",

			string_address);
@@ -1099,14 +1109,19 @@ msm_set_image_variant(struct device *dev, 
{

	char *store_address;



	if (current_image != SMEM_IMAGE_VERSION_PARTITION_APPS)

	down_read(&current_image_rwsem);

	if (current_image != SMEM_IMAGE_VERSION_PARTITION_APPS) {

		up_read(&current_image_rwsem);

		return count;

	}

	store_address = socinfo_get_image_version_base_address();

	if (IS_ERR_OR_NULL(store_address)) {

		pr_err("Failed to get image version base address");

		up_read(&current_image_rwsem);

		return count;

	}

	store_address += current_image * SMEM_IMAGE_VERSION_SINGLE_BLOCK_SIZE;

	up_read(&current_image_rwsem);

	store_address += SMEM_IMAGE_VERSION_VARIANT_OFFSET;

	snprintf(store_address, SMEM_IMAGE_VERSION_VARIANT_SIZE, "%-.20s", buf);

	return count;
@@ -1124,7 +1139,9 @@ msm_get_image_crm_version(struct device *dev, 
		pr_err("Failed to get image version base address");

		return snprintf(buf, SMEM_IMAGE_VERSION_OEM_SIZE, "Unknown");

	}

	down_read(&current_image_rwsem);

	string_address += current_image * SMEM_IMAGE_VERSION_SINGLE_BLOCK_SIZE;

	up_read(&current_image_rwsem);

	string_address += SMEM_IMAGE_VERSION_OEM_OFFSET;

	return snprintf(buf, SMEM_IMAGE_VERSION_OEM_SIZE, "%-.32s\n",

			string_address);
@@ -1138,14 +1155,19 @@ msm_set_image_crm_version(struct device *dev, 
{

	char *store_address;



	if (current_image != SMEM_IMAGE_VERSION_PARTITION_APPS)

	down_read(&current_image_rwsem);

	if (current_image != SMEM_IMAGE_VERSION_PARTITION_APPS) {

		up_read(&current_image_rwsem);

		return count;

	}

	store_address = socinfo_get_image_version_base_address();

	if (IS_ERR_OR_NULL(store_address)) {

		pr_err("Failed to get image version base address");

		up_read(&current_image_rwsem);

		return count;

	}

	store_address += current_image * SMEM_IMAGE_VERSION_SINGLE_BLOCK_SIZE;

	up_read(&current_image_rwsem);

	store_address += SMEM_IMAGE_VERSION_OEM_OFFSET;

	snprintf(store_address, SMEM_IMAGE_VERSION_OEM_SIZE, "%-.32s", buf);

	return count;
@@ -1156,8 +1178,14 @@ msm_get_image_number(struct device *dev, 
			struct device_attribute *attr,

			char *buf)

{

	return snprintf(buf, PAGE_SIZE, "%d\n",

	int ret;



	down_read(&current_image_rwsem);

	ret = snprintf(buf, PAGE_SIZE, "%d\n",

			current_image);

	up_read(&current_image_rwsem);

	return ret;



}



static ssize_t
@@ -1169,10 +1197,12 @@ msm_select_image(struct device *dev, struct device_attribute *attr, 
	ret = kstrtoint(buf, 10, &digit);

	if (ret)

		return ret;

	down_write(&current_image_rwsem);

	if (digit >= 0 && digit < SMEM_IMAGE_VERSION_BLOCKS_COUNT)

		current_image = digit;

	else

		current_image = 0;

	up_write(&current_image_rwsem);

	return count;

}