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

Re: [Xen-devel] [PATCH MM-PART1 v3 1/8] xen/arm: Don't boot Xen on platform using AIVIVT instruction caches



On Mon, 10 Jun 2019, Julien Grall wrote:
> On 6/10/19 9:40 PM, Stefano Stabellini wrote:
> > Hi Julien,
> 
> Hi Stefano,
> 
> > 
> > I expressed my preference below. We don't agree. Is there anything else
> > you would like me to add to this thread? Do you have a specific
> > question? The only question I see below is "Users of what?" but I take
> > it was just rhetorical.
> 
> No it wasn't rhetorical. It was a genuine question, because you are implying
> that:
>       1) It is possible to have user that are using AIVIVT
>       2) We have to support out of tree users
> 
> The latter is particularly critical as this implies that any change in Xen
> should be done with keeping in mind any patches that could be applied on top
> of Xen.
> 
> So I am all hear of your arguments here... At the end, we need to come to an
> agreement here as at the moment my patch can't go without your ack.

No, we don't have to support out of tree users. I didn't mean to imply
it. But it costs us very little to be courteous and polite in cases like
this, sending a more obvious [ANNOUNCE] email saying "we are dropping
AIVIVT as nobody should be using it".

Can this patch go in regardless? I wouldn't be happy about it, but if
this was a vote it would be a -1, not a -2. It is difficult to give an
ack for a thing I don't like, but I wouldn't go as far as nacking it.

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