Commit e4f226b1 authored by Sukadev Bhattiprolu's avatar Sukadev Bhattiprolu Committed by Michael Ellerman

powerpc/perf/hv-gpci: Increase request buffer size

The GPCI hcall allows for a 4K buffer but we limit the buffer to 1K.
The problem with a 1K buffer is if a request results in returning
more values than can be accomodated in the 1K buffer the request will
fail.

The buffer we are using is currently allocated on the stack and hence
limited in size. Instead use a per-CPU 4K buffer like we do with 24x7
counters (hv-24x7.c).

While here, rename the macro GPCI_MAX_DATA_BYTES to HGPCI_MAX_DATA_BYTES
for consistency with 24x7 counters.
Signed-off-by: default avatarSukadev Bhattiprolu <sukadev@linux.vnet.ibm.com>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
parent be283eeb
...@@ -127,8 +127,16 @@ static const struct attribute_group *attr_groups[] = { ...@@ -127,8 +127,16 @@ static const struct attribute_group *attr_groups[] = {
NULL, NULL,
}; };
#define GPCI_MAX_DATA_BYTES \ #define HGPCI_REQ_BUFFER_SIZE 4096
(1024 - sizeof(struct hv_get_perf_counter_info_params)) #define HGPCI_MAX_DATA_BYTES \
(HGPCI_REQ_BUFFER_SIZE - sizeof(struct hv_get_perf_counter_info_params))
DEFINE_PER_CPU(char, hv_gpci_reqb[HGPCI_REQ_BUFFER_SIZE]) __aligned(sizeof(uint64_t));
struct hv_gpci_request_buffer {
struct hv_get_perf_counter_info_params params;
uint8_t bytes[HGPCI_MAX_DATA_BYTES];
} __packed;
static unsigned long single_gpci_request(u32 req, u32 starting_index, static unsigned long single_gpci_request(u32 req, u32 starting_index,
u16 secondary_index, u8 version_in, u32 offset, u8 length, u16 secondary_index, u8 version_in, u32 offset, u8 length,
...@@ -137,24 +145,21 @@ static unsigned long single_gpci_request(u32 req, u32 starting_index, ...@@ -137,24 +145,21 @@ static unsigned long single_gpci_request(u32 req, u32 starting_index,
unsigned long ret; unsigned long ret;
size_t i; size_t i;
u64 count; u64 count;
struct hv_gpci_request_buffer *arg;
struct { arg = (void *)get_cpu_var(hv_gpci_reqb);
struct hv_get_perf_counter_info_params params; memset(arg, 0, HGPCI_REQ_BUFFER_SIZE);
uint8_t bytes[GPCI_MAX_DATA_BYTES];
} __packed __aligned(sizeof(uint64_t)) arg = { arg->params.counter_request = cpu_to_be32(req);
.params = { arg->params.starting_index = cpu_to_be32(starting_index);
.counter_request = cpu_to_be32(req), arg->params.secondary_index = cpu_to_be16(secondary_index);
.starting_index = cpu_to_be32(starting_index), arg->params.counter_info_version_in = version_in;
.secondary_index = cpu_to_be16(secondary_index),
.counter_info_version_in = version_in,
}
};
ret = plpar_hcall_norets(H_GET_PERF_COUNTER_INFO, ret = plpar_hcall_norets(H_GET_PERF_COUNTER_INFO,
virt_to_phys(&arg), sizeof(arg)); virt_to_phys(arg), HGPCI_REQ_BUFFER_SIZE);
if (ret) { if (ret) {
pr_devel("hcall failed: 0x%lx\n", ret); pr_devel("hcall failed: 0x%lx\n", ret);
return ret; goto out;
} }
/* /*
...@@ -163,9 +168,11 @@ static unsigned long single_gpci_request(u32 req, u32 starting_index, ...@@ -163,9 +168,11 @@ static unsigned long single_gpci_request(u32 req, u32 starting_index,
*/ */
count = 0; count = 0;
for (i = offset; i < offset + length; i++) for (i = offset; i < offset + length; i++)
count |= arg.bytes[i] << (i - offset); count |= arg->bytes[i] << (i - offset);
*value = count; *value = count;
out:
put_cpu_var(hv_gpci_reqb);
return ret; return ret;
} }
...@@ -245,10 +252,10 @@ static int h_gpci_event_init(struct perf_event *event) ...@@ -245,10 +252,10 @@ static int h_gpci_event_init(struct perf_event *event)
} }
/* last byte within the buffer? */ /* last byte within the buffer? */
if ((event_get_offset(event) + length) > GPCI_MAX_DATA_BYTES) { if ((event_get_offset(event) + length) > HGPCI_MAX_DATA_BYTES) {
pr_devel("request outside of buffer: %zu > %zu\n", pr_devel("request outside of buffer: %zu > %zu\n",
(size_t)event_get_offset(event) + length, (size_t)event_get_offset(event) + length,
GPCI_MAX_DATA_BYTES); HGPCI_MAX_DATA_BYTES);
return -EINVAL; return -EINVAL;
} }
......
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