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

Re: [Xen-devel] [PATCH v3 2/3] x86/ioreq server: Add new functions to get/set memory types.

On 4/26/2016 6:53 PM, Wei Liu wrote:
Hi Yu

On Mon, Apr 25, 2016 at 06:35:39PM +0800, Yu Zhang wrote:
For clarity this patch breaks the code to set/get memory types out
of do_hvm_op() into dedicated functions: hvmop_set/get_mem_type().
Also, for clarity, checks for whether a memory type change is allowed
are broken out into a separate function called by hvmop_set_mem_type().

There is no intentional functional change in this patch.

changes in v3:
  - Add Andrew's Acked-by and George's Reviewed-by.

changes in v2:
  - According to George Dunlap's comments, follow the "set rc /
    do something / goto out" pattern in hvmop_get_mem_type().

Normally we put these changelogs (or other information that is not
intended to be committed) between "---" so that they are ignored
when committing. Here is one example:


Note the Cc and some extra words inside surrounded by two "---". They
will be ignored when committing.

Oh. Thanks for your information, Wei. :)


The code itself looks good to me.


Xen-devel mailing list



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