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

Re: [Xen-devel] [RFC 0/6] Rangeset generalisation



> -----Original Message-----
> From: Andrii Anisov [mailto:andrii.anisov@xxxxxxxxx]
> Sent: 16 February 2017 12:03
> To: xen-devel@xxxxxxxxxxxxxxxxxxxx
> Cc: andrii_anisov@xxxxxxxx; Andrew Cooper
> <Andrew.Cooper3@xxxxxxxxxx>; George Dunlap
> <George.Dunlap@xxxxxxxxxx>; Ian Jackson <Ian.Jackson@xxxxxxxxxx>;
> jbeulich@xxxxxxxx; konrad.wilk@xxxxxxxxxx; Paul Durrant
> <Paul.Durrant@xxxxxxxxxx>; sstabellini@xxxxxxxxxx; Tim (Xen.org)
> <tim@xxxxxxx>; Wei Liu <wei.liu2@xxxxxxxxxx>
> Subject: [RFC 0/6] Rangeset generalisation
> 
> From: Andrii Anisov <andrii_anisov@xxxxxxxx>
> 
> Rangesets in XEN seems to be a pretty generic thing slightly poisoned with.
> domain specific funtionality in initialization and deinitialization code.
> 
> So make the rangeset code generic with moving domain specific code to
> common/domain.c

Any particular reason this series is RFC? The cleanup seems a good thing to do 
to me.

  Paul

> 
> Andrii Anisov (6):
>   rangeset_new() refactoring
>   rangeset_destroy() refactoring
>   Drop rangeset_domain_initialise()
>   rangeset_domain_destroy() refactoring
>   rangeset_domain_printk() refactoring
>   Drop domain remains from rangeset
> 
>  xen/arch/x86/domain.c      |  2 +-
>  xen/arch/x86/hvm/ioreq.c   |  6 ++---
>  xen/arch/x86/mm/p2m.c      |  6 ++---
>  xen/arch/x86/setup.c       |  4 +--
>  xen/common/domain.c        | 48 +++++++++++++++++++++++++++++++----
>  xen/common/keyhandler.c    |  2 +-
>  xen/common/rangeset.c      | 63 +++++++++++++++-----------------------------
> --
>  xen/include/xen/domain.h   |  9 +++++++
>  xen/include/xen/rangeset.h | 42 +++++++++++++++----------------
>  9 files changed, 102 insertions(+), 80 deletions(-)
> 
> --
> 2.7.4


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

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