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

Re: [Xen-devel] [v5][PATCH 03/10] xen:x86: define a new hypercall to get RMRR mappings



>>> On 28.08.14 at 04:24, <tiejun.chen@xxxxxxxxx> wrote:
> If you guys have no more comments, could I send a new series to review?

You certainly can do so at any time, but as said before I didn't get
to looking at the current version yet; briefly having looked at the
first two patches I'm already pretty convinced that the structuring
still isn't right (you shouldn't be exposing VT-d internals into
arbitrary parts of the hypervisor, but rather introduce a new
vendor IOMMU hook/actor to get at the reserved page information
- this all still goes along the lines of you apparently not
understanding that a reasonable level of abstraction helps in the
long term maintenance of such code).

Jan


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


 


Rackspace

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