From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roy Franz Subject: Re: [PATCH v2 0/5] xen: Break multiboot (v1) dependency and add multiboot2 support Date: Thu, 25 Sep 2014 08:56:39 -0700 Message-ID: References: <1411579162-27503-1-git-send-email-daniel.kiper@oracle.com> <5423F96C0200007800038CDF@mail.emea.novell.com> <20140925130140.GV3459@olila.local.net-space.pl> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta5.messagelabs.com ([195.245.231.135]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1XXBPO-0001HC-Iq for xen-devel@lists.xenproject.org; Thu, 25 Sep 2014 15:56:42 +0000 Received: by mail-vc0-f176.google.com with SMTP id la4so6426242vcb.21 for ; Thu, 25 Sep 2014 08:56:40 -0700 (PDT) In-Reply-To: <20140925130140.GV3459@olila.local.net-space.pl> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Daniel Kiper Cc: keir , Ian Campbell , Stefano Stabellini , andrew.cooper3@citrix.com, ross.philipson@citrix.com, ning.sun@intel.com, Jan Beulich , xen-devel@lists.xenproject.org, qiaowei.ren@intel.com, richard.l.maliszewski@intel.com, gang.wei@intel.com, Fu Wei List-Id: xen-devel@lists.xenproject.org On Thu, Sep 25, 2014 at 6:01 AM, Daniel Kiper wrote: > On Thu, Sep 25, 2014 at 10:15:56AM +0100, Jan Beulich wrote: >> >>> On 24.09.14 at 19:48, wrote: >> > On Wed, Sep 24, 2014 at 10:19 AM, Daniel Kiper wrote: >> >> Hi, >> >> >> >> This patch series breaks multiboot (v1) protocol dependency and adds >> >> multiboot2 support. It lays down the foundation for EFI + GRUB2 + Xen >> >> development. Detailed description of ideas and thoughts you will >> >> find in commit message for every patch. If something is not obvious >> >> please drop me a line. >> >> >> >> Most of the requested things are fixed but there are still some minor >> >> outstanding issues (multiboot2 tags generation, excessive amount of casts >> >> in xen/arch/x86/boot/reloc.c, etc.; please check commit messages for >> >> more details). If something is not fixed yet it means that I do not have >> >> good idea how to do that. In case you spot something which was mentioned >> >> during previous review and still think that your comment is valid >> >> in particular case please notify me. >> >> >> >> Below you can find reply for Konrad's questions in regards to >> >> exception request for Xen 4.5 release. >> >> >> >>> Couple of questions: >> >>> >> >>> - Since this is mostly XBI code, is there a lot of overlap with the ARM/x86 >> >>> refactoring of the EFI code? As in, will it require a lot of >> >>> rebasing/fixing it up? >> >> >> >> I did not checked that right now. However, Roy once told me that this should >> >> not be very big issue because overlap is not so big. Please check this email >> >> for more details: >> > http://lists.xen.org/archives/html/xen-devel/2014-08/msg01173.html >> >> >> >> Sadly, it was sent more then one month ago so maybe something has changed. >> >> Roy, any comments on that? >> > There aren't fundamental conceptual conflicts, but there will be quite >> > a bit of refactoring/rebasing to do. Most of my patchset is >> > re-organizing >> > x86/efi/boot.c, so rebasing it on this patchset would be a significant >> > amount of work. My guess is that the first 10 of the 13 patches >> > in my series would need be be completely redone, as the code they are >> > rearranging has been changed by your series. >> > I think that it would be much less work for you to re-base your series >> > on mine, than the other way. The x86/efi/boot.c changes >> > are a relatively small part of your series, but are the bulk of mine. >> > >> > I should have another version of my patchset out today. >> >> And when looking at both series later today in detail, I'm certainly >> going to give priority to Roy's as I expect that one to be largely >> ready to go in now (or at least a sufficiently large initial part of it). > > OK. Roy, do you expect any bigger changes in your code right now? > May I rebase safely my work on latest release of your patches? > Should I care about anything in special way? > > Daniel I don't think there should be any more major changes, but I'm still working through a few last issues which may result in some code movement. A lot of my patches are re-organizing boot.c, so even small changes can be a pain for patches based on that. I'm hoping to be able to resolve the last few issues in the next few days to get the patches merged for 4.5, so if you hold of for a few days you can hopefully take my changes from the master branch. Roy