Fix sparse warning "warning: Variable length array is used." by using
kmalloc_array to allocate the required amount of memory instead and
kfree to deallocate memory after use.
Signed-off-by: Guru Das Srinagesh
---
v4:
- Changed kmalloc_array flags from GFP_KERNEL to GFP_ATOMIC
v3:
- Fixe
* Kees Cook wrote:
> > There's the option of using GCC plugins now that the infrastructure was
> > upstreamed from grsecurity. It can be used as part of the regular build
> > process and as long as the analysis is pretty simple it shouldn't hurt
> > compile
> > time much.
>
> Well, and that
SoC platform changes (arch/arm/mach-*). This merge window, the bulk is
for a few platforms:
* Gemini:
- Legacy platform that Linus Walleij has converted to multiplatform
and DT, so a handful of various tweaks there, removal of some old
stale support, etc.
* Atmel AT91:
- Fixup of various power ma
Device-tree updates for arm64 platforms. Just as with 32-bit, a bunch of smaller
changes, but also some new platforms that are worth mentioning:
* Rockchip RK3399 platforms for Chromebooks, including Samsung Chromebook
Plus (Kevin)
* Orange Pi PC2 (Allwinner H5)
* Freescale LS2088A and LS1088A SoC
We've traditionally kept defconfig updates in a separate branch, often to
encourage submaintainers to handle those patches separately to avoid conflicts
on the shared files. The amount of changes seem to be decreasing though, so
we might rethink how we handle this going forward.
There really isn't
Device-tree continues to see lots of updates. The majority of patches
here are smaller changes for new hardware on existing platforms, and
there are a few larger changes worth pointing out.
Major new platforms:
- Gemini has been ported to DT, so a handful of "new" platforms moved over
from board
Changes to platform code for 64-bit ARM platforms.
Most of these are small changes to the one defconfig we use on arm64
(no per-platform configs there), to enable new drivers.
There are also a few other changes. Broadcom sold off their 'Vulcan'
design to Cavium, where it is now called ThunderX2.
Driver updates for ARM SoCs.
* Reset subsystem, merged through arm-soc by tradition:
- Make bool drivers explicitly non-modular
- New support for i.MX7 and Arria10 reset controllers
* PATA driver for Palmchip BK371 (acked by Tejun)
* Power domain drivers for i.MX (GPC, GPCv2)
- Moved out of mach
Smaller patches that didn't seem to find a home in other branches, and
low-priority fixes from late in the merge window. A number of these are
MAINTAINER updates, it seems.
Highlights:
* Maintainers:
- Remove Alexandre Courbot and Stephen Warren from Tegra maintainership,
add Jon Hunter
- Remove
Hi Linus,
It's been a relatively quiet release cycle here. Patch count is about
the usual (818 commits, which includes merges). Accumulated diffstat:
683 files changed, 28234 insertions(+), 7251 deletions(-)
Dirstat is also more or less as we're used to it these days. Since arm's dts
structure
Commit-ID: 4a1bec4605838fd7872ec41677585e241e256785
Gitweb: http://git.kernel.org/tip/4a1bec4605838fd7872ec41677585e241e256785
Author: Nick Desaulniers
AuthorDate: Mon, 8 May 2017 20:29:46 -0700
Committer: Ingo Molnar
CommitDate: Tue, 9 May 2017 08:16:45 +0200
x86/build: Don't add -mac
* Greg KH wrote:
> What about a simple coccinelle script to test for this type of thing?
> We write it once, add it to the in-kernel body of tests, and then 0-day
> runs it on all trees all the time. That should catch this type of
> issue, like all of the other "bad programming bus" that the to
Commit-ID: 861ce4a3244c21b0af64f880d5bfe5e6e2fb9e4a
Gitweb: http://git.kernel.org/tip/861ce4a3244c21b0af64f880d5bfe5e6e2fb9e4a
Author: Laura Abbott
AuthorDate: Mon, 8 May 2017 14:23:16 -0700
Committer: Ingo Molnar
CommitDate: Tue, 9 May 2017 08:12:27 +0200
x86/mm/32: Set the '__vmalloc
On Mon, May 08, 2017 at 10:32:30PM +0300, Konstantin Khlebnikov wrote:
> On Mon, May 8, 2017 at 10:22 PM, Daniel Axtens wrote:
> > I recently had to debug a problem with these (we hadn't backported
> > Konstantin's patches in this area) and this would have saved a lot
> > of time/pain.
>
> Yep,
Hi Linux
http://www.carpediemeventos.cl/case-studies.php?france=2k712cqa9ksz
Chris
Hi,
Darren, Dmitry, thank you for the review.
On 08-05-17 23:04, Darren Hart wrote:
On Mon, May 08, 2017 at 09:56:31PM +0200, Hans de Goede wrote:
PEAQ is a new European OEM, I've bought one of their 2-in-1 x86
devices, which is actually quite a nice device. Under Windows it has
Dolby software
On Mon, May 08, 2017 at 10:30:13PM -0300, Fabio Estevam wrote:
> Hi Peter,
>
> On Mon, May 8, 2017 at 12:02 PM, Peter Senna Tschudin
> wrote:
> > Hello Kishon,
> >
> > Our iMX6 hardware (imx6q-b850v3.dts) is not booting with latest
> > linux-next and I could bisect until:
> >
> > commit 442ec4c04
* Kees Cook wrote:
> On Mon, May 8, 2017 at 7:02 AM, Ingo Molnar wrote:
> >
> > * Kees Cook wrote:
> >
> >> > And yes, I realize that there were other such bugs and that such bugs
> >> > might
> >> > occur in the future - but why not push the overhead of the security
> >> > check to
> >> > t
Bjorn Andersson writes:
> The SMD channel is not the primary WCNSS channel and must explicitly be
> closed as the device is removed, or the channel will already by open on
> a subsequent probe call in e.g. the case of reloading the kernel module.
>
> This issue was introduced because I simplified
On Monday 08 May 2017 07:42 PM, Daniel Axtens wrote:
Hi all,
I've had a look at the API as it was a big thing I didn't like in the
earlier version.
I am much happier with this one.
Thanks to mpe for suggesting this. :)
Some comments:
- I'm no longer subscribed to skiboot but I've had
Hi Jens,
I was about to fix up this splat..
[ 445.022141] loop: module loaded
[ 445.078116] BUG: using smp_processor_id() in preemptible [] code:
loop0/3801
[ 445.085873] caller is debug_smp_processor_id+0x17/0x20
[ 445.091016] CPU: 7 PID: 3801 Comm: loop0 Tainted: GE
For all frequent late stage acpi_get_table() clone invocations, we should
only change them altogether, otherwise, excessive acpi_put_table() could
unexpectedly unmap the table used by the other users. Thus the current plan
is to change all acpi_get_table() clones together or to change none of
them.
In commit:
f28442497b5c ("x86/boot: Fix KASLR and memmap= collision")
... the memmap= option is parsed so that KASLR can avoid those reserved
regions. It uses cmdline_find_option() to get the value if memmap=
is specified, however the problem is that cmdline_find_option() can only
find the last
From: Dan Williams
Reading an ACPI table through the /sys/firmware/acpi/tables interface
more than 65,536 times leads to the following log message:
ACPI Error: Table 88033595eaa8, Validation count is zero after
increment
(20170119/tbutils-423)
Add the missing acpi_put_table() so the tabl
Option mem= will limit the max address a system can use and any memory
region above the limit will be removed.
Furthermore, memmap=nn[KMG] which has no offset specified has the same
behaviour as mem=.
KASLR needs to consider this when choosing the random position for
decompressing the kernel.
Th
In commit:
9710f581bb4c ("x86, mm: Let "memmap=" take more entries one time")
... 'memmap=' was changed to adopt multiple, comma delimited values in a
single entry, so update the related description.
In the special case of only specifying size value without an offset,
like memmap=nn[KMG], memm
This patch adds acpi_put_table() to make all acpi_get_table() clone
invocations balanced for sysfs ACPI table dump code.
Since Linux does not use all of the tables, this can help to reduce some
usless memory mappings.
While originally, all tables will be remained to be mapped after a
userspace ac
People reported kernel panic occurs during system boots up with mem boot option.
After checking code, several problems are found about memmap= and mem= in boot
stage
kaslr.
*) In commit f28442497b5c ("x86/boot: Fix KASLR and memmap= collision"), only
one memmap
entry is considered and only th
In the Linux kernel side, acpi_get_table() clones haven't been fully
balanced by acpi_put_table() invocations. In ACPICA side, due to the design
change, there are also unbalanced acpi_get_table_by_index() invocations
requiring special care.
So it is not a good timing to report acpi_get_table() cou
Hi Moritz,
[auto build test WARNING on ljones-mfd/for-mfd-next]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Moritz-Fischer/mfd-cros-ec-Add-cros_ec_readmem-helpers-for-I2C-SPI-based-ECs/201705
On 05/09/2017 07:37 AM, Amir Goldstein wrote:
On Tue, May 9, 2017 at 7:13 AM, Oleksij Rempel mailto:o...@pengutronix.de>> wrote:
On 05/02/2017 09:37 AM, Richard Weinberger wrote:
Amir,
Am 02.05.2017 um 09:19 schrieb Amir Goldstein:
On Fri, Apr 28, 2017 at
Goede dag,
Dit is het beveiligingsnetwerk van de kredietverstrekking.
SAFETY NET CREDIT biedt flexibele en betaalbare leningen voor elk doel om u
te helpen uw doelen te bereiken. We lenen tegen een lage rente van 3%. Hier
zijn enkele belangrijke kenmerken van de persoonlijke lening aangebo
On Mon, 8 May 2017 19:42:46 +0200
SF Markus Elfring wrote:
> > Which issue do you mean? I dont see any issue you fix here.
>
> Are the run time characteristics a bit nicer for the function “seq_putc”
> in comparison to the function “seq_puts” for printing a single line break
> here?
>
> Rega
On 08-05-17 19:12, Rob Herring wrote:
On Fri, May 05, 2017 at 08:38:35AM +0200, Mike Looijmans wrote:
This adds the devicetree bindings documentation for the LTC3651 battery charger.
Signed-off-by: Mike Looijmans
---
.../bindings/power/supply/ltc3651-charger.txt | 26 +++
The LTC3651 reports its status via GPIO lines. This driver translates
the GPIO levels to battery charger status information via sysfs.
It relies on devicetree to supply the IO configuration.
Signed-off-by: Mike Looijmans
---
v3: Add "lltc" vendor prefix to DT gpios properties
v2: Remove unused in
On 05/02/2017 06:41 AM, Paolo Bonzini wrote:
>
> KVM Forum 2017: Call For Participation
> October 25-27, 2017 - Hilton Prague - Prague, Czech Republic
>
> (All submissions must be received before midnight June 15, 2017)
> ===
On Mon, May 08, 2017 at 05:59:48PM -0400, Jon Mason wrote:
> It is not always possible to find a resource via the name. This may be
> due to using previous versions of device tree without the names or using
> ACPI. To allow for resources to be discovered by either the name or the
> index, a wrapp
On Tue, May 09, 2017 at 02:27:48AM +, Gregory Fong wrote:
> Add device managed devm_misc_register() to allow simplifying some
> miscdevice code.
>
> Signed-off-by: Gregory Fong
> ---
> This seemed like it would be handy for removing a large chunk of the cleanup
> code in various miscdevice us
Since the addition of the multi queue code with commit 59d0f7465644
("net: fec: init multi queue date structure") the queue selection
has been handelt by the default transmit queue selection
implementation which tries to evenly distribute the traffic across
all available queues. This selection pres
This adds the devicetree bindings documentation for the LTC3651 battery charger.
Signed-off-by: Mike Looijmans
---
v2: Add "lltc," vendor prefix to gpios
Expand irq paragraph
.../bindings/power/supply/ltc3651-charger.txt | 27 ++
1 file changed, 27 insertions(+)
cr
Assorted bits and pieces from various people. No common topic in
that pile, sorry.
The following changes since commit 4f7d029b9bf009fbee76bb10c0c4351a1870d2f3:
Linux 4.11-rc7 (2017-04-16 13:00:18 -0700)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kern
"Gustavo A. R. Silva" writes:
> The name of an array used by itself will always return the array's address.
> So these tests will always evaluate as false and therefore the _return_
> will never be executed.
>
> Signed-off-by: Gustavo A. R. Silva
I don't understand the commit log, especially wh
Braino fix for iov_iter_revert() misuse
The following changes since commit a6a5993243550b09f620941dea741b7421fdf79c:
iov_iter: don't revert iov buffer if csum error (2017-05-01 14:49:53 -0400)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/viro
On 08-05-17, 18:05, kart...@techveda.org wrote:
> From: Karthik Tummala
>
> Fixed following checkpatch.pl warning:
> * WARNING: Prefer kcalloc over kzalloc with multiply
>
> Instead of specifying no.of bytes * size as argument
> in kzalloc, prefer kcalloc.
>
> Signed-off-by: Karthik Tummala
>
On 08-05-17, 14:57, Sudeep Holla wrote:
> Yes, I followed the thread and figured that out. But Rajendra also
> raised "What if the microcontroller firmware maps the performance-index
> to voltage but expects linux to scale the frequency? There is no way to
> specify a performance-index *and* a freq
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: bf5f89463f5b3109a72ed13ca62b57e90213387d
commit: beba3a20bf90ce1b93e24592c3ebf0d0bb581bbe x86: switch to RAW_COPY_USER
date: 6 weeks ago
reproduce:
# apt-get install sparse
git checkout beba
Make the rpmsg_create_ept() return an ERR_PTR() rather than NULL, as
this is common practice throughout the kernel and I got the first two
clients of this API wrong.
Signed-off-by: Bjorn Andersson
---
drivers/rpmsg/qcom_smd.c | 8
drivers/rpmsg/rpmsg_char.c | 4 ++--
drive
On Wed, May 03, 2017 at 05:51:41PM +0300, Yury Norov wrote:
> From: Jan Glauber
>
> Ported from x86_64 with paravirtualization support removed.
>
> Signed-off-by: Jan Glauber
>
> Note. This patch removes protection from direct inclusion of
> arch/arm64/include/asm/spinlock_types.h. It's done b
The SIMATIC IOT2000 is derived from the Galileo Gen2 board and shares
its I2C frequency.
Signed-off-by: Jan Kiszka
Signed-off-by: Sascha Weisenberger
---
drivers/mfd/intel_quark_i2c_gpio.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/mfd/intel_quark_i2c_gpio.c
b/drivers/mfd
The SMD channel is not the primary WCNSS channel and must explicitly be
closed as the device is removed, or the channel will already by open on
a subsequent probe call in e.g. the case of reloading the kernel module.
This issue was introduced because I simplified the underlying SMD
implementation
The name of an array used by itself will always return the array's address.
So these tests will always evaluate as false and therefore the _return_
will never be executed.
Signed-off-by: Gustavo A. R. Silva
---
drivers/net/wireless/ath/ath10k/wmi.c | 9 -
1 file changed, 9 deletions(-)
Hello Markus,
On 8 May 2017 at 14:40, SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Mon, 8 May 2017 11:05:05 +0200
>
> A few update suggestions were taken into account
> from static source code analysis.
>
> Markus Elfring (4):
> Combine two function calls into one in dma_buf_debug_s
Hi,
The following commit e61c38d85b7 makes the uarts on i.MX6 nonfunctional (no
data transmitted or received).
With e61c38d85b7 reverted the uarts work ok.
---
commit e61c38d85b7392e033ee03bca46f1d6006156175
Author: Uwe Kleine-König
Date: Tue Apr 4 11:18:51 2017 +0200
se
On 05/02/2017 09:37 AM, Richard Weinberger wrote:
Amir,
Am 02.05.2017 um 09:19 schrieb Amir Goldstein:
On Fri, Apr 28, 2017 at 1:03 AM, Richard Weinberger wrote:
Am 24.04.2017 um 17:47 schrieb Richard Weinberger:
So, if some flag should be implemented, who should do it? :)
I'll not do it
The name of an array used by itself will always return the array's address.
So this test will always evaluate as true.
Addresses-Coverity-ID: 1364903
Signed-off-by: Gustavo A. R. Silva
---
drivers/net/wireless/ath/ath9k/eeprom.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a
Hi all,
Please do not add any v4.13 destined material in your linux-next
included branches until after v4.12-rc1 has been released.
Changes since 20170508:
Non-merge commits (relative to Linus' tree): 2374
2523 files changed, 83999 insertions(+), 42709 dele
On 05/08/2017 02:41 AM, Philipp Zabel wrote:
Hi Steve,
On Wed, 2017-04-12 at 17:45 -0700, Steve Longerbeam wrote:
This patch adds a call to imx_media_fill_empty_mbus_fields() in the
*_try_fmt() functions at the sink pads, to set empty field order and
colorimetry parameters.
If the field orde
On Mon, May 08, 2017 at 08:29:46PM -0700, Nick Desaulniers wrote:
> Clang does not support this machine dependent option.
>
> Older versions of GCC (pre 3.0) may not support this option, added in
> 2000, but it's unlikely they can still compile the kernel.
>
> Signed-off-by: Nick Desaulniers
> -
On Mon, May 08, 2017 at 07:31:50PM -0700, Andy Lutomirski wrote:
> On Mon, May 8, 2017 at 6:38 PM, Josh Poimboeuf wrote:
> >> Also, don't you need some indication of which reg is the base from
> >> which you find previous frame? After all, sometimes GCC will emit a
> >> frame pointer even in an o
On Monday 08 May 2017 04:06 PM, Colin King wrote:
> From: Colin Ian King
>
> Making thermal_emergency_poweroff static fixes sparse warning:
>
> drivers/thermal/thermal_core.c:6: warning: symbol
> 'thermal_emergency_poweroff' was not declared. Should it be static?
Acked-by: Keerthy
>
>
On 05/08/17 at 05:25pm, Borislav Petkov wrote:
> On Mon, May 08, 2017 at 09:36:49AM +0800, Baoquan He wrote:
> > Thanks for explaining, Bhupesh.
> >
> > BIOS issue of SGI uv1 is still not fixed. There's a quirk for uv1 to
> > use efi old map:
> >
> > void __init efi_apply_memmap_quirks(void)
> >
Clang does not support this machine dependent option.
Older versions of GCC (pre 3.0) may not support this option, added in
2000, but it's unlikely they can still compile the kernel.
Signed-off-by: Nick Desaulniers
---
arch/x86/Makefile | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
d
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: bf5f89463f5b3109a72ed13ca62b57e90213387d
commit: 960fdfdeb9e85a67bed136bc945c541ba61c2bdd xfrm: input: constify
xfrm_input_afinfo
date: 3 months ago
reproduce:
# apt-get install sparse
git
Hi Martin,
Quoting "Martin K. Petersen" :
Gustavo A.,
Properly update the position of the arguments in function call.
Applied to 4.12/scsi-fixes, thank you!
Awesome, glad to help. :)
--
Gustavo A. R. Silva
Hi Mathieu,
On Wed, May 03, 2017 at 04:29:37PM -0600, Mathieu Poirier wrote:
> On Tue, May 02, 2017 at 06:08:32PM +0800, Leo Yan wrote:
> > Update kernel-parameters.txt to add new parameter:
> > coresight_cpu_debug.enable is a knob to enable debugging at boot time.
> >
> > Add detailed documentat
From: Suzuki K Poulose
The of_get_coresight_platform_data iterates over the possible CPU nodes
to find a given cpu phandle. However it does not drop the reference
to the node pointer returned by the of_get_coresight_platform_data.
This patch also introduces another minor fix is to use
of_cpu_dev
Add debug unit on Qualcomm msm8916 based platforms, including the
DragonBoard 410c board.
Signed-off-by: Leo Yan
---
arch/arm64/boot/dts/qcom/msm8916.dtsi | 32
1 file changed, 32 insertions(+)
diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi
b/arch/arm64/boo
Bind debug module driver for Hi6220.
Signed-off-by: Leo Yan
---
arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 64 +++
1 file changed, 64 insertions(+)
diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi
b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi
index 470461d..467
Coresight includes debug module and usually the module connects with CPU
debug logic. ARMv8 architecture reference manual (ARM DDI 0487A.k) has
description for related info in "Part H: External Debug".
Chapter H7 "The Sample-based Profiling Extension" introduces several
sampling registers, e.g. we
This is refactor to add function of_coresight_get_cpu(), so it's used to
retrieve CPU id for coresight component. Finally can use it as a common
function for multiple places.
Suggested-by: Mathieu Poirier
Reviewed-by: Suzuki K Poulose
Signed-off-by: Leo Yan
---
drivers/hwtracing/coresight/of_c
Update document file entries for Coresight debug module.
Signed-off-by: Leo Yan
---
MAINTAINERS | 2 ++
1 file changed, 2 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index b948dfa..a4b1f60 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1191,7 +1191,9 @@ L:linux-arm-ker...@lists
Add coresight_cpu_debug.enable to kernel-parameters.txt, this flag is
used to enable/disable the CPU sampling based debugging.
Signed-off-by: Leo Yan
---
Documentation/admin-guide/kernel-parameters.txt | 7 +++
1 file changed, 7 insertions(+)
diff --git a/Documentation/admin-guide/kernel-pa
Add detailed documentation for Coresight CPU debug driver, which
contains the info for driver implementation, Mike Leach excellent
summary for "clock and power domain". At the end some examples on how
to enable the debugging functionality are provided.
Suggested-by: Mike Leach
Signed-off-by: Leo
ARMv8 architecture reference manual (ARM DDI 0487A.k) Chapter H7 "The
Sample-based Profiling Extension" has description for sampling
registers, we can utilize these registers to check program counter
value with combined CPU exception level, secure state, etc. So this is
helpful for CPU lockup bugs,
According to ARMv8 architecture reference manual (ARM DDI 0487A.k)
Chapter 'Part H: External debug', the CPU can integrate debug module
and it can support self-hosted debug and external debug. Especially
for supporting self-hosted debug, this means the program can access
the debug module from mmio
On Mon, May 8, 2017 at 10:37 PM, Sudeep Holla wrote:
>
>
> On 08/05/17 17:46, Jassi Brar wrote:
>> On Mon, May 8, 2017 at 9:40 PM, Rob Herring wrote:
>>> +Bjorn
>>>
>>> On Tue, May 02, 2017 at 02:55:49PM +0100, Sudeep Holla wrote:
The ARM MHU has mechanism to assert interrupt signals to faci
On 05/09/2017 01:40 AM, Michael S. Tsirkin wrote:
On Sun, May 07, 2017 at 04:19:28AM +, Wang, Wei W wrote:
On 05/06/2017 06:26 AM, Michael S. Tsirkin wrote:
On Thu, Apr 27, 2017 at 02:31:49PM +0800, Wei Wang wrote:
On 04/27/2017 07:20 AM, Michael S. Tsirkin wrote:
On Wed, Apr 26, 2017 at
On Mon, 2017-05-08 at 16:48 -0300, Gabriel Krisman Bertazi wrote:
> Thanks for reporting this. Can you confirm the following patch prevents
> the issue?
Nope, it still gripes.
[ 43.910362] BUG: sleeping function called from invalid context at
mm/slab.h:432
[ 43.910955] in_atomic(): 1, irqs
On Mon, May 8, 2017 at 6:38 PM, Josh Poimboeuf wrote:
> On Mon, May 08, 2017 at 05:21:24PM -0700, Andy Lutomirski wrote:
>> On Sun, May 7, 2017 at 9:55 AM, Josh Poimboeuf wrote:
>> > struct undwarf {
>> > unsigned int ip;/* instruction pointer (relative
>> > offse
Add device managed devm_misc_register() to allow simplifying some
miscdevice code.
Signed-off-by: Gregory Fong
---
This seemed like it would be handy for removing a large chunk of the cleanup
code in various miscdevice users. Let me know whether you think it'd be worth
going ahead changing this
> > + /* TODO: provide device tree option for multi-master mode.
> > */
>
>
> I am curious what everyone thinks about this. It seemed that, earlier on,
> people did not like me disabling multi-master mode, but I think that it
> would make bus recovery not work as well. Given that, I
Hi James,
On 2017/5/9 1:27, James Morse wrote:
> Hi Xiongfeng Wang,
>
> On 28/04/17 03:55, Xiongfeng Wang wrote:
>> It is ok to just ignore the process following the ESB instruction in
>> el0_sync, because the process will be sent SIGBUS signal.
I don't understand. How will Lin
Colin,
> The 2nd check to see if request_size is less than zero is redundant
> because the first check takes error exit path on this condition. So,
> since it is redundant, remove it.
Applied to 4.12/scsi-fixes.
--
Martin K. Petersen Oracle Linux Engineering
Colin,
> I believe there is a typo on the wq destroy of els_wq, currently the
> driver is checking if els_cq is not null and I think this should be a
> check on els_wq instead.
Applied to 4.12/scsi-fixes. Thanks!
--
Martin K. Petersen Oracle Linux Engineering
Guenter,
> The driver now uses IRQ_POLL and needs to select it to avoid the following
> build error.
Applied to 4.12/scsi-fixes, thank you!
--
Martin K. Petersen Oracle Linux Engineering
Kees,
> Using memcpy() from a string that is shorter than the length copied
> means the destination buffer is being filled with arbitrary data from
> the kernel rodata segment. Instead, use strncpy() which will fill the
> trailing bytes with zeros.
Applied to 4.12/scsi-fixes, thanks!
--
Martin
On Mon, May 08, 2017 at 04:31:11PM -0700, Kees Cook wrote:
> On Mon, May 8, 2017 at 3:53 PM, Josh Poimboeuf wrote:
> > On Mon, May 08, 2017 at 12:32:52PM -0700, Kees Cook wrote:
> >> +#define REFCOUNT_EXCEPTION \
> >> + "movl $0x7fff, %[counter]\n\t"
Hi,
> From: linux-acpi-ow...@vger.kernel.org
> [mailto:linux-acpi-ow...@vger.kernel.org] On Behalf Of Rafael J.
> Wysocki
> Subject: Re: [PATCH v3 2/4] ACPICA: Tables: Add mechanism to allow to balance
> late stage
> acpi_get_table() independently
>
> On Thursday, May 04, 2017 07:18:28 AM Zheng
Christoph,
> Normally we'd just pass the scsi_sense_hdr structure in from the caler
> if we care about sense data. Is this something you considered?
>
> Otherwise this looks fine to me.
I agree with Christoph that passing the sense header would be more
consistent with the rest of the SCSI code.
This patchset is for fixes reported by checkpatch.pl
Please find the following patches:
[PATCH 1/3] staging: iio: meter: Add the comment for mutex definition
[PATCH 2/3] staging: iio: meter: Fix the identations for proper
[PATCH 3/3] staging: iio: meter: Replace symbolic permissions with octal
pe
Mr.Torokhov
Hi.
I fixed source, and sent v8 patch to you.
* FF codes ('CONFIG_JOYSTICK_PSXPAD_SPI_FF' defined),
put together in one place.
* psxpad_spi_suspend() stub removed again.
Regard.
---
Tomohiro Yoshidomi
Hi Jaegeuk,
On 2017/5/9 5:23, Jaegeuk Kim wrote:
> Hi Chao,
>
> I can't see a strong reason to split meta from data/node and rename the
> existing
> function names. Instead, how about keeping the existing one while adding some
> page types to deal with log types?
Hmm.. before write this patch,
PlayStation 1/2 joypads can be connected directly to the SPI interface.
Signed-off-by: Tomohiro Yoshidomi
Acked-by: David Herrmann
---
drivers/input/joystick/Kconfig | 22 ++
drivers/input/joystick/Makefile | 1 +
drivers/input/joystick/psxpad-spi.c | 431 +++
Gustavo A.,
> Properly update the position of the arguments in function call.
Applied to 4.12/scsi-fixes, thank you!
--
Martin K. Petersen Oracle Linux Engineering
This patch fixes below checkpatch.pl warning:
CHECK: struct mutex definition without comment
Signed-off-by: Harinath Nampally
---
drivers/staging/iio/meter/ade7753.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/iio/meter/ade7753.c
b/drivers/staging/iio/m
This patch fixes below checkpatch.pl kind of warnings:
CHECK: Alignment should match open parenthesis
Signed-off-by: Harinath Nampally
---
drivers/staging/iio/meter/ade7753.c | 55 ++---
1 file changed, 27 insertions(+), 28 deletions(-)
diff --git a/drivers/stagi
This patch fixes below kind of warnings:
WARNING: Symbolic permissions 'S_IXXX | S_IXXX' are not preferred.
Below errors are false positives:
ade7753.c:382: ERROR: Use 4 digit octal (0777) not decimal permissions
ade7753.c:386: ERROR: Use 4 digit octal (0777) not decimal permissions
Signed-off-by
Konstantin Khlebnikov writes:
> On Mon, May 8, 2017 at 10:22 PM, Daniel Axtens wrote:
>> If we set a next or last buddy for a se that is not on_rq, we will
>> end up taking a NULL pointer dereference in wakeup_preempt_entity
>> via pick_next_task_fair.
>>
>> Detect when we would be about to do t
On Mon, May 8, 2017 at 9:45 PM, Dmitry Torokhov
wrote:
> + /* Make sure there is something at this address */
> + error = i2c_smbus_read_byte(client);
> + if (error < 0) {
> + dev_dbg(&client->dev, "nothing at this address: %d\n", error);
> + return -
Christoph,
> The open-osd domain doesn't exist anymore, and mails to the list lead
> to really annoying bounced that repeat every day.
>
> Also the primarydata address for Benny bounces, and while I have a new
> one for him he doesn't seem to be maintaining the OSD code any more.
>
> Which beggs
1 - 100 of 683 matches
Mail list logo
