For modules that call misc_register and misc_deregister in its module
init and exit methods without any additional code ends up being
boilerplate. This patch adds helper macro module_misc_device(), that
replaces module_init()/ module_exit() with template functions.
This patch also converts drivers
From: Markus Elfring
Date: Sun, 21 Aug 2016 09:03:21 +0200
Several update suggestions were taken into account
from static source code analysis.
Markus Elfring (7):
Use memdup_user() rather than duplicating its implementation
One function call less in aac_send_raw_srb() after error detection
On Sun, 2016-08-21 at 10:11 +0800, Chen-Yu Tsai wrote:
> Hi everyone,
>
> This series adds support for X-Powers' AXP806 PMIC. This is the
> secondary
> PMIC accompanying Allwinner's A80 SoC. For now, only the regulators
> are
> supported. While the AXP806 supports standalone operation, no
> hardwa
From: Markus Elfring
Date: Sat, 20 Aug 2016 20:05:24 +0200
Reuse existing functionality from memdup_user() instead of keeping
duplicate source code.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drivers/scsi/aacraid/commctrl.c | 12 +++-
1
>From e8187662ee30aab709a260c72fb86c51673f8e0d Mon Sep 17 00:00:00 2001
From: Markus Elfring
Date: Sat, 20 Aug 2016 20:40:47 +0200
Subject: [PATCH 2/7] aacraid: One function call less in aac_send_raw_srb()
after error detection
The kfree() function was called in a few cases by the
aac_send_raw_s
From: Markus Elfring
Date: Sat, 20 Aug 2016 21:25:20 +0200
Six local variables will be set to an appropriate value a bit later.
Thus omit the explicit initialisation at the beginning.
Signed-off-by: Markus Elfring
---
drivers/scsi/aacraid/commctrl.c | 12 ++--
1 file changed, 6 inserti
From: Markus Elfring
Date: Sun, 21 Aug 2016 07:07:08 +0200
Do not use curly brackets at some source code places
where a single statement should be sufficient.
Signed-off-by: Markus Elfring
---
drivers/scsi/aacraid/commctrl.c | 18 ++
1 file changed, 6 insertions(+), 12 deletion
From: Markus Elfring
Date: Sun, 21 Aug 2016 07:10:43 +0200
Keywords which belong to the category "control flow" in the C programming
language should be followed by a space character according to
the Linux coding style convention.
Signed-off-by: Markus Elfring
---
drivers/scsi/aacraid/commctrl.
From: Markus Elfring
Date: Sun, 21 Aug 2016 08:04:48 +0200
Replace the specification of data structures by references for variables
as the parameter for the operator "sizeof" to make the corresponding size
determination a bit safer according to the Linux coding style convention.
Signed-off-by: M
The Exynos PMU node is an interrupt, clock and PMU (Power Management Unit)
controller, and these functionalities are supported by different drivers
that matches the same compatible strings.
Since commit 15cc2ed6dcf9 ("of/irq: Mark initialised interrupt controllers
as populated") the OF core flags
From: Markus Elfring
Date: Sun, 21 Aug 2016 08:23:25 +0200
The script "checkpatch.pl" can point out that assignments should usually
not be performed within condition checks.
Thus move the assignment for the variable "srbfib" to a separate statement.
Signed-off-by: Markus Elfring
---
drivers/sc
2016-08-20 2:05 GMT+09:00 Sylwester Nawrocki :
> On 08/16/2016 08:35 AM, Chanwoo Choi wrote:
>> &spi_1 {
>> + cs-gpios = <&gpd6 3 GPIO_ACTIVE_HIGH>;
>> + status = "okay";
>> +
>> + wm5110: wm5110-codec@0 {
>> + compatible = "wlf,wm5110";
>> + reg = <0x0>;
>> +
> Am 20.08.2016 um 15:34 schrieb One Thousand Gnomes
> :
>> What it is not about are UART/RS232 converters connected through USB or
>> virtual
>> serial ports created for WWAN modems (e.g. /dev/ttyACM, /dev/ttyHSO). Or BT
>> devices
>> connected through USB (even if they also run HCI protocol).
> Am 20.08.2016 um 15:22 schrieb One Thousand Gnomes
> :
>
> On Fri, 19 Aug 2016 19:42:37 +0200
> "H. Nikolaus Schaller" wrote:
>
>>> Am 19.08.2016 um 13:06 schrieb One Thousand Gnomes
>>> :
>>>
If possible, please do a callback for every character that arrives.
And not only if the
TOROUT_STRING() macro function does not insert a space between flag and
message while other similar couterparts do. The output will be
inconsistent and weird especially when it is read by dmesg with color
option enabled. This commit adds an space between flag and message in
TOROUT_STRING() output
Few output messages of rcuperf has no space between flag and start of
message while every other messages keeps a space consistently. It makes
output messages to be inconsistent and weird especially when it be read
by dmesg with color option enabled. This commit fixes the problem by
modifying a pr
rcu_perf_writer_state is being written only while nobody reads it. This
commit removes the unnecessary variable and macro constants for it.
Signed-off-by: SeongJae Park
---
kernel/rcu/rcuperf.c | 16 ++--
1 file changed, 2 insertions(+), 14 deletions(-)
diff --git a/kernel/rcu/rcup
Hello All:
This version would move reset into phy. I wanted to implement a hard
code version but I meet some problem with ioremap reset controller,
in order to keep my word, I offer this version this time.
Changelog:
v5
A few modification at style, add the missing doc in the last
commit.
v4
The only use for this is for solving a hardware design problem in
usb of Rockchip RK3288.
Signed-off-by: Randy Li
---
include/linux/phy/phy.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/include/linux/phy/phy.h b/include/linux/phy/phy.h
index f08b672..4d34607 100644
--- a/include/linux/
It is a hardware bug in RK3288, the only way to solve it is to
reset the phy.
Signed-off-by: Randy Li
---
.../devicetree/bindings/phy/rockchip-usb-phy.txt | 3 +++
drivers/phy/phy-rockchip-usb.c | 20
2 files changed, 23 insertions(+)
diff --git a
On the rk3288 USB host-only port (the one that's not the OTG-enabled
port) the PHY can get into a bad state when a wakeup is asserted (not
just a wakeup from full system suspend but also a wakeup from
autosuspend).
We can get the PHY out of its bad state by asserting its "port reset",
but unfortun
The "host1" port (AKA the dwc2 port that isn't the OTG port) on rk3288
has a hardware errata that causes everything to get confused when we get
a remote wakeup. We'll use the reset that's in the CRU to reset the
port when it's in a bad state.
Note that we add the reset to both dwc2 controllers ev
lzma_decompress_to_file never actually close the file
pointer, let's fix it.
Signed-off-by: Shawn Lin
---
tools/perf/util/lzma.c | 19 ---
1 file changed, 12 insertions(+), 7 deletions(-)
diff --git a/tools/perf/util/lzma.c b/tools/perf/util/lzma.c
index 95a1acb..a95d473 100644
hi linux
http://fengjzn.com/clock.php?pair=eeqp1a69bz7bc8
Abhishek Rai
The 2nd parameter of 'find_first_bit' is the number of bits to search.
In this case, we are passing 'sizeof(unsigned long)' which is likely to
be 4.
It is likely that the number of bits in a long was expected here, so use
BITS_PER_LONG instead.
Signed-off-by: Christophe JAILLET
---
Other options
From: Markus Elfring
Date: Sun, 21 Aug 2016 10:39:04 +0200
Reuse existing functionality from memdup_user() instead of keeping
duplicate source code.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drivers/scsi/megaraid/megaraid_sas_base.c | 11 +++--
Window uses shared stride for UV planes and tegra_dc_window struct
defines array of 2 strides per window. That's not taken in account
during setting up of the window addresses and strides, resulting in
out-of-bounds write of the 3-rd (non-existent) V plane stride that
overwrites Y plane base addres
Am 20.08.2016 um 19:58 schrieb Mikko Rapeli:
Cc'ing lkml too.
On Fri, Aug 19, 2016 at 11:54:21PM +0100, Emil Velikov wrote:
Story time:
I was dreaming of a day were we can stop installing these headers,
thus making deprecation a bit easier process.
Yet after failing to convince Dave and Daniel
On Fri, Aug 19, 2016 at 8:41 AM, Eric Wheeler wrote:
>> On Mon, Aug 15, 2016 at 11:23:28AM -0700, Christoph Hellwig wrote:
>> > On Mon, Aug 15, 2016 at 11:11:22PM +0800, Ming Lei wrote:
>> > > After arbitrary bio size is supported, the incoming bio may
>> > > be very big. We have to split the bio
From: Markus Elfring
Date: Sun, 21 Aug 2016 11:30:57 +0200
Reuse existing functionality from memdup_user() instead of keeping
duplicate source code.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drivers/staging/lustre/lustre/llite/dir.c | 12 +++--
On Mon, Jun 13, 2016 at 6:48 PM, Theodore Ts'o wrote:
> +static inline void maybe_reseed_primary_crng(void)
> +{
> + if (crng_init > 2 &&
> + time_after(jiffies, primary_crng.init_time +
> CRNG_RESEED_INTERVAL))
> + crng_reseed(&primary_crng, &input_pool);
> +}
Hi,
On 08/19/2016 09:30 PM, Ross Zwisler wrote:
> On Fri, Aug 19, 2016 at 07:59:29AM -0700, Dan Williams wrote:
>> On Fri, Aug 19, 2016 at 4:19 AM, Xiao Guangrong
>> wrote:
>>>
>>> Hi Dan,
>>>
>>> Recently, Redhat reported that nvml test suite failed on QEMU/KVM,
>>> more detailed info please refer to
Le 21/08/2016 à 11:45, SF Markus Elfring a écrit :
From: Markus Elfring
Date: Sun, 21 Aug 2016 11:30:57 +0200
Reuse existing functionality from memdup_user() instead of keeping
duplicate source code.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
On Sun, 21 Aug 2016, Christophe JAILLET wrote:
> Le 21/08/2016 à 11:45, SF Markus Elfring a écrit :
> > From: Markus Elfring
> > Date: Sun, 21 Aug 2016 11:30:57 +0200
> >
> > Reuse existing functionality from memdup_user() instead of keeping
> > duplicate source code.
> >
> > This issue was det
On 08/20/2016 01:17 AM, Michael Kerrisk (man-pages) wrote:
On 08/20/2016 08:56 AM, Michael Kerrisk (man-pages) wrote:
On 08/19/2016 08:30 PM, Vegard Nossum wrote:
Is there any reason why we couldn't do the (size > pipe_max_size) check
before calling account_pipe_buffers()?
No reason that I ca
Cc: proper lists.
ep->desc.bInterval seems to be 0 here.
On 08/21/2016, 12:42 PM, Vittorio Zecca wrote:
> I am not sure this is the right place so please bear with me...
> From Vittorio Zecca
>
> After compiling kernel 4.7.2 with ubsan I got the following messages
> at boot time:
>
> (devio.c:1
On Sunday 21 August 2016 04:01 PM, Julia Lawall wrote:
>
>
> On Sun, 21 Aug 2016, Christophe JAILLET wrote:
>
>> Le 21/08/2016 à 11:45, SF Markus Elfring a écrit :
>>> From: Markus Elfring
>>> Date: Sun, 21 Aug 2016 11:30:57 +0200
>>>
>>> Reuse existing functionality from memdup_user() instea
On Sun, 21 Aug 2016, Vaishali Thakkar wrote:
>
>
> On Sunday 21 August 2016 04:01 PM, Julia Lawall wrote:
> >
> >
> > On Sun, 21 Aug 2016, Christophe JAILLET wrote:
> >
> >> Le 21/08/2016 à 11:45, SF Markus Elfring a écrit :
> >>> From: Markus Elfring
> >>> Date: Sun, 21 Aug 2016 11:30:57 +0200
Hello.
On 8/21/2016 10:56 AM, Randy Li wrote:
The "host1" port (AKA the dwc2 port that isn't the OTG port) on rk3288
has a hardware errata that causes everything to get confused when we get
a remote wakeup. We'll use the reset that's in the CRU to reset the
port when it's in a bad state.
Note
On 16/08/16 16:27, Lars-Peter Clausen wrote:
> On 08/15/2016 05:54 PM, Jonathan Cameron wrote:
>> On 09/08/16 01:19, Brian Norris wrote:
>>> When using CONFIG_DEBUG_ATOMIC_SLEEP, the scheduler nicely points out
>>> that we're calling sleeping primitives within the wait_event loop, which
>>> means w
On Fri, Aug 19, 2016 at 8:34 PM, Peter Zijlstra wrote:
> Why are you wanting to use nohz_full if you do syscalls?
We hope to reduce the overhead of the tick while the real time
applications run,
and these applications might do some syscalls to operate the I/O devices like
EtherCAT.
On Sun, Aug 21, 2016 at 12:53:15PM +0300, Jan Varho wrote:
> On Mon, Jun 13, 2016 at 6:48 PM, Theodore Ts'o wrote:
> > +static inline void maybe_reseed_primary_crng(void)
> > +{
> > + if (crng_init > 2 &&
> > + time_after(jiffies, primary_crng.init_time +
> > CRNG_RESEED_INTERVAL)
Hi guys,
On Sun, Aug 14, 2016 at 10:07:45PM +0200, Greg Kroah-Hartman wrote:
> 3.14-stable review patch. If anyone has any objections, please let me know.
>
> --
>
> From: Fabian Frederick
>
> commit 9b24fef9f0410fb5364245d6cc2bd044cc064007 upstream.
>
> Commit 53dad6d3a8e5 (
Hi,
I've been seeing this:
[ cut here ]
WARNING: CPU: 0 PID: 2377 at fs/inode.c:280 drop_nlink+0xc2/0x120
Kernel panic - not syncing: panic_on_warn set ...
CPU: 0 PID: 2377 Comm: syz-executor Not tainted 4.8.0-rc2+ #145
Hardware name: QEMU Standard PC (i440FX + PIIX, 199
Theodore Ts'o writes:
> On Sat, Aug 20, 2016 at 02:31:26PM +0300, Dmitry Monakhov wrote:
>> > I'm not sure I see the advantage of doing this in a container, I
>> > guess. I just do in my standard laptop environment today.
>> I can not because I laptop from famous thinkpad t430 series with
>> fla
On 18 August 2016 at 12:52, tip-bot for David Carrillo-Cisneros
wrote:
> Commit-ID: 71e7bc2bab77e64882c031c2af943c3256c1adb0
> Gitweb: http://git.kernel.org/tip/71e7bc2bab77e64882c031c2af943c3256c1adb0
> Author: David Carrillo-Cisneros
> AuthorDate: Wed, 17 Aug 2016 13:55:04 -0700
> Comm
On Sunday 21 August 2016 04:31 PM, Julia Lawall wrote:
>
>
> On Sun, 21 Aug 2016, Vaishali Thakkar wrote:
>
>>
>>
>> On Sunday 21 August 2016 04:01 PM, Julia Lawall wrote:
>>>
>>>
>>> On Sun, 21 Aug 2016, Christophe JAILLET wrote:
>>>
Le 21/08/2016 à 11:45, SF Markus Elfring a écrit :
>>>
Hi Marcel, Johan,
I am unable to unload module bluetooth to verify that the second
leak is not a false positive; however, the one in btusb is a real
memory leak.
There was a bugzilla last week with that backtrace:
https://bugzilla.kernel.org/show_bug.cgi?id=120691
At the time, I was thinkin
On 08/21/2016 07:06 PM, Sergei Shtylyov wrote:
Hello.
On 8/21/2016 10:56 AM, Randy Li wrote:
The "host1" port (AKA the dwc2 port that isn't the OTG port) on rk3288
has a hardware errata that causes everything to get confused when we get
a remote wakeup. We'll use the reset that's in the CRU
On 08/21/2016 01:21 PM, Jonathan Cameron wrote:
[...]
> I've applied to this to the fixes-togreg branch of iio.git
>
> For now I haven't marked it for stable, purely because I'm not sure
> when the first 'problem' usage was introduced. I'm happy to explicitly
> send a request for stable inclusion
Dear List, Thomas, Gregory
Commit cb4f71c4298853db0c6751b1209e4535956f136c changes the order of
the network interfaces for armada-38x. As a special exception to the
"order by register address" rule says the comment in the dtsi. The
commit messages even calls it a violation.
I can't remember havin
On 六, 2016-08-20 at 08:43 +0800, Shawn Guo wrote:
> On Wed, May 25, 2016 at 9:45 PM, Peter Robinson > wrote:
> >
> > Not much use unless the SoC is selected so depend on the ARCH_MXC
> > and COMPILE_TEST like all the other thermal drivers.
> >
> > v2: drop extraneous OF
> >
> > Signed-off-by: P
The "exist" field is only checked when "roccat_open" has already been
called or when we have made sure that the corresponding roccat_device is
not NULL. Since the value of the "open" field has been increased by the
"roccat_open" call, instead of checking "exist" we can just check if
"open" is equal
From: Markus Elfring
Date: Sun, 21 Aug 2016 15:41:23 +0200
Several update suggestions were taken into account
from static source code analysis.
Markus Elfring (7):
Use memdup_user() rather than duplicating its implementation
Delete unnecessary initialisations for the variable "dev"
Delete
From: Markus Elfring
Date: Sun, 21 Aug 2016 12:48:27 +0200
Reuse existing functionality from memdup_user() instead of keeping
duplicate source code.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
drivers/usb/misc/iowarrior.c | 11 +++
1 fil
From: Markus Elfring
Date: Sun, 21 Aug 2016 13:26:09 +0200
The local variable "dev" was initialised despite of the detail
that it was immediately reassigned by the following statement.
Thus remove such unnecessary specifications.
Signed-off-by: Markus Elfring
---
drivers/usb/misc/iowarrior.c |
From: Markus Elfring
Date: Sun, 21 Aug 2016 13:45:09 +0200
The local variable "retval" will be set to an appropriate value
a bit later. Thus omit the explicit initialisation at the beginning.
Signed-off-by: Markus Elfring
---
drivers/usb/misc/iowarrior.c | 2 +-
1 file changed, 1 insertion(+),
From: Markus Elfring
Date: Sun, 21 Aug 2016 15:15:03 +0200
Two local variables will be set to an appropriate value a bit later.
Thus omit the explicit initialisation at the beginning.
Signed-off-by: Markus Elfring
---
drivers/usb/misc/iowarrior.c | 4 ++--
1 file changed, 2 insertions(+), 2 de
From: Markus Elfring
Date: Sun, 21 Aug 2016 15:17:22 +0200
Two local variables will be set to an appropriate value a bit later.
Thus omit the explicit initialisation at the beginning.
Signed-off-by: Markus Elfring
---
drivers/usb/misc/iowarrior.c | 4 ++--
1 file changed, 2 insertions(+), 2 de
From: Markus Elfring
Date: Sun, 21 Aug 2016 15:25:30 +0200
Do not use curly brackets at a few source code places
where a single statement should be sufficient.
Signed-off-by: Markus Elfring
---
drivers/usb/misc/iowarrior.c | 22 --
1 file changed, 8 insertions(+), 14 deleti
From: Markus Elfring
Date: Sun, 21 Aug 2016 15:30:11 +0200
The script "checkpatch.pl" can point out that assignments should usually
not be performed within condition checks.
Thus move the assignments for two local variables to separate statements.
Signed-off-by: Markus Elfring
---
drivers/usb/
From: Colin Ian King
trivial typo fix in pr_debug message
Signed-off-by: Colin Ian King
---
tools/perf/tests/backward-ring-buffer.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/tests/backward-ring-buffer.c
b/tools/perf/tests/backward-ring-buffer.c
index 61578
From: Colin Ian King
trivial typo fix in pr_debug message
Signed-off-by: Colin Ian King
---
tools/perf/tests/bpf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/tests/bpf.c b/tools/perf/tests/bpf.c
index fc54064..2673e86 100644
--- a/tools/perf/tests/bpf.c
+++
Hi,
As Eduardo is quite busy recently, I will take all the thermal soc
driver changes this time.
Currently, I have cherry picked all the patches that Eduardo has
already applied in thermal-soc tree and reviewed all the pending
patches in patchwork. Now, there are four patches queued for next -rc (
From: Colin Ian King
trivial typo fix in pr_debug message
Signed-off-by: Colin Ian King
---
tools/perf/util/bpf-loader.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/util/bpf-loader.c b/tools/perf/util/bpf-loader.c
index 1f12e4e..2b2c9b8 100644
--- a/tools/per
From: Colin Ian King
trivial typo fix in pr_err message
Signed-off-by: Colin Ian King
---
drivers/net/ethernet/tehuti/tehuti.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/tehuti/tehuti.c
b/drivers/net/ethernet/tehuti/tehuti.c
index 7452b5f..7108c68
On Sun, Aug 21, 2016 at 03:02:38PM +0300, Dmitry Monakhov wrote:
> Definitely. But in your case we still need jessie64's chroot.
> It should be prepared somehow. My point is that if we give an automated build
> script this may help people to start with xfstests with less pain.
> In my experience ti
On Sun, Aug 21, 2016 at 07:26:04PM +0800, GeHao Kang wrote:
> On Fri, Aug 19, 2016 at 8:34 PM, Peter Zijlstra wrote:
>
> > Why are you wanting to use nohz_full if you do syscalls?
>
> We hope to reduce the overhead of the tick while the real time
> applications run,
> and these applications migh
On 27/07/16 10:28, Matt Ranostay wrote:
> On Mon, Jul 25, 2016 at 3:06 PM, Colin King wrote:
>> From: Colin Ian King
>>
>> Fix an incorrect assignment due to a typo on a variable name. The
>> variable val2 should be assigned 10 and not val.
>
> Yikes! Good catch..
>
> Reviewed-By: Matt Rano
On 25/07/16 11:57, Pratik Prajapati wrote:
> Signed-off-by: Pratik Prajapati
Sorry it took me so long to get to this. Wading back through my
emails post holiday still.
Small issue in the documentation.
Jonathan
> ---
> Changes v1 -> v2:
> - documented current_led files in ABI
> - masked val
From: Wei Yongjun
Fixes the following sparse warning:
drivers/gpu/drm/i915/intel_hotplug.c:480:6: warning:
symbol 'i915_hpd_poll_init_work' was not declared. Should it be static?
Also move the '{' to new line.
Signed-off-by: Wei Yongjun
---
drivers/gpu/drm/i915/intel_hotplug.c | 3 ++-
1 fi
On 25/07/16 11:57, Pratik Prajapati wrote:
> Signed-off-by: Pratik Prajapati
Why? I.e. Why replace it with regmap. I kind of assumed this was
to add spi support on a dual bus chip, but it doesn't look like
vcnl4000 has an spi version...
I wouldn't have minded if the driver had originally been wr
From: Chao Yu
During fstrim, if one of multiple write_checkpoint failed, break off and
return error number to caller.
Signed-off-by: Chao Yu
---
fs/f2fs/segment.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c
index a394012..020767c 100644
--- a/fs/
From: Chao Yu
In available_free_memory, there are two same judgement conditions which
is used for checking NAT excess, remove one of them.
Signed-off-by: Chao Yu
---
fs/f2fs/node.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/fs/f2fs/node.c b/fs/f2fs/node.c
index f75d197..8a28800 10064
On 21/08/16 13:26, Lars-Peter Clausen wrote:
> On 08/21/2016 01:21 PM, Jonathan Cameron wrote:
> [...]
>> I've applied to this to the fixes-togreg branch of iio.git
>>
>> For now I haven't marked it for stable, purely because I'm not sure
>> when the first 'problem' usage was introduced. I'm happy
From: Chao Yu
In batch discard approach of fstrim will grab/release gc_mutex lock
repeatly, it makes contention of the lock becoming more intensive.
So after one batch discards were issued in checkpoint and the lock
was released, it's better to do schedule() to increase opportunity
of grabbing g
From: Florian Westphal
commit a08e4e190b866579896c09af59b3bdca821da2cd upstream.
The target size includes the size of the xt_entry_target struct.
Signed-off-by: Florian Westphal
Signed-off-by: Pablo Neira Ayuso
Signed-off-by: Willy Tarreau
---
net/netfilter/x_tables.c | 3 +++
1 file change
From: Florian Westphal
commit 832756f6fe8e4e82a63361119b7e2384e02f upstream.
Signed-off-by: Florian Westphal
Signed-off-by: Pablo Neira Ayuso
Signed-off-by: Willy Tarreau
---
net/ipv4/netfilter/arp_tables.c | 82 ++---
1 file changed, 36 insertions(+),
From: Florian Westphal
commit ce683e5f9d045e5d67d1312a42b359cb2ab2a13c upstream.
We're currently asserting that targetoff + targetsize <= nextoff.
Extend it to also check that targetoff is >= sizeof(xt_entry).
Since this is generic code, add an argument pointing to the start of the
match/target
From: Dave Chinner
commit 51b07f30a71c27405259a0248206ed4e22adbee2 upstream.
Some careless idiot(*) wrote crap code in commit 1a3e8f3 ("xfs:
convert inode cache lookups to use RCU locking") back in late 2010,
and so xfs_iflush_cluster checks the wrong inode for whether it is
still valid under RC
From: Kangjie Lu
commit 9a47e9cff994f37f7f0dbd9ae23740d0f64f9fe6 upstream.
The stack object âr1â has a total size of 32 bytes. Its field
âeventâ and âvalâ both contain 4 bytes padding. These 8 bytes
padding bytes are sent to user without being initialized.
Signed-off-by: Kangjie Lu
From: James Hogan
commit 9b731bcfdec4c159ad2e4312e25d69221709b96a upstream.
Propagate errors from kvm_mips_handle_kseg0_tlb_fault() and
kvm_mips_handle_mapped_seg_tlb_fault(), usually triggering an internal
error since they normally indicate the guest accessed bad physical
memory or the commpage
From: Steve French
commit 4fcd1813e6404dd4420c7d12fb483f9320f0bf93 upstream.
Azure server blocks clients that open a socket and don't do anything on it.
In our reconnect scenarios, we can reconnect the tcp session and
detect the socket is available but we defer the negprot and SMB3 session
setup
From: Andrew Goodbody
commit f3eec0cf784e0d6c47822ca6b66df3d5812af7e6 upstream.
shared_fifo endpoints would only get a previous tx state cleared
out, the rx state was only cleared for non shared_fifo endpoints
Change this so that the rx state is cleared for all endpoints.
This addresses an issue
From: Mike Snitzer
commit 99f3c90d0d85708e7401a81ce3314e50bf7f2819 upstream.
When the corrupt_bio_byte feature was introduced it caused READ bios to
no longer be errored with -EIO during the down_interval. This had to do
with the complexity of needing to submit READs if the corrupt_bio_byte
fea
From: Ursula Braun
commit 7831b4ff0d926e0deeaabef9db8800ed069a2757 upstream.
A qeth_card contains a napi_struct linked to the net_device during
device probing. This struct must be deleted when removing the qeth
device, otherwise Panic on oops can occur when qeth devices are
repeatedly removed an
From: Wei Fang
commit 72d8c36ec364c82bf1bf0c64dfa1041cfaf139f7 upstream.
sas_ata_strategy_handler() adds the works of the ata error handler to
system_unbound_wq. This workqueue asynchronously runs work items, so the
ata error handler will be performed concurrently on different CPUs. In
this case
From: Oliver Hartkopp
commit 25e1ed6e64f52a692ba3191c4fde650aab3ecc07 upstream.
For 'real' hardware CAN devices the netlink interface is used to set CAN
specific communication parameters. Real CAN hardware can not be created nor
removed with the ip tool ...
This patch adds a private dellink fun
From: Lyude
commit 14ff8d48f2235295dfb3117693008e367b49cdb5 upstream.
DRM_CONNECTOR_POLL_CONNECT only enables polling for connections, not
disconnections. Because of this, we end up losing hotplug polling for
analog connectors once they get connected.
Easy way to reproduce:
- Grab a machine wi
From: Takashi Iwai
commit f388cdcdd160687c6650833f286b9c89c50960ff upstream.
snd_ctl_remove() has a notification for the removal event. It's
superfluous when done during the device got disconnected. Although
the notification itself is mostly harmless, it may potentially be
harmful, and should
From: "Charles (Chas) Williams"
commit 75ff39ccc1bd5d3c455b6822ab09e533c551f758 upstream.
From: Eric Dumazet
Yue Cao claims that current host rate limiting of challenge ACKS
(RFC 5961) could leak enough information to allow a patient attacker
to hijack TCP sessions. He will soon provide detail
From: Matthias Schiffer
commit f5b556c94c8490d42fea79d7b4ae0ecbc291e69d upstream.
This makes the ath79 bootconsole behave the same way as the generic 8250
bootconsole.
Also waiting for TEMT (transmit buffer is empty) instead of just THRE
(transmit buffer is not full) ensures that all characters
From: Jeff Mahoney
commit 78c4e172412de5d0456dc00d2b34050aa0b683b5 upstream.
This reverts commit 2f36db71009304b3f0b95afacd8eba1f9f046b87.
It fixed a local root exploit but also introduced a dependency on
the lower file system implementing an mmap operation just to open a file,
which is a bit o
From: Florian Westphal
commit 7d3f843eed2954c9feab481f55175a1afcc9 upstream.
Signed-off-by: Florian Westphal
Signed-off-by: Pablo Neira Ayuso
Signed-off-by: Willy Tarreau
---
net/ipv4/netfilter/ip_tables.c | 61 +-
1 file changed, 25 insertions(+),
From: Linus Walleij
commit 0c1f91b98552da49d9d8eed32b3132a58d2f4598 upstream.
These two spi_w8r8() calls return a value with is used by the code
following the error check. The dubious use was caused by a cleanup
patch.
Fixes: d34dbee8ac8e ("staging:iio:accel:kxsd9 cleanup and conversion to
iio
From: Theodore Ts'o
commit c9eb13a9105e2e418f72e46a2b6da3f49e696902 upstream.
If the orphaned inode list contains inode #5, ext4_iget() returns a
bad inode (since the bootloader inode should never be referenced
directly). Because of the bad inode, we end up processing the inode
repeatedly and t
From: Herbert Xu
commit 5f070e81bee35f1b7bd1477bb223a873ff657803 upstream.
When there is more data to be processed, the current test in
scatterwalk_done may prevent us from calling pagedone even when
we should.
In particular, if we're on an SG entry spanning multiple pages
where the last page i
From: Andi Kleen
commit 54c2f3fdb941204cad136024c7b854b7ad112ab6 upstream.
Signed-off-by: Andi Kleen
Link:
http://lkml.kernel.org/r/1375740170-7446-12-git-send-email-a...@firstfloor.org
Signed-off-by: H. Peter Anvin
Signed-off-by: Willy Tarreau
---
arch/x86/kernel/apm_32.c | 2 +-
1 file ch
From: Antonio Alecrim Jr
commit d6d6d1bc44362112e10a48d434e5b3c716152003 upstream.
Signed-off-by: Antonio Alecrim Jr
Signed-off-by: David S. Miller
Signed-off-by: Willy Tarreau
---
drivers/isdn/hardware/mISDN/hfcpci.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/dr
From: Alex Deucher
commit 3edc38a0facef45ee22af8afdce3737f421f36ab upstream.
Some of the checks didn't handle frev 2 tables properly.
Signed-off-by: Alex Deucher
Cc: sta...@vger.kernel.org
Signed-off-by: Willy Tarreau
---
drivers/gpu/drm/radeon/radeon_atombios.c | 4 ++--
1 file changed, 2 i
1 - 100 of 499 matches
Mail list logo
