[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2 2/3] xen/arm: vsmc: Don't implement function ID that doesn't exist



Hi Julien,

On 06.02.18 16:53, Julien Grall wrote:
Hi Volodymyr,

On 02/02/2018 01:46 PM, Volodymyr Babchuk wrote:


On 02.02.18 13:41, Julien Grall wrote:
The current implementation of SMCCC relies on the fact only function
number (bits [15:0]) is enough to identify what to implement.

However, PSCI call are only available in the range 0x84000000-0x8400001F
and 0xC4000000-0xC400001F. Furthermore, not all SMC32 functions have
equivalent in the SMC64. This is the case of:
     * PSCI_VERSION
     * CPU_OFF
     * MIGRATE_INFO_TYPE
     * SYSTEM_OFF
     * SYSTEM_RESET

Similarly call count, call uid, revision can only be query using smc32/hvc32
fast calls (See 6.2 in ARM DEN 0028B).

Xen should only implement identifier existing in the specification in
order to avoid potential clash with later revision. Therefore rework the
vsmc code to use the whole function identifier rather than only the
function number.

At the same time, the new macros for call count, call uid, revision are
renamed to better suit the spec.

Lastly, update SSSC_SMCCC_FUNCTION_COUNT to match the correct number of
funtions. Note that version is not updated because the number has always
been wrong, and nobody could properly use it.

Signed-off-by: Julien Grall <julien.grall@xxxxxxxxxx>

`Reviewed-by: Volodymyr Babchuk <volodymyr_babchuk@xxxxxxxx>`

Thank you for the reviewed-by. I noticed that you put ` at the beginning and end of the line in your reviewed-by tag so far. Is there any reason for that?

It is out of habit. You see, this is preferred way to add R-b tag at GitHub (or at least in OP-TEE repos). Github uses Markdown, and text in ` becomes formatted as a code. I do code review mostly at github, so it become a habit to add ` around R-b tags. You can ignore this, of course. I'll try to omit ` in the future.


--
Volodymyr Babchuk

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.