I''m a little confused about these patches. Could anyone clarify whether they are supposed to be applied? mkbuildtree doesn''t seem to use them. And some of the stuff in there is present in the sparse tree already, while other parts aren''t. Thanks, Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
On 27 Mar 2006, at 16:48, Jan Beulich wrote:> I''m a little confused about these patches. Could anyone clarify > whether they > are supposed to be applied? mkbuildtree doesn''t seem to use them. And > some > of the stuff in there is present in the sparse tree already, while > other parts > aren''t.They''re applied by a rule in buildconfigs/Rules.mk. -- Keir _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
>>> Keir Fraser <Keir.Fraser@cl.cam.ac.uk> 03/27/06 6:07 PM >>> > >On 27 Mar 2006, at 16:48, Jan Beulich wrote: > >> I''m a little confused about these patches. Could anyone clarify >> whether they >> are supposed to be applied? mkbuildtree doesn''t seem to use them. And >> some >> of the stuff in there is present in the sparse tree already, while >> other parts >> aren''t. > >They''re applied by a rule in buildconfigs/Rules.mk.Not exactly: That file looks for patches/*-*, but currently the directory is named _patches. Also, as I said, part of what''s in the patches there is already in the sparse tree, so these patches can''t apply cleanly I would think. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
>>> Keir Fraser <Keir.Fraser@cl.cam.ac.uk> 03/27/06 6:07 PM >>> >On 27 Mar 2006, at 16:48, Jan Beulich wrote: >> I''m a little confused about these patches. Could anyone clarify >> whether they >> are supposed to be applied? mkbuildtree doesn''t seem to use them. And >> some >> of the stuff in there is present in the sparse tree already, while >> other parts >> aren''t. > >They''re applied by a rule in buildconfigs/Rules.mk.Just as a follow-up: I was just told that the _patches thing is something that gets done to our internal tree, so forget about that inconsistency. What remains is the question why some of what''s in these patches is already in the sparse tree. Is that simply because the patches get applied before overlaying the sparse tree, and hence the files that are in the sparse tree have to duplicate the code? If so, it would seem much cleaner to remove i386-mach-io-check-nmi.patch and smp-alts.patch (as they appear to completely live in the sparse tree) and add the few files needed for fully applying the others. Thanks, Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel
On 3/27/06, Jan Beulich <jbeulich@novell.com> wrote:> >>> Keir Fraser <Keir.Fraser@cl.cam.ac.uk> 03/27/06 6:07 PM >>> > >On 27 Mar 2006, at 16:48, Jan Beulich wrote: > >> I''m a little confused about these patches. Could anyone clarify > >> whether they > >> are supposed to be applied? mkbuildtree doesn''t seem to use them. And > >> some > >> of the stuff in there is present in the sparse tree already, while > >> other parts > >> aren''t. > > > >They''re applied by a rule in buildconfigs/Rules.mk. > > Just as a follow-up: I was just told that the _patches thing is something that > gets done to our internal tree, so forget about that inconsistency. What > remains is the question why some of what''s in these patches is already in > the sparse tree. Is that simply because the patches get applied before > overlaying the sparse tree, and hence the files that are in the sparse tree > have to duplicate the code? If so, it would seem much cleaner to remove > i386-mach-io-check-nmi.patch and smp-alts.patch (as they appear to > completely live in the sparse tree) and add the few files needed for fully > applying the others.Yes, parts of these patches which touch files which are in the sparse tree are applied to the files in the sparse tree because the patches get applied before the sparse tree files overwrite the files from the reference tree. I find maintaining these patches quite painful and have considered doing what you suggest several times. I think I''ll make that change once 3.0.2 is out. christian _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel