* [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier @ 2020-10-30 18:18 Lee Jones 2020-10-31 5:09 ` Peilin Ye ` (3 more replies) 0 siblings, 4 replies; 25+ messages in thread From: Lee Jones @ 2020-10-30 18:18 UTC (permalink / raw) To: daniel.vetter, gregkh, yepeilin.cs Cc: linux-arm-kernel, linux-kernel, Lee Jones, stable, Russell King Commit 09e5b3fd5672 ("Fonts: Support FONT_EXTRA_WORDS macros for built-in fonts") introduced the following error when building rpc_defconfig (only this build appears to be affected): `acorndata_8x8' referenced in section `.text' of arch/arm/boot/compressed/ll_char_wr.o: defined in discarded section `.data' of arch/arm/boot/compressed/font.o `acorndata_8x8' referenced in section `.data.rel.ro' of arch/arm/boot/compressed/font.o: defined in discarded section `.data' of arch/arm/boot/compressed/font.o make[3]: *** [/scratch/linux/arch/arm/boot/compressed/Makefile:191: arch/arm/boot/compressed/vmlinux] Error 1 make[2]: *** [/scratch/linux/arch/arm/boot/Makefile:61: arch/arm/boot/compressed/vmlinux] Error 2 make[1]: *** [/scratch/linux/arch/arm/Makefile:317: zImage] Error 2 The .data section is discarded at link time. Reinstating acorndata_8x8 as const ensures it is still available after linking. Cc: <stable@vger.kernel.org> Cc: Russell King <linux@armlinux.org.uk> Signed-off-by: Lee Jones <lee.jones@linaro.org> --- lib/fonts/font_acorn_8x8.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/fonts/font_acorn_8x8.c b/lib/fonts/font_acorn_8x8.c index 069b3e80c4344..fb395f0d40317 100644 --- a/lib/fonts/font_acorn_8x8.c +++ b/lib/fonts/font_acorn_8x8.c @@ -5,7 +5,7 @@ #define FONTDATAMAX 2048 -static struct font_data acorndata_8x8 = { +static const struct font_data acorndata_8x8 = { { 0, 0, FONTDATAMAX, 0 }, { /* 00 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, /* ^@ */ /* 01 */ 0x7e, 0x81, 0xa5, 0x81, 0xbd, 0x99, 0x81, 0x7e, /* ^A */ -- 2.25.1 ^ permalink raw reply related [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-10-30 18:18 [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier Lee Jones @ 2020-10-31 5:09 ` Peilin Ye 2020-10-31 10:27 ` Russell King - ARM Linux admin ` (2 subsequent siblings) 3 siblings, 0 replies; 25+ messages in thread From: Peilin Ye @ 2020-10-31 5:09 UTC (permalink / raw) To: Lee Jones, daniel.vetter, gregkh Cc: linux-arm-kernel, linux-kernel, stable, Russell King, yepeilin.cs Hi Lee, On Fri, Oct 30, 2020 at 06:18:22PM +0000, Lee Jones wrote: > Commit 09e5b3fd5672 ("Fonts: Support FONT_EXTRA_WORDS macros for > built-in fonts") introduced the following error when building > rpc_defconfig (only this build appears to be affected): > > `acorndata_8x8' referenced in section `.text' of arch/arm/boot/compressed/ll_char_wr.o: > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > `acorndata_8x8' referenced in section `.data.rel.ro' of arch/arm/boot/compressed/font.o: > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > make[3]: *** [/scratch/linux/arch/arm/boot/compressed/Makefile:191: arch/arm/boot/compressed/vmlinux] Error 1 > make[2]: *** [/scratch/linux/arch/arm/boot/Makefile:61: arch/arm/boot/compressed/vmlinux] Error 2 > make[1]: *** [/scratch/linux/arch/arm/Makefile:317: zImage] Error 2 > > The .data section is discarded at link time. Reinstating > acorndata_8x8 as const ensures it is still available after linking. Thanks a lot for fixing this up! I wasn't aware that the symbol is being referenced in arch/arm/boot/compressed/ll_char_wr.S. I'm sorry for the trouble. The patch is, > Cc: <stable@vger.kernel.org> > Cc: Russell King <linux@armlinux.org.uk> > Signed-off-by: Lee Jones <lee.jones@linaro.org> Tested-by: Peilin Ye <yepeilin.cs@gmail.com> Thank you, Peilin Ye ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-10-30 18:18 [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier Lee Jones 2020-10-31 5:09 ` Peilin Ye @ 2020-10-31 10:27 ` Russell King - ARM Linux admin 2020-11-01 13:11 ` Lee Jones 2020-11-02 10:56 ` Daniel Vetter 2020-11-02 18:32 ` [PATCH v2 1/1] Fonts: " Peilin Ye 3 siblings, 1 reply; 25+ messages in thread From: Russell King - ARM Linux admin @ 2020-10-31 10:27 UTC (permalink / raw) To: Lee Jones Cc: daniel.vetter, gregkh, yepeilin.cs, linux-arm-kernel, linux-kernel, stable On Fri, Oct 30, 2020 at 06:18:22PM +0000, Lee Jones wrote: > Commit 09e5b3fd5672 ("Fonts: Support FONT_EXTRA_WORDS macros for Your commit ID does not exist in mainline kernels, which makes this confusing. The commit ID you should be using is 6735b4632def. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last! ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-10-31 10:27 ` Russell King - ARM Linux admin @ 2020-11-01 13:11 ` Lee Jones 2020-11-02 10:23 ` Russell King - ARM Linux admin 0 siblings, 1 reply; 25+ messages in thread From: Lee Jones @ 2020-11-01 13:11 UTC (permalink / raw) To: Russell King - ARM Linux admin Cc: daniel.vetter, gregkh, yepeilin.cs, linux-arm-kernel, linux-kernel, stable On Sat, 31 Oct 2020, Russell King - ARM Linux admin wrote: > On Fri, Oct 30, 2020 at 06:18:22PM +0000, Lee Jones wrote: > > Commit 09e5b3fd5672 ("Fonts: Support FONT_EXTRA_WORDS macros for > > Your commit ID does not exist in mainline kernels, which makes this > confusing. The commit ID you should be using is 6735b4632def. Ah yes, quite right. That is the ID from android-3.18 where this issue was first seen and fixed against. I will fix it up for Mainline. Does the fix look okay to you though Russell? -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-11-01 13:11 ` Lee Jones @ 2020-11-02 10:23 ` Russell King - ARM Linux admin 2020-11-02 10:30 ` Russell King - ARM Linux admin 2020-11-02 16:12 ` Peilin Ye 0 siblings, 2 replies; 25+ messages in thread From: Russell King - ARM Linux admin @ 2020-11-02 10:23 UTC (permalink / raw) To: Lee Jones Cc: daniel.vetter, gregkh, yepeilin.cs, linux-arm-kernel, linux-kernel, stable On Sun, Nov 01, 2020 at 01:11:22PM +0000, Lee Jones wrote: > On Sat, 31 Oct 2020, Russell King - ARM Linux admin wrote: > > > On Fri, Oct 30, 2020 at 06:18:22PM +0000, Lee Jones wrote: > > > Commit 09e5b3fd5672 ("Fonts: Support FONT_EXTRA_WORDS macros for > > > > Your commit ID does not exist in mainline kernels, which makes this > > confusing. The commit ID you should be using is 6735b4632def. > > Ah yes, quite right. That is the ID from android-3.18 where this > issue was first seen and fixed against. I will fix it up for > Mainline. > > Does the fix look okay to you though Russell? Frankly, I don't know. Looking at the commit itself, it looks safe, but it depends what this "extra" data is being used for. From what I can see, the commit in question just adds the additional opaque data as a member named "extra", and one is left to guess what it's use as. I'd have thought a small structure with named members would have been the minimum given our standards for in-kernel code. Why was the "const" dropped in the first place? Does this "extra" member get written to somewhere? So, sorry, no idea. This looks to me like a very unsatisfactory commit, and probably something that got a very poor review. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last! ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-11-02 10:23 ` Russell King - ARM Linux admin @ 2020-11-02 10:30 ` Russell King - ARM Linux admin 2020-11-02 16:12 ` Peilin Ye 1 sibling, 0 replies; 25+ messages in thread From: Russell King - ARM Linux admin @ 2020-11-02 10:30 UTC (permalink / raw) To: Lee Jones Cc: daniel.vetter, linux-kernel, stable, gregkh, yepeilin.cs, linux-arm-kernel On Mon, Nov 02, 2020 at 10:23:43AM +0000, Russell King - ARM Linux admin wrote: > On Sun, Nov 01, 2020 at 01:11:22PM +0000, Lee Jones wrote: > > On Sat, 31 Oct 2020, Russell King - ARM Linux admin wrote: > > > > > On Fri, Oct 30, 2020 at 06:18:22PM +0000, Lee Jones wrote: > > > > Commit 09e5b3fd5672 ("Fonts: Support FONT_EXTRA_WORDS macros for > > > > > > Your commit ID does not exist in mainline kernels, which makes this > > > confusing. The commit ID you should be using is 6735b4632def. > > > > Ah yes, quite right. That is the ID from android-3.18 where this > > issue was first seen and fixed against. I will fix it up for > > Mainline. > > > > Does the fix look okay to you though Russell? > > Frankly, I don't know. Looking at the commit itself, it looks safe, > but it depends what this "extra" data is being used for. From what > I can see, the commit in question just adds the additional opaque > data as a member named "extra", and one is left to guess what it's > use as. > > I'd have thought a small structure with named members would have > been the minimum given our standards for in-kernel code. > > Why was the "const" dropped in the first place? Does this "extra" > member get written to somewhere? > > So, sorry, no idea. This looks to me like a very unsatisfactory > commit, and probably something that got a very poor review. Also, the commit description is missing a chunk: For user-provided fonts, the framebuffer layer resolves this issue by reserving four extra words at the beginning of data buffers. Later, whenever a function needs to access them, it simply uses the following macros: Recently we have gathered all the above macros to <linux/font.h>. So what were these macros that have been nicely removed from the commit description? I guess they started with a '#' character and git thought they were a comment. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last! ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-11-02 10:23 ` Russell King - ARM Linux admin 2020-11-02 10:30 ` Russell King - ARM Linux admin @ 2020-11-02 16:12 ` Peilin Ye 1 sibling, 0 replies; 25+ messages in thread From: Peilin Ye @ 2020-11-02 16:12 UTC (permalink / raw) To: Russell King - ARM Linux admin Cc: Lee Jones, daniel.vetter, gregkh, linux-arm-kernel, linux-kernel, stable Hi Russell, On Mon, Nov 02, 2020 at 10:23:43AM +0000, Russell King - ARM Linux admin wrote: > On Sun, Nov 01, 2020 at 01:11:22PM +0000, Lee Jones wrote: > > On Sat, 31 Oct 2020, Russell King - ARM Linux admin wrote: > > > > > On Fri, Oct 30, 2020 at 06:18:22PM +0000, Lee Jones wrote: > > > > Commit 09e5b3fd5672 ("Fonts: Support FONT_EXTRA_WORDS macros for > > > > > > Your commit ID does not exist in mainline kernels, which makes this > > > confusing. The commit ID you should be using is 6735b4632def. > > > > Ah yes, quite right. That is the ID from android-3.18 where this > > issue was first seen and fixed against. I will fix it up for > > Mainline. > > > > Does the fix look okay to you though Russell? > > Frankly, I don't know. Looking at the commit itself, it looks safe, > but it depends what this "extra" data is being used for. From what > I can see, the commit in question just adds the additional opaque > data as a member named "extra", and one is left to guess what it's > use as. Thank you very much for looking into this. I apologize for the trouble and confusion it has caused. The motivation behind this commit, and commit 5af08640795b ("fbcon: Fix global-out-of-bounds read in fbcon_get_font()") was to fix a decades-old out-of-bounds access bug in the framebuffer layer. However the framebuffer layer is doing bounds checking in a very strange way, by hiding the buffer length before the buffer, then access it using a negative-indexing macro: #define FNTSIZE(fd) (((int *)(fd))[-2]) Other "extra" (so-called by the framebuffer layer) fields include: #define REFCOUNT(fd) (((int *)(fd))[-1]) #define FNTCHARCNT(fd) (((int *)(fd))[-3]) #define FNTSUM(fd) (((int *)(fd))[-4]) ...representing reference count, character count and checksum, respectively. The commit in question (6735b4632def) prepends the buffer length to each of the built-in font buffers, so other functions in the framebuffer layer can use FNTSIZE() on them. 5af08640795b uses it to fix that out-of-bounds bug. > I'd have thought a small structure with named members would have > been the minimum given our standards for in-kernel code. Yes, this is a temporary bug fix, and is far from satisfactory. We are trying to replace these magic macros using a structure with properly named members. It is taking more time than I imagined, but one day this temporary fix will disappear from the kernel, I hope. > Why was the "const" dropped in the first place? Does this "extra" > member get written to somewhere? No, I will try to come up with a solution without these fields being writable. > So, sorry, no idea. This looks to me like a very unsatisfactory > commit, and probably something that got a very poor review. I hope this helps explain it. Again, I apologize for all the troubles. I will do more thorough testing and practice writing a commit message. Thank you! Sincerely, Peilin Ye ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-10-30 18:18 [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier Lee Jones 2020-10-31 5:09 ` Peilin Ye 2020-10-31 10:27 ` Russell King - ARM Linux admin @ 2020-11-02 10:56 ` Daniel Vetter 2020-11-02 11:09 ` Lee Jones 2020-11-02 18:32 ` [PATCH v2 1/1] Fonts: " Peilin Ye 3 siblings, 1 reply; 25+ messages in thread From: Daniel Vetter @ 2020-11-02 10:56 UTC (permalink / raw) To: Lee Jones Cc: Greg KH, Peilin Ye, Linux ARM, Linux Kernel Mailing List, stable, Russell King On Fri, Oct 30, 2020 at 7:18 PM Lee Jones <lee.jones@linaro.org> wrote: > > Commit 09e5b3fd5672 ("Fonts: Support FONT_EXTRA_WORDS macros for > built-in fonts") introduced the following error when building > rpc_defconfig (only this build appears to be affected): > > `acorndata_8x8' referenced in section `.text' of arch/arm/boot/compressed/ll_char_wr.o: > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > `acorndata_8x8' referenced in section `.data.rel.ro' of arch/arm/boot/compressed/font.o: > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > make[3]: *** [/scratch/linux/arch/arm/boot/compressed/Makefile:191: arch/arm/boot/compressed/vmlinux] Error 1 > make[2]: *** [/scratch/linux/arch/arm/boot/Makefile:61: arch/arm/boot/compressed/vmlinux] Error 2 > make[1]: *** [/scratch/linux/arch/arm/Makefile:317: zImage] Error 2 > > The .data section is discarded at link time. Reinstating > acorndata_8x8 as const ensures it is still available after linking. > > Cc: <stable@vger.kernel.org> > Cc: Russell King <linux@armlinux.org.uk> > Signed-off-by: Lee Jones <lee.jones@linaro.org> Shouldn't we add the const to all of them, for consistency? -Daniel > --- > lib/fonts/font_acorn_8x8.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/fonts/font_acorn_8x8.c b/lib/fonts/font_acorn_8x8.c > index 069b3e80c4344..fb395f0d40317 100644 > --- a/lib/fonts/font_acorn_8x8.c > +++ b/lib/fonts/font_acorn_8x8.c > @@ -5,7 +5,7 @@ > > #define FONTDATAMAX 2048 > > -static struct font_data acorndata_8x8 = { > +static const struct font_data acorndata_8x8 = { > { 0, 0, FONTDATAMAX, 0 }, { > /* 00 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, /* ^@ */ > /* 01 */ 0x7e, 0x81, 0xa5, 0x81, 0xbd, 0x99, 0x81, 0x7e, /* ^A */ > -- > 2.25.1 > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-11-02 10:56 ` Daniel Vetter @ 2020-11-02 11:09 ` Lee Jones 2020-11-02 11:18 ` Daniel Vetter 0 siblings, 1 reply; 25+ messages in thread From: Lee Jones @ 2020-11-02 11:09 UTC (permalink / raw) To: Daniel Vetter Cc: Greg KH, Peilin Ye, Linux ARM, Linux Kernel Mailing List, stable, Russell King On Mon, 02 Nov 2020, Daniel Vetter wrote: > On Fri, Oct 30, 2020 at 7:18 PM Lee Jones <lee.jones@linaro.org> wrote: > > > > Commit 09e5b3fd5672 ("Fonts: Support FONT_EXTRA_WORDS macros for > > built-in fonts") introduced the following error when building > > rpc_defconfig (only this build appears to be affected): > > > > `acorndata_8x8' referenced in section `.text' of arch/arm/boot/compressed/ll_char_wr.o: > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > `acorndata_8x8' referenced in section `.data.rel.ro' of arch/arm/boot/compressed/font.o: > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > make[3]: *** [/scratch/linux/arch/arm/boot/compressed/Makefile:191: arch/arm/boot/compressed/vmlinux] Error 1 > > make[2]: *** [/scratch/linux/arch/arm/boot/Makefile:61: arch/arm/boot/compressed/vmlinux] Error 2 > > make[1]: *** [/scratch/linux/arch/arm/Makefile:317: zImage] Error 2 > > > > The .data section is discarded at link time. Reinstating > > acorndata_8x8 as const ensures it is still available after linking. > > > > Cc: <stable@vger.kernel.org> > > Cc: Russell King <linux@armlinux.org.uk> > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > > Shouldn't we add the const to all of them, for consistency? The thought did cross my mind. However, I do not see any further issues which need addressing. Nor do I have any visibility into what issues may be caused by doing so. The only thing I know for sure is that this patch fixes the compile error pertained to in the commit message, and I'd like for this fix to be as atomic as possible, as it's designed to be routed through the Stable/LTS trees. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-11-02 11:09 ` Lee Jones @ 2020-11-02 11:18 ` Daniel Vetter 2020-11-02 11:30 ` Lee Jones 0 siblings, 1 reply; 25+ messages in thread From: Daniel Vetter @ 2020-11-02 11:18 UTC (permalink / raw) To: Lee Jones Cc: Greg KH, Peilin Ye, Linux ARM, Linux Kernel Mailing List, stable, Russell King On Mon, Nov 2, 2020 at 12:09 PM Lee Jones <lee.jones@linaro.org> wrote: > > On Mon, 02 Nov 2020, Daniel Vetter wrote: > > > On Fri, Oct 30, 2020 at 7:18 PM Lee Jones <lee.jones@linaro.org> wrote: > > > > > > Commit 09e5b3fd5672 ("Fonts: Support FONT_EXTRA_WORDS macros for > > > built-in fonts") introduced the following error when building > > > rpc_defconfig (only this build appears to be affected): > > > > > > `acorndata_8x8' referenced in section `.text' of arch/arm/boot/compressed/ll_char_wr.o: > > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > > `acorndata_8x8' referenced in section `.data.rel.ro' of arch/arm/boot/compressed/font.o: > > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > > make[3]: *** [/scratch/linux/arch/arm/boot/compressed/Makefile:191: arch/arm/boot/compressed/vmlinux] Error 1 > > > make[2]: *** [/scratch/linux/arch/arm/boot/Makefile:61: arch/arm/boot/compressed/vmlinux] Error 2 > > > make[1]: *** [/scratch/linux/arch/arm/Makefile:317: zImage] Error 2 > > > > > > The .data section is discarded at link time. Reinstating > > > acorndata_8x8 as const ensures it is still available after linking. > > > > > > Cc: <stable@vger.kernel.org> > > > Cc: Russell King <linux@armlinux.org.uk> > > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > > > > Shouldn't we add the const to all of them, for consistency? > > The thought did cross my mind. However, I do not see any further > issues which need addressing. Nor do I have any visibility into what > issues may be caused by doing so. The only thing I know for sure is > that this patch fixes the compile error pertained to in the commit > message, and I'd like for this fix to be as atomic as possible, as > it's designed to be routed through the Stable/LTS trees. The trouble is that if we only make one of them const, then it'll take so much longer to hit any issues due to code not handling this correctly. Being consistent with all fonts sounds like the best approach. And the original patch that lost the const for the additional data also went through cc: stable for all fonts together. So that shouldn't be the hold-up. -Daniel > > -- > Lee Jones [李琼斯] > Senior Technical Lead - Developer Services > Linaro.org │ Open source software for Arm SoCs > Follow Linaro: Facebook | Twitter | Blog -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-11-02 11:18 ` Daniel Vetter @ 2020-11-02 11:30 ` Lee Jones 2020-11-02 14:50 ` Daniel Vetter 0 siblings, 1 reply; 25+ messages in thread From: Lee Jones @ 2020-11-02 11:30 UTC (permalink / raw) To: Daniel Vetter Cc: Greg KH, Peilin Ye, Linux ARM, Linux Kernel Mailing List, stable, Russell King On Mon, 02 Nov 2020, Daniel Vetter wrote: > On Mon, Nov 2, 2020 at 12:09 PM Lee Jones <lee.jones@linaro.org> wrote: > > > > On Mon, 02 Nov 2020, Daniel Vetter wrote: > > > > > On Fri, Oct 30, 2020 at 7:18 PM Lee Jones <lee.jones@linaro.org> wrote: > > > > > > > > Commit 09e5b3fd5672 ("Fonts: Support FONT_EXTRA_WORDS macros for > > > > built-in fonts") introduced the following error when building > > > > rpc_defconfig (only this build appears to be affected): > > > > > > > > `acorndata_8x8' referenced in section `.text' of arch/arm/boot/compressed/ll_char_wr.o: > > > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > > > `acorndata_8x8' referenced in section `.data.rel.ro' of arch/arm/boot/compressed/font.o: > > > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > > > make[3]: *** [/scratch/linux/arch/arm/boot/compressed/Makefile:191: arch/arm/boot/compressed/vmlinux] Error 1 > > > > make[2]: *** [/scratch/linux/arch/arm/boot/Makefile:61: arch/arm/boot/compressed/vmlinux] Error 2 > > > > make[1]: *** [/scratch/linux/arch/arm/Makefile:317: zImage] Error 2 > > > > > > > > The .data section is discarded at link time. Reinstating > > > > acorndata_8x8 as const ensures it is still available after linking. > > > > > > > > Cc: <stable@vger.kernel.org> > > > > Cc: Russell King <linux@armlinux.org.uk> > > > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > > > > > > Shouldn't we add the const to all of them, for consistency? > > > > The thought did cross my mind. However, I do not see any further > > issues which need addressing. Nor do I have any visibility into what > > issues may be caused by doing so. The only thing I know for sure is > > that this patch fixes the compile error pertained to in the commit > > message, and I'd like for this fix to be as atomic as possible, as > > it's designed to be routed through the Stable/LTS trees. > > The trouble is that if we only make one of them const, then it'll take > so much longer to hit any issues due to code not handling this > correctly. Being consistent with all fonts sounds like the best > approach. > > And the original patch that lost the const for the additional data > also went through cc: stable for all fonts together. So that shouldn't > be the hold-up. My plan was to keep the fix as simple as possible. This is only an issue due to the odd handling of the compressed Arm image which exclusively references 'acorndata_8x8' and discards it's .data section. I am happy to go with the majority on this though. Does anyone else have an opinion? -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-11-02 11:30 ` Lee Jones @ 2020-11-02 14:50 ` Daniel Vetter 2020-11-02 16:17 ` Peilin Ye 0 siblings, 1 reply; 25+ messages in thread From: Daniel Vetter @ 2020-11-02 14:50 UTC (permalink / raw) To: Lee Jones Cc: Greg KH, Peilin Ye, Linux ARM, Linux Kernel Mailing List, stable, Russell King On Mon, Nov 2, 2020 at 12:30 PM Lee Jones <lee.jones@linaro.org> wrote: > > On Mon, 02 Nov 2020, Daniel Vetter wrote: > > > On Mon, Nov 2, 2020 at 12:09 PM Lee Jones <lee.jones@linaro.org> wrote: > > > > > > On Mon, 02 Nov 2020, Daniel Vetter wrote: > > > > > > > On Fri, Oct 30, 2020 at 7:18 PM Lee Jones <lee.jones@linaro.org> wrote: > > > > > > > > > > Commit 09e5b3fd5672 ("Fonts: Support FONT_EXTRA_WORDS macros for > > > > > built-in fonts") introduced the following error when building > > > > > rpc_defconfig (only this build appears to be affected): > > > > > > > > > > `acorndata_8x8' referenced in section `.text' of arch/arm/boot/compressed/ll_char_wr.o: > > > > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > > > > `acorndata_8x8' referenced in section `.data.rel.ro' of arch/arm/boot/compressed/font.o: > > > > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > > > > make[3]: *** [/scratch/linux/arch/arm/boot/compressed/Makefile:191: arch/arm/boot/compressed/vmlinux] Error 1 > > > > > make[2]: *** [/scratch/linux/arch/arm/boot/Makefile:61: arch/arm/boot/compressed/vmlinux] Error 2 > > > > > make[1]: *** [/scratch/linux/arch/arm/Makefile:317: zImage] Error 2 > > > > > > > > > > The .data section is discarded at link time. Reinstating > > > > > acorndata_8x8 as const ensures it is still available after linking. > > > > > > > > > > Cc: <stable@vger.kernel.org> > > > > > Cc: Russell King <linux@armlinux.org.uk> > > > > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > > > > > > > > Shouldn't we add the const to all of them, for consistency? > > > > > > The thought did cross my mind. However, I do not see any further > > > issues which need addressing. Nor do I have any visibility into what > > > issues may be caused by doing so. The only thing I know for sure is > > > that this patch fixes the compile error pertained to in the commit > > > message, and I'd like for this fix to be as atomic as possible, as > > > it's designed to be routed through the Stable/LTS trees. > > > > The trouble is that if we only make one of them const, then it'll take > > so much longer to hit any issues due to code not handling this > > correctly. Being consistent with all fonts sounds like the best > > approach. > > > > And the original patch that lost the const for the additional data > > also went through cc: stable for all fonts together. So that shouldn't > > be the hold-up. > > My plan was to keep the fix as simple as possible. > > This is only an issue due to the odd handling of the compressed Arm > image which exclusively references 'acorndata_8x8' and discards it's > .data section. > > I am happy to go with the majority on this though. > > Does anyone else have an opinion? Oh I don't want to hold up the fix, I'm just semi desperately looking for people who care beyond "this is the most minimal thing for my use case" since this entire area is orphaned. With the other things fixed feel free to smash my ack onto this. Maybe Peilin is going to include the full re-cosntification in a cleanup series, dunno. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-11-02 14:50 ` Daniel Vetter @ 2020-11-02 16:17 ` Peilin Ye 2020-11-02 16:24 ` Lee Jones 0 siblings, 1 reply; 25+ messages in thread From: Peilin Ye @ 2020-11-02 16:17 UTC (permalink / raw) To: Daniel Vetter Cc: Lee Jones, Greg KH, Linux ARM, Linux Kernel Mailing List, stable, Russell King On Mon, Nov 02, 2020 at 03:50:49PM +0100, Daniel Vetter wrote: > Maybe Peilin is going to include the full re-cosntification in a > cleanup series, dunno. Sure, I will do it in a separate patch. Thank you, Peilin Ye ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-11-02 16:17 ` Peilin Ye @ 2020-11-02 16:24 ` Lee Jones 2020-11-02 16:25 ` Lee Jones 2020-11-02 16:34 ` Peilin Ye 0 siblings, 2 replies; 25+ messages in thread From: Lee Jones @ 2020-11-02 16:24 UTC (permalink / raw) To: Peilin Ye Cc: Daniel Vetter, Greg KH, Linux ARM, Linux Kernel Mailing List, stable, Russell King On Mon, 02 Nov 2020, Peilin Ye wrote: > On Mon, Nov 02, 2020 at 03:50:49PM +0100, Daniel Vetter wrote: > > Maybe Peilin is going to include the full re-cosntification in a > > cleanup series, dunno. > > Sure, I will do it in a separate patch. Are you happy to conduct a proper clean-up on top of this patch? This is currently broken in all of the LTS kernels, which I would like to have fixed post-haste. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-11-02 16:24 ` Lee Jones @ 2020-11-02 16:25 ` Lee Jones 2020-11-02 16:34 ` Peilin Ye 1 sibling, 0 replies; 25+ messages in thread From: Lee Jones @ 2020-11-02 16:25 UTC (permalink / raw) To: Peilin Ye Cc: Daniel Vetter, Greg KH, Linux ARM, Linux Kernel Mailing List, stable, Russell King On Mon, 02 Nov 2020, Lee Jones wrote: > On Mon, 02 Nov 2020, Peilin Ye wrote: > > > On Mon, Nov 02, 2020 at 03:50:49PM +0100, Daniel Vetter wrote: > > > Maybe Peilin is going to include the full re-cosntification in a > > > cleanup series, dunno. > > > > Sure, I will do it in a separate patch. > > Are you happy to conduct a proper clean-up on top of this patch? > > This is currently broken in all of the LTS kernels, which I would like > to have fixed post-haste. Of course, if it's *just* a matter of making all of the structures const again, I will do that myself and post a fix either this evening or tomorrow morning. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier 2020-11-02 16:24 ` Lee Jones 2020-11-02 16:25 ` Lee Jones @ 2020-11-02 16:34 ` Peilin Ye 1 sibling, 0 replies; 25+ messages in thread From: Peilin Ye @ 2020-11-02 16:34 UTC (permalink / raw) To: Lee Jones Cc: Daniel Vetter, Greg KH, Linux ARM, Linux Kernel Mailing List, stable, Russell King On Mon, Nov 02, 2020 at 04:24:47PM +0000, Lee Jones wrote: > On Mon, 02 Nov 2020, Peilin Ye wrote: > > > On Mon, Nov 02, 2020 at 03:50:49PM +0100, Daniel Vetter wrote: > > > Maybe Peilin is going to include the full re-cosntification in a > > > cleanup series, dunno. > > > > Sure, I will do it in a separate patch. > > Are you happy to conduct a proper clean-up on top of this patch? > > This is currently broken in all of the LTS kernels, which I would like > to have fixed post-haste. Sure I will do it now. Thank you, Peilin Ye ^ permalink raw reply [flat|nested] 25+ messages in thread
* [PATCH v2 1/1] Fonts: Replace discarded const qualifier 2020-10-30 18:18 [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier Lee Jones ` (2 preceding siblings ...) 2020-11-02 10:56 ` Daniel Vetter @ 2020-11-02 18:32 ` Peilin Ye 2020-11-03 8:53 ` Lee Jones 3 siblings, 1 reply; 25+ messages in thread From: Peilin Ye @ 2020-11-02 18:32 UTC (permalink / raw) To: lee.jones, daniel.vetter, gregkh Cc: linux, linux-arm-kernel, linux-kernel, stable, Peilin Ye From: Lee Jones <lee.jones@linaro.org> Commit 6735b4632def ("Fonts: Support FONT_EXTRA_WORDS macros for built-in fonts") introduced the following error when building rpc_defconfig (only this build appears to be affected): `acorndata_8x8' referenced in section `.text' of arch/arm/boot/compressed/ll_char_wr.o: defined in discarded section `.data' of arch/arm/boot/compressed/font.o `acorndata_8x8' referenced in section `.data.rel.ro' of arch/arm/boot/compressed/font.o: defined in discarded section `.data' of arch/arm/boot/compressed/font.o make[3]: *** [/scratch/linux/arch/arm/boot/compressed/Makefile:191: arch/arm/boot/compressed/vmlinux] Error 1 make[2]: *** [/scratch/linux/arch/arm/boot/Makefile:61: arch/arm/boot/compressed/vmlinux] Error 2 make[1]: *** [/scratch/linux/arch/arm/Makefile:317: zImage] Error 2 The .data section is discarded at link time. Reinstating acorndata_8x8 as const ensures it is still available after linking. Do the same for the other 12 built-in fonts as well, for consistency purposes. Cc: <stable@vger.kernel.org> Cc: Russell King <linux@armlinux.org.uk> Fixes: 6735b4632def ("Fonts: Support FONT_EXTRA_WORDS macros for built-in fonts") Signed-off-by: Lee Jones <lee.jones@linaro.org> Co-developed-by: Peilin Ye <yepeilin.cs@gmail.com> Signed-off-by: Peilin Ye <yepeilin.cs@gmail.com> --- Changes in v2: - Fix commit ID to 6735b4632def in commit message (Russell King <linux@armlinux.org.uk>) - Add `const` back for all 13 built-in fonts (Daniel Vetter <daniel.vetter@ffwll.ch>) - Add a Fixes: tag lib/fonts/font_10x18.c | 2 +- lib/fonts/font_6x10.c | 2 +- lib/fonts/font_6x11.c | 2 +- lib/fonts/font_6x8.c | 2 +- lib/fonts/font_7x14.c | 2 +- lib/fonts/font_8x16.c | 2 +- lib/fonts/font_8x8.c | 2 +- lib/fonts/font_acorn_8x8.c | 2 +- lib/fonts/font_mini_4x6.c | 2 +- lib/fonts/font_pearl_8x8.c | 2 +- lib/fonts/font_sun12x22.c | 2 +- lib/fonts/font_sun8x16.c | 2 +- lib/fonts/font_ter16x32.c | 2 +- 13 files changed, 13 insertions(+), 13 deletions(-) diff --git a/lib/fonts/font_10x18.c b/lib/fonts/font_10x18.c index 0e2deac97da0..e02f9df24d1e 100644 --- a/lib/fonts/font_10x18.c +++ b/lib/fonts/font_10x18.c @@ -8,7 +8,7 @@ #define FONTDATAMAX 9216 -static struct font_data fontdata_10x18 = { +static const struct font_data fontdata_10x18 = { { 0, 0, FONTDATAMAX, 0 }, { /* 0 0x00 '^@' */ 0x00, 0x00, /* 0000000000 */ diff --git a/lib/fonts/font_6x10.c b/lib/fonts/font_6x10.c index 87da8acd07db..6e3c4b7691c8 100644 --- a/lib/fonts/font_6x10.c +++ b/lib/fonts/font_6x10.c @@ -3,7 +3,7 @@ #define FONTDATAMAX 2560 -static struct font_data fontdata_6x10 = { +static const struct font_data fontdata_6x10 = { { 0, 0, FONTDATAMAX, 0 }, { /* 0 0x00 '^@' */ 0x00, /* 00000000 */ diff --git a/lib/fonts/font_6x11.c b/lib/fonts/font_6x11.c index 5e975dfa10a5..2d22a24e816f 100644 --- a/lib/fonts/font_6x11.c +++ b/lib/fonts/font_6x11.c @@ -9,7 +9,7 @@ #define FONTDATAMAX (11*256) -static struct font_data fontdata_6x11 = { +static const struct font_data fontdata_6x11 = { { 0, 0, FONTDATAMAX, 0 }, { /* 0 0x00 '^@' */ 0x00, /* 00000000 */ diff --git a/lib/fonts/font_6x8.c b/lib/fonts/font_6x8.c index 700039a9ceae..e7442a0d183d 100644 --- a/lib/fonts/font_6x8.c +++ b/lib/fonts/font_6x8.c @@ -3,7 +3,7 @@ #define FONTDATAMAX 2048 -static struct font_data fontdata_6x8 = { +static const struct font_data fontdata_6x8 = { { 0, 0, FONTDATAMAX, 0 }, { /* 0 0x00 '^@' */ 0x00, /* 000000 */ diff --git a/lib/fonts/font_7x14.c b/lib/fonts/font_7x14.c index 86d298f38505..9cc7ae2e03f7 100644 --- a/lib/fonts/font_7x14.c +++ b/lib/fonts/font_7x14.c @@ -8,7 +8,7 @@ #define FONTDATAMAX 3584 -static struct font_data fontdata_7x14 = { +static const struct font_data fontdata_7x14 = { { 0, 0, FONTDATAMAX, 0 }, { /* 0 0x00 '^@' */ 0x00, /* 0000000 */ diff --git a/lib/fonts/font_8x16.c b/lib/fonts/font_8x16.c index 37cedd36ca5e..bab25dc59e8d 100644 --- a/lib/fonts/font_8x16.c +++ b/lib/fonts/font_8x16.c @@ -10,7 +10,7 @@ #define FONTDATAMAX 4096 -static struct font_data fontdata_8x16 = { +static const struct font_data fontdata_8x16 = { { 0, 0, FONTDATAMAX, 0 }, { /* 0 0x00 '^@' */ 0x00, /* 00000000 */ diff --git a/lib/fonts/font_8x8.c b/lib/fonts/font_8x8.c index 8ab695538395..109d0572368f 100644 --- a/lib/fonts/font_8x8.c +++ b/lib/fonts/font_8x8.c @@ -9,7 +9,7 @@ #define FONTDATAMAX 2048 -static struct font_data fontdata_8x8 = { +static const struct font_data fontdata_8x8 = { { 0, 0, FONTDATAMAX, 0 }, { /* 0 0x00 '^@' */ 0x00, /* 00000000 */ diff --git a/lib/fonts/font_acorn_8x8.c b/lib/fonts/font_acorn_8x8.c index 069b3e80c434..fb395f0d4031 100644 --- a/lib/fonts/font_acorn_8x8.c +++ b/lib/fonts/font_acorn_8x8.c @@ -5,7 +5,7 @@ #define FONTDATAMAX 2048 -static struct font_data acorndata_8x8 = { +static const struct font_data acorndata_8x8 = { { 0, 0, FONTDATAMAX, 0 }, { /* 00 */ 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, /* ^@ */ /* 01 */ 0x7e, 0x81, 0xa5, 0x81, 0xbd, 0x99, 0x81, 0x7e, /* ^A */ diff --git a/lib/fonts/font_mini_4x6.c b/lib/fonts/font_mini_4x6.c index 1449876c6a27..592774a90917 100644 --- a/lib/fonts/font_mini_4x6.c +++ b/lib/fonts/font_mini_4x6.c @@ -43,7 +43,7 @@ __END__; #define FONTDATAMAX 1536 -static struct font_data fontdata_mini_4x6 = { +static const struct font_data fontdata_mini_4x6 = { { 0, 0, FONTDATAMAX, 0 }, { /*{*/ /* Char 0: ' ' */ diff --git a/lib/fonts/font_pearl_8x8.c b/lib/fonts/font_pearl_8x8.c index 32d65551e7ed..a6f95ebce950 100644 --- a/lib/fonts/font_pearl_8x8.c +++ b/lib/fonts/font_pearl_8x8.c @@ -14,7 +14,7 @@ #define FONTDATAMAX 2048 -static struct font_data fontdata_pearl8x8 = { +static const struct font_data fontdata_pearl8x8 = { { 0, 0, FONTDATAMAX, 0 }, { /* 0 0x00 '^@' */ 0x00, /* 00000000 */ diff --git a/lib/fonts/font_sun12x22.c b/lib/fonts/font_sun12x22.c index 641a6b4dca42..a5b65bd49604 100644 --- a/lib/fonts/font_sun12x22.c +++ b/lib/fonts/font_sun12x22.c @@ -3,7 +3,7 @@ #define FONTDATAMAX 11264 -static struct font_data fontdata_sun12x22 = { +static const struct font_data fontdata_sun12x22 = { { 0, 0, FONTDATAMAX, 0 }, { /* 0 0x00 '^@' */ 0x00, 0x00, /* 000000000000 */ diff --git a/lib/fonts/font_sun8x16.c b/lib/fonts/font_sun8x16.c index 193fe6d988e0..e577e76a6a7c 100644 --- a/lib/fonts/font_sun8x16.c +++ b/lib/fonts/font_sun8x16.c @@ -3,7 +3,7 @@ #define FONTDATAMAX 4096 -static struct font_data fontdata_sun8x16 = { +static const struct font_data fontdata_sun8x16 = { { 0, 0, FONTDATAMAX, 0 }, { /* */ 0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00, /* */ 0x00,0x00,0x7e,0x81,0xa5,0x81,0x81,0xbd,0x99,0x81,0x81,0x7e,0x00,0x00,0x00,0x00, diff --git a/lib/fonts/font_ter16x32.c b/lib/fonts/font_ter16x32.c index 91b9c283bd9c..f7c3abb6b99e 100644 --- a/lib/fonts/font_ter16x32.c +++ b/lib/fonts/font_ter16x32.c @@ -4,7 +4,7 @@ #define FONTDATAMAX 16384 -static struct font_data fontdata_ter16x32 = { +static const struct font_data fontdata_ter16x32 = { { 0, 0, FONTDATAMAX, 0 }, { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x7f, 0xfc, 0x7f, 0xfc, -- 2.25.1 ^ permalink raw reply related [flat|nested] 25+ messages in thread
* Re: [PATCH v2 1/1] Fonts: Replace discarded const qualifier 2020-11-02 18:32 ` [PATCH v2 1/1] Fonts: " Peilin Ye @ 2020-11-03 8:53 ` Lee Jones 2020-11-03 8:58 ` Daniel Vetter 0 siblings, 1 reply; 25+ messages in thread From: Lee Jones @ 2020-11-03 8:53 UTC (permalink / raw) To: Peilin Ye Cc: daniel.vetter, gregkh, linux, linux-arm-kernel, linux-kernel, stable On Mon, 02 Nov 2020, Peilin Ye wrote: > From: Lee Jones <lee.jones@linaro.org> > > Commit 6735b4632def ("Fonts: Support FONT_EXTRA_WORDS macros for built-in > fonts") introduced the following error when building rpc_defconfig (only > this build appears to be affected): > > `acorndata_8x8' referenced in section `.text' of arch/arm/boot/compressed/ll_char_wr.o: > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > `acorndata_8x8' referenced in section `.data.rel.ro' of arch/arm/boot/compressed/font.o: > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > make[3]: *** [/scratch/linux/arch/arm/boot/compressed/Makefile:191: arch/arm/boot/compressed/vmlinux] Error 1 > make[2]: *** [/scratch/linux/arch/arm/boot/Makefile:61: arch/arm/boot/compressed/vmlinux] Error 2 > make[1]: *** [/scratch/linux/arch/arm/Makefile:317: zImage] Error 2 > > The .data section is discarded at link time. Reinstating acorndata_8x8 as > const ensures it is still available after linking. Do the same for the > other 12 built-in fonts as well, for consistency purposes. > > Cc: <stable@vger.kernel.org> > Cc: Russell King <linux@armlinux.org.uk> > Fixes: 6735b4632def ("Fonts: Support FONT_EXTRA_WORDS macros for built-in fonts") > Signed-off-by: Lee Jones <lee.jones@linaro.org> > Co-developed-by: Peilin Ye <yepeilin.cs@gmail.com> > Signed-off-by: Peilin Ye <yepeilin.cs@gmail.com> > --- > Changes in v2: > - Fix commit ID to 6735b4632def in commit message (Russell King > <linux@armlinux.org.uk>) > - Add `const` back for all 13 built-in fonts (Daniel Vetter > <daniel.vetter@ffwll.ch>) > - Add a Fixes: tag > > lib/fonts/font_10x18.c | 2 +- > lib/fonts/font_6x10.c | 2 +- > lib/fonts/font_6x11.c | 2 +- > lib/fonts/font_6x8.c | 2 +- > lib/fonts/font_7x14.c | 2 +- > lib/fonts/font_8x16.c | 2 +- > lib/fonts/font_8x8.c | 2 +- > lib/fonts/font_acorn_8x8.c | 2 +- > lib/fonts/font_mini_4x6.c | 2 +- > lib/fonts/font_pearl_8x8.c | 2 +- > lib/fonts/font_sun12x22.c | 2 +- > lib/fonts/font_sun8x16.c | 2 +- > lib/fonts/font_ter16x32.c | 2 +- > 13 files changed, 13 insertions(+), 13 deletions(-) LGTM. Thanks for keeping my authorship. Much appreciated. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH v2 1/1] Fonts: Replace discarded const qualifier 2020-11-03 8:53 ` Lee Jones @ 2020-11-03 8:58 ` Daniel Vetter 2020-11-03 9:15 ` Greg KH 0 siblings, 1 reply; 25+ messages in thread From: Daniel Vetter @ 2020-11-03 8:58 UTC (permalink / raw) To: Lee Jones Cc: Peilin Ye, Greg KH, Russell King - ARM Linux, Linux ARM, Linux Kernel Mailing List, stable On Tue, Nov 3, 2020 at 9:53 AM Lee Jones <lee.jones@linaro.org> wrote: > > On Mon, 02 Nov 2020, Peilin Ye wrote: > > > From: Lee Jones <lee.jones@linaro.org> > > > > Commit 6735b4632def ("Fonts: Support FONT_EXTRA_WORDS macros for built-in > > fonts") introduced the following error when building rpc_defconfig (only > > this build appears to be affected): > > > > `acorndata_8x8' referenced in section `.text' of arch/arm/boot/compressed/ll_char_wr.o: > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > `acorndata_8x8' referenced in section `.data.rel.ro' of arch/arm/boot/compressed/font.o: > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > make[3]: *** [/scratch/linux/arch/arm/boot/compressed/Makefile:191: arch/arm/boot/compressed/vmlinux] Error 1 > > make[2]: *** [/scratch/linux/arch/arm/boot/Makefile:61: arch/arm/boot/compressed/vmlinux] Error 2 > > make[1]: *** [/scratch/linux/arch/arm/Makefile:317: zImage] Error 2 > > > > The .data section is discarded at link time. Reinstating acorndata_8x8 as > > const ensures it is still available after linking. Do the same for the > > other 12 built-in fonts as well, for consistency purposes. > > > > Cc: <stable@vger.kernel.org> > > Cc: Russell King <linux@armlinux.org.uk> > > Fixes: 6735b4632def ("Fonts: Support FONT_EXTRA_WORDS macros for built-in fonts") > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > > Co-developed-by: Peilin Ye <yepeilin.cs@gmail.com> > > Signed-off-by: Peilin Ye <yepeilin.cs@gmail.com> > > --- > > Changes in v2: > > - Fix commit ID to 6735b4632def in commit message (Russell King > > <linux@armlinux.org.uk>) > > - Add `const` back for all 13 built-in fonts (Daniel Vetter > > <daniel.vetter@ffwll.ch>) > > - Add a Fixes: tag > > > > lib/fonts/font_10x18.c | 2 +- > > lib/fonts/font_6x10.c | 2 +- > > lib/fonts/font_6x11.c | 2 +- > > lib/fonts/font_6x8.c | 2 +- > > lib/fonts/font_7x14.c | 2 +- > > lib/fonts/font_8x16.c | 2 +- > > lib/fonts/font_8x8.c | 2 +- > > lib/fonts/font_acorn_8x8.c | 2 +- > > lib/fonts/font_mini_4x6.c | 2 +- > > lib/fonts/font_pearl_8x8.c | 2 +- > > lib/fonts/font_sun12x22.c | 2 +- > > lib/fonts/font_sun8x16.c | 2 +- > > lib/fonts/font_ter16x32.c | 2 +- > > 13 files changed, 13 insertions(+), 13 deletions(-) > > LGTM. > > Thanks for keeping my authorship. Much appreciated. Should I stuff this into drm-misc-fixes? Or will someone else pick this up? Greg? I guess drm-misc-fixes might be easiest since there's a bunch of other fbcon/font stuff in the queue in drm-misc from Peilin. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH v2 1/1] Fonts: Replace discarded const qualifier 2020-11-03 8:58 ` Daniel Vetter @ 2020-11-03 9:15 ` Greg KH 2020-11-03 9:52 ` Daniel Vetter 0 siblings, 1 reply; 25+ messages in thread From: Greg KH @ 2020-11-03 9:15 UTC (permalink / raw) To: Daniel Vetter Cc: Lee Jones, Peilin Ye, Russell King - ARM Linux, Linux ARM, Linux Kernel Mailing List, stable On Tue, Nov 03, 2020 at 09:58:18AM +0100, Daniel Vetter wrote: > On Tue, Nov 3, 2020 at 9:53 AM Lee Jones <lee.jones@linaro.org> wrote: > > > > On Mon, 02 Nov 2020, Peilin Ye wrote: > > > > > From: Lee Jones <lee.jones@linaro.org> > > > > > > Commit 6735b4632def ("Fonts: Support FONT_EXTRA_WORDS macros for built-in > > > fonts") introduced the following error when building rpc_defconfig (only > > > this build appears to be affected): > > > > > > `acorndata_8x8' referenced in section `.text' of arch/arm/boot/compressed/ll_char_wr.o: > > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > > `acorndata_8x8' referenced in section `.data.rel.ro' of arch/arm/boot/compressed/font.o: > > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > > make[3]: *** [/scratch/linux/arch/arm/boot/compressed/Makefile:191: arch/arm/boot/compressed/vmlinux] Error 1 > > > make[2]: *** [/scratch/linux/arch/arm/boot/Makefile:61: arch/arm/boot/compressed/vmlinux] Error 2 > > > make[1]: *** [/scratch/linux/arch/arm/Makefile:317: zImage] Error 2 > > > > > > The .data section is discarded at link time. Reinstating acorndata_8x8 as > > > const ensures it is still available after linking. Do the same for the > > > other 12 built-in fonts as well, for consistency purposes. > > > > > > Cc: <stable@vger.kernel.org> > > > Cc: Russell King <linux@armlinux.org.uk> > > > Fixes: 6735b4632def ("Fonts: Support FONT_EXTRA_WORDS macros for built-in fonts") > > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > > > Co-developed-by: Peilin Ye <yepeilin.cs@gmail.com> > > > Signed-off-by: Peilin Ye <yepeilin.cs@gmail.com> > > > --- > > > Changes in v2: > > > - Fix commit ID to 6735b4632def in commit message (Russell King > > > <linux@armlinux.org.uk>) > > > - Add `const` back for all 13 built-in fonts (Daniel Vetter > > > <daniel.vetter@ffwll.ch>) > > > - Add a Fixes: tag > > > > > > lib/fonts/font_10x18.c | 2 +- > > > lib/fonts/font_6x10.c | 2 +- > > > lib/fonts/font_6x11.c | 2 +- > > > lib/fonts/font_6x8.c | 2 +- > > > lib/fonts/font_7x14.c | 2 +- > > > lib/fonts/font_8x16.c | 2 +- > > > lib/fonts/font_8x8.c | 2 +- > > > lib/fonts/font_acorn_8x8.c | 2 +- > > > lib/fonts/font_mini_4x6.c | 2 +- > > > lib/fonts/font_pearl_8x8.c | 2 +- > > > lib/fonts/font_sun12x22.c | 2 +- > > > lib/fonts/font_sun8x16.c | 2 +- > > > lib/fonts/font_ter16x32.c | 2 +- > > > 13 files changed, 13 insertions(+), 13 deletions(-) > > > > LGTM. > > > > Thanks for keeping my authorship. Much appreciated. > > Should I stuff this into drm-misc-fixes? Or will someone else pick > this up? Greg? > > I guess drm-misc-fixes might be easiest since there's a bunch of other > fbcon/font stuff in the queue in drm-misc from Peilin. You can take it: Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH v2 1/1] Fonts: Replace discarded const qualifier 2020-11-03 9:15 ` Greg KH @ 2020-11-03 9:52 ` Daniel Vetter 2020-11-03 10:55 ` Lee Jones 0 siblings, 1 reply; 25+ messages in thread From: Daniel Vetter @ 2020-11-03 9:52 UTC (permalink / raw) To: Greg KH Cc: Daniel Vetter, Lee Jones, Peilin Ye, Russell King - ARM Linux, Linux ARM, Linux Kernel Mailing List, stable On Tue, Nov 03, 2020 at 10:15:38AM +0100, Greg KH wrote: > On Tue, Nov 03, 2020 at 09:58:18AM +0100, Daniel Vetter wrote: > > On Tue, Nov 3, 2020 at 9:53 AM Lee Jones <lee.jones@linaro.org> wrote: > > > > > > On Mon, 02 Nov 2020, Peilin Ye wrote: > > > > > > > From: Lee Jones <lee.jones@linaro.org> > > > > > > > > Commit 6735b4632def ("Fonts: Support FONT_EXTRA_WORDS macros for built-in > > > > fonts") introduced the following error when building rpc_defconfig (only > > > > this build appears to be affected): > > > > > > > > `acorndata_8x8' referenced in section `.text' of arch/arm/boot/compressed/ll_char_wr.o: > > > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > > > `acorndata_8x8' referenced in section `.data.rel.ro' of arch/arm/boot/compressed/font.o: > > > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > > > make[3]: *** [/scratch/linux/arch/arm/boot/compressed/Makefile:191: arch/arm/boot/compressed/vmlinux] Error 1 > > > > make[2]: *** [/scratch/linux/arch/arm/boot/Makefile:61: arch/arm/boot/compressed/vmlinux] Error 2 > > > > make[1]: *** [/scratch/linux/arch/arm/Makefile:317: zImage] Error 2 > > > > > > > > The .data section is discarded at link time. Reinstating acorndata_8x8 as > > > > const ensures it is still available after linking. Do the same for the > > > > other 12 built-in fonts as well, for consistency purposes. > > > > > > > > Cc: <stable@vger.kernel.org> > > > > Cc: Russell King <linux@armlinux.org.uk> > > > > Fixes: 6735b4632def ("Fonts: Support FONT_EXTRA_WORDS macros for built-in fonts") > > > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > > > > Co-developed-by: Peilin Ye <yepeilin.cs@gmail.com> > > > > Signed-off-by: Peilin Ye <yepeilin.cs@gmail.com> > > > > --- > > > > Changes in v2: > > > > - Fix commit ID to 6735b4632def in commit message (Russell King > > > > <linux@armlinux.org.uk>) > > > > - Add `const` back for all 13 built-in fonts (Daniel Vetter > > > > <daniel.vetter@ffwll.ch>) > > > > - Add a Fixes: tag > > > > > > > > lib/fonts/font_10x18.c | 2 +- > > > > lib/fonts/font_6x10.c | 2 +- > > > > lib/fonts/font_6x11.c | 2 +- > > > > lib/fonts/font_6x8.c | 2 +- > > > > lib/fonts/font_7x14.c | 2 +- > > > > lib/fonts/font_8x16.c | 2 +- > > > > lib/fonts/font_8x8.c | 2 +- > > > > lib/fonts/font_acorn_8x8.c | 2 +- > > > > lib/fonts/font_mini_4x6.c | 2 +- > > > > lib/fonts/font_pearl_8x8.c | 2 +- > > > > lib/fonts/font_sun12x22.c | 2 +- > > > > lib/fonts/font_sun8x16.c | 2 +- > > > > lib/fonts/font_ter16x32.c | 2 +- > > > > 13 files changed, 13 insertions(+), 13 deletions(-) > > > > > > LGTM. > > > > > > Thanks for keeping my authorship. Much appreciated. > > > > Should I stuff this into drm-misc-fixes? Or will someone else pick > > this up? Greg? > > > > I guess drm-misc-fixes might be easiest since there's a bunch of other > > fbcon/font stuff in the queue in drm-misc from Peilin. > > You can take it: > > Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Applied to drm-misc-fixes, thanks everyone! Cheers, Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH v2 1/1] Fonts: Replace discarded const qualifier 2020-11-03 9:52 ` Daniel Vetter @ 2020-11-03 10:55 ` Lee Jones 2020-11-03 11:42 ` Peilin Ye 2020-11-07 5:19 ` Peilin Ye 0 siblings, 2 replies; 25+ messages in thread From: Lee Jones @ 2020-11-03 10:55 UTC (permalink / raw) To: Greg KH, Peilin Ye, Russell King - ARM Linux, Linux ARM, Linux Kernel Mailing List, stable On Tue, 03 Nov 2020, Daniel Vetter wrote: > On Tue, Nov 03, 2020 at 10:15:38AM +0100, Greg KH wrote: > > On Tue, Nov 03, 2020 at 09:58:18AM +0100, Daniel Vetter wrote: > > > On Tue, Nov 3, 2020 at 9:53 AM Lee Jones <lee.jones@linaro.org> wrote: > > > > > > > > On Mon, 02 Nov 2020, Peilin Ye wrote: > > > > > > > > > From: Lee Jones <lee.jones@linaro.org> > > > > > > > > > > Commit 6735b4632def ("Fonts: Support FONT_EXTRA_WORDS macros for built-in > > > > > fonts") introduced the following error when building rpc_defconfig (only > > > > > this build appears to be affected): > > > > > > > > > > `acorndata_8x8' referenced in section `.text' of arch/arm/boot/compressed/ll_char_wr.o: > > > > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > > > > `acorndata_8x8' referenced in section `.data.rel.ro' of arch/arm/boot/compressed/font.o: > > > > > defined in discarded section `.data' of arch/arm/boot/compressed/font.o > > > > > make[3]: *** [/scratch/linux/arch/arm/boot/compressed/Makefile:191: arch/arm/boot/compressed/vmlinux] Error 1 > > > > > make[2]: *** [/scratch/linux/arch/arm/boot/Makefile:61: arch/arm/boot/compressed/vmlinux] Error 2 > > > > > make[1]: *** [/scratch/linux/arch/arm/Makefile:317: zImage] Error 2 > > > > > > > > > > The .data section is discarded at link time. Reinstating acorndata_8x8 as > > > > > const ensures it is still available after linking. Do the same for the > > > > > other 12 built-in fonts as well, for consistency purposes. > > > > > > > > > > Cc: <stable@vger.kernel.org> > > > > > Cc: Russell King <linux@armlinux.org.uk> > > > > > Fixes: 6735b4632def ("Fonts: Support FONT_EXTRA_WORDS macros for built-in fonts") > > > > > Signed-off-by: Lee Jones <lee.jones@linaro.org> > > > > > Co-developed-by: Peilin Ye <yepeilin.cs@gmail.com> > > > > > Signed-off-by: Peilin Ye <yepeilin.cs@gmail.com> > > > > > --- > > > > > Changes in v2: > > > > > - Fix commit ID to 6735b4632def in commit message (Russell King > > > > > <linux@armlinux.org.uk>) > > > > > - Add `const` back for all 13 built-in fonts (Daniel Vetter > > > > > <daniel.vetter@ffwll.ch>) > > > > > - Add a Fixes: tag > > > > > > > > > > lib/fonts/font_10x18.c | 2 +- > > > > > lib/fonts/font_6x10.c | 2 +- > > > > > lib/fonts/font_6x11.c | 2 +- > > > > > lib/fonts/font_6x8.c | 2 +- > > > > > lib/fonts/font_7x14.c | 2 +- > > > > > lib/fonts/font_8x16.c | 2 +- > > > > > lib/fonts/font_8x8.c | 2 +- > > > > > lib/fonts/font_acorn_8x8.c | 2 +- > > > > > lib/fonts/font_mini_4x6.c | 2 +- > > > > > lib/fonts/font_pearl_8x8.c | 2 +- > > > > > lib/fonts/font_sun12x22.c | 2 +- > > > > > lib/fonts/font_sun8x16.c | 2 +- > > > > > lib/fonts/font_ter16x32.c | 2 +- > > > > > 13 files changed, 13 insertions(+), 13 deletions(-) > > > > > > > > LGTM. > > > > > > > > Thanks for keeping my authorship. Much appreciated. > > > > > > Should I stuff this into drm-misc-fixes? Or will someone else pick > > > this up? Greg? > > > > > > I guess drm-misc-fixes might be easiest since there's a bunch of other > > > fbcon/font stuff in the queue in drm-misc from Peilin. > > > > You can take it: > > > > Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> > > Applied to drm-misc-fixes, thanks everyone! Would you be kind enough to let us know when this lands in Mainline please? We'll need to back-port it to start fixing up our Stable kernels ASAP. TIA. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH v2 1/1] Fonts: Replace discarded const qualifier 2020-11-03 10:55 ` Lee Jones @ 2020-11-03 11:42 ` Peilin Ye 2020-11-07 5:19 ` Peilin Ye 1 sibling, 0 replies; 25+ messages in thread From: Peilin Ye @ 2020-11-03 11:42 UTC (permalink / raw) To: Lee Jones Cc: Greg KH, Russell King - ARM Linux, Linux ARM, Linux Kernel Mailing List, stable On Tue, Nov 03, 2020 at 10:55:23AM +0000, Lee Jones wrote: > Would you be kind enough to let us know when this lands in Mainline > please? We'll need to back-port it to start fixing up our Stable > kernels ASAP. Sure, I will keep track of it, and update here when it happens. Thank you, and sorry again for the trouble, Peilin Ye ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH v2 1/1] Fonts: Replace discarded const qualifier 2020-11-03 10:55 ` Lee Jones 2020-11-03 11:42 ` Peilin Ye @ 2020-11-07 5:19 ` Peilin Ye 2020-11-07 15:36 ` Greg KH 1 sibling, 1 reply; 25+ messages in thread From: Peilin Ye @ 2020-11-07 5:19 UTC (permalink / raw) To: Lee Jones Cc: Greg KH, Russell King - ARM Linux, Linux ARM, Linux Kernel Mailing List, stable Hi all, On Tue, Nov 03, 2020 at 10:55:23AM +0000, Lee Jones wrote: > Would you be kind enough to let us know when this lands in Mainline > please? We'll need to back-port it to start fixing up our Stable > kernels ASAP. Patch is in mainline now: 9522750c66c689b739e151fcdf895420dc81efc0 Fonts: Replace discarded const qualifier Thank you, Peilin Ye ^ permalink raw reply [flat|nested] 25+ messages in thread
* Re: [PATCH v2 1/1] Fonts: Replace discarded const qualifier 2020-11-07 5:19 ` Peilin Ye @ 2020-11-07 15:36 ` Greg KH 0 siblings, 0 replies; 25+ messages in thread From: Greg KH @ 2020-11-07 15:36 UTC (permalink / raw) To: Peilin Ye Cc: Lee Jones, Russell King - ARM Linux, Linux ARM, Linux Kernel Mailing List, stable On Sat, Nov 07, 2020 at 12:19:18AM -0500, Peilin Ye wrote: > Hi all, > > On Tue, Nov 03, 2020 at 10:55:23AM +0000, Lee Jones wrote: > > Would you be kind enough to let us know when this lands in Mainline > > please? We'll need to back-port it to start fixing up our Stable > > kernels ASAP. > > Patch is in mainline now: > > 9522750c66c689b739e151fcdf895420dc81efc0 Fonts: Replace discarded const qualifier Now queued up for stable, thanks! greg k-h ^ permalink raw reply [flat|nested] 25+ messages in thread
end of thread, other threads:[~2020-11-07 15:35 UTC | newest] Thread overview: 25+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-10-30 18:18 [PATCH 1/1] Fonts: font_acorn_8x8: Replace discarded const qualifier Lee Jones 2020-10-31 5:09 ` Peilin Ye 2020-10-31 10:27 ` Russell King - ARM Linux admin 2020-11-01 13:11 ` Lee Jones 2020-11-02 10:23 ` Russell King - ARM Linux admin 2020-11-02 10:30 ` Russell King - ARM Linux admin 2020-11-02 16:12 ` Peilin Ye 2020-11-02 10:56 ` Daniel Vetter 2020-11-02 11:09 ` Lee Jones 2020-11-02 11:18 ` Daniel Vetter 2020-11-02 11:30 ` Lee Jones 2020-11-02 14:50 ` Daniel Vetter 2020-11-02 16:17 ` Peilin Ye 2020-11-02 16:24 ` Lee Jones 2020-11-02 16:25 ` Lee Jones 2020-11-02 16:34 ` Peilin Ye 2020-11-02 18:32 ` [PATCH v2 1/1] Fonts: " Peilin Ye 2020-11-03 8:53 ` Lee Jones 2020-11-03 8:58 ` Daniel Vetter 2020-11-03 9:15 ` Greg KH 2020-11-03 9:52 ` Daniel Vetter 2020-11-03 10:55 ` Lee Jones 2020-11-03 11:42 ` Peilin Ye 2020-11-07 5:19 ` Peilin Ye 2020-11-07 15:36 ` Greg KH
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).