Commit aa838896 authored by Joe Perches's avatar Joe Perches Committed by Greg Kroah-Hartman
Browse files

drivers core: Use sysfs_emit and sysfs_emit_at for show(device *...) functions 



Convert the various sprintf fmaily calls in sysfs device show functions
to sysfs_emit and sysfs_emit_at for PAGE_SIZE buffer safety.

Done with:

$ spatch -sp-file sysfs_emit_dev.cocci --in-place --max-width=80 .

And cocci script:

$ cat sysfs_emit_dev.cocci
@@
identifier d_show;
identifier dev, attr, buf;
@@

ssize_t d_show(struct device *dev, struct device_attribute *attr, char *buf)
{
	<...
	return
-	sprintf(buf,
+	sysfs_emit(buf,
	...);
	...>
}

@@
identifier d_show;
identifier dev, attr, buf;
@@

ssize_t d_show(struct device *dev, struct device_attribute *attr, char *buf)
{
	<...
	return
-	snprintf(buf, PAGE_SIZE,
+	sysfs_emit(buf,
	...);
	...>
}

@@
identifier d_show;
identifier dev, attr, buf;
@@

ssize_t d_show(struct device *dev, struct device_attribute *attr, char *buf)
{
	<...
	return
-	scnprintf(buf, PAGE_SIZE,
+	sysfs_emit(buf,
	...);
	...>
}

@@
identifier d_show;
identifier dev, attr, buf;
expression chr;
@@

ssize_t d_show(struct device *dev, struct device_attribute *attr, char *buf)
{
	<...
	return
-	strcpy(buf, chr);
+	sysfs_emit(buf, chr);
	...>
}

@@
identifier d_show;
identifier dev, attr, buf;
identifier len;
@@

ssize_t d_show(struct device *dev, struct device_attribute *attr, char *buf)
{
	<...
	len =
-	sprintf(buf,
+	sysfs_emit(buf,
	...);
	...>
	return len;
}

@@
identifier d_show;
identifier dev, attr, buf;
identifier len;
@@

ssize_t d_show(struct device *dev, struct device_attribute *attr, char *buf)
{
	<...
	len =
-	snprintf(buf, PAGE_SIZE,
+	sysfs_emit(buf,
	...);
	...>
	return len;
}

@@
identifier d_show;
identifier dev, attr, buf;
identifier len;
@@

ssize_t d_show(struct device *dev, struct device_attribute *attr, char *buf)
{
	<...
	len =
-	scnprintf(buf, PAGE_SIZE,
+	sysfs_emit(buf,
	...);
	...>
	return len;
}

@@
identifier d_show;
identifier dev, attr, buf;
identifier len;
@@

ssize_t d_show(struct device *dev, struct device_attribute *attr, char *buf)
{
	<...
-	len += scnprintf(buf + len, PAGE_SIZE - len,
+	len += sysfs_emit_at(buf, len,
	...);
	...>
	return len;
}

@@
identifier d_show;
identifier dev, attr, buf;
expression chr;
@@

ssize_t d_show(struct device *dev, struct device_attribute *attr, char *buf)
{
	...
-	strcpy(buf, chr);
-	return strlen(buf);
+	return sysfs_emit(buf, chr);
}

Signed-off-by: default avatarJoe Perches <joe@perches.com>
Link: https://lore.kernel.org/r/3d033c33056d88bbe34d4ddb62afd05ee166ab9a.1600285923.git.joe@perches.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 2efc459d

drivers/base/arch_topology.c

+1 −1
Original line number Diff line number Diff line
@@ -71,7 +71,7 @@ static ssize_t cpu_capacity_show(struct device *dev, 
{

	struct cpu *cpu = container_of(dev, struct cpu, dev);



	return sprintf(buf, "%lu\n", topology_get_cpu_scale(cpu->dev.id));

	return sysfs_emit(buf, "%lu\n", topology_get_cpu_scale(cpu->dev.id));

}



static void update_topology_flags_workfn(struct work_struct *work);

drivers/base/cacheinfo.c

+9 −9
Original line number Diff line number Diff line
@@ -377,7 +377,7 @@ static ssize_t size_show(struct device *dev, 
{

	struct cacheinfo *this_leaf = dev_get_drvdata(dev);



	return sprintf(buf, "%uK\n", this_leaf->size >> 10);

	return sysfs_emit(buf, "%uK\n", this_leaf->size >> 10);

}



static ssize_t shared_cpumap_show_func(struct device *dev, bool list, char *buf)
@@ -407,11 +407,11 @@ static ssize_t type_show(struct device *dev, 


	switch (this_leaf->type) {

	case CACHE_TYPE_DATA:

		return sprintf(buf, "Data\n");

		return sysfs_emit(buf, "Data\n");

	case CACHE_TYPE_INST:

		return sprintf(buf, "Instruction\n");

		return sysfs_emit(buf, "Instruction\n");

	case CACHE_TYPE_UNIFIED:

		return sprintf(buf, "Unified\n");

		return sysfs_emit(buf, "Unified\n");

	default:

		return -EINVAL;

	}
@@ -425,11 +425,11 @@ static ssize_t allocation_policy_show(struct device *dev, 
	int n = 0;



	if ((ci_attr & CACHE_READ_ALLOCATE) && (ci_attr & CACHE_WRITE_ALLOCATE))

		n = sprintf(buf, "ReadWriteAllocate\n");

		n = sysfs_emit(buf, "ReadWriteAllocate\n");

	else if (ci_attr & CACHE_READ_ALLOCATE)

		n = sprintf(buf, "ReadAllocate\n");

		n = sysfs_emit(buf, "ReadAllocate\n");

	else if (ci_attr & CACHE_WRITE_ALLOCATE)

		n = sprintf(buf, "WriteAllocate\n");

		n = sysfs_emit(buf, "WriteAllocate\n");

	return n;

}
@@ -441,9 +441,9 @@ static ssize_t write_policy_show(struct device *dev, 
	int n = 0;



	if (ci_attr & CACHE_WRITE_THROUGH)

		n = sprintf(buf, "WriteThrough\n");

		n = sysfs_emit(buf, "WriteThrough\n");

	else if (ci_attr & CACHE_WRITE_BACK)

		n = sprintf(buf, "WriteBack\n");

		n = sysfs_emit(buf, "WriteBack\n");

	return n;

}

drivers/base/core.c

+10 −9
Original line number Diff line number Diff line
@@ -260,7 +260,7 @@ static ssize_t status_show(struct device *dev, 
	default:

		status = "unknown"; break;

	}

	return sprintf(buf, "%s\n", status);

	return sysfs_emit(buf, "%s\n", status);

}

static DEVICE_ATTR_RO(status);
@@ -277,7 +277,7 @@ static ssize_t auto_remove_on_show(struct device *dev, 
	else

		str = "never";



	return sprintf(buf, "%s\n", str);

	return sysfs_emit(buf, "%s\n", str);

}

static DEVICE_ATTR_RO(auto_remove_on);
@@ -286,7 +286,7 @@ static ssize_t runtime_pm_show(struct device *dev, 
{

	struct device_link *link = to_devlink(dev);



	return sprintf(buf, "%d\n", !!(link->flags & DL_FLAG_PM_RUNTIME));

	return sysfs_emit(buf, "%d\n", !!(link->flags & DL_FLAG_PM_RUNTIME));

}

static DEVICE_ATTR_RO(runtime_pm);
@@ -295,7 +295,8 @@ static ssize_t sync_state_only_show(struct device *dev, 
{

	struct device_link *link = to_devlink(dev);



	return sprintf(buf, "%d\n", !!(link->flags & DL_FLAG_SYNC_STATE_ONLY));

	return sysfs_emit(buf, "%d\n",

			  !!(link->flags & DL_FLAG_SYNC_STATE_ONLY));

}

static DEVICE_ATTR_RO(sync_state_only);
@@ -1060,7 +1061,7 @@ static ssize_t waiting_for_supplier_show(struct device *dev, 
	      && dev->links.need_for_probe;

	mutex_unlock(&wfs_lock);

	device_unlock(dev);

	return sprintf(buf, "%u\n", val);

	return sysfs_emit(buf, "%u\n", val);

}

static DEVICE_ATTR_RO(waiting_for_supplier);
@@ -1710,7 +1711,7 @@ ssize_t device_show_ulong(struct device *dev, 
			  char *buf)

{

	struct dev_ext_attribute *ea = to_ext_attr(attr);

	return snprintf(buf, PAGE_SIZE, "%lx\n", *(unsigned long *)(ea->var));

	return sysfs_emit(buf, "%lx\n", *(unsigned long *)(ea->var));

}

EXPORT_SYMBOL_GPL(device_show_ulong);
@@ -1740,7 +1741,7 @@ ssize_t device_show_int(struct device *dev, 
{

	struct dev_ext_attribute *ea = to_ext_attr(attr);



	return snprintf(buf, PAGE_SIZE, "%d\n", *(int *)(ea->var));

	return sysfs_emit(buf, "%d\n", *(int *)(ea->var));

}

EXPORT_SYMBOL_GPL(device_show_int);
@@ -1761,7 +1762,7 @@ ssize_t device_show_bool(struct device *dev, struct device_attribute *attr, 
{

	struct dev_ext_attribute *ea = to_ext_attr(attr);



	return snprintf(buf, PAGE_SIZE, "%d\n", *(bool *)(ea->var));

	return sysfs_emit(buf, "%d\n", *(bool *)(ea->var));

}

EXPORT_SYMBOL_GPL(device_show_bool);
@@ -1993,7 +1994,7 @@ static ssize_t online_show(struct device *dev, struct device_attribute *attr, 
	device_lock(dev);

	val = !dev->offline;

	device_unlock(dev);

	return sprintf(buf, "%u\n", val);

	return sysfs_emit(buf, "%u\n", val);

}



static ssize_t online_store(struct device *dev, struct device_attribute *attr,

drivers/base/cpu.c

+16 −16
Original line number Diff line number Diff line
@@ -156,7 +156,7 @@ static ssize_t show_crash_notes(struct device *dev, struct device_attribute *att 
	 * operation should be safe. No locking required.

	 */

	addr = per_cpu_ptr_to_phys(per_cpu_ptr(crash_notes, cpunum));

	rc = sprintf(buf, "%Lx\n", addr);

	rc = sysfs_emit(buf, "%Lx\n", addr);

	return rc;

}

static DEVICE_ATTR(crash_notes, 0400, show_crash_notes, NULL);
@@ -167,7 +167,7 @@ static ssize_t show_crash_notes_size(struct device *dev, 
{

	ssize_t rc;



	rc = sprintf(buf, "%zu\n", sizeof(note_buf_t));

	rc = sysfs_emit(buf, "%zu\n", sizeof(note_buf_t));

	return rc;

}

static DEVICE_ATTR(crash_notes_size, 0400, show_crash_notes_size, NULL);
@@ -231,7 +231,7 @@ static struct cpu_attr cpu_attrs[] = { 
static ssize_t print_cpus_kernel_max(struct device *dev,

				     struct device_attribute *attr, char *buf)

{

	return sprintf(buf, "%d\n", NR_CPUS - 1);

	return sysfs_emit(buf, "%d\n", NR_CPUS - 1);

}

static DEVICE_ATTR(kernel_max, 0444, print_cpus_kernel_max, NULL);
@@ -279,7 +279,7 @@ static ssize_t print_cpus_isolated(struct device *dev, 


	cpumask_andnot(isolated, cpu_possible_mask,

		       housekeeping_cpumask(HK_FLAG_DOMAIN));

	n = sprintf(buf, "%*pbl\n", cpumask_pr_args(isolated));

	n = sysfs_emit(buf, "%*pbl\n", cpumask_pr_args(isolated));



	free_cpumask_var(isolated);
@@ -291,7 +291,7 @@ static DEVICE_ATTR(isolated, 0444, print_cpus_isolated, NULL); 
static ssize_t print_cpus_nohz_full(struct device *dev,

				  struct device_attribute *attr, char *buf)

{

	return sprintf(buf, "%*pbl\n", cpumask_pr_args(tick_nohz_full_mask));

	return sysfs_emit(buf, "%*pbl\n", cpumask_pr_args(tick_nohz_full_mask));

}

static DEVICE_ATTR(nohz_full, 0444, print_cpus_nohz_full, NULL);

#endif
@@ -323,7 +323,7 @@ static ssize_t print_cpu_modalias(struct device *dev, 
	ssize_t n;

	u32 i;



	n = sprintf(buf, "cpu:type:" CPU_FEATURE_TYPEFMT ":feature:",

	n = sysfs_emit(buf, "cpu:type:" CPU_FEATURE_TYPEFMT ":feature:",

		       CPU_FEATURE_TYPEVAL);



	for (i = 0; i < MAX_CPU_FEATURES; i++)
@@ -516,56 +516,56 @@ static void __init cpu_dev_register_generic(void) 
ssize_t __weak cpu_show_meltdown(struct device *dev,

				 struct device_attribute *attr, char *buf)

{

	return sprintf(buf, "Not affected\n");

	return sysfs_emit(buf, "Not affected\n");

}



ssize_t __weak cpu_show_spectre_v1(struct device *dev,

				   struct device_attribute *attr, char *buf)

{

	return sprintf(buf, "Not affected\n");

	return sysfs_emit(buf, "Not affected\n");

}



ssize_t __weak cpu_show_spectre_v2(struct device *dev,

				   struct device_attribute *attr, char *buf)

{

	return sprintf(buf, "Not affected\n");

	return sysfs_emit(buf, "Not affected\n");

}



ssize_t __weak cpu_show_spec_store_bypass(struct device *dev,

					  struct device_attribute *attr, char *buf)

{

	return sprintf(buf, "Not affected\n");

	return sysfs_emit(buf, "Not affected\n");

}



ssize_t __weak cpu_show_l1tf(struct device *dev,

			     struct device_attribute *attr, char *buf)

{

	return sprintf(buf, "Not affected\n");

	return sysfs_emit(buf, "Not affected\n");

}



ssize_t __weak cpu_show_mds(struct device *dev,

			    struct device_attribute *attr, char *buf)

{

	return sprintf(buf, "Not affected\n");

	return sysfs_emit(buf, "Not affected\n");

}



ssize_t __weak cpu_show_tsx_async_abort(struct device *dev,

					struct device_attribute *attr,

					char *buf)

{

	return sprintf(buf, "Not affected\n");

	return sysfs_emit(buf, "Not affected\n");

}



ssize_t __weak cpu_show_itlb_multihit(struct device *dev,

			    struct device_attribute *attr, char *buf)

{

	return sprintf(buf, "Not affected\n");

	return sysfs_emit(buf, "Not affected\n");

}



ssize_t __weak cpu_show_srbds(struct device *dev,

			      struct device_attribute *attr, char *buf)

{

	return sprintf(buf, "Not affected\n");

	return sysfs_emit(buf, "Not affected\n");

}



static DEVICE_ATTR(meltdown, 0444, cpu_show_meltdown, NULL);

drivers/base/dd.c

+1 −1
Original line number Diff line number Diff line
@@ -486,7 +486,7 @@ static ssize_t state_synced_show(struct device *dev, 
	device_lock(dev);

	val = dev->state_synced;

	device_unlock(dev);

	return sprintf(buf, "%u\n", val);

	return sysfs_emit(buf, "%u\n", val);

}

static DEVICE_ATTR_RO(state_synced);

CRA Git | Maintained and supported by SUSTech CRA and CCSE