linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* wfi and wfe redundancy
@ 2022-10-10  4:34 Min Lee
  0 siblings, 0 replies; only message in thread
From: Min Lee @ 2022-10-10  4:34 UTC (permalink / raw)
  To: suzuki.poulose; +Cc: catalin.marinas, will, linux-arm-kernel

Hi Suzuki,

I am writing to ask you whether the code you wrote is including redundancy or not.
The commit is https://github.com/iamroot19/linux/commit/bb9052744f4b7ae11d0061ac9492dd2949981b49.
wfi has been added in __no_granule_support function through your commit. Thus, there are wfe and also wfi.
The function saves an error code into one of the memory spaces, then falls into an infinite loop. In the loop, wfe and wfi are used to save power.
There is no possibility that generates an event on the sequence, so wfe is enough to keep the core asleep. I think that's why the previous code used only wfe.
Interrupts can also wake up the core that is sleeping by wfe, but this is also not enough to explain why wfi is needed because only one, wfe or wfi, is needed for saving power.
Would you please tell me why both wfe and wfi are needed?

Best,
Min

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2022-10-10  4:35 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-10  4:34 wfi and wfe redundancy Min Lee

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).