Commit e7a9b05c authored by Jan Glauber's avatar Jan Glauber Committed by Herbert Xu

crypto: cavium - Fix smp_processor_id() warnings

Switch to raw_smp_processor_id() to prevent a number of
warnings from kernel debugging. We do not care about
preemption here, as the CPU number is only used as a
poor mans load balancing or device selection. If preemption
happens during a compress/decompress operation a small performance
hit will occur but everything will continue to work, so just
ignore it.
Signed-off-by: default avatarJan Glauber <jglauber@cavium.com>
Reviewed-by: default avatarRobert Richter <rrichter@cavium.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent 1cc7e01f
...@@ -87,12 +87,12 @@ u32 zip_load_instr(union zip_inst_s *instr, ...@@ -87,12 +87,12 @@ u32 zip_load_instr(union zip_inst_s *instr,
* Distribute the instructions between the enabled queues based on * Distribute the instructions between the enabled queues based on
* the CPU id. * the CPU id.
*/ */
if (smp_processor_id() % 2 == 0) if (raw_smp_processor_id() % 2 == 0)
queue = 0; queue = 0;
else else
queue = 1; queue = 1;
zip_dbg("CPU Core: %d Queue number:%d", smp_processor_id(), queue); zip_dbg("CPU Core: %d Queue number:%d", raw_smp_processor_id(), queue);
/* Take cmd buffer lock */ /* Take cmd buffer lock */
spin_lock(&zip_dev->iq[queue].lock); spin_lock(&zip_dev->iq[queue].lock);
......
...@@ -113,7 +113,7 @@ struct zip_device *zip_get_device(int node) ...@@ -113,7 +113,7 @@ struct zip_device *zip_get_device(int node)
*/ */
int zip_get_node_id(void) int zip_get_node_id(void)
{ {
return cpu_to_node(smp_processor_id()); return cpu_to_node(raw_smp_processor_id());
} }
/* Initializes the ZIP h/w sub-system */ /* Initializes the ZIP h/w sub-system */
......
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