Re: [PATCH] rtlwifi: Fix size of wireless mode variable

Larry Finger writes: >> I can't find commit acc6907b87a9 from any of my trees. And oddly enough >> I can't either any commits with title "rtlwifi: Fix warning from >> ieee80211_get_tx_rates() when using 5G". I can fix it before commiting >> but what should I use? >> >> Also the fixes line should

Re: rtlwifi: Fix size of wireless mode variable

> Smatch reports the following warning: > > CHECK drivers/net/wireless/realtek/rtlwifi/rc.c > drivers/net/wireless/realtek/rtlwifi/rc.c:144 _rtl_rc_rate_set_series() warn: > impossible condition '(wireless_mode == 256) => (0-255 == 256)' > > This warning arises because commit acc6907b87a9 (

[PATCH] android,lowmemorykiller: Don't abuse TIF_MEMDIE.

Currently, lowmemorykiller (LMK) is using TIF_MEMDIE for two purposes. One is to remember processes killed by LMK, and the other is to accelerate termination of processes killed by LMK. But since LMK is invoked as a memory shrinker function, there still should be some memory available. It is very

Re: [PATCH] android,lowmemorykiller: Don't abuse TIF_MEMDIE.

On Tue 08-03-16 20:01:32, Tetsuo Handa wrote: > Currently, lowmemorykiller (LMK) is using TIF_MEMDIE for two purposes. > One is to remember processes killed by LMK, and the other is to > accelerate termination of processes killed by LMK. > > But since LMK is invoked as a memory shrinker function,

Re: [PATCH 1/1] staging: android: Fixed coding style issues reported by checkpatch

On Sun, Mar 06, 2016 at 05:21:08PM -0800, Greg KH wrote: > You need to do only one thing per patch, please split this up into > multiple patches. Created two separate patches out of it having same logical change in each. > Also verify that these are all really needed, please always work against >

[PATCH 0/4] Support lustre routers with stand alone LNet stack

Often lustre file systems will employ LNet routers in their set ups. Those routers tend to be very light weight so we want to avoid installing additional software that will burden the system. Installing the entire lustre client can be pretty heavy so we want to give the ability to only enable the L

[PATCH 3/4] staging: lustre: add help section of Kconfig config LNET

Include a help section for Kconfig LNET. Signed-off-by: James Simmons --- drivers/staging/lustre/lnet/Kconfig |8 +++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/drivers/staging/lustre/lnet/Kconfig b/drivers/staging/lustre/lnet/Kconfig index 9c85091..980ce44 100644 -

[PATCH 1/4] staging: lustre: move libcfs to lnet layer

The lustre file system has a layered architecture with libcfs as the lowest layer and LNet layered on top. Then on top of LNet we run the lustre client. This patch moves the libcfs module code out of lustre into the lnet tree. This fits into the long term goal of eventually merging libcfs into LNet

[PATCH 4/4] staging: lustre: make lustre dependent on LNet

In the case of lustre routers you only need a functioning LNet stack. Especially since often the routers are very light weight and want to avoid any addition software that would create additional pressures on the system. Signed-off-by: James Simmons --- drivers/staging/Makefile |

[PATCH 2/4] staging: lustre: fixup kernel Kconfig option LNET_MAX_PAYLOAD

A few errors exist for the Kconfig option LNET_MAX_PAYLOAD. First mistake is the default size is 1MB not 2MB as it is shown to the person configuring the kernel. Second the LNET_MAX_PAYLOAD option is more closely related to LNET than the LUSTRE_FS option. Signed-off-by: James Simmons --- drivers

[PATCH 0/6] staging: unisys: visorbus: cleanup gotos in visorbus_main.c

This patchset cleans up the gotos found in visorbus_main.c David Kershner (6): staging: unisys: visorbus: fix up gotos in visorbus_init staging: unisys: visorbus: Remove gotos in visorbus_match staging: unisys: visorbus: git rid of gotos in devmajorminor_create_file staging: unisys: v

[PATCH 2/6] staging: unisys: visorbus: Remove gotos in visorbus_match

Gotos in visorbus_match are not needed. Signed-off-by: David Kershner Signed-off-by: Timothy Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 24 ++-- 1 file changed, 10 insertions(+), 14 deletions(-) diff --git a/drivers/staging/unisys/visorbus/visorbus_main.c b

[PATCH 3/6] staging: unisys: visorbus: git rid of gotos in devmajorminor_create_file

The gotos in devmajorminor_create_file aren't needed, get rid of them. Signed-off-by: David Kershner Signed-off-by: Timothy Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 25 - 1 file changed, 8 insertions(+), 17 deletions(-) diff --git a/drivers/staging/uni

[PATCH 1/6] staging: unisys: visorbus: fix up gotos in visorbus_init

This patch fixes the gotos in visorbus_init Signed-off-by: David Kershner Signed-off-by: Timothy Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 24 +++- 1 file changed, 11 insertions(+), 13 deletions(-) diff --git a/drivers/staging/unisys/visorbus/visorbus_main.

[PATCH 4/6] staging: unisys: visorbus: git rid of gotos in register_devmajorminor_attributes

Gotos are not needed in register_devmajorminor_attributes, get rid of them. Signed-off-by: David Kershner Signed-off-by: Timothy Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 13 + 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/drivers/staging/unisys/viso

[PATCH 5/6] staging: unisys: visorbus: Cleanup gotos in visordriver_probe_device

Visordriver_probe_device gotos were messy, clean them up. Signed-off-by: David Kershner Signed-off-by: Timothy Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/drivers/staging/unisys/visorbus/visorbus_main

[PATCH 6/6] staging: unisys: visorbus: rename visordriver_probe_device gotos

Away is ambiguous when specifying error vs success. Make return labels more meaningful by marking them as error paths. Signed-off-by: David Kershner Signed-off-by: Timothy Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-)

[PATCH] staging:vt6655: fix coding style violations for lines over 80 characters

This patch addresses line length errors reported by checkpatch.pl that could be fixed with simple line breaks. Signed-off-by: Kathryn Hampton --- drivers/staging/vt6655/baseband.c | 20 ++--- drivers/staging/vt6655/baseband.h | 6 ++- drivers/staging/vt6655/card.c | 95 +

Re: [PATCH] scsi: storvsc: fix SRB_STATUS_ABORTED handling

> "Vitaly" == Vitaly Kuznetsov writes: Vitaly> Commit 3209f9d780d1 ("scsi: storvsc: Fix a bug in the handling Vitaly> of SRB status flags") filtered SRB_STATUS_AUTOSENSE_VALID out Vitaly> effectively making the (SRB_STATUS_ABORTED | Vitaly> SRB_STATUS_AUTOSENSE_VALID) case a dead code. The lo

[PATCH 2/2] staging: unisys: visorbus: clean up visorchannel_clear

Goto label was ambiguous change to out_free_page to signify that we will always go through the path even on success. Signed-off-by: David Kershner Signed-off-by: Timothy Sell --- drivers/staging/unisys/visorbus/visorchannel.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 0/2] staging: unisys: visorbus: cleanup gotos in visorchannel.c

This patchset cleans up the gotos in the file drivers/staging/unisys/visorbus/visorchannel.c. David Kershner (2): staging: unisys: visorbus: cleanup gotos in visorchannel_create_guts staging: unisys: visorbus: clean up visorchannel_clear drivers/staging/unisys/visorbus/visorchannel.c | 18 +

[PATCH 1/2] staging: unisys: visorbus: cleanup gotos in visorchannel_create_guts

Away label is ambiguous change to it err_destroy_channel to make it clear this is an error path. Signed-off-by: David Kershner Signed-off-by: Timothy Sell --- drivers/staging/unisys/visorbus/visorchannel.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/driver

[PATCH] staging: unisys: visorbus: remove unused sysfs attribute devmajorminor/*

From: Tim Sell The sysfs attribute directory at: /sys/bus/visorbus/devices/vbus:dev/devmajorminor/* or /sys/devices/visorbus/vbus:dev/devmajorminor/* previously provided a location where a visorbus function driver could publish information (for usermode use) about possibly-multiple ma