Re: [PATCH 1/1] powerpc: Increase COMMAND_LINE_SIZE to 2048 from 512.
On 04/15/2014 05:49 AM, Benjamin Herrenschmidt wrote: On Mon, 2014-04-14 at 14:58 -0400, Joseph Salisbury wrote: After further review, it appears ppc does not actually use the define in the ppc headers but uses the common generic default(include/uapi/asm-generic/setup.h). COMMAND_LINE_SIZE should probably become a kernel config option. Do folks agree that is the correct thing to do? If so, I can re-work the patch. No objection on my side. Make sure you remove any unused arch define while at it. Cheers, Ben. Hi Ben, I can think of two ways to add the new config option. One would be to have a large entry in ~/arch/Kconfig, with a default COMMAND_LINE_SIZE line for each architecture. The other way would be to have the default value for COMMAND_LINE_SIZE in the architecture sub-directory Kconfig file: ~/arch/powerpc/Kconfig for example. Do you have a preference for either way? Thanks, Joe ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 1/1] powerpc: Increase COMMAND_LINE_SIZE to 2048 from 512.
On Mon, 2014-04-14 at 14:58 -0400, Joseph Salisbury wrote: After further review, it appears ppc does not actually use the define in the ppc headers but uses the common generic default(include/uapi/asm-generic/setup.h). COMMAND_LINE_SIZE should probably become a kernel config option. Do folks agree that is the correct thing to do? If so, I can re-work the patch. No objection on my side. Make sure you remove any unused arch define while at it. Cheers, Ben. ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
[PATCH 1/1] powerpc: Increase COMMAND_LINE_SIZE to 2048 from 512.
BugLink: http://bugs.launchpad.net/bugs/1306677 While booting the PPC64EL kernel, the command line gets truncated to 512 characters. This is due to a hard limit of 512 defined for COMMAND_LINE_SIZE. It would be beneficial to have a command line longer than 512 characters, as iscsi targets and cloud-init parameters are passed through the kernel command line. Signed-off-by: Joseph Salisbury joseph.salisb...@canonical.com Cc: Benjamin Herrenschmidt b...@kernel.crashing.org (supporter:LINUX FOR POWERPC...) Cc: Paul Mackerras pau...@samba.org (supporter:LINUX FOR POWERPC...) Cc: linuxppc-dev@lists.ozlabs.org (open list:LINUX FOR POWERPC...) Cc: linux-ker...@vger.kernel.org (open list) Cc: sta...@vger.kernel.org --- arch/powerpc/boot/ops.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/boot/ops.h b/arch/powerpc/boot/ops.h index b3218ce..5a01fb1 100644 --- a/arch/powerpc/boot/ops.h +++ b/arch/powerpc/boot/ops.h @@ -15,7 +15,7 @@ #include types.h #include string.h -#defineCOMMAND_LINE_SIZE 512 +#defineCOMMAND_LINE_SIZE 2048 #defineMAX_PATH_LEN256 #defineMAX_PROP_LEN256 /* What should this be? */ -- 1.7.9.5 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev
Re: [PATCH 1/1] powerpc: Increase COMMAND_LINE_SIZE to 2048 from 512.
On 04/14/2014 11:30 AM, Joseph Salisbury wrote: BugLink: http://bugs.launchpad.net/bugs/1306677 While booting the PPC64EL kernel, the command line gets truncated to 512 characters. This is due to a hard limit of 512 defined for COMMAND_LINE_SIZE. It would be beneficial to have a command line longer than 512 characters, as iscsi targets and cloud-init parameters are passed through the kernel command line. Signed-off-by: Joseph Salisbury joseph.salisb...@canonical.com Cc: Benjamin Herrenschmidt b...@kernel.crashing.org (supporter:LINUX FOR POWERPC...) Cc: Paul Mackerras pau...@samba.org (supporter:LINUX FOR POWERPC...) Cc: linuxppc-dev@lists.ozlabs.org (open list:LINUX FOR POWERPC...) Cc: linux-ker...@vger.kernel.org (open list) Cc: sta...@vger.kernel.org --- arch/powerpc/boot/ops.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/boot/ops.h b/arch/powerpc/boot/ops.h index b3218ce..5a01fb1 100644 --- a/arch/powerpc/boot/ops.h +++ b/arch/powerpc/boot/ops.h @@ -15,7 +15,7 @@ #include types.h #include string.h -#define COMMAND_LINE_SIZE 512 +#define COMMAND_LINE_SIZE 2048 #define MAX_PATH_LEN256 #define MAX_PROP_LEN256 /* What should this be? */ After further review, it appears ppc does not actually use the define in the ppc headers but uses the common generic default(include/uapi/asm-generic/setup.h). COMMAND_LINE_SIZE should probably become a kernel config option. Do folks agree that is the correct thing to do? If so, I can re-work the patch. Thanks, Joe ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev