diff options
author | Sudeep Holla <sudeep.holla@arm.com> | 2014-09-30 09:48:22 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-11-07 14:45:00 -0500 |
commit | 5aaba36318e5995e8c95d077a46d9a4d00fcc1cd (patch) | |
tree | edf9eb1ea758b82f1f04963a0fc54fab8bc8bc50 /drivers | |
parent | 0372ffb35d00288802265586a29c117911d02fb8 (diff) |
cpumask: factor out show_cpumap into separate helper function
Many sysfs *_show function use cpu{list,mask}_scnprintf to copy cpumap
to the buffer aligned to PAGE_SIZE, append '\n' and '\0' to return null
terminated buffer with newline.
This patch creates a new helper function cpumap_print_to_pagebuf in
cpumask.h using newly added bitmap_print_to_pagebuf and consolidates
most of those sysfs functions using the new helper function.
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
Suggested-by: Stephen Boyd <sboyd@codeaurora.org>
Tested-by: Stephen Boyd <sboyd@codeaurora.org>
Acked-by: "Rafael J. Wysocki" <rjw@rjwysocki.net>
Acked-by: Bjorn Helgaas <bhelgaas@google.com>
Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: x86@kernel.org
Cc: linux-acpi@vger.kernel.org
Cc: linux-pci@vger.kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/acpi/acpi_pad.c | 8 | ||||
-rw-r--r-- | drivers/base/cpu.c | 5 | ||||
-rw-r--r-- | drivers/base/node.c | 14 | ||||
-rw-r--r-- | drivers/base/topology.c | 22 | ||||
-rw-r--r-- | drivers/pci/pci-sysfs.c | 39 |
5 files changed, 20 insertions, 68 deletions
diff --git a/drivers/acpi/acpi_pad.c b/drivers/acpi/acpi_pad.c index f148a0580e04..c7b105c0e1d3 100644 --- a/drivers/acpi/acpi_pad.c +++ b/drivers/acpi/acpi_pad.c | |||
@@ -350,12 +350,10 @@ static ssize_t acpi_pad_idlecpus_store(struct device *dev, | |||
350 | static ssize_t acpi_pad_idlecpus_show(struct device *dev, | 350 | static ssize_t acpi_pad_idlecpus_show(struct device *dev, |
351 | struct device_attribute *attr, char *buf) | 351 | struct device_attribute *attr, char *buf) |
352 | { | 352 | { |
353 | int n = 0; | 353 | return cpumap_print_to_pagebuf(false, buf, |
354 | n = cpumask_scnprintf(buf, PAGE_SIZE-2, to_cpumask(pad_busy_cpus_bits)); | 354 | to_cpumask(pad_busy_cpus_bits)); |
355 | buf[n++] = '\n'; | ||
356 | buf[n] = '\0'; | ||
357 | return n; | ||
358 | } | 355 | } |
356 | |||
359 | static DEVICE_ATTR(idlecpus, S_IRUGO|S_IWUSR, | 357 | static DEVICE_ATTR(idlecpus, S_IRUGO|S_IWUSR, |
360 | acpi_pad_idlecpus_show, | 358 | acpi_pad_idlecpus_show, |
361 | acpi_pad_idlecpus_store); | 359 | acpi_pad_idlecpus_store); |
diff --git a/drivers/base/cpu.c b/drivers/base/cpu.c index 006b1bc5297d..4d8a56406fbb 100644 --- a/drivers/base/cpu.c +++ b/drivers/base/cpu.c | |||
@@ -207,11 +207,8 @@ static ssize_t show_cpus_attr(struct device *dev, | |||
207 | char *buf) | 207 | char *buf) |
208 | { | 208 | { |
209 | struct cpu_attr *ca = container_of(attr, struct cpu_attr, attr); | 209 | struct cpu_attr *ca = container_of(attr, struct cpu_attr, attr); |
210 | int n = cpulist_scnprintf(buf, PAGE_SIZE-2, *(ca->map)); | ||
211 | 210 | ||
212 | buf[n++] = '\n'; | 211 | return cpumap_print_to_pagebuf(true, buf, *ca->map); |
213 | buf[n] = '\0'; | ||
214 | return n; | ||
215 | } | 212 | } |
216 | 213 | ||
217 | #define _CPU_ATTR(name, map) \ | 214 | #define _CPU_ATTR(name, map) \ |
diff --git a/drivers/base/node.c b/drivers/base/node.c index 472168cd0c97..a3b82e9c7f20 100644 --- a/drivers/base/node.c +++ b/drivers/base/node.c | |||
@@ -25,32 +25,26 @@ static struct bus_type node_subsys = { | |||
25 | }; | 25 | }; |
26 | 26 | ||
27 | 27 | ||
28 | static ssize_t node_read_cpumap(struct device *dev, int type, char *buf) | 28 | static ssize_t node_read_cpumap(struct device *dev, bool list, char *buf) |
29 | { | 29 | { |
30 | struct node *node_dev = to_node(dev); | 30 | struct node *node_dev = to_node(dev); |
31 | const struct cpumask *mask = cpumask_of_node(node_dev->dev.id); | 31 | const struct cpumask *mask = cpumask_of_node(node_dev->dev.id); |
32 | int len; | ||
33 | 32 | ||
34 | /* 2008/04/07: buf currently PAGE_SIZE, need 9 chars per 32 bits. */ | 33 | /* 2008/04/07: buf currently PAGE_SIZE, need 9 chars per 32 bits. */ |
35 | BUILD_BUG_ON((NR_CPUS/32 * 9) > (PAGE_SIZE-1)); | 34 | BUILD_BUG_ON((NR_CPUS/32 * 9) > (PAGE_SIZE-1)); |
36 | 35 | ||
37 | len = type? | 36 | return cpumap_print_to_pagebuf(list, buf, mask); |
38 | cpulist_scnprintf(buf, PAGE_SIZE-2, mask) : | ||
39 | cpumask_scnprintf(buf, PAGE_SIZE-2, mask); | ||
40 | buf[len++] = '\n'; | ||
41 | buf[len] = '\0'; | ||
42 | return len; | ||
43 | } | 37 | } |
44 | 38 | ||
45 | static inline ssize_t node_read_cpumask(struct device *dev, | 39 | static inline ssize_t node_read_cpumask(struct device *dev, |
46 | struct device_attribute *attr, char *buf) | 40 | struct device_attribute *attr, char *buf) |
47 | { | 41 | { |
48 | return node_read_cpumap(dev, 0, buf); | 42 | return node_read_cpumap(dev, false, buf); |
49 | } | 43 | } |
50 | static inline ssize_t node_read_cpulist(struct device *dev, | 44 | static inline ssize_t node_read_cpulist(struct device *dev, |
51 | struct device_attribute *attr, char *buf) | 45 | struct device_attribute *attr, char *buf) |
52 | { | 46 | { |
53 | return node_read_cpumap(dev, 1, buf); | 47 | return node_read_cpumap(dev, true, buf); |
54 | } | 48 | } |
55 | 49 | ||
56 | static DEVICE_ATTR(cpumap, S_IRUGO, node_read_cpumask, NULL); | 50 | static DEVICE_ATTR(cpumap, S_IRUGO, node_read_cpumask, NULL); |
diff --git a/drivers/base/topology.c b/drivers/base/topology.c index be7c1fb7c0c9..f7c353843ddf 100644 --- a/drivers/base/topology.c +++ b/drivers/base/topology.c | |||
@@ -42,29 +42,11 @@ static ssize_t show_##name(struct device *dev, \ | |||
42 | return sprintf(buf, "%d\n", topology_##name(dev->id)); \ | 42 | return sprintf(buf, "%d\n", topology_##name(dev->id)); \ |
43 | } | 43 | } |
44 | 44 | ||
45 | #if defined(topology_thread_cpumask) || defined(topology_core_cpumask) || \ | ||
46 | defined(topology_book_cpumask) | ||
47 | static ssize_t show_cpumap(int type, const struct cpumask *mask, char *buf) | ||
48 | { | ||
49 | ptrdiff_t len = PTR_ALIGN(buf + PAGE_SIZE - 1, PAGE_SIZE) - buf; | ||
50 | int n = 0; | ||
51 | |||
52 | if (len > 1) { | ||
53 | n = type? | ||
54 | cpulist_scnprintf(buf, len-2, mask) : | ||
55 | cpumask_scnprintf(buf, len-2, mask); | ||
56 | buf[n++] = '\n'; | ||
57 | buf[n] = '\0'; | ||
58 | } | ||
59 | return n; | ||
60 | } | ||
61 | #endif | ||
62 | |||
63 | #define define_siblings_show_map(name) \ | 45 | #define define_siblings_show_map(name) \ |
64 | static ssize_t show_##name(struct device *dev, \ | 46 | static ssize_t show_##name(struct device *dev, \ |
65 | struct device_attribute *attr, char *buf) \ | 47 | struct device_attribute *attr, char *buf) \ |
66 | { \ | 48 | { \ |
67 | return show_cpumap(0, topology_##name(dev->id), buf); \ | 49 | return cpumap_print_to_pagebuf(false, buf, topology_##name(dev->id));\ |
68 | } | 50 | } |
69 | 51 | ||
70 | #define define_siblings_show_list(name) \ | 52 | #define define_siblings_show_list(name) \ |
@@ -72,7 +54,7 @@ static ssize_t show_##name##_list(struct device *dev, \ | |||
72 | struct device_attribute *attr, \ | 54 | struct device_attribute *attr, \ |
73 | char *buf) \ | 55 | char *buf) \ |
74 | { \ | 56 | { \ |
75 | return show_cpumap(1, topology_##name(dev->id), buf); \ | 57 | return cpumap_print_to_pagebuf(true, buf, topology_##name(dev->id));\ |
76 | } | 58 | } |
77 | 59 | ||
78 | #define define_siblings_show_func(name) \ | 60 | #define define_siblings_show_func(name) \ |
diff --git a/drivers/pci/pci-sysfs.c b/drivers/pci/pci-sysfs.c index 2c6643fdc0cf..368bdac42603 100644 --- a/drivers/pci/pci-sysfs.c +++ b/drivers/pci/pci-sysfs.c | |||
@@ -77,11 +77,10 @@ static ssize_t broken_parity_status_store(struct device *dev, | |||
77 | } | 77 | } |
78 | static DEVICE_ATTR_RW(broken_parity_status); | 78 | static DEVICE_ATTR_RW(broken_parity_status); |
79 | 79 | ||
80 | static ssize_t pci_dev_show_local_cpu(struct device *dev, int type, | 80 | static ssize_t pci_dev_show_local_cpu(struct device *dev, bool list, |
81 | struct device_attribute *attr, char *buf) | 81 | struct device_attribute *attr, char *buf) |
82 | { | 82 | { |
83 | const struct cpumask *mask; | 83 | const struct cpumask *mask; |
84 | int len; | ||
85 | 84 | ||
86 | #ifdef CONFIG_NUMA | 85 | #ifdef CONFIG_NUMA |
87 | mask = (dev_to_node(dev) == -1) ? cpu_online_mask : | 86 | mask = (dev_to_node(dev) == -1) ? cpu_online_mask : |
@@ -89,59 +88,41 @@ static ssize_t pci_dev_show_local_cpu(struct device *dev, int type, | |||
89 | #else | 88 | #else |
90 | mask = cpumask_of_pcibus(to_pci_dev(dev)->bus); | 89 | mask = cpumask_of_pcibus(to_pci_dev(dev)->bus); |
91 | #endif | 90 | #endif |
92 | len = type ? | 91 | return cpumap_print_to_pagebuf(list, buf, mask); |
93 | cpumask_scnprintf(buf, PAGE_SIZE-2, mask) : | ||
94 | cpulist_scnprintf(buf, PAGE_SIZE-2, mask); | ||
95 | |||
96 | buf[len++] = '\n'; | ||
97 | buf[len] = '\0'; | ||
98 | return len; | ||
99 | } | 92 | } |
100 | 93 | ||
101 | static ssize_t local_cpus_show(struct device *dev, | 94 | static ssize_t local_cpus_show(struct device *dev, |
102 | struct device_attribute *attr, char *buf) | 95 | struct device_attribute *attr, char *buf) |
103 | { | 96 | { |
104 | return pci_dev_show_local_cpu(dev, 1, attr, buf); | 97 | return pci_dev_show_local_cpu(dev, false, attr, buf); |
105 | } | 98 | } |
106 | static DEVICE_ATTR_RO(local_cpus); | 99 | static DEVICE_ATTR_RO(local_cpus); |
107 | 100 | ||
108 | static ssize_t local_cpulist_show(struct device *dev, | 101 | static ssize_t local_cpulist_show(struct device *dev, |
109 | struct device_attribute *attr, char *buf) | 102 | struct device_attribute *attr, char *buf) |
110 | { | 103 | { |
111 | return pci_dev_show_local_cpu(dev, 0, attr, buf); | 104 | return pci_dev_show_local_cpu(dev, true, attr, buf); |
112 | } | 105 | } |
113 | static DEVICE_ATTR_RO(local_cpulist); | 106 | static DEVICE_ATTR_RO(local_cpulist); |
114 | 107 | ||
115 | /* | 108 | /* |
116 | * PCI Bus Class Devices | 109 | * PCI Bus Class Devices |
117 | */ | 110 | */ |
118 | static ssize_t pci_bus_show_cpuaffinity(struct device *dev, int type, | ||
119 | struct device_attribute *attr, | ||
120 | char *buf) | ||
121 | { | ||
122 | int ret; | ||
123 | const struct cpumask *cpumask; | ||
124 | |||
125 | cpumask = cpumask_of_pcibus(to_pci_bus(dev)); | ||
126 | ret = type ? | ||
127 | cpulist_scnprintf(buf, PAGE_SIZE-2, cpumask) : | ||
128 | cpumask_scnprintf(buf, PAGE_SIZE-2, cpumask); | ||
129 | buf[ret++] = '\n'; | ||
130 | buf[ret] = '\0'; | ||
131 | return ret; | ||
132 | } | ||
133 | |||
134 | static ssize_t cpuaffinity_show(struct device *dev, | 111 | static ssize_t cpuaffinity_show(struct device *dev, |
135 | struct device_attribute *attr, char *buf) | 112 | struct device_attribute *attr, char *buf) |
136 | { | 113 | { |
137 | return pci_bus_show_cpuaffinity(dev, 0, attr, buf); | 114 | const struct cpumask *cpumask = cpumask_of_pcibus(to_pci_bus(dev)); |
115 | |||
116 | return cpumap_print_to_pagebuf(false, buf, cpumask); | ||
138 | } | 117 | } |
139 | static DEVICE_ATTR_RO(cpuaffinity); | 118 | static DEVICE_ATTR_RO(cpuaffinity); |
140 | 119 | ||
141 | static ssize_t cpulistaffinity_show(struct device *dev, | 120 | static ssize_t cpulistaffinity_show(struct device *dev, |
142 | struct device_attribute *attr, char *buf) | 121 | struct device_attribute *attr, char *buf) |
143 | { | 122 | { |
144 | return pci_bus_show_cpuaffinity(dev, 1, attr, buf); | 123 | const struct cpumask *cpumask = cpumask_of_pcibus(to_pci_bus(dev)); |
124 | |||
125 | return cpumap_print_to_pagebuf(true, buf, cpumask); | ||
145 | } | 126 | } |
146 | static DEVICE_ATTR_RO(cpulistaffinity); | 127 | static DEVICE_ATTR_RO(cpulistaffinity); |
147 | 128 | ||