Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1278424 > unrolled thread
| Started by | Scott Branden <sbranden@broadcom.com> |
|---|---|
| First post | 2015-11-26 21:10 +0100 |
| Last post | 2015-11-28 00:50 +0100 |
| Articles | 5 — 3 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
[PATCH 1/1] kconfig: add CPU endian selection beconfig and leconfig Scott Branden <sbranden@broadcom.com> - 2015-11-26 21:10 +0100
Re: [PATCH 1/1] kconfig: add CPU endian selection beconfig and leconfig Arnd Bergmann <arnd@arndb.de> - 2015-11-27 17:10 +0100
Re: [PATCH 1/1] kconfig: add CPU endian selection beconfig and leconfig Scott Branden <sbranden@broadcom.com> - 2015-11-28 00:50 +0100
Re: [PATCH 1/1] kconfig: add CPU endian selection beconfig and leconfig Florian Fainelli <f.fainelli@gmail.com> - 2015-11-27 19:50 +0100
Re: [PATCH 1/1] kconfig: add CPU endian selection beconfig and leconfig Scott Branden <sbranden@broadcom.com> - 2015-11-28 00:50 +0100
| From | Scott Branden <sbranden@broadcom.com> |
|---|---|
| Date | 2015-11-26 21:10 +0100 |
| Subject | [PATCH 1/1] kconfig: add CPU endian selection beconfig and leconfig |
| Message-ID | <qzb1D-1pH-11@gated-at.bofh.it> |
Add support for switching defconfig between big and little endian CPU. Various CPU types have ability to select big and little endian CPU in the kernel configuration. "make beconfig" will set CONFIG_CPU_BIG_ENDIAN "make leconfig" will unset CONFIG_CPU_BIG_ENDIAN Signed-off-by: Scott Branden <sbranden@broadcom.com> --- kernel/configs/be.config | 1 + kernel/configs/le.config | 1 + scripts/kconfig/Makefile | 10 ++++++++++ 3 files changed, 12 insertions(+) create mode 100644 kernel/configs/be.config create mode 100644 kernel/configs/le.config diff --git a/kernel/configs/be.config b/kernel/configs/be.config new file mode 100644 index 0000000..c5cdc99 --- /dev/null +++ b/kernel/configs/be.config @@ -0,0 +1 @@ +CONFIG_CPU_BIG_ENDIAN=y diff --git a/kernel/configs/le.config b/kernel/configs/le.config new file mode 100644 index 0000000..be479cf --- /dev/null +++ b/kernel/configs/le.config @@ -0,0 +1 @@ +# CONFIG_CPU_BIG_ENDIAN is not set diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile index d79cba4..2e8007e 100644 --- a/scripts/kconfig/Makefile +++ b/scripts/kconfig/Makefile @@ -126,6 +126,14 @@ PHONY += tinyconfig tinyconfig: $(Q)$(MAKE) -f $(srctree)/Makefile allnoconfig tiny.config +PHONY += beconfig +beconfig: be.config + @: + +PHONY += leconfig +leconfig: le.config + @: + # Help text used by make help help: @echo ' config - Update current config utilising a line-oriented program' @@ -151,6 +159,8 @@ help: @echo ' kvmconfig - Enable additional options for kvm guest kernel support' @echo ' xenconfig - Enable additional options for xen dom0 and guest kernel support' @echo ' tinyconfig - Configure the tiniest possible kernel' + @echo ' beconfig - Update current config to use big endian CPU' + @echo ' leconfig - Update current config to use little endian CPU' # lxdialog stuff check-lxdialog := $(srctree)/$(src)/lxdialog/check-lxdialog.sh -- 2.5.0 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Arnd Bergmann <arnd@arndb.de> |
|---|---|
| Date | 2015-11-27 17:10 +0100 |
| Message-ID | <qztKV-58r-11@gated-at.bofh.it> |
| In reply to | #1278424 |
On Thursday 26 November 2015 11:59:55 Scott Branden wrote: > @@ -151,6 +159,8 @@ help: > @echo ' kvmconfig - Enable additional options for kvm guest kernel support' > @echo ' xenconfig - Enable additional options for xen dom0 and guest kernel support' > @echo ' tinyconfig - Configure the tiniest possible kernel' > + @echo ' beconfig - Update current config to use big endian CPU' > + @echo ' leconfig - Update current config to use little endian CPU' > > Most architectures don't actually support picking the endianess, I think it would be better to keep this architecture specific in some form. If we limit it to ARM/ARM64, we might not need the leconfig target at all, because that is the default for all defconfig files in the kernel aside from ixp4xx, which we can probably ignore here. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Scott Branden <sbranden@broadcom.com> |
|---|---|
| Date | 2015-11-28 00:50 +0100 |
| Subject | Re: [PATCH 1/1] kconfig: add CPU endian selection beconfig and leconfig |
| Message-ID | <qzAW6-WL-1@gated-at.bofh.it> |
| In reply to | #1278869 |
Hi Arnd, Thanks for comment - I forgot to directly add you to CC list so was going to add you in a couple days if you didn't find this patch proposal. Response below. On 15-11-27 07:59 AM, Arnd Bergmann wrote: > On Thursday 26 November 2015 11:59:55 Scott Branden wrote: >> @@ -151,6 +159,8 @@ help: >> @echo ' kvmconfig - Enable additional options for kvm guest kernel support' >> @echo ' xenconfig - Enable additional options for xen dom0 and guest kernel support' >> @echo ' tinyconfig - Configure the tiniest possible kernel' >> + @echo ' beconfig - Update current config to use big endian CPU' >> + @echo ' leconfig - Update current config to use little endian CPU' >> >> > > Most architectures don't actually support picking the endianess, I think > it would be better to keep this architecture specific in some form. If we > limit it to ARM/ARM64, we might not need the leconfig target at all, because > that is the default for all defconfig files in the kernel aside from ixp4xx, > which we can probably ignore here. Yes, I thought about placing le.config in arm64/configs. Then I would need it in arm/configs also. CONFIG_CPU_BIG_ENDIAN is currently used by arc, sh, c6x, powerpc, arm, arm64, and mips. Then the next architecture comes along. So to have the same feature on those architectures they would also need le.config in their configs directory. running "make leconfig" or "make beconfig" on architectures that do not support CPU_BIG_ENDIAN has no harmful side affect other than the endianness selection will not change. For these reasons placing the functionality in kernel/configs alongside xen.config seemed like the sensible thing to do. kernel/configs seems like a very good place to add more .config selection options to as well. Of course, whatever method is selected to switch between big and little endian sharing a single defconfig I'm open to. > > Arnd > Thanks for comment. Scott -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Florian Fainelli <f.fainelli@gmail.com> |
|---|---|
| Date | 2015-11-27 19:50 +0100 |
| Subject | Re: [PATCH 1/1] kconfig: add CPU endian selection beconfig and leconfig |
| Message-ID | <qzwfL-6wG-1@gated-at.bofh.it> |
| In reply to | #1278424 |
Le 26/11/2015 11:59, Scott Branden a écrit : > Add support for switching defconfig between big and little endian CPU. > Various CPU types have ability to select big and little endian > CPU in the kernel configuration. > > "make beconfig" will set CONFIG_CPU_BIG_ENDIAN > "make leconfig" will unset CONFIG_CPU_BIG_ENDIAN I believe I understand what you are trying to achieve here, which is to have an identical defconfig file that you can share between a big-endian and little-endian kernel? Is not this something better left to a build system which understands what config fragments are in general? Since we seem to have support for fragments now with kvm and friends, having this does not seem to be a big stretch though. > > Signed-off-by: Scott Branden <sbranden@broadcom.com> > --- > kernel/configs/be.config | 1 + > kernel/configs/le.config | 1 + > scripts/kconfig/Makefile | 10 ++++++++++ > 3 files changed, 12 insertions(+) > create mode 100644 kernel/configs/be.config > create mode 100644 kernel/configs/le.config > > diff --git a/kernel/configs/be.config b/kernel/configs/be.config > new file mode 100644 > index 0000000..c5cdc99 > --- /dev/null > +++ b/kernel/configs/be.config > @@ -0,0 +1 @@ > +CONFIG_CPU_BIG_ENDIAN=y > diff --git a/kernel/configs/le.config b/kernel/configs/le.config > new file mode 100644 > index 0000000..be479cf > --- /dev/null > +++ b/kernel/configs/le.config > @@ -0,0 +1 @@ > +# CONFIG_CPU_BIG_ENDIAN is not set > diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile > index d79cba4..2e8007e 100644 > --- a/scripts/kconfig/Makefile > +++ b/scripts/kconfig/Makefile > @@ -126,6 +126,14 @@ PHONY += tinyconfig > tinyconfig: > $(Q)$(MAKE) -f $(srctree)/Makefile allnoconfig tiny.config > > +PHONY += beconfig > +beconfig: be.config > + @: > + > +PHONY += leconfig > +leconfig: le.config > + @: > + > # Help text used by make help > help: > @echo ' config - Update current config utilising a line-oriented program' > @@ -151,6 +159,8 @@ help: > @echo ' kvmconfig - Enable additional options for kvm guest kernel support' > @echo ' xenconfig - Enable additional options for xen dom0 and guest kernel support' > @echo ' tinyconfig - Configure the tiniest possible kernel' > + @echo ' beconfig - Update current config to use big endian CPU' > + @echo ' leconfig - Update current config to use little endian CPU' > > # lxdialog stuff > check-lxdialog := $(srctree)/$(src)/lxdialog/check-lxdialog.sh > -- Florian -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Scott Branden <sbranden@broadcom.com> |
|---|---|
| Date | 2015-11-28 00:50 +0100 |
| Subject | Re: [PATCH 1/1] kconfig: add CPU endian selection beconfig and leconfig |
| Message-ID | <qzAW6-WL-3@gated-at.bofh.it> |
| In reply to | #1278919 |
Hi Florian, On 15-11-27 10:39 AM, Florian Fainelli wrote: > Le 26/11/2015 11:59, Scott Branden a écrit : >> Add support for switching defconfig between big and little endian CPU. >> Various CPU types have ability to select big and little endian >> CPU in the kernel configuration. >> >> "make beconfig" will set CONFIG_CPU_BIG_ENDIAN >> "make leconfig" will unset CONFIG_CPU_BIG_ENDIAN > > I believe I understand what you are trying to achieve here, which is to > have an identical defconfig file that you can share between a big-endian > and little-endian kernel? You understand correct - ARM64 maintainers have only allowed 1 defconfig upstream and it supports little endian. And it doesn't make sense to have another defconfig with a single line difference upstreamed. Yet the defconfig does not allow the system to boot in big endian mode. > > Is not this something better left to a build system which understands > what config fragments are in general? Every build system would need to invent their own procedure. By adding this into the upstream kernel there is no need to. make defconfig make beconfig (or make leconfig) make > > Since we seem to have support for fragments now with kvm and friends, > having this does not seem to be a big stretch though. Yes, that is why I proposed place it in a common location. > >> >> Signed-off-by: Scott Branden <sbranden@broadcom.com> >> --- >> kernel/configs/be.config | 1 + >> kernel/configs/le.config | 1 + >> scripts/kconfig/Makefile | 10 ++++++++++ >> 3 files changed, 12 insertions(+) >> create mode 100644 kernel/configs/be.config >> create mode 100644 kernel/configs/le.config >> >> diff --git a/kernel/configs/be.config b/kernel/configs/be.config >> new file mode 100644 >> index 0000000..c5cdc99 >> --- /dev/null >> +++ b/kernel/configs/be.config >> @@ -0,0 +1 @@ >> +CONFIG_CPU_BIG_ENDIAN=y >> diff --git a/kernel/configs/le.config b/kernel/configs/le.config >> new file mode 100644 >> index 0000000..be479cf >> --- /dev/null >> +++ b/kernel/configs/le.config >> @@ -0,0 +1 @@ >> +# CONFIG_CPU_BIG_ENDIAN is not set >> diff --git a/scripts/kconfig/Makefile b/scripts/kconfig/Makefile >> index d79cba4..2e8007e 100644 >> --- a/scripts/kconfig/Makefile >> +++ b/scripts/kconfig/Makefile >> @@ -126,6 +126,14 @@ PHONY += tinyconfig >> tinyconfig: >> $(Q)$(MAKE) -f $(srctree)/Makefile allnoconfig tiny.config >> >> +PHONY += beconfig >> +beconfig: be.config >> + @: >> + >> +PHONY += leconfig >> +leconfig: le.config >> + @: >> + >> # Help text used by make help >> help: >> @echo ' config - Update current config utilising a line-oriented program' >> @@ -151,6 +159,8 @@ help: >> @echo ' kvmconfig - Enable additional options for kvm guest kernel support' >> @echo ' xenconfig - Enable additional options for xen dom0 and guest kernel support' >> @echo ' tinyconfig - Configure the tiniest possible kernel' >> + @echo ' beconfig - Update current config to use big endian CPU' >> + @echo ' leconfig - Update current config to use little endian CPU' >> >> # lxdialog stuff >> check-lxdialog := $(srctree)/$(src)/lxdialog/check-lxdialog.sh >> > > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web