From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH v12 0/8] Xen VMware tools support Date: Tue, 28 Jul 2015 10:32:20 -0400 Message-ID: <20150728143209.GA26623@x230.dumpdata.com> References: <1435447665-5433-1-git-send-email-Don.Slutz@Gmail.com> <559FA13B020000780008F41F@mail.emea.novell.com> <20150710090729.GE26853@zion.uk.xensource.com> <55B6F46B.2060906@Gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <55B6F46B.2060906@Gmail.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Don Slutz Cc: Jun Nakajima , Tim Deegan , Kevin Tian , Wei Liu , Ian Campbell , Stefano Stabellini , George Dunlap , Andrew Cooper , Ian Jackson , xen-devel@lists.xen.org, Jan Beulich , Eddie Dong , Aravind Gopalakrishnan , Suravee Suthikulpanit , Keir Fraser , Boris Ostrovsky List-Id: xen-devel@lists.xenproject.org On Mon, Jul 27, 2015 at 11:18:03PM -0400, Don Slutz wrote: > On 07/10/15 05:07, Wei Liu wrote: > > On Fri, Jul 10, 2015 at 09:40:59AM +0100, Jan Beulich wrote: > >>>>> On 28.06.15 at 01:27, wrote: > >>> Changes v11 to v12: > >>> Rebased on staging. > >>> > > > >> > >> I was actually meaning to apply this (at least until patch 6; I'm > >> not convinced patch 7 makes sense to go in at this point), but > >> noticed that comments by Konrad were still un-addressed. I'm > >> also expecting some re-basing to be needed on top of Paul's > >> emulation re-work. > >> > > My understanding of Konrad's comments are: > > For patch #2: > adjust a comment that is removed in patch #4. > I did this, but failed to get it posted before leaving on > vacation (sigh). > > Adjust copyright date from 2012 to 2015 (but not required). > > > For patch #3: > Perhaps add 'Recommended value is 7' ? > I did not take this as a required change. > > > For patch #7 (which also need a lot of merge work) I was waiting till > Paul's changes had been committed. Now that they are I will be working > on the merge, > > >> Wei, considering that this had already missed 4.5, would you > >> be okay to take at least first 6 patches (which seem reasonably > >> risk free) after the freeze? > >> > > > > Yes. I think it is fine, but please do it as soon as possible, > > preferably within next week. > > > > I see that this did not happen. Do you want me to re-post just 1 to 6 > that I have now rebased? (the improvement to the doc, Ian said could be > a follow on patch) Welcome back from vacation! I think Wei was hoping to get your patches before the feature freeze. Now that ship has sailed ... > > -Don Slutz > > > Wei. > > > >> Jan