On 09/14/15 22:07, Sedat Dilek wrote:
On Tue, Sep 15, 2015 at 5:40 AM, Behan Webster
wrote:
I haven't upstreamed this patch yet because we are still characterizing its
effects on other architectures.
Then embed this information into the changelog of *your* patch.
It's in our tree/build system
either.
NAK
Behan
On 09/14/15 21:19, Sedat Dilek wrote:
Based on a patch of Behan Webster (see [1]).
CLANG (here: v3.7) requires '-Oz' as OptLevel to be set.
A Linux v4.3-rc1 kernel built fine with CONFIG_CC_OPTIMIZE_FOR_SIZE=y
and boots on bare metal.
This is a Ubuntu/precise AMD64 system
On 09/14/15 22:07, Sedat Dilek wrote:
On Tue, Sep 15, 2015 at 5:40 AM, Behan Webster
<beh...@converseincode.com> wrote:
I haven't upstreamed this patch yet because we are still characterizing its
effects on other architectures.
Then embed this information into the changelog of *your*
either.
NAK
Behan
On 09/14/15 21:19, Sedat Dilek wrote:
Based on a patch of Behan Webster (see [1]).
CLANG (here: v3.7) requires '-Oz' as OptLevel to be set.
A Linux v4.3-rc1 kernel built fine with CONFIG_CC_OPTIMIZE_FOR_SIZE=y
and boots on bare metal.
This is a Ubuntu/precise AMD64 system
lang.
Solve this by using an deferred variable. This costs us a few shell
invocations, but this is only a constant number.
Reported-by: Behan Webster
Reported-by: Anton Blanchard
Signed-off-by: Michal Marek
---
Makefile | 9 +
arch/powerpc/Makefile | 8 --
. This costs us a few shell
invocations, but this is only a constant number.
Reported-by: Behan Webster beh...@converseincode.com
Reported-by: Anton Blanchard an...@samba.org
Signed-off-by: Michal Marek mma...@suse.com
---
Makefile | 9 +
arch/powerpc/Makefile | 8
he include?
I'm not sure that being able to define the default compiler per arch is
necessary. But I know I need to be able to add arch specific flags for
clang.
Behan
--
Behan Webster
beh...@converseincode.com <mailto:beh...@converseincode.com>
--
To unsubscribe from this list: send the lin
compiler per arch is
necessary. But I know I need to be able to add arch specific flags for
clang.
Behan
--
Behan Webster
beh...@converseincode.com mailto:beh...@converseincode.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord
On 01/28/15 22:42, David Miller wrote:
> From: Behan Webster
> Date: Wed, 28 Jan 2015 17:36:14 -0800
>
>> Missing MODULE_DEVICE_TABLE for pci ids from benet driver found by clang.
>>
>> Signed-off-by: Behan Webster
>> Suggested-by: Arnd Bergmann
> W
On 01/29/15 01:10, Arnd Bergmann wrote:
> On Wednesday 28 January 2015 22:42:28 David Miller wrote:
>> From: Behan Webster
>> Date: Wed, 28 Jan 2015 17:36:14 -0800
>>
>>> Missing MODULE_DEVICE_TABLE for pci ids from benet driver found by clang.
>>>
>>
On 01/28/15 22:42, David Miller wrote:
From: Behan Webster beh...@converseincode.com
Date: Wed, 28 Jan 2015 17:36:14 -0800
Missing MODULE_DEVICE_TABLE for pci ids from benet driver found by clang.
Signed-off-by: Behan Webster beh...@converseincode.com
Suggested-by: Arnd Bergmann
On 01/29/15 01:10, Arnd Bergmann wrote:
On Wednesday 28 January 2015 22:42:28 David Miller wrote:
From: Behan Webster beh...@converseincode.com
Date: Wed, 28 Jan 2015 17:36:14 -0800
Missing MODULE_DEVICE_TABLE for pci ids from benet driver found by clang.
Signed-off-by: Behan Webster beh
Missing MODULE_DEVICE_TABLE for pci ids from benet driver found by clang.
Signed-off-by: Behan Webster
Suggested-by: Arnd Bergmann
Cc: Arnd Bergmann
---
drivers/net/ethernet/emulex/benet/be_main.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/net/ethernet/emulex/benet/be_main.c
Missing MODULE_DEVICE_TABLE for pci ids from be2iscsi driver found by clang.
Signed-off-by: Behan Webster
Reviewed-by: Mark Charlebois
Suggested-by: Arnd Bergmann
Cc: Arnd Bergmann
---
drivers/scsi/be2iscsi/be_main.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/scsi/be2iscsi
e:
>>>>> On 28 January 2015 at 17:08, Alex Elder wrote:
>>>>>> On 01/28/2015 10:17 AM, Ard Biesheuvel wrote:
>>>>>>> On 28 January 2015 at 14:11, Alex Elder wrote:
>>>>>>>> On 01/28/2015 05:15 AM, A
gt;>>> On 01/28/2015 10:17 AM, Ard Biesheuvel wrote:
>>>>>> On 28 January 2015 at 14:11, Alex Elder wrote:
>>>>>>> On 01/28/2015 05:15 AM, Ard Biesheuvel wrote:
>>>>>>>> On 28 January 2015 at 05:18, Behan Webster
>
t;> On 01/28/2015 05:15 AM, Ard Biesheuvel wrote:
>>>>>> On 28 January 2015 at 05:18, Behan Webster
>>>>>> wrote:
>>>>>>> From: Alex Elder
>>>>>>>
>>>>>>> My GCC-based build environment likes to
:
On 01/28/2015 05:15 AM, Ard Biesheuvel wrote:
On 28 January 2015 at 05:18, Behan Webster beh...@converseincode.com
wrote:
From: Alex Elder el...@linaro.org
My GCC-based build environment likes to call register r12 by the
name ip in inline asm. Behan Webster informed me that his Clang-
based
:
On 01/28/2015 10:17 AM, Ard Biesheuvel wrote:
On 28 January 2015 at 14:11, Alex Elder el...@linaro.org wrote:
On 01/28/2015 05:15 AM, Ard Biesheuvel wrote:
On 28 January 2015 at 05:18, Behan Webster beh...@converseincode.com
wrote:
From: Alex Elder el...@linaro.org
My GCC-based build
Missing MODULE_DEVICE_TABLE for pci ids from be2iscsi driver found by clang.
Signed-off-by: Behan Webster beh...@converseincode.com
Reviewed-by: Mark Charlebois charl...@gmail.com
Suggested-by: Arnd Bergmann a...@arndb.de
Cc: Arnd Bergmann a...@arndb.de
---
drivers/scsi/be2iscsi/be_main.c | 1
Missing MODULE_DEVICE_TABLE for pci ids from benet driver found by clang.
Signed-off-by: Behan Webster beh...@converseincode.com
Suggested-by: Arnd Bergmann a...@arndb.de
Cc: Arnd Bergmann a...@arndb.de
---
drivers/net/ethernet/emulex/benet/be_main.c | 1 -
1 file changed, 1 deletion(-)
diff
...@linaro.org
wrote:
On 28 January 2015 at 17:08, Alex Elder el...@linaro.org wrote:
On 01/28/2015 10:17 AM, Ard Biesheuvel wrote:
On 28 January 2015 at 14:11, Alex Elder el...@linaro.org wrote:
On 01/28/2015 05:15 AM, Ard Biesheuvel wrote:
On 28 January 2015 at 05:18, Behan Webster
beh
clang warns if inline functions aren't used. By making them __maybe_unused
there is no warning for either gcc nor clang.
Signed-off-by: Behan Webster
Suggested-by: Arnd Bergmann
Cc: Arnd Bergmann
Cc: "Christopher Li"
---
include/linux/compiler-gcc.h | 12 ++--
1 file
From: Alex Elder
My GCC-based build environment likes to call register r12 by the
name "ip" in inline asm. Behan Webster informed me that his Clang-
based build environment likes "r12" instead.
Try to make them both happy.
Signed-off-by: Alex Elder
Signed-off-by: Behan We
From: Alex Elder el...@linaro.org
My GCC-based build environment likes to call register r12 by the
name ip in inline asm. Behan Webster informed me that his Clang-
based build environment likes r12 instead.
Try to make them both happy.
Signed-off-by: Alex Elder el...@linaro.org
Signed-off
clang warns if inline functions aren't used. By making them __maybe_unused
there is no warning for either gcc nor clang.
Signed-off-by: Behan Webster beh...@converseincode.com
Suggested-by: Arnd Bergmann a...@arndb.de
Cc: Arnd Bergmann a...@arndb.de
Cc: Christopher Li spa...@chrisli.org
AironetIeOui[] = {0x00, 0x01, 0x66};
^
The solution is making them "static const".
However doing so requires dropping const when being used with struct
octet_string. However the value is used in a const fashion thereafter, so no
harm done.
Signed-off-by: Behan Webster
Suggested-by: Arnd Be
rtllib_probe_req is defined as "static inline" in rtllib_softmac.c however it
is declared differently as "extern inline" in rtllib_softmac.h. Since it isn't
used outside of the scope of rtllib_softmac, it makes sense to remove the
incorrect declaration.
Signed-off-by: Beha
and defining the rtllib_softmac
version of this routine static fixes the warning.
Signed-off-by: Behan Webster
Suggested-by: Arnd Bergmann
Cc: Arnd Bergmann
---
drivers/staging/rtl8192e/rtllib.h | 1 -
drivers/staging/rtl8192e/rtllib_softmac.c | 2 +-
2 files changed, 1 insertion(+), 2
Removing a number of warnings generated from compiling stl8192e with clang.
The LLVMLinux project aims to fully build the Linux kernel using both gcc and
clang (the C front end for the LLVM compiler infrastructure project).
Behan Webster (4):
staging, rtl8192e, LLVMLinux: Change extern inline
ses instead.
Signed-off-by: Behan Webster
Suggested-by: Arnd Bergmann
Cc: Arnd Bergmann
---
drivers/staging/rtl8192e/rtllib.h | 4 ++--
drivers/staging/rtl8192e/rtllib_softmac.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rtl8192e/rtllib.
c and clang.
Signed-off-by: Behan Webster
Suggested-by: Arnd Bergmann
Cc: Arnd Bergmann
---
drivers/staging/rts5208/rtsx_transport.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rts5208/rtsx_transport.h
b/drivers/staging/rts5208/rtsx_transport.h
index b4b
-by: Behan Webster beh...@converseincode.com
Suggested-by: Arnd Bergmann a...@arndb.de
Cc: Arnd Bergmann a...@arndb.de
---
drivers/staging/rts5208/rtsx_transport.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/rts5208/rtsx_transport.h
b/drivers/staging/rts5208
-by: Behan Webster beh...@converseincode.com
Suggested-by: Arnd Bergmann a...@arndb.de
Cc: Arnd Bergmann a...@arndb.de
---
drivers/staging/rtl8192e/rtllib.h | 4 ++--
drivers/staging/rtl8192e/rtllib_softmac.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers
Removing a number of warnings generated from compiling stl8192e with clang.
The LLVMLinux project aims to fully build the Linux kernel using both gcc and
clang (the C front end for the LLVM compiler infrastructure project).
Behan Webster (4):
staging, rtl8192e, LLVMLinux: Change extern inline
and defining the rtllib_softmac
version of this routine static fixes the warning.
Signed-off-by: Behan Webster beh...@converseincode.com
Suggested-by: Arnd Bergmann a...@arndb.de
Cc: Arnd Bergmann a...@arndb.de
---
drivers/staging/rtl8192e/rtllib.h | 1 -
drivers/staging/rtl8192e
rtllib_probe_req is defined as static inline in rtllib_softmac.c however it
is declared differently as extern inline in rtllib_softmac.h. Since it isn't
used outside of the scope of rtllib_softmac, it makes sense to remove the
incorrect declaration.
Signed-off-by: Behan Webster beh
AironetIeOui[] = {0x00, 0x01, 0x66};
^
The solution is making them static const.
However doing so requires dropping const when being used with struct
octet_string. However the value is used in a const fashion thereafter, so no
harm done.
Signed-off-by: Behan Webster beh...@converseincode.com
to 4c5c30249452aaebf258751ea4222eba3dd3da4c:
crypto: LLVMLinux: Remove VLAIS usage from crypto/testmgr.c
(2014-10-14 10:51:24 +0200)
LLVMLinux patches for v3.18
Behan Webster (6
to 4c5c30249452aaebf258751ea4222eba3dd3da4c:
crypto: LLVMLinux: Remove VLAIS usage from crypto/testmgr.c
(2014-10-14 10:51:24 +0200)
LLVMLinux patches for v3.18
Behan Webster (6
On 09/27/14 09:46, Felipe Balbi wrote:
On Fri, Sep 26, 2014 at 06:10:52PM -0700, Behan Webster wrote:
Replace the use of nested functions where a normal function will suffice.
Nested functions are not liked by upstream kernel developers in general. Their
use breaks the use of clang
On 09/27/14 09:46, Felipe Balbi wrote:
On Fri, Sep 26, 2014 at 06:10:52PM -0700, Behan Webster wrote:
Replace the use of nested functions where a normal function will suffice.
Nested functions are not liked by upstream kernel developers in general. Their
use breaks the use of clang
-by: Behan Webster
Suggested-by: Arnd Bergmann
Cc: Arnd Bergmann
---
drivers/media/platform/ti-vpe/csc.c | 8 ++--
drivers/media/platform/ti-vpe/sc.c | 8 ++--
2 files changed, 4 insertions(+), 12 deletions(-)
diff --git a/drivers/media/platform/ti-vpe/csc.c
b/drivers/media/platform/ti-vpe
-by: Behan Webster
Suggested-by: Arnd Bergmann
Cc: Arnd Bergmann
---
drivers/video/fbdev/omap2/omapfb/omapfb-main.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/video/fbdev/omap2/omapfb/omapfb-main.c
b/drivers/video/fbdev/omap2/omapfb/omapfb-main.c
index
project aims to fully build the Linux kernel using both gcc and
clang (the C front end for the LLVM compiler infrastructure project).
Behan Webster (2):
arm, fbdev, omap2, LLVMLinux: Remove nested function from omap2 dss
arm, fbdev, omap2, LLVMLinux: Remove nested function from omapfb
drivers
-by: Behan Webster
Suggested-by: Arnd Bergmann
Cc: Arnd Bergmann
---
drivers/video/fbdev/omap2/dss/dispc-compat.c | 9 +
drivers/video/fbdev/omap2/dss/manager-sysfs.c | 16 +---
2 files changed, 14 insertions(+), 11 deletions(-)
diff --git a/drivers/video/fbdev/omap2/dss/dispc
-by: Behan Webster
Suggested-by: Arnd Bergmann
Cc: Arnd Bergmann
---
drivers/md/bcache/sysfs.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/drivers/md/bcache/sysfs.c b/drivers/md/bcache/sysfs.c
index b3ff57d..53d8baa 100644
--- a/drivers/md/bcache/sysfs.c
+++ b/drivers
On 09/26/14 17:55, Felipe Balbi wrote:
On Fri, Sep 26, 2014 at 05:31:48PM -0700, Behan Webster wrote:
As written, the __init for ti_clk_get_div_table is in the middle of the return
type.
The gcc documentation indicates that section attributes should be added to the
end of the function
ts section attributes between the
return type and function name. In the case where the return type is a pointer,
it's important to place the '*' on left of the __init.
This updated code works for both gcc and clang.
Signed-off-by: Behan Webster
Reviewed-by: Mark Charlebois
---
drivers/clk/ti/div
attributes between the
return type and function name. In the case where the return type is a pointer,
it's important to place the '*' on left of the __init.
This updated code works for both gcc and clang.
Signed-off-by: Behan Webster beh...@converseincode.com
Reviewed-by: Mark Charlebois charl
On 09/26/14 17:55, Felipe Balbi wrote:
On Fri, Sep 26, 2014 at 05:31:48PM -0700, Behan Webster wrote:
As written, the __init for ti_clk_get_div_table is in the middle of the return
type.
The gcc documentation indicates that section attributes should be added to the
end of the function
-by: Behan Webster beh...@converseincode.com
Suggested-by: Arnd Bergmann a...@arndb.de
Cc: Arnd Bergmann a...@arndb.de
---
drivers/md/bcache/sysfs.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/drivers/md/bcache/sysfs.c b/drivers/md/bcache/sysfs.c
index b3ff57d..53d8baa
-by: Behan Webster beh...@converseincode.com
Suggested-by: Arnd Bergmann a...@arndb.de
Cc: Arnd Bergmann a...@arndb.de
---
drivers/video/fbdev/omap2/dss/dispc-compat.c | 9 +
drivers/video/fbdev/omap2/dss/manager-sysfs.c | 16 +---
2 files changed, 14 insertions(+), 11 deletions
-by: Behan Webster beh...@converseincode.com
Suggested-by: Arnd Bergmann a...@arndb.de
Cc: Arnd Bergmann a...@arndb.de
---
drivers/video/fbdev/omap2/omapfb/omapfb-main.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/video/fbdev/omap2/omapfb/omapfb-main.c
b
project aims to fully build the Linux kernel using both gcc and
clang (the C front end for the LLVM compiler infrastructure project).
Behan Webster (2):
arm, fbdev, omap2, LLVMLinux: Remove nested function from omap2 dss
arm, fbdev, omap2, LLVMLinux: Remove nested function from omapfb
drivers
-by: Behan Webster beh...@converseincode.com
Suggested-by: Arnd Bergmann a...@arndb.de
Cc: Arnd Bergmann a...@arndb.de
---
drivers/media/platform/ti-vpe/csc.c | 8 ++--
drivers/media/platform/ti-vpe/sc.c | 8 ++--
2 files changed, 4 insertions(+), 12 deletions(-)
diff --git a/drivers/media
On 09/25/14 06:34, Michal Marek wrote:
On 2014-09-24 20:50, Behan Webster wrote:
Getting clang to error on unused flags wasn't trivial (this change broke
a lot of builds apparently). Fortunately we weren't the only ones who
wanted it to behave like gcc in this case. I think it's going
The __initconst is in the wrong place, and when moved to the correct place
it uncovers an error where the variable is used by non-init data structures.
Instead merely make them const and put the const in the right spot.
Signed-off-by: Behan Webster
Reviewed-by: Mark Charlebois
Acked-by: Arnd
The __initconst is in the wrong place, and when moved to the correct place
it uncovers an error where the variable is used by non-init data structures.
Instead merely make them const and put the const in the right spot.
Signed-off-by: Behan Webster beh...@converseincode.com
Reviewed-by: Mark
On 09/25/14 06:34, Michal Marek wrote:
On 2014-09-24 20:50, Behan Webster wrote:
Getting clang to error on unused flags wasn't trivial (this change broke
a lot of builds apparently). Fortunately we weren't the only ones who
wanted it to behave like gcc in this case. I think it's going
ut it would not work.
Is there any tips I am missing here?
Try building from our kernel repo listed on http://llvm.linuxfoundation.org
Thanks,
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
to fix these sorts of things in
clang, but some changes are more likely than others.
Thanks,
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo inf
build test robot
indicates that this patch breaks MIPS. So indeed your concern is justified.
Gotta love the kbuild test robot. Thank you Fengguang Wu!
If this makes it work on ARM, what about other architectures?
For now, I think I need to try something else.
Thanks,
Behan
--
Behan Webster
ou like me to respin the "gpio, bcm-kona,
LLVMLinux: Remove use of __initconst" patch as well?
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More m
On 09/24/14 02:16, Arnd Bergmann wrote:
On Tuesday 23 September 2014 20:44:44 Behan Webster wrote:
The ASM below does not compile with clang and is not the way that the mcr
command is used in other parts of the kernel.
arch/arm/mach-vt8500/vt8500.c:72:11: error: invalid % escape in inline
On 09/24/14 02:16, Arnd Bergmann wrote:
On Tuesday 23 September 2014 20:44:44 Behan Webster wrote:
The ASM below does not compile with clang and is not the way that the mcr
command is used in other parts of the kernel.
arch/arm/mach-vt8500/vt8500.c:72:11: error: invalid % escape in inline
, bcm-kona,
LLVMLinux: Remove use of __initconst patch as well?
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo
is justified.
Gotta love the kbuild test robot. Thank you Fengguang Wu!
If this makes it work on ARM, what about other architectures?
For now, I think I need to try something else.
Thanks,
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this list: send the line
is pushing hard to fix these sorts of things in
clang, but some changes are more likely than others.
Thanks,
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More
tips I am missing here?
Try building from our kernel repo listed on http://llvm.linuxfoundation.org
Thanks,
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More
From: Mark Charlebois
cmd in COMPATIBLE_IOCTL is always a u32, so cast it so there isn't a warning
about an overflow in XFORM.
Signed-off-by: Mark Charlebois
Signed-off-by: Behan Webster
Acked-by: Arnd Bergmann
---
fs/compat_ioctl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
: : "r" (0));
~^~~~
1 error generated.
There are other forms that are supported on different ARM instruction sets but
generally the kernel just uses mcr as it is supported in all ARM instruction
sets.
Signed-off-by: Behan Webster
Reviewed-by: Mark Charlebois
Acked-by: Will Deacon
---
On 09/23/14 14:29, Matt Porter wrote:
On Tue, Sep 23, 2014 at 12:30:16PM -0700, beh...@converseincode.com wrote:
From: Behan Webster
The __initconst is in the wrong place, and when moved to the correct place
it uncovers an error where the variable is used by non-init data structures.
Instead
On 09/23/14 14:29, Matt Porter wrote:
On Tue, Sep 23, 2014 at 12:30:16PM -0700, beh...@converseincode.com wrote:
From: Behan Webster beh...@converseincode.com
The __initconst is in the wrong place, and when moved to the correct place
it uncovers an error where the variable is used by non-init
));
~^~~~
1 error generated.
There are other forms that are supported on different ARM instruction sets but
generally the kernel just uses mcr as it is supported in all ARM instruction
sets.
Signed-off-by: Behan Webster beh...@converseincode.com
Reviewed-by: Mark Charlebois charl...@gmail.com
Acked
From: Mark Charlebois charl...@gmail.com
cmd in COMPATIBLE_IOCTL is always a u32, so cast it so there isn't a warning
about an overflow in XFORM.
Signed-off-by: Mark Charlebois charl...@gmail.com
Signed-off-by: Behan Webster beh...@converseincode.com
Acked-by: Arnd Bergmann a...@arndb.de
---
fs
On 09/17/14 04:30, Herbert Xu wrote:
On Wed, Sep 17, 2014 at 02:15:40PM +0300, Dmitry Kasatkin wrote:
On 17/09/14 12:22, Herbert Xu wrote:
On Mon, Sep 15, 2014 at 12:30:23AM -0700, beh...@converseincode.com wrote:
From: Behan Webster
Add a macro which replaces the use of a Variable Length
On 09/17/14 04:30, Herbert Xu wrote:
On Wed, Sep 17, 2014 at 02:15:40PM +0300, Dmitry Kasatkin wrote:
On 17/09/14 12:22, Herbert Xu wrote:
On Mon, Sep 15, 2014 at 12:30:23AM -0700, beh...@converseincode.com wrote:
From: Behan Webster beh...@converseincode.com
Add a macro which replaces
On 09/15/14 07:21, Linus Torvalds wrote:
On Mon, Sep 15, 2014 at 12:30 AM, wrote:
From: Behan Webster
Replaced the use of a Variable Length Array In Struct (VLAIS) with a C99
compliant equivalent. This patch allocates the appropriate amount of memory
using a char array using
nux-next. I actually just had to pull it out of my
own builds due to the same merge issue.
Hmm. Seems I put a couple of patches into the for-next branch late last
night which shouldn't have been. Fixed.
Apologies,
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this
; \
+ struct shash_desc *shash = (struct shash_desc *)__desc
+
char shash##__desc[] or similar? Otherwise it won't work if you use
this macro twice in the same block.
Best Regards,
Michał Mirosław
Good thinking. Will fix.
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from
)] CRYPTO_MINALIGN_ATTR; \
+ struct shash_desc *shash = (struct shash_desc *)__desc
+
char shash##__desc[] or similar? Otherwise it won't work if you use
this macro twice in the same block.
Best Regards,
Michał Mirosław
Good thinking. Will fix.
Behan
--
Behan Webster
beh...@converseincode.com
had to pull it out of my
own builds due to the same merge issue.
Hmm. Seems I put a couple of patches into the for-next branch late last
night which shouldn't have been. Fixed.
Apologies,
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this list: send the line
On 09/15/14 07:21, Linus Torvalds wrote:
On Mon, Sep 15, 2014 at 12:30 AM, beh...@converseincode.com wrote:
From: Behan Webster beh...@converseincode.com
Replaced the use of a Variable Length Array In Struct (VLAIS) with a C99
compliant equivalent. This patch allocates the appropriate amount
headers? Honestly this setup seems
really error prone.
You're not the only person to ask for this. I've got one already coded
up based on suggestions from tglx.
I've regenerated all the patches. I will resubmit soon.
Will fix.
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe
it into the crypto api headers? Honestly this setup seems
really error prone.
You're not the only person to ask for this. I've got one already coded
up based on suggestions from tglx.
I've regenerated all the patches. I will resubmit soon.
Will fix.
Behan
--
Behan Webster
beh
said, I prefer to see patches no longer required, than
needing them to be upstreamed, so in that this is a win.
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kerne
the explanation as well with the new patch set so everyone
else in other subsystems sees those reasons as well.
If this works for everyone I'll also go back and update the crypto
patches for the subsystems that have already accepted my previous patches.
Thanks,
Behan
--
Behan Webster
beh
On 09/08/14 04:15, Dmitry Kasatkin wrote:
On 07/09/14 05:06, Behan Webster wrote:
On 09/06/14 03:11, Thomas Gleixner wrote:
On Fri, 5 Sep 2014, Behan Webster wrote:
On 09/05/14 17:18, Thomas Gleixner wrote:
Signed-off-by: Behan Webster
Signed-off-by: Mark Charlebois
Signed-off-by: Jan
On 09/08/14 04:15, Dmitry Kasatkin wrote:
On 07/09/14 05:06, Behan Webster wrote:
On 09/06/14 03:11, Thomas Gleixner wrote:
On Fri, 5 Sep 2014, Behan Webster wrote:
On 09/05/14 17:18, Thomas Gleixner wrote:
Signed-off-by: Behan Webster beh...@converseincode.com
Signed-off-by: Mark Charlebois
the explanation as well with the new patch set so everyone
else in other subsystems sees those reasons as well.
If this works for everyone I'll also go back and update the crypto
patches for the subsystems that have already accepted my previous patches.
Thanks,
Behan
--
Behan Webster
beh
that step this time. My apologies as well.
That all being said, I prefer to see patches no longer required, than
needing them to be upstreamed, so in that this is a win.
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel
(AEABI). They aren't random.
Memcpy and memmove *could* might be satisfied with linker magic instead.
But memset uses the reverse parameter list.
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of
VLAIS removal patches. Happy to use macros in
this case.
Thanks,
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kern
On 09/06/14 03:11, Thomas Gleixner wrote:
On Fri, 5 Sep 2014, Behan Webster wrote:
On 09/05/14 17:18, Thomas Gleixner wrote:
Signed-off-by: Behan Webster
Signed-off-by: Mark Charlebois
Signed-off-by: Jan-Simon Möller
This SOB chain is completely ass backwards. See Documentation
On 09/06/14 03:11, Thomas Gleixner wrote:
On Fri, 5 Sep 2014, Behan Webster wrote:
On 09/05/14 17:18, Thomas Gleixner wrote:
Signed-off-by: Behan Webster beh...@converseincode.com
Signed-off-by: Mark Charlebois charl...@gmail.com
Signed-off-by: Jan-Simon Möller dl...@gmx.de
This SOB chain
-crypto VLAIS removal patches. Happy to use macros in
this case.
Thanks,
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org
(AEABI). They aren't random.
Memcpy and memmove *could* might be satisfied with linker magic instead.
But memset uses the reverse parameter list.
Behan
--
Behan Webster
beh...@converseincode.com
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message
On 09/05/14 17:18, Thomas Gleixner wrote:
On Fri, 5 Sep 2014, beh...@converseincode.com wrote:
From: Behan Webster
Replaced the use of a Variable Length Array In Struct (VLAIS) with a C99
compliant equivalent. This patch allocates the appropriate amount of memory
using an char array
On 09/05/14 17:18, Thomas Gleixner wrote:
On Fri, 5 Sep 2014, beh...@converseincode.com wrote:
From: Behan Webster beh...@converseincode.com
Replaced the use of a Variable Length Array In Struct (VLAIS) with a C99
compliant equivalent. This patch allocates the appropriate amount of memory
1 - 100 of 236 matches
Mail list logo