* [PATCH] build: save generated config in /boot @ 2015-12-18 21:35 Doug Goldstein 2015-12-21 12:11 ` Jan Beulich 0 siblings, 1 reply; 25+ messages in thread From: Doug Goldstein @ 2015-12-18 21:35 UTC (permalink / raw) To: xen-devel Cc: Keir Fraser, Ian Campbell, Ian Jackson, Doug Goldstein, Tim Deegan, Jan Beulich Since we now support changing Xen options with Kconfig, we should save the configuration that was used to build up Xen. This will save it in /boot alongside the installed xen.gz and call it xen-$(FULLVERSION).config Suggested-by: Ian Campbell <ian.campbell@citrix.com> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> --- xen/Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/xen/Makefile b/xen/Makefile index 9023863..460b977 100644 --- a/xen/Makefile +++ b/xen/Makefile @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) + $(INSTALL_DATA) $(KCONFIG_CONFIG) $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config if [ -r $(TARGET).efi -a -n '$(EFI_DIR)' ]; then \ [ -d $(D)$(EFI_DIR) ] || $(INSTALL_DIR) $(D)$(EFI_DIR); \ $(INSTALL_DATA) $(TARGET).efi $(D)$(EFI_DIR)/$(T)-$(XEN_FULLVERSION).efi; \ -- 2.4.10 ^ permalink raw reply related [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-18 21:35 [PATCH] build: save generated config in /boot Doug Goldstein @ 2015-12-21 12:11 ` Jan Beulich 2015-12-21 12:40 ` Andrew Cooper 2015-12-21 14:35 ` Doug Goldstein 0 siblings, 2 replies; 25+ messages in thread From: Jan Beulich @ 2015-12-21 12:11 UTC (permalink / raw) To: Doug Goldstein Cc: Keir Fraser, Tim Deegan, Ian Jackson, Ian Campbell, xen-devel >>> On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: > Since we now support changing Xen options with Kconfig, we should save > the configuration that was used to build up Xen. This will save it in > /boot alongside the installed xen.gz and call it > xen-$(FULLVERSION).config > > Suggested-by: Ian Campbell <ian.campbell@citrix.com> > Signed-off-by: Doug Goldstein <cardoe@cardoe.com> > --- > xen/Makefile | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/xen/Makefile b/xen/Makefile > index 9023863..460b977 100644 > --- a/xen/Makefile > +++ b/xen/Makefile > @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) > ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) > [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) > $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) > + $(INSTALL_DATA) $(KCONFIG_CONFIG) $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config Was it really suggested to put this into /boot? It has no business being there... Jan ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-21 12:11 ` Jan Beulich @ 2015-12-21 12:40 ` Andrew Cooper 2015-12-21 12:53 ` Jan Beulich 2015-12-21 14:35 ` Doug Goldstein 1 sibling, 1 reply; 25+ messages in thread From: Andrew Cooper @ 2015-12-21 12:40 UTC (permalink / raw) To: Jan Beulich, Doug Goldstein Cc: Tim Deegan, xen-devel, Keir Fraser, Ian Jackson, Ian Campbell On 21/12/15 12:11, Jan Beulich wrote: >>>> On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: >> Since we now support changing Xen options with Kconfig, we should save >> the configuration that was used to build up Xen. This will save it in >> /boot alongside the installed xen.gz and call it >> xen-$(FULLVERSION).config >> >> Suggested-by: Ian Campbell <ian.campbell@citrix.com> >> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> >> --- >> xen/Makefile | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/xen/Makefile b/xen/Makefile >> index 9023863..460b977 100644 >> --- a/xen/Makefile >> +++ b/xen/Makefile >> @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) >> ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) >> [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) >> $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) >> + $(INSTALL_DATA) $(KCONFIG_CONFIG) $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config > Was it really suggested to put this into /boot? It has no business > being there... A typical Linux has: andrewcoop@andrewcoop:/local/xen.git/xen$ ls -lA /boot/ total 21044 -rw-r--r-- 1 root root 157726 Dec 15 15:40 config-3.16.0-4-amd64 drwxr-xr-x 5 root root 4096 Dec 18 07:53 grub -rw-r--r-- 1 root root 15535038 Dec 18 07:52 initrd.img-3.16.0-4-amd64 drwx------ 2 root root 16384 Oct 19 11:11 lost+found -rw-r--r-- 1 root root 2676277 Dec 15 15:40 System.map-3.16.0-4-amd64 -rw-r--r-- 1 root root 3118928 Dec 15 15:37 vmlinuz-3.16.0-4-amd64 which at the very least is consistent between Debian and RHEL derivatives. IMO, doing the same for Xen is sensible. ~Andrew ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-21 12:40 ` Andrew Cooper @ 2015-12-21 12:53 ` Jan Beulich 2015-12-22 12:45 ` Doug Goldstein 0 siblings, 1 reply; 25+ messages in thread From: Jan Beulich @ 2015-12-21 12:53 UTC (permalink / raw) To: Doug Goldstein, Andrew Cooper Cc: Keir Fraser, Tim Deegan, Ian Jackson, Ian Campbell, xen-devel >>> On 21.12.15 at 13:40, <andrew.cooper3@citrix.com> wrote: > On 21/12/15 12:11, Jan Beulich wrote: >>>>> On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: >>> Since we now support changing Xen options with Kconfig, we should save >>> the configuration that was used to build up Xen. This will save it in >>> /boot alongside the installed xen.gz and call it >>> xen-$(FULLVERSION).config >>> >>> Suggested-by: Ian Campbell <ian.campbell@citrix.com> >>> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> >>> --- >>> xen/Makefile | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/xen/Makefile b/xen/Makefile >>> index 9023863..460b977 100644 >>> --- a/xen/Makefile >>> +++ b/xen/Makefile >>> @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) >>> ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) >>> [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) >>> $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) >>> + $(INSTALL_DATA) $(KCONFIG_CONFIG) > $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config >> Was it really suggested to put this into /boot? It has no business >> being there... > > A typical Linux has: > > andrewcoop@andrewcoop:/local/xen.git/xen$ ls -lA /boot/ > total 21044 > -rw-r--r-- 1 root root 157726 Dec 15 15:40 config-3.16.0-4-amd64 > drwxr-xr-x 5 root root 4096 Dec 18 07:53 grub > -rw-r--r-- 1 root root 15535038 Dec 18 07:52 initrd.img-3.16.0-4-amd64 > drwx------ 2 root root 16384 Oct 19 11:11 lost+found > -rw-r--r-- 1 root root 2676277 Dec 15 15:40 System.map-3.16.0-4-amd64 > -rw-r--r-- 1 root root 3118928 Dec 15 15:37 vmlinuz-3.16.0-4-amd64 > > which at the very least is consistent between Debian and RHEL derivatives. > > IMO, doing the same for Xen is sensible. I'm afraid I have to disagree - just because Linux does things a certain way doesn't mean that the only (sensible) way. Imo /boot should hold exclusively stuff needed for booting. Remember how we moved xen-syms out of there not so long ago? You could have objected to that change too, considering that Linux puts System.map and sometimes also the uncompressed vmlinux there. Yet I think it was a correct move, and the change here should follow that model instead of Linux'es. Jan ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-21 12:53 ` Jan Beulich @ 2015-12-22 12:45 ` Doug Goldstein 2015-12-22 12:50 ` Jan Beulich 0 siblings, 1 reply; 25+ messages in thread From: Doug Goldstein @ 2015-12-22 12:45 UTC (permalink / raw) To: Jan Beulich, Andrew Cooper Cc: Keir Fraser, Tim Deegan, Ian Jackson, Ian Campbell, xen-devel [-- Attachment #1.1: Type: text/plain, Size: 2699 bytes --] On 12/21/15 6:53 AM, Jan Beulich wrote: >>>> On 21.12.15 at 13:40, <andrew.cooper3@citrix.com> wrote: >> On 21/12/15 12:11, Jan Beulich wrote: >>>>>> On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: >>>> Since we now support changing Xen options with Kconfig, we should save >>>> the configuration that was used to build up Xen. This will save it in >>>> /boot alongside the installed xen.gz and call it >>>> xen-$(FULLVERSION).config >>>> >>>> Suggested-by: Ian Campbell <ian.campbell@citrix.com> >>>> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> >>>> --- >>>> xen/Makefile | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/xen/Makefile b/xen/Makefile >>>> index 9023863..460b977 100644 >>>> --- a/xen/Makefile >>>> +++ b/xen/Makefile >>>> @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) >>>> ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) >>>> [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) >>>> $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) >>>> + $(INSTALL_DATA) $(KCONFIG_CONFIG) >> $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config >>> Was it really suggested to put this into /boot? It has no business >>> being there... >> >> A typical Linux has: >> >> andrewcoop@andrewcoop:/local/xen.git/xen$ ls -lA /boot/ >> total 21044 >> -rw-r--r-- 1 root root 157726 Dec 15 15:40 config-3.16.0-4-amd64 >> drwxr-xr-x 5 root root 4096 Dec 18 07:53 grub >> -rw-r--r-- 1 root root 15535038 Dec 18 07:52 initrd.img-3.16.0-4-amd64 >> drwx------ 2 root root 16384 Oct 19 11:11 lost+found >> -rw-r--r-- 1 root root 2676277 Dec 15 15:40 System.map-3.16.0-4-amd64 >> -rw-r--r-- 1 root root 3118928 Dec 15 15:37 vmlinuz-3.16.0-4-amd64 >> >> which at the very least is consistent between Debian and RHEL derivatives. >> >> IMO, doing the same for Xen is sensible. > > I'm afraid I have to disagree - just because Linux does things a > certain way doesn't mean that the only (sensible) way. Imo /boot > should hold exclusively stuff needed for booting. Remember how > we moved xen-syms out of there not so long ago? You could have > objected to that change too, considering that Linux puts > System.map and sometimes also the uncompressed vmlinux there. > Yet I think it was a correct move, and the change here should > follow that model instead of Linux'es. > > Jan > I don't think you're correct here Jan. Looking at most distros I have at hand (I'll admit I don't have SuSE available to me). The uncompressed kernel is not in /boot. In fact all of them use /usr/lib/debug so Xen did follow Linux here. -- Doug Goldstein [-- Attachment #1.2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 959 bytes --] [-- Attachment #2: Type: text/plain, Size: 126 bytes --] _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-22 12:45 ` Doug Goldstein @ 2015-12-22 12:50 ` Jan Beulich 0 siblings, 0 replies; 25+ messages in thread From: Jan Beulich @ 2015-12-22 12:50 UTC (permalink / raw) To: Doug Goldstein Cc: Keir Fraser, Ian Campbell, Andrew Cooper, Tim Deegan, xen-devel, Ian Jackson >>> On 22.12.15 at 13:45, <cardoe@cardoe.com> wrote: > On 12/21/15 6:53 AM, Jan Beulich wrote: >>>>> On 21.12.15 at 13:40, <andrew.cooper3@citrix.com> wrote: >>> On 21/12/15 12:11, Jan Beulich wrote: >>>>>>> On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: >>>>> Since we now support changing Xen options with Kconfig, we should save >>>>> the configuration that was used to build up Xen. This will save it in >>>>> /boot alongside the installed xen.gz and call it >>>>> xen-$(FULLVERSION).config >>>>> >>>>> Suggested-by: Ian Campbell <ian.campbell@citrix.com> >>>>> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> >>>>> --- >>>>> xen/Makefile | 1 + >>>>> 1 file changed, 1 insertion(+) >>>>> >>>>> diff --git a/xen/Makefile b/xen/Makefile >>>>> index 9023863..460b977 100644 >>>>> --- a/xen/Makefile >>>>> +++ b/xen/Makefile >>>>> @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) >>>>> ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) >>>>> [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) >>>>> $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) >>>>> + $(INSTALL_DATA) $(KCONFIG_CONFIG) >>> $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config >>>> Was it really suggested to put this into /boot? It has no business >>>> being there... >>> >>> A typical Linux has: >>> >>> andrewcoop@andrewcoop:/local/xen.git/xen$ ls -lA /boot/ >>> total 21044 >>> -rw-r--r-- 1 root root 157726 Dec 15 15:40 config-3.16.0-4-amd64 >>> drwxr-xr-x 5 root root 4096 Dec 18 07:53 grub >>> -rw-r--r-- 1 root root 15535038 Dec 18 07:52 initrd.img-3.16.0-4-amd64 >>> drwx------ 2 root root 16384 Oct 19 11:11 lost+found >>> -rw-r--r-- 1 root root 2676277 Dec 15 15:40 System.map-3.16.0-4-amd64 >>> -rw-r--r-- 1 root root 3118928 Dec 15 15:37 vmlinuz-3.16.0-4-amd64 >>> >>> which at the very least is consistent between Debian and RHEL derivatives. >>> >>> IMO, doing the same for Xen is sensible. >> >> I'm afraid I have to disagree - just because Linux does things a >> certain way doesn't mean that the only (sensible) way. Imo /boot >> should hold exclusively stuff needed for booting. Remember how >> we moved xen-syms out of there not so long ago? You could have >> objected to that change too, considering that Linux puts >> System.map and sometimes also the uncompressed vmlinux there. >> Yet I think it was a correct move, and the change here should >> follow that model instead of Linux'es. > > I don't think you're correct here Jan. Looking at most distros I have at > hand (I'll admit I don't have SuSE available to me). The uncompressed > kernel is not in /boot. Note how I said "sometimes"? And I suppose you will agree that System.map is unnecessary for booting, despite it getting put in /boot quite commonly as it looks. Jan ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-21 12:11 ` Jan Beulich 2015-12-21 12:40 ` Andrew Cooper @ 2015-12-21 14:35 ` Doug Goldstein 2015-12-21 14:51 ` Jan Beulich 2016-01-04 11:27 ` Ian Campbell 1 sibling, 2 replies; 25+ messages in thread From: Doug Goldstein @ 2015-12-21 14:35 UTC (permalink / raw) To: Jan Beulich; +Cc: Keir Fraser, Tim Deegan, Ian Jackson, Ian Campbell, xen-devel [-- Attachment #1.1: Type: text/plain, Size: 1242 bytes --] On 12/21/15 6:11 AM, Jan Beulich wrote: >>>> On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: >> Since we now support changing Xen options with Kconfig, we should save >> the configuration that was used to build up Xen. This will save it in >> /boot alongside the installed xen.gz and call it >> xen-$(FULLVERSION).config >> >> Suggested-by: Ian Campbell <ian.campbell@citrix.com> >> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> >> --- >> xen/Makefile | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/xen/Makefile b/xen/Makefile >> index 9023863..460b977 100644 >> --- a/xen/Makefile >> +++ b/xen/Makefile >> @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) >> ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) >> [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) >> $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) >> + $(INSTALL_DATA) $(KCONFIG_CONFIG) $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config > > Was it really suggested to put this into /boot? It has no business > being there... > > Jan > Yes. By multiple people. Ian Campbell was the first person to suggest it in that location. -- Doug Goldstein [-- Attachment #1.2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 959 bytes --] [-- Attachment #2: Type: text/plain, Size: 126 bytes --] _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-21 14:35 ` Doug Goldstein @ 2015-12-21 14:51 ` Jan Beulich 2015-12-21 15:20 ` Doug Goldstein 2016-01-04 11:27 ` Ian Campbell 1 sibling, 1 reply; 25+ messages in thread From: Jan Beulich @ 2015-12-21 14:51 UTC (permalink / raw) To: Doug Goldstein Cc: Keir Fraser, Tim Deegan, Ian Jackson, Ian Campbell, xen-devel >>> On 21.12.15 at 15:35, <cardoe@cardoe.com> wrote: > On 12/21/15 6:11 AM, Jan Beulich wrote: >>>>> On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: >>> Since we now support changing Xen options with Kconfig, we should save >>> the configuration that was used to build up Xen. This will save it in >>> /boot alongside the installed xen.gz and call it >>> xen-$(FULLVERSION).config >>> >>> Suggested-by: Ian Campbell <ian.campbell@citrix.com> >>> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> >>> --- >>> xen/Makefile | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/xen/Makefile b/xen/Makefile >>> index 9023863..460b977 100644 >>> --- a/xen/Makefile >>> +++ b/xen/Makefile >>> @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) >>> ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) >>> [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) >>> $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) >>> + $(INSTALL_DATA) $(KCONFIG_CONFIG) > $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config >> >> Was it really suggested to put this into /boot? It has no business >> being there... > > Yes. By multiple people. Ian Campbell was the first person to suggest it > in that location. Okay, so I've looked it up, and no, he didn't. He just gave this as one possibility: "It occurred to me this morning that we probably ought to stash the .config somewhere on install in such a way that it can be associated with the Xen binary (i.e. with the same full suffix as the binary itself, not the abridged symlink names), maybe as $(BOOT_DIR)/$(T)- $(XEN_FULLVERSION).config?" But yes, I'm sorry for not noticing this as an undesirable place right away. Jan ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-21 14:51 ` Jan Beulich @ 2015-12-21 15:20 ` Doug Goldstein 2015-12-21 15:35 ` Jan Beulich 0 siblings, 1 reply; 25+ messages in thread From: Doug Goldstein @ 2015-12-21 15:20 UTC (permalink / raw) To: Jan Beulich; +Cc: Keir Fraser, Tim Deegan, Ian Jackson, Ian Campbell, xen-devel [-- Attachment #1.1: Type: text/plain, Size: 2111 bytes --] On 12/21/15 8:51 AM, Jan Beulich wrote: >>>> On 21.12.15 at 15:35, <cardoe@cardoe.com> wrote: >> On 12/21/15 6:11 AM, Jan Beulich wrote: >>>>>> On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: >>>> Since we now support changing Xen options with Kconfig, we should save >>>> the configuration that was used to build up Xen. This will save it in >>>> /boot alongside the installed xen.gz and call it >>>> xen-$(FULLVERSION).config >>>> >>>> Suggested-by: Ian Campbell <ian.campbell@citrix.com> >>>> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> >>>> --- >>>> xen/Makefile | 1 + >>>> 1 file changed, 1 insertion(+) >>>> >>>> diff --git a/xen/Makefile b/xen/Makefile >>>> index 9023863..460b977 100644 >>>> --- a/xen/Makefile >>>> +++ b/xen/Makefile >>>> @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) >>>> ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) >>>> [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) >>>> $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) >>>> + $(INSTALL_DATA) $(KCONFIG_CONFIG) >> $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config >>> >>> Was it really suggested to put this into /boot? It has no business >>> being there... >> >> Yes. By multiple people. Ian Campbell was the first person to suggest it >> in that location. > > Okay, so I've looked it up, and no, he didn't. He just gave this as one > possibility: > > "It occurred to me this morning that we probably ought to stash the .config > somewhere on install in such a way that it can be associated with the Xen > binary (i.e. with the same full suffix as the binary itself, not the > abridged symlink names), maybe as $(BOOT_DIR)/$(T)- > $(XEN_FULLVERSION).config?" > > But yes, I'm sorry for not noticing this as an undesirable place right > away. > > Jan > > Ok well I'm at a loss here because the quote clearly shows him suggesting that location. Do you have a suggested location because so far I've just got a no from you on the only suggested location. -- Doug Goldstein [-- Attachment #1.2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 959 bytes --] [-- Attachment #2: Type: text/plain, Size: 126 bytes --] _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-21 15:20 ` Doug Goldstein @ 2015-12-21 15:35 ` Jan Beulich 2015-12-22 12:45 ` Doug Goldstein 0 siblings, 1 reply; 25+ messages in thread From: Jan Beulich @ 2015-12-21 15:35 UTC (permalink / raw) To: Doug Goldstein Cc: Keir Fraser, Tim Deegan, Ian Jackson, Ian Campbell, xen-devel >>> On 21.12.15 at 16:20, <cardoe@cardoe.com> wrote: > On 12/21/15 8:51 AM, Jan Beulich wrote: >>>>> On 21.12.15 at 15:35, <cardoe@cardoe.com> wrote: >>> On 12/21/15 6:11 AM, Jan Beulich wrote: >>>>>>> On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: >>>>> Since we now support changing Xen options with Kconfig, we should save >>>>> the configuration that was used to build up Xen. This will save it in >>>>> /boot alongside the installed xen.gz and call it >>>>> xen-$(FULLVERSION).config >>>>> >>>>> Suggested-by: Ian Campbell <ian.campbell@citrix.com> >>>>> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> >>>>> --- >>>>> xen/Makefile | 1 + >>>>> 1 file changed, 1 insertion(+) >>>>> >>>>> diff --git a/xen/Makefile b/xen/Makefile >>>>> index 9023863..460b977 100644 >>>>> --- a/xen/Makefile >>>>> +++ b/xen/Makefile >>>>> @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) >>>>> ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) >>>>> [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) >>>>> $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) >>>>> + $(INSTALL_DATA) $(KCONFIG_CONFIG) >>> $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config >>>> >>>> Was it really suggested to put this into /boot? It has no business >>>> being there... >>> >>> Yes. By multiple people. Ian Campbell was the first person to suggest it >>> in that location. >> >> Okay, so I've looked it up, and no, he didn't. He just gave this as one >> possibility: >> >> "It occurred to me this morning that we probably ought to stash the .config >> somewhere on install in such a way that it can be associated with the Xen >> binary (i.e. with the same full suffix as the binary itself, not the >> abridged symlink names), maybe as $(BOOT_DIR)/$(T)- >> $(XEN_FULLVERSION).config?" >> >> But yes, I'm sorry for not noticing this as an undesirable place right >> away. > > Ok well I'm at a loss here because the quote clearly shows him > suggesting that location. Do you have a suggested location because so > far I've just got a no from you on the only suggested location. Match the xen-syms location? Jan ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-21 15:35 ` Jan Beulich @ 2015-12-22 12:45 ` Doug Goldstein 2015-12-22 12:52 ` Jan Beulich 0 siblings, 1 reply; 25+ messages in thread From: Doug Goldstein @ 2015-12-22 12:45 UTC (permalink / raw) To: Jan Beulich; +Cc: Keir Fraser, Tim Deegan, Ian Jackson, Ian Campbell, xen-devel [-- Attachment #1.1: Type: text/plain, Size: 2407 bytes --] On 12/21/15 9:35 AM, Jan Beulich wrote: >>>> On 21.12.15 at 16:20, <cardoe@cardoe.com> wrote: >> On 12/21/15 8:51 AM, Jan Beulich wrote: >>>>>> On 21.12.15 at 15:35, <cardoe@cardoe.com> wrote: >>>> On 12/21/15 6:11 AM, Jan Beulich wrote: >>>>>>>> On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: >>>>>> Since we now support changing Xen options with Kconfig, we should save >>>>>> the configuration that was used to build up Xen. This will save it in >>>>>> /boot alongside the installed xen.gz and call it >>>>>> xen-$(FULLVERSION).config >>>>>> >>>>>> Suggested-by: Ian Campbell <ian.campbell@citrix.com> >>>>>> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> >>>>>> --- >>>>>> xen/Makefile | 1 + >>>>>> 1 file changed, 1 insertion(+) >>>>>> >>>>>> diff --git a/xen/Makefile b/xen/Makefile >>>>>> index 9023863..460b977 100644 >>>>>> --- a/xen/Makefile >>>>>> +++ b/xen/Makefile >>>>>> @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) >>>>>> ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) >>>>>> [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) >>>>>> $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) >>>>>> + $(INSTALL_DATA) $(KCONFIG_CONFIG) >>>> $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config >>>>> >>>>> Was it really suggested to put this into /boot? It has no business >>>>> being there... >>>> >>>> Yes. By multiple people. Ian Campbell was the first person to suggest it >>>> in that location. >>> >>> Okay, so I've looked it up, and no, he didn't. He just gave this as one >>> possibility: >>> >>> "It occurred to me this morning that we probably ought to stash the .config >>> somewhere on install in such a way that it can be associated with the Xen >>> binary (i.e. with the same full suffix as the binary itself, not the >>> abridged symlink names), maybe as $(BOOT_DIR)/$(T)- >>> $(XEN_FULLVERSION).config?" >>> >>> But yes, I'm sorry for not noticing this as an undesirable place right >>> away. >> >> Ok well I'm at a loss here because the quote clearly shows him >> suggesting that location. Do you have a suggested location because so >> far I've just got a no from you on the only suggested location. > > Match the xen-syms location? > > Jan > I guess I fail to grasp the rationale behind not putting it in /boot. -- Doug Goldstein [-- Attachment #1.2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 959 bytes --] [-- Attachment #2: Type: text/plain, Size: 126 bytes --] _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-22 12:45 ` Doug Goldstein @ 2015-12-22 12:52 ` Jan Beulich 2015-12-22 14:46 ` Andrew Cooper 0 siblings, 1 reply; 25+ messages in thread From: Jan Beulich @ 2015-12-22 12:52 UTC (permalink / raw) To: Doug Goldstein Cc: Keir Fraser, Tim Deegan, Ian Jackson, Ian Campbell, xen-devel >>> On 22.12.15 at 13:45, <cardoe@cardoe.com> wrote: > On 12/21/15 9:35 AM, Jan Beulich wrote: >>>>> On 21.12.15 at 16:20, <cardoe@cardoe.com> wrote: >>> On 12/21/15 8:51 AM, Jan Beulich wrote: >>>>>>> On 21.12.15 at 15:35, <cardoe@cardoe.com> wrote: >>>>> On 12/21/15 6:11 AM, Jan Beulich wrote: >>>>>>>>> On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: >>>>>>> Since we now support changing Xen options with Kconfig, we should save >>>>>>> the configuration that was used to build up Xen. This will save it in >>>>>>> /boot alongside the installed xen.gz and call it >>>>>>> xen-$(FULLVERSION).config >>>>>>> >>>>>>> Suggested-by: Ian Campbell <ian.campbell@citrix.com> >>>>>>> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> >>>>>>> --- >>>>>>> xen/Makefile | 1 + >>>>>>> 1 file changed, 1 insertion(+) >>>>>>> >>>>>>> diff --git a/xen/Makefile b/xen/Makefile >>>>>>> index 9023863..460b977 100644 >>>>>>> --- a/xen/Makefile >>>>>>> +++ b/xen/Makefile >>>>>>> @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) >>>>>>> ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) >>>>>>> [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) >>>>>>> $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) >>>>>>> + $(INSTALL_DATA) $(KCONFIG_CONFIG) >>>>> $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config >>>>>> >>>>>> Was it really suggested to put this into /boot? It has no business >>>>>> being there... >>>>> >>>>> Yes. By multiple people. Ian Campbell was the first person to suggest it >>>>> in that location. >>>> >>>> Okay, so I've looked it up, and no, he didn't. He just gave this as one >>>> possibility: >>>> >>>> "It occurred to me this morning that we probably ought to stash the .config >>>> somewhere on install in such a way that it can be associated with the Xen >>>> binary (i.e. with the same full suffix as the binary itself, not the >>>> abridged symlink names), maybe as $(BOOT_DIR)/$(T)- >>>> $(XEN_FULLVERSION).config?" >>>> >>>> But yes, I'm sorry for not noticing this as an undesirable place right >>>> away. >>> >>> Ok well I'm at a loss here because the quote clearly shows him >>> suggesting that location. Do you have a suggested location because so >>> far I've just got a no from you on the only suggested location. >> >> Match the xen-syms location? > > I guess I fail to grasp the rationale behind not putting it in /boot. It's the other way around really - you'd have to provide a reason (other than "Linux does so too") for putting it in /boot. Jan ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-22 12:52 ` Jan Beulich @ 2015-12-22 14:46 ` Andrew Cooper 2015-12-22 15:59 ` Jan Beulich 0 siblings, 1 reply; 25+ messages in thread From: Andrew Cooper @ 2015-12-22 14:46 UTC (permalink / raw) To: Jan Beulich, Doug Goldstein Cc: Tim Deegan, xen-devel, Keir Fraser, Ian Jackson, Ian Campbell On 22/12/15 12:52, Jan Beulich wrote: >>>> On 22.12.15 at 13:45, <cardoe@cardoe.com> wrote: >> On 12/21/15 9:35 AM, Jan Beulich wrote: >>>>>> On 21.12.15 at 16:20, <cardoe@cardoe.com> wrote: >>>> On 12/21/15 8:51 AM, Jan Beulich wrote: >>>>>>>> On 21.12.15 at 15:35, <cardoe@cardoe.com> wrote: >>>>>> On 12/21/15 6:11 AM, Jan Beulich wrote: >>>>>>>>>> On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: >>>>>>>> Since we now support changing Xen options with Kconfig, we should save >>>>>>>> the configuration that was used to build up Xen. This will save it in >>>>>>>> /boot alongside the installed xen.gz and call it >>>>>>>> xen-$(FULLVERSION).config >>>>>>>> >>>>>>>> Suggested-by: Ian Campbell <ian.campbell@citrix.com> >>>>>>>> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> >>>>>>>> --- >>>>>>>> xen/Makefile | 1 + >>>>>>>> 1 file changed, 1 insertion(+) >>>>>>>> >>>>>>>> diff --git a/xen/Makefile b/xen/Makefile >>>>>>>> index 9023863..460b977 100644 >>>>>>>> --- a/xen/Makefile >>>>>>>> +++ b/xen/Makefile >>>>>>>> @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) >>>>>>>> ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) >>>>>>>> [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) >>>>>>>> $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) >>>>>>>> + $(INSTALL_DATA) $(KCONFIG_CONFIG) >>>>>> $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config >>>>>>> Was it really suggested to put this into /boot? It has no business >>>>>>> being there... >>>>>> Yes. By multiple people. Ian Campbell was the first person to suggest it >>>>>> in that location. >>>>> Okay, so I've looked it up, and no, he didn't. He just gave this as one >>>>> possibility: >>>>> >>>>> "It occurred to me this morning that we probably ought to stash the .config >>>>> somewhere on install in such a way that it can be associated with the Xen >>>>> binary (i.e. with the same full suffix as the binary itself, not the >>>>> abridged symlink names), maybe as $(BOOT_DIR)/$(T)- >>>>> $(XEN_FULLVERSION).config?" >>>>> >>>>> But yes, I'm sorry for not noticing this as an undesirable place right >>>>> away. >>>> Ok well I'm at a loss here because the quote clearly shows him >>>> suggesting that location. Do you have a suggested location because so >>>> far I've just got a no from you on the only suggested location. >>> Match the xen-syms location? >> I guess I fail to grasp the rationale behind not putting it in /boot. > It's the other way around really - you'd have to provide a reason > (other than "Linux does so too") for putting it in /boot. I disagree. Xen being consistent with Linux in this regard is in the best interest of the users of Xen, as they end up finding similar information in similar places. The onus is on you to provide a reason why we should deliberately do something different. ~Andrew ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-22 14:46 ` Andrew Cooper @ 2015-12-22 15:59 ` Jan Beulich 2015-12-22 16:02 ` Doug Goldstein 0 siblings, 1 reply; 25+ messages in thread From: Jan Beulich @ 2015-12-22 15:59 UTC (permalink / raw) To: Andrew Cooper Cc: Keir Fraser, Ian Campbell, Tim Deegan, Ian Jackson, xen-devel, Doug Goldstein >>> On 22.12.15 at 15:46, <andrew.cooper3@citrix.com> wrote: > On 22/12/15 12:52, Jan Beulich wrote: >>>>> On 22.12.15 at 13:45, <cardoe@cardoe.com> wrote: >>> On 12/21/15 9:35 AM, Jan Beulich wrote: >>>>>>> On 21.12.15 at 16:20, <cardoe@cardoe.com> wrote: >>>>> On 12/21/15 8:51 AM, Jan Beulich wrote: >>>>>>>>> On 21.12.15 at 15:35, <cardoe@cardoe.com> wrote: >>>>>>> On 12/21/15 6:11 AM, Jan Beulich wrote: >>>>>>>>>>> On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: >>>>>>>>> Since we now support changing Xen options with Kconfig, we should save >>>>>>>>> the configuration that was used to build up Xen. This will save it in >>>>>>>>> /boot alongside the installed xen.gz and call it >>>>>>>>> xen-$(FULLVERSION).config >>>>>>>>> >>>>>>>>> Suggested-by: Ian Campbell <ian.campbell@citrix.com> >>>>>>>>> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> >>>>>>>>> --- >>>>>>>>> xen/Makefile | 1 + >>>>>>>>> 1 file changed, 1 insertion(+) >>>>>>>>> >>>>>>>>> diff --git a/xen/Makefile b/xen/Makefile >>>>>>>>> index 9023863..460b977 100644 >>>>>>>>> --- a/xen/Makefile >>>>>>>>> +++ b/xen/Makefile >>>>>>>>> @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) >>>>>>>>> ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) >>>>>>>>> [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) >>>>>>>>> $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) >>>>>>>>> + $(INSTALL_DATA) $(KCONFIG_CONFIG) >>>>>>> $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config >>>>>>>> Was it really suggested to put this into /boot? It has no business >>>>>>>> being there... >>>>>>> Yes. By multiple people. Ian Campbell was the first person to suggest it >>>>>>> in that location. >>>>>> Okay, so I've looked it up, and no, he didn't. He just gave this as one >>>>>> possibility: >>>>>> >>>>>> "It occurred to me this morning that we probably ought to stash the .config >>>>>> somewhere on install in such a way that it can be associated with the Xen >>>>>> binary (i.e. with the same full suffix as the binary itself, not the >>>>>> abridged symlink names), maybe as $(BOOT_DIR)/$(T)- >>>>>> $(XEN_FULLVERSION).config?" >>>>>> >>>>>> But yes, I'm sorry for not noticing this as an undesirable place right >>>>>> away. >>>>> Ok well I'm at a loss here because the quote clearly shows him >>>>> suggesting that location. Do you have a suggested location because so >>>>> far I've just got a no from you on the only suggested location. >>>> Match the xen-syms location? >>> I guess I fail to grasp the rationale behind not putting it in /boot. >> It's the other way around really - you'd have to provide a reason >> (other than "Linux does so too") for putting it in /boot. > > I disagree. > > Xen being consistent with Linux in this regard is in the best interest > of the users of Xen, as they end up finding similar information in > similar places. > > The onus is on you to provide a reason why we should deliberately do > something different. I'm sorry, but no - why would we slavishly follow what Linux does, no matter whether it makes sense? Jan ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-22 15:59 ` Jan Beulich @ 2015-12-22 16:02 ` Doug Goldstein 2015-12-22 16:09 ` Jan Beulich 0 siblings, 1 reply; 25+ messages in thread From: Doug Goldstein @ 2015-12-22 16:02 UTC (permalink / raw) To: Jan Beulich, Andrew Cooper Cc: Keir Fraser, Tim Deegan, Ian Jackson, Ian Campbell, xen-devel [-- Attachment #1.1: Type: text/plain, Size: 3406 bytes --] On 12/22/15 9:59 AM, Jan Beulich wrote: >>>> On 22.12.15 at 15:46, <andrew.cooper3@citrix.com> wrote: >> On 22/12/15 12:52, Jan Beulich wrote: >>>>>> On 22.12.15 at 13:45, <cardoe@cardoe.com> wrote: >>>> On 12/21/15 9:35 AM, Jan Beulich wrote: >>>>>>>> On 21.12.15 at 16:20, <cardoe@cardoe.com> wrote: >>>>>> On 12/21/15 8:51 AM, Jan Beulich wrote: >>>>>>>>>> On 21.12.15 at 15:35, <cardoe@cardoe.com> wrote: >>>>>>>> On 12/21/15 6:11 AM, Jan Beulich wrote: >>>>>>>>>>>> On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: >>>>>>>>>> Since we now support changing Xen options with Kconfig, we should save >>>>>>>>>> the configuration that was used to build up Xen. This will save it in >>>>>>>>>> /boot alongside the installed xen.gz and call it >>>>>>>>>> xen-$(FULLVERSION).config >>>>>>>>>> >>>>>>>>>> Suggested-by: Ian Campbell <ian.campbell@citrix.com> >>>>>>>>>> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> >>>>>>>>>> --- >>>>>>>>>> xen/Makefile | 1 + >>>>>>>>>> 1 file changed, 1 insertion(+) >>>>>>>>>> >>>>>>>>>> diff --git a/xen/Makefile b/xen/Makefile >>>>>>>>>> index 9023863..460b977 100644 >>>>>>>>>> --- a/xen/Makefile >>>>>>>>>> +++ b/xen/Makefile >>>>>>>>>> @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) >>>>>>>>>> ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) $(D)$(BOOT_DIR)/$(T)$(Z) >>>>>>>>>> [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) >>>>>>>>>> $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms-$(XEN_FULLVERSION) >>>>>>>>>> + $(INSTALL_DATA) $(KCONFIG_CONFIG) >>>>>>>> $(D)$(BOOT_DIR)/$(T)-$(XEN_FULLVERSION).config >>>>>>>>> Was it really suggested to put this into /boot? It has no business >>>>>>>>> being there... >>>>>>>> Yes. By multiple people. Ian Campbell was the first person to suggest it >>>>>>>> in that location. >>>>>>> Okay, so I've looked it up, and no, he didn't. He just gave this as one >>>>>>> possibility: >>>>>>> >>>>>>> "It occurred to me this morning that we probably ought to stash the .config >>>>>>> somewhere on install in such a way that it can be associated with the Xen >>>>>>> binary (i.e. with the same full suffix as the binary itself, not the >>>>>>> abridged symlink names), maybe as $(BOOT_DIR)/$(T)- >>>>>>> $(XEN_FULLVERSION).config?" >>>>>>> >>>>>>> But yes, I'm sorry for not noticing this as an undesirable place right >>>>>>> away. >>>>>> Ok well I'm at a loss here because the quote clearly shows him >>>>>> suggesting that location. Do you have a suggested location because so >>>>>> far I've just got a no from you on the only suggested location. >>>>> Match the xen-syms location? >>>> I guess I fail to grasp the rationale behind not putting it in /boot. >>> It's the other way around really - you'd have to provide a reason >>> (other than "Linux does so too") for putting it in /boot. >> >> I disagree. >> >> Xen being consistent with Linux in this regard is in the best interest >> of the users of Xen, as they end up finding similar information in >> similar places. >> >> The onus is on you to provide a reason why we should deliberately do >> something different. > > I'm sorry, but no - why would we slavishly follow what Linux does, no > matter whether it makes sense? > > Jan > How does it not make sense in this case? That's what Andrew and I are asking you to explain. -- Doug Goldstein [-- Attachment #1.2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 959 bytes --] [-- Attachment #2: Type: text/plain, Size: 126 bytes --] _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-22 16:02 ` Doug Goldstein @ 2015-12-22 16:09 ` Jan Beulich 2016-01-05 16:53 ` Doug Goldstein 0 siblings, 1 reply; 25+ messages in thread From: Jan Beulich @ 2015-12-22 16:09 UTC (permalink / raw) To: Doug Goldstein Cc: Keir Fraser, Ian Campbell, Andrew Cooper, Tim Deegan, xen-devel, Ian Jackson >>> On 22.12.15 at 17:02, <cardoe@cardoe.com> wrote: > How does it not make sense in this case? That's what Andrew and I are > asking you to explain. But I already explained it: The file isn't needed for booting. Jan ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-22 16:09 ` Jan Beulich @ 2016-01-05 16:53 ` Doug Goldstein 2016-01-06 9:22 ` Jan Beulich 0 siblings, 1 reply; 25+ messages in thread From: Doug Goldstein @ 2016-01-05 16:53 UTC (permalink / raw) To: Jan Beulich Cc: Keir Fraser, Ian Campbell, Andrew Cooper, Tim Deegan, xen-devel, Ian Jackson [-- Attachment #1.1: Type: text/plain, Size: 470 bytes --] On 12/22/15 10:09 AM, Jan Beulich wrote: >>>> On 22.12.15 at 17:02, <cardoe@cardoe.com> wrote: >> How does it not make sense in this case? That's what Andrew and I are >> asking you to explain. > > But I already explained it: The file isn't needed for booting. > > Jan > Jan, One use of it in /boot would be to check the file in 20_linux_xen like we check the kernel config for different options to add different parameters. -- Doug Goldstein [-- Attachment #1.2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 959 bytes --] [-- Attachment #2: Type: text/plain, Size: 126 bytes --] _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2016-01-05 16:53 ` Doug Goldstein @ 2016-01-06 9:22 ` Jan Beulich 2016-01-06 10:31 ` Ian Campbell 0 siblings, 1 reply; 25+ messages in thread From: Jan Beulich @ 2016-01-06 9:22 UTC (permalink / raw) To: Doug Goldstein Cc: Keir Fraser, Ian Campbell, Andrew Cooper, Tim Deegan, xen-devel, Ian Jackson >>> On 05.01.16 at 17:53, <cardoe@cardoe.com> wrote: > On 12/22/15 10:09 AM, Jan Beulich wrote: >>>>> On 22.12.15 at 17:02, <cardoe@cardoe.com> wrote: >>> How does it not make sense in this case? That's what Andrew and I are >>> asking you to explain. >> >> But I already explained it: The file isn't needed for booting. > > One use of it in /boot would be to check the file in 20_linux_xen like > we check the kernel config for different options to add different > parameters. Which, if you search for it, is a very controversial thing: I agree with the side demanding for grub to not inspect the kernel's .config. Jan ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2016-01-06 9:22 ` Jan Beulich @ 2016-01-06 10:31 ` Ian Campbell 2016-01-06 10:38 ` Jan Beulich 0 siblings, 1 reply; 25+ messages in thread From: Ian Campbell @ 2016-01-06 10:31 UTC (permalink / raw) To: Jan Beulich, Doug Goldstein Cc: Andrew Cooper, xen-devel, Keir Fraser, Ian Jackson, Tim Deegan On Wed, 2016-01-06 at 02:22 -0700, Jan Beulich wrote: > > > > On 05.01.16 at 17:53, <cardoe@cardoe.com> wrote: > > On 12/22/15 10:09 AM, Jan Beulich wrote: > > > > > > On 22.12.15 at 17:02, <cardoe@cardoe.com> wrote: > > > > How does it not make sense in this case? That's what Andrew and I > > > > are > > > > asking you to explain. > > > > > > But I already explained it: The file isn't needed for booting. > > > > One use of it in /boot would be to check the file in 20_linux_xen like > > we check the kernel config for different options to add different > > parameters. > > Which, if you search for it, is a very controversial thing: I agree > with the side demanding for grub to not inspect the kernel's > .config. What is the preferred mechanism for determining from the contents of /boot whether an XSM entry should be produced or not? Ian. ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2016-01-06 10:31 ` Ian Campbell @ 2016-01-06 10:38 ` Jan Beulich 2016-01-06 11:58 ` Ian Jackson 2016-01-06 12:17 ` Wei Liu 0 siblings, 2 replies; 25+ messages in thread From: Jan Beulich @ 2016-01-06 10:38 UTC (permalink / raw) To: Ian Campbell Cc: Keir Fraser, Andrew Cooper, Tim Deegan, xen-devel, Ian Jackson, Doug Goldstein >>> On 06.01.16 at 11:31, <ian.campbell@citrix.com> wrote: > On Wed, 2016-01-06 at 02:22 -0700, Jan Beulich wrote: >> > > > On 05.01.16 at 17:53, <cardoe@cardoe.com> wrote: >> > On 12/22/15 10:09 AM, Jan Beulich wrote: >> > > > > > On 22.12.15 at 17:02, <cardoe@cardoe.com> wrote: >> > > > How does it not make sense in this case? That's what Andrew and I >> > > > are >> > > > asking you to explain. >> > > >> > > But I already explained it: The file isn't needed for booting. >> > >> > One use of it in /boot would be to check the file in 20_linux_xen like >> > we check the kernel config for different options to add different >> > parameters. >> >> Which, if you search for it, is a very controversial thing: I agree >> with the side demanding for grub to not inspect the kernel's >> .config. > > What is the preferred mechanism for determining from the contents of /boot > whether an XSM entry should be produced or not? I don't think this can be inferred - iirc even an XSM-enabled build can be booted without unless in some kind of strict mode? IOW this needs to be configured on the host, not guessed by inspecting .config. (And as a side note, inspecting .config doesn't truly require .config to be on the boot partition, unless such inspection would happen at boot time, which I don't think is the case.) Jan ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2016-01-06 10:38 ` Jan Beulich @ 2016-01-06 11:58 ` Ian Jackson 2016-01-06 12:56 ` Jan Beulich 2016-01-06 12:17 ` Wei Liu 1 sibling, 1 reply; 25+ messages in thread From: Ian Jackson @ 2016-01-06 11:58 UTC (permalink / raw) To: Jan Beulich Cc: Keir Fraser, Ian Campbell, Andrew Cooper, Tim Deegan, xen-devel, Doug Goldstein Jan Beulich writes ("Re: [Xen-devel] [PATCH] build: save generated config in /boot"): > On 06.01.16 at 11:31, <ian.campbell@citrix.com> wrote: > > What is the preferred mechanism for determining from the contents of /boot > > whether an XSM entry should be produced or not? > > I don't think this can be inferred - iirc even an XSM-enabled build > can be booted without unless in some kind of strict mode? IOW > this needs to be configured on the host, not guessed by > inspecting .config. (And as a side note, inspecting .config doesn't > truly require .config to be on the boot partition, unless such > inspection would happen at boot time, which I don't think is the > case.) Well, actually, it _can_ be inferred. Therefore it should be. Ian. ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2016-01-06 11:58 ` Ian Jackson @ 2016-01-06 12:56 ` Jan Beulich 0 siblings, 0 replies; 25+ messages in thread From: Jan Beulich @ 2016-01-06 12:56 UTC (permalink / raw) To: Ian Jackson Cc: Keir Fraser, Ian Campbell, Andrew Cooper, Tim Deegan, xen-devel, Doug Goldstein >>> On 06.01.16 at 12:58, <Ian.Jackson@eu.citrix.com> wrote: > Jan Beulich writes ("Re: [Xen-devel] [PATCH] build: save generated config in > /boot"): >> On 06.01.16 at 11:31, <ian.campbell@citrix.com> wrote: >> > What is the preferred mechanism for determining from the contents of /boot >> > whether an XSM entry should be produced or not? >> >> I don't think this can be inferred - iirc even an XSM-enabled build >> can be booted without unless in some kind of strict mode? IOW >> this needs to be configured on the host, not guessed by >> inspecting .config. (And as a side note, inspecting .config doesn't >> truly require .config to be on the boot partition, unless such >> inspection would happen at boot time, which I don't think is the >> case.) > > Well, actually, it _can_ be inferred. Therefore it should be. You mean from the presence of the policy file (which ends up being a container of garbage for a non-XSM-Xen)? If not - what else do you think the need for an XSM entry can be derived from? Jan ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2016-01-06 10:38 ` Jan Beulich 2016-01-06 11:58 ` Ian Jackson @ 2016-01-06 12:17 ` Wei Liu 2016-01-06 13:01 ` Jan Beulich 1 sibling, 1 reply; 25+ messages in thread From: Wei Liu @ 2016-01-06 12:17 UTC (permalink / raw) To: Jan Beulich Cc: Tim Deegan, Keir Fraser, Ian Campbell, Andrew Cooper, Ian Jackson, xen-devel, wei.liu2, Doug Goldstein On Wed, Jan 06, 2016 at 03:38:36AM -0700, Jan Beulich wrote: > >>> On 06.01.16 at 11:31, <ian.campbell@citrix.com> wrote: > > On Wed, 2016-01-06 at 02:22 -0700, Jan Beulich wrote: > >> > > > On 05.01.16 at 17:53, <cardoe@cardoe.com> wrote: > >> > On 12/22/15 10:09 AM, Jan Beulich wrote: > >> > > > > > On 22.12.15 at 17:02, <cardoe@cardoe.com> wrote: > >> > > > How does it not make sense in this case? That's what Andrew and I > >> > > > are > >> > > > asking you to explain. > >> > > > >> > > But I already explained it: The file isn't needed for booting. > >> > > >> > One use of it in /boot would be to check the file in 20_linux_xen like > >> > we check the kernel config for different options to add different > >> > parameters. > >> > >> Which, if you search for it, is a very controversial thing: I agree > >> with the side demanding for grub to not inspect the kernel's > >> .config. > > > > What is the preferred mechanism for determining from the contents of /boot > > whether an XSM entry should be produced or not? > > I don't think this can be inferred - iirc even an XSM-enabled build > can be booted without unless in some kind of strict mode? IOW > this needs to be configured on the host, not guessed by > inspecting .config. (And as a side note, inspecting .config doesn't It's better to lower the hurdle for XSM adoption by instructing grub to do sensible default than requiring users to produce entries by hand, isn't it? > truly require .config to be on the boot partition, unless such > inspection would happen at boot time, which I don't think is the > case.) > Yes and no. It's true that it can be placed anywhere on the file system as long as grub (or any other tools that need to inspect it) can find it. But how would you propose grub to deal with this? The location needs to be "fixed" from then on and present forever. Either we need to persuade every distro to accept our new less-well-known location to always place .config there or we make them patch grub as they package Xen. This is just unnecessary work. Anywhere other than /boot is inferior IMHO. Wei. > Jan > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xen.org > http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2016-01-06 12:17 ` Wei Liu @ 2016-01-06 13:01 ` Jan Beulich 0 siblings, 0 replies; 25+ messages in thread From: Jan Beulich @ 2016-01-06 13:01 UTC (permalink / raw) To: wei.liu2 Cc: Tim Deegan, Keir Fraser, Ian Campbell, Andrew Cooper, Ian Jackson, xen-devel, DougGoldstein >>> On 06.01.16 at 13:17, <wei.liu2@citrix.com> wrote: > On Wed, Jan 06, 2016 at 03:38:36AM -0700, Jan Beulich wrote: >> >>> On 06.01.16 at 11:31, <ian.campbell@citrix.com> wrote: >> > On Wed, 2016-01-06 at 02:22 -0700, Jan Beulich wrote: >> >> > > > On 05.01.16 at 17:53, <cardoe@cardoe.com> wrote: >> >> > On 12/22/15 10:09 AM, Jan Beulich wrote: >> >> > > > > > On 22.12.15 at 17:02, <cardoe@cardoe.com> wrote: >> >> > > > How does it not make sense in this case? That's what Andrew and I >> >> > > > are >> >> > > > asking you to explain. >> >> > > >> >> > > But I already explained it: The file isn't needed for booting. >> >> > >> >> > One use of it in /boot would be to check the file in 20_linux_xen like >> >> > we check the kernel config for different options to add different >> >> > parameters. >> >> >> >> Which, if you search for it, is a very controversial thing: I agree >> >> with the side demanding for grub to not inspect the kernel's >> >> .config. >> > >> > What is the preferred mechanism for determining from the contents of /boot >> > whether an XSM entry should be produced or not? >> >> I don't think this can be inferred - iirc even an XSM-enabled build >> can be booted without unless in some kind of strict mode? IOW >> this needs to be configured on the host, not guessed by >> inspecting .config. (And as a side note, inspecting .config doesn't > > It's better to lower the hurdle for XSM adoption by instructing grub to > do sensible default than requiring users to produce entries by hand, > isn't it? > >> truly require .config to be on the boot partition, unless such >> inspection would happen at boot time, which I don't think is the >> case.) >> > > Yes and no. > > It's true that it can be placed anywhere on the file system as long as > grub (or any other tools that need to inspect it) can find it. > > But how would you propose grub to deal with this? The location needs to > be "fixed" from then on and present forever. Either we need to persuade > every distro to accept our new less-well-known location to always place > .config there or we make them patch grub as they package Xen. This is > just unnecessary work. Anywhere other than /boot is inferior IMHO. I think we need to separate the two issues: First we need to settle on whether inspecting .config for XSM purposes is (a) necessary and if so (b) the right approach (which I doubt). See also my other reply to Ian sent a minute or two ago. Only then do we need to get concerned about whether .config really has a use when placed on /boot. Jan ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH] build: save generated config in /boot 2015-12-21 14:35 ` Doug Goldstein 2015-12-21 14:51 ` Jan Beulich @ 2016-01-04 11:27 ` Ian Campbell 1 sibling, 0 replies; 25+ messages in thread From: Ian Campbell @ 2016-01-04 11:27 UTC (permalink / raw) To: Doug Goldstein, Jan Beulich Cc: Keir Fraser, Tim Deegan, Ian Jackson, xen-devel On Mon, 2015-12-21 at 08:35 -0600, Doug Goldstein wrote: > On 12/21/15 6:11 AM, Jan Beulich wrote: > > > > > On 18.12.15 at 22:35, <cardoe@cardoe.com> wrote: > > > Since we now support changing Xen options with Kconfig, we should > > > save > > > the configuration that was used to build up Xen. This will save it in > > > /boot alongside the installed xen.gz and call it > > > xen-$(FULLVERSION).config > > > > > > Suggested-by: Ian Campbell <ian.campbell@citrix.com> > > > Signed-off-by: Doug Goldstein <cardoe@cardoe.com> > > > --- > > > xen/Makefile | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/xen/Makefile b/xen/Makefile > > > index 9023863..460b977 100644 > > > --- a/xen/Makefile > > > +++ b/xen/Makefile > > > @@ -58,6 +58,7 @@ _install: $(TARGET)$(CONFIG_XEN_INSTALL_SUFFIX) > > > ln -f -s $(T)-$(XEN_FULLVERSION)$(Z) > > > $(D)$(BOOT_DIR)/$(T)$(Z) > > > [ -d "$(D)$(DEBUG_DIR)" ] || $(INSTALL_DIR) $(D)$(DEBUG_DIR) > > > $(INSTALL_DATA) $(TARGET)-syms $(D)$(DEBUG_DIR)/$(T)-syms- > > > $(XEN_FULLVERSION) > > > + $(INSTALL_DATA) $(KCONFIG_CONFIG) $(D)$(BOOT_DIR)/$(T)- > > > $(XEN_FULLVERSION).config > > > > Was it really suggested to put this into /boot? It has no business > > being there... > > > > Jan > > > > Yes. By multiple people. Ian Campbell was the first person to suggest it > in that location. I don't really care where it goes so long as it is somewhere which is (or can become) well known. IOW: I don't care to join in the argument about the location. Ian. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel ^ permalink raw reply [flat|nested] 25+ messages in thread
end of thread, other threads:[~2016-01-06 13:01 UTC | newest] Thread overview: 25+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2015-12-18 21:35 [PATCH] build: save generated config in /boot Doug Goldstein 2015-12-21 12:11 ` Jan Beulich 2015-12-21 12:40 ` Andrew Cooper 2015-12-21 12:53 ` Jan Beulich 2015-12-22 12:45 ` Doug Goldstein 2015-12-22 12:50 ` Jan Beulich 2015-12-21 14:35 ` Doug Goldstein 2015-12-21 14:51 ` Jan Beulich 2015-12-21 15:20 ` Doug Goldstein 2015-12-21 15:35 ` Jan Beulich 2015-12-22 12:45 ` Doug Goldstein 2015-12-22 12:52 ` Jan Beulich 2015-12-22 14:46 ` Andrew Cooper 2015-12-22 15:59 ` Jan Beulich 2015-12-22 16:02 ` Doug Goldstein 2015-12-22 16:09 ` Jan Beulich 2016-01-05 16:53 ` Doug Goldstein 2016-01-06 9:22 ` Jan Beulich 2016-01-06 10:31 ` Ian Campbell 2016-01-06 10:38 ` Jan Beulich 2016-01-06 11:58 ` Ian Jackson 2016-01-06 12:56 ` Jan Beulich 2016-01-06 12:17 ` Wei Liu 2016-01-06 13:01 ` Jan Beulich 2016-01-04 11:27 ` Ian Campbell
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.