Re: [bug report] staging: lustre: replace simple cases of LIBCFS_ALLOC with kzalloc.

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

Darlehen

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

[staging:staging-next 468/569] drivers/staging/lustre/lnet/selftest/module.c:125:47: error: 'struct cfs_cpt_table' has no member named 'ctb_parts'

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

[PATCH v2 char-misc 1/1] Drivers: hv: vmbus: Implement Direct Mode for stimer0

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

Re: [PATCH net-next] hv_netvsc: Use the num_online_cpus() for channel limit

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

[PATCH] Staging: iio: ade7758: Expand buf_lock to cover both buffer

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

[PATCH] staging: sm750fb: fix CamelCase for dispSet var

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

[PATCH v2] staging: android: ion: Zero CMA allocated memory

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:

Re: [PATCH] staging: android: ion: Zero CMA allocated memory

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.

Re: [PATCH] staging: android: ion: Zero CMA allocated memory

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

Re: [PATCH-next] hal_com_phycfg: Remove unneeded semicolons

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:

Re: [PATCH-next] hal_com_phycfg: Remove unneeded semicolons

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 > --- >

[PATCH] staging: rtl8192u: remove redundant initialization of 'timeout'

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:

Re: [PATCH v3 3/7] x86/hyper-v: reenlightenment notifications support

"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:

Re: [PATCH v2 00/14] fix to remove unnecessary parenthesis,typedef and avoid camelCase

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

Re: [PATCH v2 02/14] staging: wilc1000: removed unnecessary defined enums typedef

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 >>

Re: [PATCH v2 02/14] staging: wilc1000: removed unnecessary defined enums typedef

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 >

Re: [PATCH v2 13/14] staging: wilc1000: rename Handle_Connect() to avoid camelCase

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 > --- >

Re: [PATCH v2 02/14] staging: wilc1000: removed unnecessary defined enums typedef

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:

Re: [PATCH v2 00/14] fix to remove unnecessary parenthesis,typedef and avoid camelCase

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

Re: [PATCH v2 00/14] fix to remove unnecessary parenthesis,typedef and avoid camelCase

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

Re: [PATCH v2 04/14] staging: wilc1000: rename host_int_ParseJoinBssParam() and it's variable using camelCase

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

Re: [PATCH v2 12/14] staging: wilc1000: rename Handle_ScanDone() to avoid camelCase

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 > --- >

Re: [PATCH] staging: most: replace function name to __func__

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 > --- >

Re: [PATCH v3 3/7] x86/hyper-v: reenlightenment notifications support

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

Re: [PATCH] staging: lustre: Fix avoid intensive reconnecting for ko2iblnd patch

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 >

[PATCH] staging: rts5208: Fix "seg_no" calculation in reset_ms_card()

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

Re: [PATCH 1/2] staging: lustre: Fix comment style

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

[PATCH] staging: lustre: libcfs: Prevent harmless read underflow

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

[PATCH v2 08/14] staging: wilc1000: rename pu8CurrByte variable to avoid camelCase

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

[PATCH v2 06/14] staging: wilc1000: rename strWIDList variable to wid_list to avoid camelCase

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

[PATCH v2 12/14] staging: wilc1000: rename Handle_ScanDone() to avoid camelCase

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(-)

[PATCH v2 14/14] staging: wilc1000: rename Handle_ConnectTimeout() and Handle_Key()

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

[PATCH v2 13/14] staging: wilc1000: rename Handle_Connect() to avoid camelCase

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

[PATCH v2 10/14] staging: wilc1000: rename ptstrJoinBssParam variable to avoid camelCase

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

[PATCH v2 11/14] staging: wilc1000: rename strConnectInfo variable to avoid camelCase

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 ---

[PATCH v2 09/14] staging: wilc1000: rename pstrHostIFconnectAttr argument to avoid camelCase

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

[PATCH v2 00/14] fix to remove unnecessary parenthesis, typedef and avoid camelCase

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

[PATCH v2 05/14] staging: wilc1000: rename camelCase used in Handle_DelAllSta() and its variable

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

[PATCH v2 04/14] staging: wilc1000: rename host_int_ParseJoinBssParam() and it's variable using camelCase

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 |

[PATCH v2 07/14] staging: wilc1000: rename u32WidsCount variable to avoid camelCase

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 ---

[PATCH v2 01/14] staging: wilc1000: remove unnecessary use of parentheses

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 ---

[PATCH v2 02/14] staging: wilc1000: removed unnecessary defined enums typedef

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

[PATCH v2 03/14] staging: wilc1000: fix alignments to match open parenthesis

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

[PATCH v2 5/7] crypto: ccree: add AEAD support

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 ++

[PATCH v2 4/7] crypto: ccree: add ahash support

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 +

[PATCH v2 6/7] crypto: ccree: add FIPS support

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

[PATCH v2 3/7] crypto: ccree: add skcipher support

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

[PATCH v2 2/7] crypto: ccree: introduce CryptoCell driver

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 ---

[PATCH v2 7/7] MAINTAINERS: update ccree entry

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 +++

[PATCH v2 0/7] crypto: ccree: Introduce Arm TrustZone CryptoCell

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

[PATCH v2 1/7] staging: ccree: rename staging ver and mark as broken

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 ++--