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

Re: [Xen-devel] [PATCH v3 1/3] x86/ioreq server(patch for 4.7): Rename p2m_mmio_write_dm to p2m_ioreq_server.



>>> On 25.04.16 at 15:39, <George.Dunlap@xxxxxxxxxxxxx> wrote:
> On Mon, Apr 25, 2016 at 11:35 AM, Yu Zhang <yu.c.zhang@xxxxxxxxxxxxxxx> wrote:
>> --- a/xen/include/public/hvm/hvm_op.h
>> +++ b/xen/include/public/hvm/hvm_op.h
>> @@ -83,7 +83,13 @@ typedef enum {
>>      HVMMEM_ram_rw,             /* Normal read/write guest RAM */
>>      HVMMEM_ram_ro,             /* Read-only; writes are discarded */
>>      HVMMEM_mmio_dm,            /* Reads and write go to the device model */
>> -    HVMMEM_mmio_write_dm       /* Read-only; writes go to the device model 
>> */
>> +#if __XEN_INTERFACE_VERSION__ < 0x00040700
>> +    HVMMEM_mmio_write_dm,      /* Read-only; writes go to the device model 
>> */
>> +#else
>> +    HVMMEM_unused,             /* Placeholder; setting memory to this type
>> +                                  will fail for code after 4.7.0 */
>> +#endif
>> +    HVMMEM_ioreq_server
> 
> Also, I don't think we've had a convincing argument for why this patch
> needs to be in 4.7.  The p2m name changes are internal only, and so
> don't need to be made at all; and the old functionality will work as
> well as it ever did.  Furthermore, the whole reason we're in this
> situation is that we checked in a publicly-visible change to the
> interface before it was properly ready; I think we should avoid making
> the same mistake this time.

Good point.

> So personally I'd just leave this patch entirely for 4.8; but if Paul
> and/or Jan have strong opinions, then I would say check in only a
> patch to do the #if/#else/#endif, and leave both the p2m type changes
> and the new HVMMEM_ioreq_server enum for when the 4.8 development
> window opens.

Doing that alone would break the build - it would need to be a
little more than that.

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®.