Commit beee23ae authored by Prakash, Prashanth's avatar Prakash, Prashanth Committed by Rafael J. Wysocki

ACPI / CPPC: replace writeX/readX to PCC with relaxed version

We do not have a strict read/write order requirement while accessing
PCC subspace. The only requirement is all access should be committed
before triggering the PCC doorbell to transfer the ownership of PCC
to the platform and this requirement is enforced by the PCC driver.

Profiling on a many core system shows improvement of about 1.8us on
average per freq change request(about 10% improvement on average).
Since these operations are executed while holding the pcc_lock,
reducing this time helps the CPPC implementation to scale much
better as the number of cores increases.
Signed-off-by: default avatarPrashanth Prakash <pprakash@codeaurora.org>
Acked-by: default avatarAshwin Chaugule <ashwin.chaugule@linaro.org>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent 8b0f5788
...@@ -116,10 +116,10 @@ static int send_pcc_cmd(u16 cmd) ...@@ -116,10 +116,10 @@ static int send_pcc_cmd(u16 cmd)
} }
/* Write to the shared comm region. */ /* Write to the shared comm region. */
writew(cmd, &generic_comm_base->command); writew_relaxed(cmd, &generic_comm_base->command);
/* Flip CMD COMPLETE bit */ /* Flip CMD COMPLETE bit */
writew(0, &generic_comm_base->status); writew_relaxed(0, &generic_comm_base->status);
/* Ring doorbell */ /* Ring doorbell */
ret = mbox_send_message(pcc_channel, &cmd); ret = mbox_send_message(pcc_channel, &cmd);
...@@ -601,16 +601,16 @@ static int cpc_read(struct cpc_reg *reg, u64 *val) ...@@ -601,16 +601,16 @@ static int cpc_read(struct cpc_reg *reg, u64 *val)
switch (reg->bit_width) { switch (reg->bit_width) {
case 8: case 8:
*val = readb(vaddr); *val = readb_relaxed(vaddr);
break; break;
case 16: case 16:
*val = readw(vaddr); *val = readw_relaxed(vaddr);
break; break;
case 32: case 32:
*val = readl(vaddr); *val = readl_relaxed(vaddr);
break; break;
case 64: case 64:
*val = readq(vaddr); *val = readq_relaxed(vaddr);
break; break;
default: default:
pr_debug("Error: Cannot read %u bit width from PCC\n", pr_debug("Error: Cannot read %u bit width from PCC\n",
...@@ -632,16 +632,16 @@ static int cpc_write(struct cpc_reg *reg, u64 val) ...@@ -632,16 +632,16 @@ static int cpc_write(struct cpc_reg *reg, u64 val)
switch (reg->bit_width) { switch (reg->bit_width) {
case 8: case 8:
writeb(val, vaddr); writeb_relaxed(val, vaddr);
break; break;
case 16: case 16:
writew(val, vaddr); writew_relaxed(val, vaddr);
break; break;
case 32: case 32:
writel(val, vaddr); writel_relaxed(val, vaddr);
break; break;
case 64: case 64:
writeq(val, vaddr); writeq_relaxed(val, vaddr);
break; break;
default: default:
pr_debug("Error: Cannot write %u bit width to PCC\n", pr_debug("Error: Cannot write %u bit width to PCC\n",
......
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