Re: [PATCH v2 3/3] iio: hmc5843: Move hmc5843 out of staging

2016-02-22 Thread Daniel Baluta
On Sun, Feb 21, 2016 at 10:29 PM, Jonathan Cameron wrote: > On 14/02/16 22:37, Cristina Moraru wrote: >> This patch moves hmc5843 driver from staging/iio/magnetometer >> to iio/magnetometer, updates the corresponding Makefiles and >> moves the hmc5843* entries to the 'Industrial I/O support -> >>

[PATCH V4 06/24] staging: wilc1000: host_interface.c: removes unnecessary log message

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unnecessary log message and relate variables. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/host_interface.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/staging

[PATCH V4 04/24] staging: wilc1000: linux_mon.c: add to check routine

2016-02-22 Thread Leo Kim
This patch adds the return value which needs the check and prints the error message when it fails. Signed-off-by: Leo Kim --- drivers/staging/wilc1000/linux_mon.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/linux_mon.c b/drivers/staging/wilc10

[PATCH V4 08/24] staging: wilc1000: linux_mon.c: removes comments

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unnecessary comments. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/linux_mon.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/wilc1000/linux_mon.c b/drivers/staging/wilc1000/linux_mon.c index f6349e5

[PATCH V4 13/24] staging: wilc1000: wilc_wfi_cfgoperations.c: removes unnecessary log messages

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unnecessary log messages. And, it adds a 'break' statement in accordance with the switch-case rule. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH V4 11/24] staging: wilc1000: host_interface.c: removes unnecessary log messages

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unnecessary log message and relate variables. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/host_interface.c | 31 --- 1 file changed, 4 insertions(+), 27 deletions(-) diff --git a/drivers/staging

[PATCH V4 10/24] staging: wilc1000: removes unused HOSTAPD_DBG tag

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unused HOSTAPD_DBG tag. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/linux_wlan_common.h | 4 +--- drivers/staging/wilc1000/wilc_debugfs.c | 5 ++--- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/driver

[PATCH V4 09/24] staging: wilc1000: wilc_wfi_cfgoperations.c: removes unnecessary log messages

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes the unnecessary log messages which is used with the HOSTAPD_DBG tag. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 88 +-- 1 file changed, 1 insertion(+), 87 deletions(-) diff -

[PATCH V4 12/24] staging: wilc1000: wilc_wfi_cfgoperations.c: removes unnecessary log messages

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unnecessary log messages and relate variables. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 233 +- 1 file changed, 11 insertions(+), 222 deletions(-) diff --git a/drivers/stag

[PATCH V4 14/24] staging: wilc1000: removes unused CFG80211_DBG tag

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unused CFG80211_DBG tag. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/linux_wlan_common.h | 4 +--- drivers/staging/wilc1000/wilc_debugfs.c | 6 +++--- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/driv

[PATCH V4 15/24] staging: wilc1000: coreconfigurator.c: removes unnecessary log messages

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unnecessary log messages. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/coreconfigurator.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/staging/wilc1000/coreconfigurator.c b/drivers/staging/

[PATCH V4 18/24] staging: wilc1000: linux_wlan.c: removes unnecessary log messages

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unnecessary log messages. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/linux_wlan.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/staging/wilc1000/linux_wlan.c index 274f

[PATCH V4 16/24] staging: wilc1000: host_interface.c: removes unnecessary log messages

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unnecessary log messages. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/host_interface.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/drivers/staging/wilc1000/host_interface.c b/drivers/s

[PATCH V4 17/24] staging: wilc1000: host_interface.c: replace PRINT_ER with netdev_err

2016-02-22 Thread Leo Kim
From: Chris Park This patch replaces PRINT_ER with netdev_err. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/host_interface.c | 284 +++--- 1 file changed, 142 insertions(+), 142 deletions(-) diff --git a/drivers/staging/wilc1000/host_i

[PATCH V4 19/24] staging: wilc1000: wilc_wfi_cfgoperations.c: removes unnecessary log messages

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unnecessary log messages. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 14 +++--- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wfi_cfgoperati

[PATCH V4 21/24] staging: wilc1000: removes unused region feature

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unused region feature on debug message. Also, removes the functions of related in this feature. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/linux_wlan_common.h | 17 ++- drivers/staging/wilc1000/wilc_debugfs.c

[PATCH V4 20/24] staging: wilc1000: wilc_wlan.c: removes unnecessary log messages

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unnecessary log messages. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/wilc_wlan.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wlan.c b/drivers/staging/wilc1000/wilc_wlan.c index c6d2

[PATCH V4 22/24] staging: wilc1000: removes usused PRINT_XX(region...)

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unused PRINT_XX(region...) because already removed in used to region feature of whole wilc source. Also, removes unnecessary related region extern variable. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/linux_wlan_common.h |

[PATCH V4 24/24] staging: wilc1000: moves LINUX_RX_SIZE, LINUX_TX_SIZE

2016-02-22 Thread Leo Kim
From: Chris Park This patch moves LINUX_RX_SIZE and LINUX_TX_SIZE to wilc_wlan.h file because there are only used to wilc_wlan.c file. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/linux_wlan_common.h | 4 drivers/staging/wilc1000/wilc_wlan.h | 3 +

[PATCH V4 23/24] staging: wilc1000: moves to define values

2016-02-22 Thread Leo Kim
From: Chris Park This patch moves DEBUG, INFO, WRN and ERR values to wilc_debugfs.c file because this is only used to wilc_debugfs.c file. And, removes unnecessary WILC_DEBUGFS and else-case variables. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/linux_wlan_co

[PATCH V4 05/24] staging: wilc1000: removes potential null dereference

2016-02-22 Thread Leo Kim
This patch removes the error reported by smatch. - wilc_wfi_cfgoperations.c:674 scan() error: potential null dereference 'strHiddenNetwork.net_info'. (kmalloc returns null) Signed-off-by: Leo Kim --- drivers/staging/wilc1000/wilc_wfi_cfgoperations.c | 2 ++ 1 file changed, 2 insertions(+)

[PATCH V4 01/24] staging: wilc1000: wilc_wlan.c: removes PRINT_ER

2016-02-22 Thread Leo Kim
This patches removes PRINT_ER that is unnecessary debug logs. Signed-off-by: Leo Kim --- drivers/staging/wilc1000/wilc_wlan.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/wilc1000/wilc_wlan.c b/drivers/staging/wilc1000/wilc_wlan.c index 391173b..c6d206d 100644 --- a/driv

[PATCH V4 03/24] staging: wilc1000: linux_mon.c: replaces PRINT_ER with netdev_err

2016-02-22 Thread Leo Kim
This patches replaces PRINT_ER with netdev_err. Signed-off-by: Leo Kim --- drivers/staging/wilc1000/linux_mon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/wilc1000/linux_mon.c b/drivers/staging/wilc1000/linux_mon.c index 558fb50..2b575d1 100644 --- a/dri

[PATCH V4 00/24] staging: wilc1000: Resend V4

2016-02-22 Thread Leo Kim
Dear Greg This is the 4th fix patch about the mistake merging. Mistake merging is revert the below commit when rebase V3 patches. (already applied commit) - 6c2ab2398b88 ("staging: wilc1000: remove spaces around '->'") (V3 mistake patch) - [PATCH V3 09/24] staging: wilc1000: wilc_wfi_cfgoperatio

[PATCH V4 07/24] staging: wilc1000: linux_mon.c: removes unnecessary log message

2016-02-22 Thread Leo Kim
From: Chris Park This patch removes unnecessary log message and relate variables. Signed-off-by: Chris Park Signed-off-by: Leo Kim --- drivers/staging/wilc1000/linux_mon.c | 42 1 file changed, 4 insertions(+), 38 deletions(-) diff --git a/drivers/staging

[PATCH V4 02/24] staging: wilc1000: linux_mon.c: removes debug logs

2016-02-22 Thread Leo Kim
This patches removes unnecessary debug logs. Signed-off-by: Leo Kim --- drivers/staging/wilc1000/linux_mon.c | 24 ++-- 1 file changed, 6 insertions(+), 18 deletions(-) diff --git a/drivers/staging/wilc1000/linux_mon.c b/drivers/staging/wilc1000/linux_mon.c index 9fcb497..5

Re: [PATCH RESEND 04/10] staging: wilc1000: removes unnecessary wilc_debug print log

2016-02-22 Thread Dan Carpenter
On Mon, Feb 22, 2016 at 01:41:13PM +0900, Leo Kim wrote: > This patch removes unnecessary wilc_debug print log. > The print log was written when if condition fail. > The condition is chip-id check function. > Also, replaces this condition with normal function. > > Signed-off-by: Leo Kim > --- >

RE: [PATCH RESEND 04/10] staging: wilc1000: removes unnecessary wilc_debug print log

2016-02-22 Thread Kim, Leo
Dear Dan, This patch is subject "removes unnecessary wilc_debug print log". Your advice are correct but in this patch is not applied. I'll be applied to next patch about the your good advice. Would be okay? Always appreciate your positive advice & great information.  Thanks, BR  Leo -Origina

Re: [PATCH] staging: xgifb: Fix comment style

2016-02-22 Thread Bo YU
Oh, I am miss ccing list in previous email. On Sun, Feb 21, 2016 at 8:57 PM, Roger H Newell wrote: > Hello: > > I think you need to make sure your staging-testing tree is current > before creating a patch. > > git checkout staging-testing && git fetch origin > git checkout my-branch && git r

Re: [PATCH RESEND 04/10] staging: wilc1000: removes unnecessary wilc_debug print log

2016-02-22 Thread Dan Carpenter
On Mon, Feb 22, 2016 at 11:42:42AM +, Kim, Leo wrote: > Dear Dan, > > This patch is subject "removes unnecessary wilc_debug print log". I'm fine with you fixing it up in a later patch, but you should not be defending this patch as valid way to do things. The rule is "do one thing at a time",

[RFC] Staging: goldfish_audio.c: Modify goldfish audio driver to use ALSA framework

2016-02-22 Thread PrasannaKumar Muralidharan
From: PrasannaKumar Muralidharan Modified goldfish audio driver to use ALSA framework. Only playback support is implemented. After fixing review comments capture support will be implemented. Only able to compile test so sending as RFC. Signed-off-by: PrasannaKumar Muralidharan --- drivers/stag

Re: [RFC] Staging: goldfish_audio.c: Modify goldfish audio driver to use ALSA framework

2016-02-22 Thread Greg KH
On Mon, Feb 22, 2016 at 09:20:24PM +0530, PrasannaKumar Muralidharan wrote: > From: PrasannaKumar Muralidharan > > Modified goldfish audio driver to use ALSA framework. Only playback support > is implemented. After fixing review comments capture support will be > implemented. Only able to compile

Re: [RFC] Staging: goldfish_audio.c: Modify goldfish audio driver to use ALSA framework

2016-02-22 Thread PrasannaKumar Muralidharan
Tried testing this with Android user space. But it did not work as the user space HAL layer does not use ALSA by default. Need to modify the Android tree to use tinyalsa to test this change. I did not try out any Linux distribution with Android emulator. I will try any of the above way and report b

Re: [RFC] Staging: goldfish_audio.c: Modify goldfish audio driver to use ALSA framework

2016-02-22 Thread Greg KH
A: No. Q: Should I include quotations after my reply? http://daringfireball.net/2007/07/on_top On Mon, Feb 22, 2016 at 10:50:09PM +0530, PrasannaKumar Muralidharan wrote: > Tried testing this with Android user space. But it did not work as the > user space HAL layer does not use ALSA by default.

Re: [RFC] Staging: goldfish_audio.c: Modify goldfish audio driver to use ALSA framework

2016-02-22 Thread PrasannaKumar Muralidharan
According to Android porting guide tinyalsa will work for ALSA drivers, it is included in the Android open source project tree. Change in their build configuration should be enough to use this patch. I think it is better to get your question answered before proceeding further. I don't have a clue

Re: [PATCH 1/1] staging: ion: debugfs sparse warning, proper mask

2016-02-22 Thread Laura Abbott
On 02/20/2016 07:15 PM, Derek Yerger wrote: From drivers/staging/android TODO file - sparse errors. The current code attempts assignment of -1 to unsigned type gfp_t. Assignment should be an enumerated type of GFP_KERNEL, GFP_ATOMIC, GFP_HIGHMEM, or __GFP_HIGH. The original 2014 patch by Gioh

[patch 1/6] staging: gdm72xx: underflow in netlink_rcv_cb()

2016-02-22 Thread Dan Carpenter
If nlh->nlmsg_len is less than ND_IFINDEX_LEN we end up trying to memcpy a negative size. I also re-ordered slighty the condition to make it more uniform. Signed-off-by: Dan Carpenter diff --git a/drivers/staging/gdm72xx/netlink_k.c b/drivers/staging/gdm72xx/netlink_k.c index cf0b47c..4089b17

[patch 2/6] Staging: gdm72xx: silence underflow warning in netlink_send()

2016-02-22 Thread Dan Carpenter
The value of "group" comes from "idx" in __gdm_wimax_event_send(): if (sscanf(e->dev->name, "wm%d", &idx) == 1) Smatch marks sscanf values as user controlled. It's supposed to be a number in 0-30 range. We cap the upper bound but allow negatives. Fix this by making it type u16 instead.

[patch 3/6] Staging: gdm72xx: make "len" unsigned

2016-02-22 Thread Dan Carpenter
We had an underflow bug here and I think I fixed it but we may as well be proactive and make "len" unsigned to be double sure. Signed-off-by: Dan Carpenter diff --git a/drivers/staging/gdm72xx/gdm_wimax.h b/drivers/staging/gdm72xx/gdm_wimax.h index 3330cd79..ed12813 100644 --- a/drivers/staging

[patch 4/6] staging: gdm72xx: zero out padding

2016-02-22 Thread Dan Carpenter
We pad the start of this buffer with 256 bytes of padding. It's not clear to me exactly what's going on or how it's used but let's zero it out. Signed-off-by: Dan Carpenter diff --git a/drivers/staging/gdm72xx/usb_boot.c b/drivers/staging/gdm72xx/usb_boot.c index 3082987..4431a80 100644 --- a

[patch 5/6] Staging: gdm72xx: underflow bug in gdm_wimax_ioctl_get_data()

2016-02-22 Thread Dan Carpenter
"size" here should be unsigned, otherwise we might end up trying to copy negative bytes in gdm_wimax_ioctl_get_data() resulting in an information leak. Reported-by: Alan Cox Signed-off-by: Dan Carpenter diff --git a/drivers/staging/gdm72xx/wm_ioctl.h b/drivers/staging/gdm72xx/wm_ioctl.h index

[patch 6/6] Staging: gdm72xx: remove duplicate condition

2016-02-22 Thread Dan Carpenter
We know "len" is not zero because we tested for that at the beginning of the function so this test can be removed. Signed-off-by: Dan Carpenter diff --git a/drivers/staging/gdm72xx/gdm_wimax.c b/drivers/staging/gdm72xx/gdm_wimax.c index 6d647d6..09bf64e 100644 --- a/drivers/staging/gdm72xx/gdm_

Re: [PATCH V4 00/24] staging: wilc1000: Resend V4

2016-02-22 Thread Greg KH
On Mon, Feb 22, 2016 at 07:14:07PM +0900, Leo Kim wrote: > Dear Greg > > This is the 4th fix patch about the mistake merging. > Mistake merging is revert the below commit when rebase V3 patches. > (already applied commit) - 6c2ab2398b88 ("staging: wilc1000: remove spaces > around '->'") > (V3 mis

Re: [PATCH 1/1] staging: ion: debugfs sparse warning, proper mask

2016-02-22 Thread Derek Yerger
I'm sorry, it seems I am mistaken. This was in reference to patches which weren't committed. Please see: http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/2014-October/060203.html for the patch with this specific mask used. The reason __GFP_HIGHMEM may have been chosen is because

[RFC] staging: iio: isl29018: use regmap to retrieve struct device

2016-02-22 Thread Alison Schofield
Remove struct device from drivers global data and use regmap API to retrieve device info instead. This replacement can be done for drivers that include regmap in their global data. Signed-off-by: Alison Schofield --- drivers/staging/iio/light/isl29018.c | 28 +++- 1 file

Re: [patch 3/6] Staging: gdm72xx: make "len" unsigned

2016-02-22 Thread kbuild test robot
Hi Dan, [auto build test WARNING on staging/staging-testing] [also build test WARNING on v4.5-rc5 next-20160222] [if your patch is applied to the wrong git tree, please drop us a note to help improving the system] url: https://github.com/0day-ci/linux/commits/Dan-Carpenter/staging-gdm72xx

Re: [PATCH] staging: rtl8188eu: Remove RF_PATH_C & RF_PATH_D

2016-02-22 Thread Larry Finger
On 02/17/2016 09:14 PM, Andrew Bradford wrote: RTL8188EE has a maximum of 2 RF paths (chains) so paths C and D are not needed to support this part. Signed-off-by: Andrew Bradford --- drivers/staging/rtl8188eu/hal/bb_cfg.c | 26 -- drivers/staging/rtl8188eu/hal

[PATCH 00/24] Second batch of LNet updates

2016-02-22 Thread James Simmons
This patch set fixes many of the LNet issues encounter run in production environments. One of the long standing issues was not being able to reconfigure LNet after initialization. Doing so left it in a broken state. Several other issues are also addressed in this patch set. Merged back into this pa

[PATCH 01/24] staging: lustre: Dynamic LNet Configuration (DLC) IOCTL changes

2016-02-22 Thread James Simmons
From: Amir Shehata This is the fourth patch of a set of patches that enables DLC. This patch changes the IOCTL infrastructure in preparation of adding extra IOCTL communication between user and kernel space. The changes include: - adding a common header to be passed to ioctl infra functions in

[PATCH 06/24] staging: lustre: remove LUSTRE_{,SRV_}LNET_PID

2016-02-22 Thread James Simmons
From: John L. Hammond Remove LUSTRE_LNET_PID (12354) and LUSTRE_SRV_LNET_PID (12345) from the libcfs headers and replace their uses with a new macro LNET_PID_LUSTRE (also 12345) in lnet/types.h. Signed-off-by: John L. Hammond Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2675 Reviewed-on:

[PATCH 08/24] staging: lustre: return appropriate errno when adding route

2016-02-22 Thread James Simmons
From: Amir Shehata When adding route it ignored specific scenarios, namely: 1. route already exists 2. route is on a local net 3. route is unreacheable This patch returns the appropriate return codes from the lower level function lnet_add_route(), and then ignores the above case from the calling

[PATCH 03/24] staging: lustre: fix crash due to NULL networks string

2016-02-22 Thread James Simmons
From: Amir Shehata If there is an invalid networks or ip2nets lnet_parse_networks() gets called with a NULL 'network' string parameter lnet_parse_networks() needs to sanitize its input string now that it's being called from multiple places. Instead, check for a NULL string everytime the functio

[PATCH 02/24] staging: lustre: Dynamic LNet Configuration (DLC) show command

2016-02-22 Thread James Simmons
From: Amir Shehata This is the fifth patch of a set of patches that enables DLC. This patch adds the new structures which will be used in the IOCTL communication. It also added a set of show operations to show buffers, networks, statistics and peer information. Signed-off-by: Amir Shehata Int

[PATCH 05/24] staging: lustre: make local functions static for LNet ni

2016-02-22 Thread James Simmons
From: Frank Zago The function lnet_unprepare can be made static. Signed-off-by: Frank Zago Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5396 Reviewed-on: http://review.whamcloud.com/11306 Reviewed-by: James Simmons Reviewed-by: Patrick Farrell Reviewed-by: Amir Shehata Reviewed-by: Jo

[PATCH 10/24] staging: lustre: missed a few cases of using NULL instead of 0

2016-02-22 Thread James Simmons
From: Frank Zago It is preferable to use NULL instead of 0 for pointers. This fixes sparse warnings such as: lustre/fld/fld_request.c:126:17: warning: Using plain integer as NULL pointer The second parameter of class_match_param() was changed to a const, to be able to remove a cast in one user,

[PATCH 11/24] staging: lustre: startup lnet acceptor thread dynamically

2016-02-22 Thread James Simmons
From: Amir Shehata With DLC it's possible to start up a system with no NIs that require the acceptor thread, and thus it won't start. Later on the user can add an NI that requires the acceptor thread to start, it is then necessary to start it up. If the user removes a NI and as a result there a

[PATCH 14/24] staging: lustre: handle lnet_check_routes() errors

2016-02-22 Thread James Simmons
From: Amir Shehata After adding a route, lnet_check_routes() is called to ensure that the route added doesn't invalidate the routing configuration. If lnet_check_routes() fails then the route just added, which caused the current configuration to be invalidated is deleted, and an error is returne

[PATCH 21/24] staging: lustre: use sock.h in only acceptor.c

2016-02-22 Thread James Simmons
On some platforms having sock.h in lib-types.h would collide with other included header files being used in the LNet layer. Looking at what was needed from sock.h only acceptor.c is dependent on it. To avoid these issues we just use sock.h only in acceptor.c. Signed-off-by: James Simmons Intel-bu

[PATCH 09/24] staging: lustre: make some lnet functions static

2016-02-22 Thread James Simmons
From: Frank Zago Some functions and variables are only used in their C file, so reduce their scope. This reduces the code size, and fixes sparse warnings such as: warning: symbol 'proc_lnet_routes' was not declared. Should it be static? warning: symbol 'proc_lnet_routers' was not declare

[PATCH 15/24] staging: lustre: improvement to router checker

2016-02-22 Thread James Simmons
From: Amir Shehata This patch starts router checker thread all the time. The router checker only checks routes by ping if live_router_check_interval or dead_router_check_interval are set to something other than 0, and there are routes configured. If these conditions are not met the router check

[PATCH 17/24] staging: lustre: prevent assert on LNet module unload

2016-02-22 Thread James Simmons
From: Amir Shehata There is a use case where lnet can be unloaded while there are no NIs configured. Removing lnet in this case will cause LNetFini() to be called without a prior call to LNetNIFini(). This will cause the LASSERT(the_lnet.ln_refcount == 0) to be triggered. To deal with this use

[PATCH 20/24] staging: lustre: avoid race during lnet acceptor thread termination

2016-02-22 Thread James Simmons
From: Bruno Faccini This patch will avoid potential race, around socket sleepers wait list, during acceptor thread termination and using sk_callback_lock RW-Lock protection. Signed-off-by: Bruno Faccini Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6476 Reviewed-on: http://review.whamclou

[PATCH 22/24] staging: lustre: Allocate the correct number of rtr buffers

2016-02-22 Thread James Simmons
From: Amir Shehata This patch ensures that the correct number of router buffers are allocated. It keeps a count that keeps track of the number of buffers allocated. Another count keeps the number of buffers requested. The number of buffers allocated is set when creating new buffers and reduced

[PATCH 13/24] staging: lustre: return -EEXIST if NI is not unique

2016-02-22 Thread James Simmons
From: Amir Shehata Return -EEXIST and not -EINVAL when trying to add a network interface which is not unique. Some minor cleanup in api-ni.c Signed-off-by: Amir Shehata Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5875 Reviewed-on: http://review.whamcloud.com/13056 Reviewed-by: Isaac Hu

[PATCH 04/24] staging: lustre: DLC user/kernel space glue code

2016-02-22 Thread James Simmons
From: Amir Shehata This is the sixth patch of a set of patches that enables DLC. This patch enables the user space to call into the kernel space DLC code. Added handlers in the LNetCtl function to call the new functions added for Dynamic Lnet Configuration Signed-off-by: Amir Shehata ntel-bug

[PATCH 07/24] staging: lustre: improve LNet clean up code and API

2016-02-22 Thread James Simmons
From: Amir Shehata This patch addresses a set of related issues: LU-5568, LU-5734, LU-5839, LU-5849, LU-5850. Create the local lnet_startup_lndni() API. This function starts up one LND. lnet_startup_lndnis() calls this function in a loop on every ni in the list passed in. lnet_startup_lndni()

[PATCH 18/24] staging: lustre: remove messages from lazy portal on NI shutdown

2016-02-22 Thread James Simmons
From: Amir Shehata When shutting down an NI in a busy system, some messages received on this NI, might be on the lazy portal. They would have grabbed a ref count on the NI. Therefore NI will not be removed until messages are processed. In order to avoid this scenario, when an NI is shutdown go

[PATCH 23/24] staging: lustre: Use lnet_is_route_alive for router aliveness

2016-02-22 Thread James Simmons
From: Chris Horn lctl show_route and lctl route_list will output router aliveness information via lnet_get_route(). lnet_get_route() should use the lnet_is_route_alive() function, introduced in e8a1124 http://review.whamcloud.com/7857, to determine route aliveness. Signed-off-by: Chris Horn Int

[PATCH 12/24] staging: lustre: reject invalid net configuration for lnet

2016-02-22 Thread James Simmons
From: Amir Shehata Currently if there exists a route that goes over a remote net and then this net is added dynamically as a local net, then traffic stops because the code in lnet_send() determines that the destination nid can be reached from another local_ni, but the src_nid is still stuck on th

[PATCH 19/24] staging: lustre: remove unnecessary EXPORT_SYMBOL from lnet layer

2016-02-22 Thread James Simmons
From: Frank Zago A lot of symbols don't need to be exported at all because they are only used in the module they belong to. Signed-off-by: Frank Zago Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-5829 Reviewed-on: http://review.whamcloud.com/13320 Reviewed-by: James Simmons Reviewed-by:

[PATCH 24/24] staging: lustre: Remove LASSERTS from router checker

2016-02-22 Thread James Simmons
From: Doug Oucharek In lnet_router_checker(), there are two LASSERTS. Neither protects us from anything and one of them triggered for a customer crashing the system unecessarily. This patch removes them. Signed-off-by: Doug Oucharek Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-7362 Rev

[PATCH 16/24] staging: lustre: assume a kernel build

2016-02-22 Thread James Simmons
From: John L. Hammond In lnet/lnet/ and lnet/selftest/ assume a kernel build (assume that __KERNEL__ is defined). Remove some common code only needed for user space LNet. Only part of the work of this patch got merged. This is the final bits. Signed-off-by: John L. Hammond Intel-bug-id: https

[PATCH] Staging: comedi: mite: added spaces around | and *

2016-02-22 Thread Philippe Loctaux
Added spaces around | and *, fixing 2 checkpatch checks. Signed-off-by: Philippe Loctaux --- drivers/staging/comedi/drivers/mite.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/comedi/drivers/mite.c b/drivers/staging/comedi/drivers/mite.c index 8f24702.

[PATCH] staging: rtl8723au: use list_first_entry*

2016-02-22 Thread Geliang Tang
Use list_first_entry*() instead of container_of() to simplify the code. Signed-off-by: Geliang Tang --- drivers/staging/rtl8723au/core/rtw_recv.c | 49 +-- drivers/staging/rtl8723au/core/rtw_xmit.c | 26 +--- 2 files changed, 22 insertions(+), 53 deletions

Re: [PATCH 4/4] staging: rtl8192u: r8192U_core: use kmalloc_array instead of kmalloc

2016-02-22 Thread Geliang Tang
On Sun, Feb 07, 2016 at 07:55:09PM -0800, Greg Kroah-Hartman wrote: > On Sun, Nov 08, 2015 at 10:17:54PM +0800, Geliang Tang wrote: > > Use kmalloc_array instead of kmalloc to allocate memory for an array. > > > > Signed-off-by: Geliang Tang > > --- > > drivers/staging/rtl8192u/r8192U_core.c | 4

Administrador do sistema

2016-02-22 Thread ADMIN
Sua caixa de correio excedeu o limite de armazenamento, que é de 20 GB como conjunto pelo administrador, você está atualmente em execução no 20,9 GB, você pode não ser capaz de Enviar ou receber novas mensagens até que você re-validar sua caixa de correio. Para Revalide sua caixa de correio

Re: [PATCH 4/4] staging: rtl8192u: r8192U_core: use kmalloc_array instead of kmalloc

2016-02-22 Thread Greg Kroah-Hartman
On Tue, Feb 23, 2016 at 08:55:04AM +0800, Geliang Tang wrote: > On Sun, Feb 07, 2016 at 07:55:09PM -0800, Greg Kroah-Hartman wrote: > > On Sun, Nov 08, 2015 at 10:17:54PM +0800, Geliang Tang wrote: > > > Use kmalloc_array instead of kmalloc to allocate memory for an array. > > > > > > Signed-off-b

Re: [PATCH] Staging: comedi: mite: added spaces around | and *

2016-02-22 Thread Joe Perches
On Tue, 2016-02-23 at 00:38 +0100, Philippe Loctaux wrote: > Added spaces around | and *, fixing 2 checkpatch checks. [] > diff --git a/drivers/staging/comedi/drivers/mite.c > b/drivers/staging/comedi/drivers/mite.c [] > @@ -51,7 +51,7 @@ >   >  #include "mite.h" >   > -#define TOP_OF_PAGE(x) ((x)

[PATCH] Staging: comedi: contec_pci_dio: fixed comment blocks coding style issues

2016-02-22 Thread Philippe Loctaux
Makes the comment blocks start with /* on separate lines, and end with */ on separate lines as well, starting with * for each comment lines. Signed-off-by: Philippe Loctaux --- drivers/staging/comedi/drivers/contec_pci_dio.c | 47 + 1 file changed, 24 insertions(+), 23 de

Re: [PATCH] staging/lustre: proper support of NFS anonymous dentries

2016-02-22 Thread Oleg Drokin
On Feb 14, 2016, at 7:58 PM, Greg Kroah-Hartman wrote: > On Sun, Feb 14, 2016 at 07:13:52PM -0500, gr...@linuxhacker.ru wrote: >> From: Dmitry Eremin >> >> NFS can ask to encode dentries that are not connected to the root. >> The fix check for parent is NULL and encode a file handle accordingly

Re: [PATCH] staging: rtl8723au: use list_first_entry*

2016-02-22 Thread Jes Sorensen
Geliang Tang writes: > Use list_first_entry*() instead of container_of() to simplify the code. > > Signed-off-by: Geliang Tang > --- > drivers/staging/rtl8723au/core/rtw_recv.c | 49 > +-- > drivers/staging/rtl8723au/core/rtw_xmit.c | 26 +--- > 2 files c

[PATCH 01/56] staging/lustre/fld: Remove useless typedefs

2016-02-22 Thread green
From: Oleg Drokin Typedefs fld_hash_func_t and fld_scan_func_t are used exectly once in the following structure so don't really help anything at all. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/fld/fld_internal.h | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-)

[PATCH 02/56] staging/lustre: Get rid of client_obd_lock_t typedef

2016-02-22 Thread green
From: Oleg Drokin Replace it with struct client_obd_lock Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/linux/obd.h | 12 ++-- drivers/staging/lustre/lustre/include/obd.h | 4 ++-- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/stagi

[PATCH 05/56] staging/lustre: Get rid of ost_cmd_t typedef

2016-02-22 Thread green
From: Oleg Drokin Replace it with enum ost_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lu

[PATCH 04/56] stagig/lustre: Get rid of cksum_type_t typedef

2016-02-22 Thread green
From: Oleg Drokin Replace it with enum cksum_type Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- drivers/staging/lustre/lustre/include/obd.h | 2 +- drivers/staging/lustre/lustre/include/obd_cksum.h | 12 ++--

[PATCH 03/56] staging/lustre: Get rid of loc_flags_t typedef

2016-02-22 Thread green
From: Oleg Drokin Replace it with direct reference of enum loc_flags Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lu_object.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lu_object.h b/drivers/staging/l

[PATCH 07/56] staging/lustre: Get rid of mds_cmd_t typedef

2016-02-22 Thread green
From: Oleg Drokin Replace it with enum mds_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lu

[PATCH 00/56] More Lustre cleanups.

2016-02-22 Thread green
From: Oleg Drokin This is another round of Lustre cleanups mostly aimed at fixing checkpatch warnings, but also removing unneeded code and fixign other things I am noticing along the way. Oleg Drokin (56): staging/lustre/fld: Remove useless typedefs staging/lustre: Get rid of client_obd_lock

[PATCH 08/56] staging/lustre: Get rid of mds_reint_t, mdt_reint_t typedefs

2016-02-22 Thread green
From: Oleg Drokin Replace with enum mdt_reint_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre

[PATCH 13/56] staging/lustre: Get rid of llog_op_type typedef

2016-02-22 Thread green
From: Oleg Drokin Replace with enum llog_op_type Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/

[PATCH 22/56] staging/lustre: Get rid of ldlm_cancel_flags_t typedef

2016-02-22 Thread green
From: Oleg Drokin Replace it with enum ldlm_cancel_flags Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre_dlm.h | 18 ++ drivers/staging/lustre/lustre/include/obd.h| 2 +- drivers/staging/lustre/lustre/include/obd_class.h | 2 +- drivers/st

[PATCH 06/56] staging/lustre: Get rid of quota_cmd_t typedef

2016-02-22 Thread green
From: Oleg Drokin Replace it with enum quota_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/

[PATCH 11/56] staging/lustre: Get rid of mgs_cmd_t typedef

2016-02-22 Thread green
From: Oleg Drokin Replace it with enum mgs_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lu

[PATCH 09/56] staging/lustre: Get rid of ldlm_cmd_t typedef

2016-02-22 Thread green
From: Oleg Drokin Replace it with enum ldlm_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/l

[PATCH 15/56] staging/lustre/ldlm: Remove unused lr_most_restr from struct ldlm_resource

2016-02-22 Thread green
From: Oleg Drokin This member comes from the dawn of time and is no longer needed Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre_dlm.h | 3 --- drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 3 --- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 1 - 3 f

[PATCH 17/56] staging/lustre: Get rid of lustre_fid typedef

2016-02-22 Thread green
From: Oleg Drokin Replace it with struct lu_fid Signed-off-by: Oleg Drokin --- .../lustre/lustre/include/lustre/lustre_idl.h | 2 +- .../lustre/lustre/include/lustre/lustre_user.h | 24 +++--- 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/drivers/st

[PATCH 12/56] staging/lustre: Get rid of obd_cmd_t typedef

2016-02-22 Thread green
From: Oleg Drokin Replace with enum obd_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lustr

[PATCH 10/56] staging/lustre: Remove ldlm type/mode typedefs

2016-02-22 Thread green
From: Oleg Drokin Replace ldlm_mode_t with enum ldlm_mode, ldlm_type_t with enum ldlm_type Signed-off-by: Oleg Drokin --- .../lustre/lustre/include/lustre/lustre_idl.h | 14 drivers/staging/lustre/lustre/include/lustre_dlm.h | 39 -- drivers/staging/lustre/lus

[PATCH 21/56] staging/lustre: Get rid of ldlm_ns_type_t typedef

2016-02-22 Thread green
From: Oleg Drokin Replace it with enum ldlm_ns_type Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre_dlm.h | 6 +++--- drivers/staging/lustre/lustre/ldlm/ldlm_lib.c | 2 +- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 4 ++-- 3 files changed, 6 insertion

[PATCH 37/56] staging/lustre: Remove last_rcvd-file related data

2016-02-22 Thread green
From: Oleg Drokin last_rcvd (last received) is a special server-side file on the Lustre FS that stores information about server filesystem and also list of connected clients and their state. There's no point in having any of tha ton the client, though. Signed-off-by: Oleg Drokin --- .../stagin

  1   2   >