On Mon, Jan 15 2018, Dan Carpenter wrote:
> [ This code was already buggy, it's just that Neil's change made it
>show up in static analysis. - dan ]
Thanks!
This bug was introduced by
Commit: 4d99b2581eff ("staging: lustre: avoid intensive reconnecting for
ko2iblnd")
which added a "fre
Benötigen Sie Privat- oder Geschäftskredite ohne Stress und schnelle
Zustimmung? Wenn ja, kontaktieren Sie uns bitte alexgr...@gmail.com
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverde
Hi NeilBrown,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git
staging-next
head: 9fb8b101a7529a7d882df16448f20c1879535f0b
commit: 6106c0f82481e686b337ee0c403821fb5c3c17ef [468/569] staging: lustre:
lnet: convert selftest to use
From: Michael Kelley
The 2016 version of Hyper-V offers the option to operate the guest VM
per-vcpu stimer's in Direct Mode, which means the timer interupts on its
own vector rather than queueing a VMbus message. Direct Mode reduces
timer processing overhead in both the hypervisor and the guest,
From: Haiyang Zhang
Date: Fri, 19 Jan 2018 13:26:43 -0700
> From: Haiyang Zhang
>
> Since we no longer localize channel/CPU affiliation within one NUMA
> node, num_online_cpus() is used as the number of channel cap, instead of
> the number of processors in a NUMA node.
>
> This patch allows a
iio_dev->mlock is to be used only by the IIO core for protecting
device mode changes between INDIO_DIRECT and INDIO_BUFFER.
This patch replaces the use of mlock with the already established
buf_lock mutex.
Introducing 'unlocked' __ade7758_spi_write_reg_8 and
__ade7758_spi_read_reg_8 functions to
Fix warning reported by checkpatch:
CHECK: Avoid CamelCase:
By renaming dispSet to disp_set
Signed-off-by: Petr Sedlak
---
drivers/staging/sm750fb/sm750_hw.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/staging/sm750fb/sm750_hw.c
b/drivers/stag
Since commit 204f672255c2 ("staging: android: ion: Use CMA APIs directly")
the CMA API is now used directly and therefore the allocated memory is no
longer automatically zeroed.
Explicitly zero CMA allocated memory to ensure that no data is exposed to
userspace.
Fixes: 204f672255c2 ("staging: and
On Sat, 20 Jan 2018, Greg KH wrote:
> On Fri, Jan 19, 2018 at 11:16:47AM -0800, Liam Mark wrote:
> > Since the CMA API is now used directly the allocated memory is no longer
> > automatically zeroed.
> >
> > Explicitly zero CMA allocated memory to ensure that no data is exposed
> > to userspace.
On Fri, 19 Jan 2018, Dan Carpenter wrote:
> On Fri, Jan 19, 2018 at 11:16:47AM -0800, Liam Mark wrote:
> > Since the CMA API is now used directly the allocated memory is no longer
> > automatically zeroed.
> >
> > Explicitly zero CMA allocated memory to ensure that no data is exposed
> > to users
El lun, 22-01-2018 a las 17:03 +0100, Greg KH escribió:
> On Wed, Jan 17, 2018 at 08:29:21PM -0500, Christopher Díaz Riveros
> wrote:
> > Trivial fix removes unneeded semicolons after switch blocks.
> >
> > This issue was detected by using the Coccinelle software.
> >
> > Signed-off-by: Christoph
On Wed, Jan 17, 2018 at 08:29:21PM -0500, Christopher Díaz Riveros wrote:
> Trivial fix removes unneeded semicolons after switch blocks.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Christopher Díaz Riveros
> ---
> drivers/staging/rtl8723bs/hal/hal_com_phycfg
From: Colin Ian King
Variable timeout is initialized however this value is never read and
it is reassigned a new value a little later, hence the initialization
is redundant and can be removed.
Cleans up clang warning:
drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c:1728:6: warning:
Value
"Michael Kelley (EOSG)" writes:
> On Fri, 19 Jan 2018, Thomas Gleixner wrote:
>
>> -Original Message-
>> From: Thomas Gleixner [mailto:t...@linutronix.de]
>> Sent: Friday, January 19, 2018 11:48 PM
>> To: Vitaly Kuznetsov
>> Cc: kbuild test robot ; kbuild-...@01.org;
>> k...@vger.kernel
My comments to this series were only notes to be taken
into account for future patches.
Thank you,
Claudiu Beznea
On 22.01.2018 12:22, Ajay Singh wrote:
> This patch series contains changes to fix issues reported by checkpatch.pl
> script.
>
> Changes since v1:
> * Patch0004: Used "host_int_par
On 22.01.2018 13:22, Dan Carpenter wrote:
> On Mon, Jan 22, 2018 at 01:06:59PM +0200, Claudiu Beznea wrote:
>>
>>
>> On 22.01.2018 12:22, Ajay Singh wrote:
>>> Fix the "do not add new typedefs" issue found by checkpatch.pl
>>> script
>>>
>> >From
>> >https://www.kernel.org/doc/html/latest/proces
On Mon, Jan 22, 2018 at 01:06:59PM +0200, Claudiu Beznea wrote:
>
>
> On 22.01.2018 12:22, Ajay Singh wrote:
> > Fix the "do not add new typedefs" issue found by checkpatch.pl
> > script
> >
> >From
> >https://www.kernel.org/doc/html/latest/process/submitting-patches.html#the-canonical-patch-fo
On 22.01.2018 12:22, Ajay Singh wrote:
> Fix "Avoid camelcase" issue found by checkpatch.pl
> script.
>
Same here as in patch 02 of this series.
> Signed-off-by: Ajay Singh
> Reviewed-by: Claudiu Beznea
> ---
> drivers/staging/wilc1000/host_interface.c | 4 ++--
> 1 file changed, 2 insertion
On 22.01.2018 12:22, Ajay Singh wrote:
> Fix the "do not add new typedefs" issue found by checkpatch.pl
> script
>
>From
>https://www.kernel.org/doc/html/latest/process/submitting-patches.html#the-canonical-patch-format
the the message body line is wrapped at 75 columns.
> Signed-off-by: Ajay
On 22.01.2018 13:10, Claudiu Beznea wrote:
>
>
> On 22.01.2018 12:22, Ajay Singh wrote:
>> This patch series contains changes to fix issues reported by checkpatch.pl
>> script.
>>
>> Changes since v1:
>> * Patch0004: Used "host_int_parse_join_bss_param" for function name.
>> * Patch0009: Put m
On 22.01.2018 12:22, Ajay Singh wrote:
> This patch series contains changes to fix issues reported by checkpatch.pl
> script.
>
> Changes since v1:
> * Patch0004: Used "host_int_parse_join_bss_param" for function name.
> * Patch0009: Put memcpy statement in single line instead of using two line
On 22.01.2018 12:22, Ajay Singh wrote:
> Fix "Avoid CamelCase:" issue reported by checkpatch.pl script
Forgot a dot here?
> Rename host_int_ParseJoinBssParam() & its variables name using
> camelCase.
>
> Signed-off-by: Ajay Singh
> Reviewed-by: Claudiu Beznea
> ---
> drivers/staging/wilc1000/
On 22.01.2018 12:22, Ajay Singh wrote:
> Fix "Avoid camelCase" issue reported by checkpatch.pl
> script.
Same here as in patch 02 of this series.
>
> Signed-off-by: Ajay Singh
> Reviewed-by: Claudiu Beznea
> ---
> drivers/staging/wilc1000/host_interface.c | 16
> 1 file chan
On Sun, Jan 21, 2018 at 04:05:26PM +, Sidong Yang wrote:
> Fix checkpatch.pl warning message about logging code. Previous code
> contains hard coded function name. Fix this code by using __func__
> macro.
>
> Signed-off-by: Sidong Yang
> ---
> drivers/staging/most/dim2/dim2.c | 2 +-
> 1 fil
Thomas Gleixner writes:
> On Fri, 19 Jan 2018, Vitaly Kuznetsov wrote:
>> kbuild test robot writes:
>>
>> > Hi Vitaly,
>> >
>> > Thank you for the patch! Perhaps something to improve:
>> >
>> > [auto build test WARNING on tip/auto-latest]
>> > [also build test WARNING on v4.15-rc8 next-20180118
On Tue, Jan 16, 2018 at 06:02:07PM +, Eremin, Dmitry wrote:
> The logic of the original commit 4d99b2581eff ("staging: lustre: avoid
> intensive reconnecting for ko2iblnd")
> was assumed conditional free of struct kib_conn if the second argument
> free_conn in function
> kiblnd_destroy_conn(s
I get some static checker warnings like this:
drivers/staging/rts5208/ms.c:2607 ms_build_l2p_tbl()
error: buffer underflow 'ms_card->segment' (-1)-16
The problem is that we memset "ms_card" to zero at the start of the
reset_ms_card() function. That means that when we try to calculate
"ms
On Fri, Jan 19, 2018 at 05:53:59PM -0800, Joe Perches wrote:
> On Thu, 2018-01-18 at 16:51 +0100, Fabian Huegel wrote:
> > Most multi-line comments started on the first line, but the preferred
> > linux kernel style is to start multi-line comments on the second line.
> > Some comments became less r
Because this is a post-op instead of a pre-op, then it means we check
if knl_buffer[-1] is a space. It doesn't really hurt anything, but
it causes a static checker warning so let's fix it.
Fixes: d7e09d0397e8 ("staging: add Lustre file system client support")
Signed-off-by: Dan Carpenter
diff -
Fix "Avoid camelCase" issue reported by checkpatch.pl script.
rename variable name from "pu8CurrByte" to "cur_byte"
Signed-off-by: Ajay Singh
Reviewed-by: Claudiu Beznea
---
drivers/staging/wilc1000/host_interface.c | 232 +++---
1 file changed, 116 insertions(+), 116 de
Fix "Avoid camelCase" issue found by checkpatch.pl script.
Rename variable name from strWIDList to wid_list.
Signed-off-by: Ajay Singh
Reviewed-by: Claudiu Beznea
---
drivers/staging/wilc1000/host_interface.c | 176 +++---
1 file changed, 88 insertions(+), 88 deletions(-
Fix "Avoid camelCase" issue reported by checkpatch.pl
script.
Signed-off-by: Ajay Singh
Reviewed-by: Claudiu Beznea
---
drivers/staging/wilc1000/host_interface.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/dri
Fix "Avoid camelCase" issue found by checkpatch.pl script.
Signed-off-by: Ajay Singh
Reviewed-by: Claudiu Beznea
---
drivers/staging/wilc1000/host_interface.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging
Fix "Avoid camelcase" issue found by checkpatch.pl
script.
Signed-off-by: Ajay Singh
Reviewed-by: Claudiu Beznea
---
drivers/staging/wilc1000/host_interface.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interface.c
b/drivers/staging/wil
Fix "Avoid camelCase" issue reported by checkpatch.pl script.
Signed-off-by: Ajay Singh
Reviewed-by: Claudiu Beznea
---
drivers/staging/wilc1000/host_interface.c | 78 +++
1 file changed, 39 insertions(+), 39 deletions(-)
diff --git a/drivers/staging/wilc1000/host_i
Fix "Avoid camelCase" issue reported by checkpatch.pl script.
In this patch rename "strConnectInfo" variable to resolve
issue reported by checkpatch.pl script.
Signed-off-by: Ajay Singh
Reviewed-by: Claudiu Beznea
---
drivers/staging/wilc1000/host_interface.c | 74 +++---
Fix "Avoid camelCase" issue found by checkpatch.pl script.
Signed-off-by: Ajay Singh
Reviewed-by: Claudiu Beznea
---
drivers/staging/wilc1000/host_interface.c | 105 ++
1 file changed, 49 insertions(+), 56 deletions(-)
diff --git a/drivers/staging/wilc1000/host_inte
This patch series contains changes to fix issues reported by checkpatch.pl
script.
Changes since v1:
* Patch0004: Used "host_int_parse_join_bss_param" for function name.
* Patch0009: Put memcpy statement in single line instead of using two lines.
* Patch0010: Used "bss_param" for variable name in
Fix "Avoid camelCase" issue reported by checkpatch.pl script.
Signed-off-by: Ajay Singh
Reviewed-by: Claudiu Beznea
---
drivers/staging/wilc1000/host_interface.c | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/wilc1000/host_interfac
Fix "Avoid CamelCase:" issue reported by checkpatch.pl script
Rename host_int_ParseJoinBssParam() & its variables name using
camelCase.
Signed-off-by: Ajay Singh
Reviewed-by: Claudiu Beznea
---
drivers/staging/wilc1000/host_interface.c | 230 +++---
1 file changed, 115 i
Fix "Avoid camleCase" issue reported by checkpatch.pl script.
rename variable name from u32WidsCount to wid_cnt to fix
checkpatch.pl report issue.
Signed-off-by: Ajay Singh
Reviewed-by: Claudiu Beznea
---
drivers/staging/wilc1000/host_interface.c | 146 +++---
1 file cha
Fix "Unnecessary parentheses around" issue found by checkpatch.pl
script.Remove the unnecessary parentheses to follow linux coding style.
Signed-off-by: Ajay Singh
Reviewed-by: Claudiu Beznea
---
drivers/staging/wilc1000/coreconfigurator.c | 2 +-
drivers/staging/wilc1000/host_interface.
Fix the "do not add new typedefs" issue found by checkpatch.pl
script
Signed-off-by: Ajay Singh
Reviewed-by: Claudiu Beznea
---
drivers/staging/wilc1000/wilc_wlan_if.h | 72 -
1 file changed, 36 insertions(+), 36 deletions(-)
diff --git a/drivers/staging/wilc100
Fix "Alignment should match open parenthesis" issues reported by
checkpatch.pl script. changes to comply with linux coding style.
Signed-off-by: Ajay Singh
Reviewed-by: Claudiu Beznea
---
drivers/staging/wilc1000/host_interface.c | 4 +--
drivers/staging/wilc1000/linux_wlan.c
Add CryptoCell AEAD support
Signed-off-by: Gilad Ben-Yossef
---
drivers/crypto/ccree/Makefile|2 +-
drivers/crypto/ccree/cc_aead.c | 2702 ++
drivers/crypto/ccree/cc_aead.h | 109 ++
drivers/crypto/ccree/cc_buffer_mgr.c | 882 +++
Add CryptoCell async. hash and HMAC support.
Signed-off-by: Gilad Ben-Yossef
---
drivers/crypto/ccree/Makefile|2 +-
drivers/crypto/ccree/cc_buffer_mgr.c | 261 +++-
drivers/crypto/ccree/cc_driver.c | 13 +
drivers/crypto/ccree/cc_driver.h |1 +
drivers/crypto/ccree/cc
Add FIPS mode support to CryptoCell driver
Signed-off-by: Gilad Ben-Yossef
---
drivers/crypto/ccree/Makefile| 1 +
drivers/crypto/ccree/cc_driver.c | 29 +-
drivers/crypto/ccree/cc_driver.h | 1 +
drivers/crypto/ccree/cc_fips.c | 111 +++
dr
Add CryptoCell skcipher support
Signed-off-by: Gilad Ben-Yossef
---
drivers/crypto/ccree/Makefile|2 +-
drivers/crypto/ccree/cc_buffer_mgr.c | 125
drivers/crypto/ccree/cc_buffer_mgr.h |8 +
drivers/crypto/ccree/cc_cipher.c | 1130 ++
dri
Introduce basic low level Arm TrustZone CryptoCell HW support.
This first patch doesn't actually register any Crypto API
transformations, these will follow up in the next patch.
This first revision supports the CC 712 REE component.
Signed-off-by: Gilad Ben-Yossef
---
drivers/crypto/Kconfig
Update Arm TrustZone CryptoCell driver entry move into drivers/crypto/
Signed-off-by: Gilad Ben-Yossef
---
MAINTAINERS | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 1082846..560e068 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -3273,12
Arm TrustZone CryptoCell is a security hardware IP that
includes support for hardware based hash, digest, cipher
and AEAD operations. This driver provides support for
these as part of the Linux Crypto sub-system.
The driver spent some time now in the staging tree being
cleaned up and is now submit
Rename the Kconfig var of the staging tree version of the driver
in preparation of introducing the final version of the driver
into the cryptodev tree to avoid link time symbol collisions.
Signed-off-by: Gilad Ben-Yossef
---
drivers/staging/ccree/Kconfig | 4 ++--
drivers/staging/ccree/Makefile
52 matches
Mail list logo