As the initial value of the return variable result is never used, it can
be removed.
Issue found with Coccinelle.
Signed-off-by: Nishka Dasgupta
---
drivers/staging/ks7010/ks7010_sdio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ks7010/ks7010_sdio.c
On Thu, 23 May 2019 22:29:17 +0200,
Gustavo A. R. Silva wrote:
>
> Make use of the struct_size() helper instead of an open-coded version
> in order to avoid any potential type mistakes, in particular in the
> context in which this code is being used.
>
> So, replace the following form:
>
>
On Thu, May 23, 2019 at 01:49:27PM -0700, Mark Salyzyn wrote:
> Cherry pick security-related fix 592acbf16821288ecdc4192c47e3774a4c48bb64
> ("ext4: zero out the unused memory region in the extent tree block") to
> 3.18.y and 4.4.y
>
> The cherry-pick is clean and requires no back-porting. Is
Hi Hariprasad,
On Thu, May 23, 2019 at 8:35 PM Hariprasad Kelam
wrote:
>
> devm_request_pci_bus_resources function will return -EBUSY/-ENOMEM
> in fail case and returns 0 on success.
>
> So no need to store return value in err variable.
>
> Signed-off-by: Hariprasad Kelam
> ---
>
Hi, Shawn
> -Original Message-
> From: Shawn Guo [mailto:shawn...@kernel.org]
> Sent: Thursday, May 23, 2019 8:41 PM
> To: Anson Huang
> Cc: s.ha...@pengutronix.de; ker...@pengutronix.de; feste...@gmail.com;
> Leonard Crestez ; Abel Vesa
> ; viresh.ku...@linaro.org; linux-arm-
>
On Fri, May 24, 2019 at 09:28:05AM +0900, Nobuhiro Iwamatsu wrote:
> Hi.
>
> 2019年5月24日(金) 4:11 Greg Kroah-Hartman :
> >
> > This is the start of the stable review cycle for the 4.14.122 release.
> > There are 77 patches in this series, all will be posted as a response
> > to this one. If anyone
The initial value of return variable ret is never used, so it can be
removed.
Issue found with Coccinelle.
Signed-off-by: Nishka Dasgupta
---
drivers/staging/ks7010/ks_hostif.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/ks7010/ks_hostif.c
On Thu, May 23, 2019 at 06:01:11PM -0700, Saravana Kannan wrote:
> Add a generic "depends-on" property that allows specifying mandatory
> functional dependencies between devices. Add device-links after the
> devices are created (but before they are probed) by looking at this
> "depends-on"
Hi, Shawn
> -Original Message-
> From: Shawn Guo [mailto:shawn...@kernel.org]
> Sent: Thursday, May 23, 2019 9:03 AM
> To: Anson Huang
> Cc: robh...@kernel.org; mark.rutl...@arm.com; s.ha...@pengutronix.de;
> ker...@pengutronix.de; feste...@gmail.com; mturque...@baylibre.com;
>
* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * *
* This automated bisection report was sent to you on the basis *
* that you may be involved with the breaking commit it has *
* found. No manual investigation has been done to verify it, *
* and the root cause of the
On 2019/5/24 下午1:36, Peter Hutterer wrote:
On Fri, May 24, 2019 at 01:25:52PM +0800, Hui Wang wrote:
On 2019/5/23 下午2:01, Peter Hutterer wrote:
On Wed, May 22, 2019 at 09:40:30AM +0200, Pali Rohár wrote:
On Wednesday 22 May 2019 07:30:43 Xiaoxiao Liu wrote:
Hi Pali,
Ok, and cannot you set
From: Anson Huang
of_node_put() is called after of_match_node() successfully called,
then in the following error handling, of_node_put() is called again
which is unnecessary, this patch adjusts the location of of_node_put()
to avoid such scenario.
Signed-off-by: Anson Huang
Reviewed-by:
From: Anson Huang
Current implementation of i.MX8 SoC driver returns -ENODEV
for all cases of error during initialization, this is incorrect.
This patch fixes them using correct return value according
to different errors.
Signed-off-by: Anson Huang
Reviewed-by: Leonard Crestez
---
On Thu, May 23, 2019 at 06:01:16PM -0700, Saravana Kannan wrote:
> This sync_state driver/bus callback is called once all the consumers
> of a supplier have probed successfully.
>
> This allows the supplier device's driver/bus to sync the supplier
> device's state to the software state with the
On Thu, May 23, 2019 at 06:01:13PM -0700, Saravana Kannan wrote:
> When consumer devices are added, they might not have a supplier device
> to link to despite needing mandatory resources/functionality from one
> or more suppliers. Add a waiting_for_suppliers list to track such
> consumers and add
On 05/23/2019 02:10 PM, David Hildenbrand wrote:
> On 16.05.19 13:16, Michal Hocko wrote:
>> On Thu 16-05-19 16:36:12, Anshuman Khandual wrote:
>>> On 05/16/2019 03:53 PM, Mark Rutland wrote:
Hi Michal,
On Wed, May 15, 2019 at 06:58:47PM +0200, Michal Hocko wrote:
> On Tue
From: Anson Huang
i.MX8MQ has clock gate for SNVS module, add clock info to SNVS
RTC node for clock management.
Signed-off-by: Anson Huang
---
arch/arm64/boot/dts/freescale/imx8mq.dtsi | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/freescale/imx8mq.dtsi
This is in-kernel C code, so there is no reason to use _BITUL().
Replace it with equivalent BIT().
I added #include explicitly although it has been included
by other headers eventually.
Signed-off-by: Masahiro Yamada
---
drivers/clocksource/arc_timer.c | 3 ++-
1 file changed, 2
On 5/24/19 12:15 PM, Hillf Danton wrote:
On Thu, 23 May 2019 10:27:37 +0800 Yang Shi wrote:
The commit 9092c71bb724 ("mm: use sc->priority for slab shrink targets")
has broken up the relationship between sc->nr_scanned and slab pressure.
The sc->nr_scanned can't double slab pressure anymore.
On Fri, May 24, 2019 at 01:25:52PM +0800, Hui Wang wrote:
>
> On 2019/5/23 下午2:01, Peter Hutterer wrote:
> > On Wed, May 22, 2019 at 09:40:30AM +0200, Pali Rohár wrote:
> > > On Wednesday 22 May 2019 07:30:43 Xiaoxiao Liu wrote:
> > > > Hi Pali,
> > > >
> > > > Ok, and cannot you set
From: Esben Haabendal
Date: Fri, 24 May 2019 07:24:42 +0200
> Fixes: 1b3fa5cf859b ("net: ll_temac: Cleanup multicast filter on change")
> Signed-off-by: Esben Haabendal
Applied.
stable-rc/linux-4.9.y boot: 107 boots: 0 failed, 94 passed with 13 offline
(v4.9.178-54-gf6bc31d8c3be)
Full Boot Summary:
https://kernelci.org/boot/all/job/stable-rc/branch/linux-4.9.y/kernel/v4.9.178-54-gf6bc31d8c3be/
Full Build Summary:
Hello,
syzbot found the following crash on:
HEAD commit:e6f6cd0d bpf: sockmap, fix use after free from sleep in ps..
git tree: bpf
console output: https://syzkaller.appspot.com/x/log.txt?x=16f116e4a0
kernel config: https://syzkaller.appspot.com/x/.config?x=fc045131472947d7
On Thu, May 23, 2019 at 07:08:51PM -0700, Alexei Starovoitov wrote:
> On Thu, May 23, 2019 at 09:57:37PM -0400, Steven Rostedt wrote:
> > On Thu, 23 May 2019 17:31:50 -0700
> > Alexei Starovoitov wrote:
> >
> >
> > > > Now from what I'm reading, it seams that the Dtrace layer may be
> > > >
On 2019/5/23 下午2:01, Peter Hutterer wrote:
On Wed, May 22, 2019 at 09:40:30AM +0200, Pali Rohár wrote:
On Wednesday 22 May 2019 07:30:43 Xiaoxiao Liu wrote:
Hi Pali,
Ok, and cannot you set ALPS_DUALPOINT flag based on that
alps_check_is_trackpoint() result and then update
Fixes: 1b3fa5cf859b ("net: ll_temac: Cleanup multicast filter on change")
Signed-off-by: Esben Haabendal
---
drivers/net/ethernet/xilinx/ll_temac_main.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/xilinx/ll_temac_main.c
On Thu, May 23, 2019 at 10:01:38PM -0700, Scott Branden wrote:
>
> On 2019-05-23 9:54 a.m., Greg Kroah-Hartman wrote:
> > On Thu, May 23, 2019 at 09:36:02AM -0700, Scott Branden wrote:
> > > Hi Greg,
> > >
> > > On 2019-05-22 10:52 p.m., Greg Kroah-Hartman wrote:
> > > > On Wed, May 22, 2019 at
On Thu, May 23, 2019 at 07:02:43PM -0400, Steven Rostedt wrote:
> On Thu, 23 May 2019 14:13:31 -0700
> Alexei Starovoitov wrote:
>
> > > In DTrace, people write scripts based on UAPI-style interfaces and they
> > > don't
> > > have to concern themselves with e.g. knowing how to get the value of
On Thu, May 23, 2019 at 7:23 PM Wolfram Sang wrote:
>
>
> > Ok, Great. Do we need to write to him about this patchset?
>
> Nope. Hint: You can dig in the mail archives studying older patches to
> see how things work. I do this as well because things may work
> differently per subsystem.
>
Thanks
On Thu, May 23, 2019 at 9:27 PM Fred Klassen wrote:
>
> Willem, this is only my 2nd patch, and my last one was a one liner.
> I’ll try to work through this, but let me know if I am doing a rookie
> mistake (learning curve and all).
Not at all. The fix makes perfect sense.
The test patches 2 and
On Thu, May 23, 2019 at 09:29:11AM +0200, walter harms wrote:
>
> No,
> the company i am working for has a custom build arm-board.
> We bought the kernel from the assembler but found it has
> some problems that need fixing.
> Basically we want to improve the linux-kernel so it can run
> native on
On 2019-05-23 9:54 a.m., Greg Kroah-Hartman wrote:
On Thu, May 23, 2019 at 09:36:02AM -0700, Scott Branden wrote:
Hi Greg,
On 2019-05-22 10:52 p.m., Greg Kroah-Hartman wrote:
On Wed, May 22, 2019 at 07:51:12PM -0700, Scott Branden wrote:
Add offset to request_firmware_into_buf to allow for
On Fri, May 24, 2019 at 2:20 AM Rob Herring wrote:
>
> On Thu, May 23, 2019 at 6:46 AM Yash Shah wrote:
> >
> > Add the compatibility string documentation for SiFive FU540-C
> > interface.
> > On the FU540, this driver also needs to read and write registers in a
> > management IP block that
On Wed, May 22, 2019 at 03:58:04PM -0400, Konstantin Ryabitsev wrote:
> > If the report is public, and lists like vger are public,
> > then using a Reported-by: and/or a Link: are simply useful
> > history and tracking information.
>
> I'm perfectly fine with Link:, however Reported-By: usually
On Thu, May 23, 2019 at 9:58 PM David Miller wrote:
>
>
> Please be consistent in your subsystem prefixes used in your Subject lines.
> You use "net: macb:" then "net/macb:" Really, plain "macb: " is sufficient.
Sure, Will take care of this in the next revision of this patch.
Thanks for your
On Thu, May 23, 2019 at 9:38 PM Fred Klassen wrote:
>
> > Thanks for the report.
> >
> > Zerocopy notification reference count is managed in skb_segment. That
> > should work.
> >
> > Support for timestamping with the new GSO feature is indeed an
> > oversight. The solution is similar to how TCP
Commit 8bd9cb51daac ("locking/atomics, asm-generic: Move some macros
from to a new file") moved BIT_ULL()
into . It only includes , so there is
no longer "include file recursion hell".
Signed-off-by: Masahiro Yamada
---
arch/mips/include/asm/cpu.h | 125 +---
On 05/21/2019 03:44 PM, Michal Hocko wrote:
> On Mon 20-05-19 10:48:35, Anshuman Khandual wrote:
>> The arm64 page table dump code can race with concurrent modification of the
>> kernel page tables. When a leaf entries are modified concurrently, the dump
>> code may log stale or inconsistent
On Thu, May 23, 2019 at 8:24 PM Andrew Lunn wrote:
>
> > +static int fu540_macb_tx_set_rate(struct clk_hw *hw, unsigned long rate,
> > + unsigned long parent_rate)
> > +{
> > + rate = fu540_macb_tx_round_rate(hw, rate, _rate);
> > + iowrite32(rate !=
Hi Andreas,
On Thu, May 23, 2019 at 6:19 PM Andreas Schwab wrote:
>
> On Mai 23 2019, Yash Shah wrote:
>
> > On FU540, the management IP block is tightly coupled with the Cadence
> > MACB IP block. It manages many of the boundary signals from the MACB IP
> > This patchset controls the tx_clk
Hi all,
In commit
f875253b5fe6 ("fs/cifs/smb2pdu.c: fix buffer free in SMB2_ioctl_free")
Fixes tag
Fixes: 2c87d6a ("cifs: Allocate memory for all iovs in smb2_ioctl")
has these problem(s):
- SHA1 should be at least 12 digits long
Can be fixed by setting core.abbrev to 12 (or more)
On Thu, May 23, 2019 at 2:58 PM Miguel Ojeda
wrote:
>
> Taking a quick look now, by the way, why does vm_map_pages_zero() (and
> __vm_map_pages() etc.) get a pointer to an array instead of a pointer
> to the first element?
Also, in __vm_map_pages(), semantically w.r.t. to the comment,
shouldn't
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
master
head: 4dde821e4296e156d133b98ddc4c45861935a4fb
commit: 81a56f6dcd20325607d6008f4bb560c96f4c821a gcc-plugins: structleak:
Generalize to all variable types
date: 3 months ago
config: s390-allmodconfig (attached
Hi all,
News: there will be no linux-next release on Monday.
Changes since 20190523:
The input-current tree gained a build failure so I reverted a commit.
The drm-fixes tree gained a build failure so I reverted a commit.
The v4l-dvb tree gained a conflict against Linus' tree.
Non-merge
On Thu, May 23, 2019 at 02:13:31PM -0700, Alexei Starovoitov wrote:
> On Thu, May 23, 2019 at 01:46:10AM -0400, Kris Van Hees wrote:
> >
> > I think there is a difference between a solution and a good solution.
> > Adding
> > a lot of knowledge in the userspace component about how things are
>
On Fri, May 24, 2019 at 4:12 AM Steven Rostedt wrote:
>
> On Thu, 23 May 2019 14:45:35 +0200
> Miguel Ojeda wrote:
>
> I still prefer the typecast of void *, as that's used a bit more in the
> kernel, but since char * is also used (not as much), I'll leave it. But
> the parenthesis around iter
On Thu, May 23, 2019 at 3:37 PM wrote:
>
> From: Ira Weiny
>
> Device pages can be more than type MEMORY_DEVICE_PUBLIC.
>
> Handle all device pages within release_pages()
>
> This was found via code inspection while determining if release_pages()
> and the new put_user_pages() could be
stable-rc/linux-4.19.y boot: 125 boots: 0 failed, 112 passed with 13 offline
(v4.19.45-115-g071ff9cc9849)
Full Boot Summary:
https://kernelci.org/boot/all/job/stable-rc/branch/linux-4.19.y/kernel/v4.19.45-115-g071ff9cc9849/
Full Build Summary:
On Thu, May 23, 2019 at 08:35:23PM -0700, Andrew Morton wrote:
> On Fri, 24 May 2019 11:30:45 +0800 Gen Zhang wrote:
>
> > In dir_add() and do_name(), de->name and vcollected are allocated by
> > kstrdup(). And de->name and vcollected are dereferenced in the following
> > codes. However, memory
From: Randy Dunlap
This is a partial revert of 164a263bf8d0, which causes build errors
on non-X86 platforms (specifically seen on IA64) when COMPILE_TEST
is set/enabled.
Fixes these build errors (on ia64):
i../sound/soc/intel/boards/bxt_da7219_max98357a.c:19:10: fatal error:
On Fri, 24 May 2019 11:30:45 +0800 Gen Zhang wrote:
> In dir_add() and do_name(), de->name and vcollected are allocated by
> kstrdup(). And de->name and vcollected are dereferenced in the following
> codes. However, memory allocation functions such as kstrdup() may fail.
> Dereferencing this
Media Device Allocator API supports just USB devices. Enhance it
adding a genetic device allocate interface to support other media
drivers.
The new interface takes pointer to struct device instead and creates
media device. This interface allows a group of drivers that have a
common root device to
media_device is embedded in struct vimc_device and when vimc is removed
vimc_device and the embedded media_device goes with it, while the active
stream and vimc_capture continue to access it.
Fix the media_device lifetime problem by changing vimc to create shared
media_device using Media Device
In dir_add() and do_name(), de->name and vcollected are allocated by
kstrdup(). And de->name and vcollected are dereferenced in the following
codes. However, memory allocation functions such as kstrdup() may fail.
Dereferencing this null pointer may cause the kernel go wrong. Thus we
should check
If vimc module is removed while streaming is active, vimc_exit runs
into NULL pointer dereference error when streaming thread tries to
access and lock graph_mutex in the struct media_device.
media_device is embedded in struct vimc_device and when vimc is removed
vimc_device and the embedded
This patch adds of_match_table entry in device driver in order to
enable spi fallback probing.
Signed-off-by: Rodrigo Ribeiro
Reviewed-by: Marcelo Schmitt
---
drivers/staging/iio/accel/adis16240.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/staging/iio/accel/adis16240.c
In function ip_ra_control(), the pointer new_ra is allocated a memory
space via kmalloc(). And it is used in the following codes. However,
when there is a memory allocation error, kmalloc() fails. Thus null
pointer dereference may happen. And it will cause the kernel to crash.
Therefore, we
Hi Benjamin,
Thanks so much for all you do for Elan touchpad.
For the width_*, I have a question for it.
Our antenna sensors fully occupied the whole touchpad PCB.
The Gap between 2 sensors are 7.5 mil (0.19mm).
That's why we did not minus one trace.
Thanks
KT
-Original Message-
From:
> -Original Message-
> From: Eduardo Valentin
> Sent: 2019年5月24日 10:50
> To: Andy Tang
> Cc: rui.zh...@intel.com; robh...@kernel.org; daniel.lezc...@linaro.org;
> mark.rutl...@arm.com; linux...@vger.kernel.org;
> devicet...@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: [EXT]
In function ip6_ra_control(), the pointer new_ra is allocated a memory
space via kmalloc(). And it is used in the following codes. However,
when there is a memory allocation error, kmalloc() fails. Thus null
pointer dereference may happen. And it will cause the kernel to crash.
Therefore, we
On Thu, 25 Apr 2019 22:32:17 +0800 Aubrey Li wrote:
> The architecture specific information of the running processes
> could be useful to the userland. Add /proc//arch_status
> interface support to examine process architecture specific
> information externally.
I'll give this an
Acked-by:
When task isn't current task, this task's state have
chance to be changed during printing this task's
backtrace, so it is possible that task's fp and fp+8
have the same vaule, so cannot break the while loop.
To fix this issue, we first save the task's state, sp
and fp, then we will get the task's
Matthias, ping? Any suggestions?
Thanks,
Pingfan
On Thu, May 2, 2019 at 2:22 PM Pingfan Liu wrote:
>
> On Thu, Apr 25, 2019 at 4:20 PM Pingfan Liu wrote:
> >
> > On Wed, Apr 24, 2019 at 4:31 PM Matthias Brugger wrote:
> > >
> > >
> > [...]
> > > > @@ -139,6 +141,8 @@ static int __init
On 2019-05-23, Aleksa Sarai wrote:
> On 2019-05-22, Andy Lutomirski wrote:
> > What are actual examples of uses for this exception? Breaking
> > selftests is not, in and of itself, a huge problem.
>
> Not as far as I know. All of the re-opening users I know of do re-opens
> of O_PATH or are
stable-rc/linux-5.0.y boot: 133 boots: 0 failed, 118 passed with 14 offline, 1
untried/unknown (v5.0.18-140-gc7802929531b)
Full Boot Summary:
https://kernelci.org/boot/all/job/stable-rc/branch/linux-5.0.y/kernel/v5.0.18-140-gc7802929531b/
Full Build Summary:
stable-rc/linux-4.14.y boot: 125 boots: 2 failed, 108 passed with 15 offline
(v4.14.121-78-g64cb9b0bb7de)
Full Boot Summary:
https://kernelci.org/boot/all/job/stable-rc/branch/linux-4.14.y/kernel/v4.14.121-78-g64cb9b0bb7de/
Full Build Summary:
On Tue, 02 Apr 2019 08:44:31 +0300
Felipe Balbi wrote:
> Hi,
>
> Felipe Balbi writes:
Hi Felipe,
I somehow missed both your original email and this gentle reminder.
> > Hi guys,
> >
> > I've been thinking about turning my f-trace.c USB function [1] into an
> > implementation of the USB
When fail to request DMA TX/RX channel, use PIO instead for layerscape.
Signed-off-by: Zhao Qiang
---
drivers/i2c/busses/i2c-imx.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/drivers/i2c/busses/i2c-imx.c b/drivers/i2c/busses/i2c-imx.c
index 98b278613cf2..39a5ab4cf332 100644
On 5/23/19 8:31 PM, Steven Rostedt wrote:
Shuah,
Want to take this or would you want me to?
-- Steve
I can take them through my tree.
thanks,
-- Shuah
On Fri, May 24, 2019 at 2:38 AM Geordan Neukum wrote:
> + depends on MFD_CORE
In order for this to work in menuconfig, this either needs to be a
select or I need to
add a prompt to MFD_CORE. I don't have strong feelings either way, but all other
Kconfig options which are related to the
In wl1271_probe(), 'glue->core' is allocated by platform_device_alloc(),
when this allocation fails, ENOMEM is returned. However, 'pdev_data'
and 'glue' are allocated by devm_kzalloc() before 'glue->core'. When
platform_device_alloc() returns NULL, we should also free 'pdev_data'
and 'glue' before
On 5/23/19 4:28 AM, Pierre-Louis Bossart wrote:
>
>
> On 5/22/19 10:58 PM, Randy Dunlap wrote:
>> From: Randy Dunlap
>>
>> skl-ssp-clk.c does not build on IA64 because the driver
>> uses the common clock interface, so make the driver depend
>> on COMMON_CLK.
>>
>> Fixes this build error:
>>
On Mon, 1 Apr 2019 06:49:07 -0700
Jason Behmer wrote:
Hi Jason,
I just noticed this email. I know it's a late response, but since you
Cc'd LKML, I figured I would respond anyway, and at least have an
answer in the archives ;-)
> Hi Steven,
> We're wondering what the correct number of bits to
On 2019/5/24 9:58, syzbot wrote:
> syzbot has bisected this bug to:
>
> commit 7e27e8d6130c5e88fac9ddec4249f7f2337fe7f8
> Author: Junwei Hu
> Date: Thu May 16 02:51:15 2019 +
>
> tipc: switch order of device registration to fix a crash>
> bisection log:
Add checksum support for gue encapsulation with the tun_flags parameter,
which could be one of the values below:
IP_VS_TUNNEL_ENCAP_FLAG_NOCSUM
IP_VS_TUNNEL_ENCAP_FLAG_CSUM
IP_VS_TUNNEL_ENCAP_FLAG_REMCSUM
Signed-off-by: Jacky Hu
---
include/net/ip_vs.h | 2 +
On Thu, 9 May 2019 19:26:33 -0700 Yury Norov wrote:
> Hi Andy,
>
> Thanks for thorough review.
>
> On Wed, May 08, 2019 at 11:46:32AM +0300, Andy Shevchenko wrote:
> > On Tue, Apr 30, 2019 at 06:06:34PM -0700, Yury Norov wrote:
> > > bitmap_parse() is ineffective and full of opaque variables
On Wed, May 15, 2019 at 05:36:47PM +0800, Yuantian Tang wrote:
> There may be no cooling device on system, or there are no enough
> cooling devices for each thermal zone in multiple thermal zone
> cases since cooling devices can't be shared.
> So make this property optional to remove such
On Mon, Apr 29, 2019 at 9:51 AM Daniel Lezcano
wrote:
>
> On 24/04/2019 01:08, Daniel Lezcano wrote:
> > On 23/04/2019 17:44, Eduardo Valentin wrote:
> >> Hello,
> >>
> >> On Tue, Apr 16, 2019 at 07:22:03PM +0200, Daniel Lezcano wrote:
> >>> Currently when we register a sensor, we specify the
Commit c42d3240990814eec1e4b2b93fa0487fc4873aed
("md: return -ENODEV if rdev has no mddev assigned") changed rdev_attr_store to
return -ENODEV when rdev->mddev is NULL, now do the same to rdev_attr_show.
Signed-off-by: Marcos Paulo de Souza
---
drivers/md/md.c | 2 +-
1 file changed, 1
In wl1271_probe(), 'glue->core' is allocated by platform_device_alloc(),
when this allocation fails, ENOMEM is returned. However, 'pdev_data'
and 'glue' are allocated by devm_kzalloc() before 'glue->core'. When
platform_device_alloc() returns NULL, we should also free 'pdev_data'
and 'glue' before
Thanks for interest, the design idea is from:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/i2c/i2c-slave-eeprom.c?h=v5.2-rc1
and
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/i2c/slave-interface
Then you will get the
[ Added Linus and Al ]
On Thu, 23 May 2019 19:08:51 -0700
Alexei Starovoitov wrote:
> > > >
> > > > I wish that was totally true, but tracepoints *can* be an abi. I had
> > > > code reverted because powertop required one to be a specific
> > > > format. To this day, the wakeup event has a
On Thu, May 23, 2019 at 11:46:47AM +0200, Tomeu Vizoso wrote:
> Hi Eduardo,
>
> I saw that for 5.1 [0] you included a kernelci boot report for your
> tree, but not for 5.2. Have you found anything that should be improved
> in KernelCI for it to be more useful to maintainers like you?
Honestly, I
The kpc2000 core makes calls against functions conditionally exported
upon selection of the kconfig symbol MFD_CORE. Therefore, the kpc2000
core depends upon the mfd_core, and that dependency must be tracked in
Kconfig to avoid potential build issues.
Signed-off-by: Geordan Neukum
---
v2 changes
On Thu, May 16, 2019 at 09:55:33AM -0700, Guenter Roeck wrote:
> On 5/16/19 8:07 AM, Linus Torvalds wrote:
> >On Wed, May 15, 2019 at 9:43 PM Eduardo Valentin wrote:
> >>
> >>- thermal core has a new devm_* API for registering cooling devices, thanks
> >>to Guenter R.
> >> I took the entire
Shuah,
Want to take this or would you want me to?
-- Steve
On Sat, 23 Mar 2019 19:27:46 +0900
Masami Hiramatsu wrote:
> Hi Shuah,
>
> Could you pick this to your tree? I think this can prevent further
> simillar errors before release...
>
> Thank you,
>
> On Fri, 22 Feb 2019 10:26:27
In function con_init(), the pointer variable vc_cons[currcons].d, vc and
vc->vc_screenbuf is allocated a memory space via kzalloc(). And they are
used in the following codes.
However, when there is a memory allocation error, kzalloc() can fail.
Thus null pointer (vc_cons[currcons].d, vc and
On Thu, May 23, 2019 at 09:34:37AM +0800, elaine.zhang wrote:
> hi, Heiko & Enric:
>
> 在 2019/5/22 下午8:27, Heiko Stuebner 写道:
> >Hi Enric,
> >
> >Am Montag, 20. Mai 2019, 15:38:32 CEST schrieb Enric Balletbo Serra:
> >>Hi all,
> >>
> >>As pointed by [1] and [2] this commit, that now is upstream,
On Fri, May 24, 2019 at 1:27 AM Josh Poimboeuf wrote:
>
> On Fri, May 24, 2019 at 12:41:59AM +0800, Kairui Song wrote:
> > On Thu, May 23, 2019 at 11:24 PM Josh Poimboeuf
> > wrote:
> > >
> > > On Thu, May 23, 2019 at 10:50:24PM +0800, Kairui Song wrote:
> > > > > > Hi Josh, this still won't
On Wed, May 22, 2019 at 08:34:34AM -0700, Doug Anderson wrote:
> Hi,
>
> On Wed, May 22, 2019 at 7:12 AM Heiko Stuebner wrote:
> >
> > This reverts commit 28694e009e512451ead5519dd801f9869acb1f60.
> >
> > The commit causes multiple issues in that:
> > - the added call to ->control does
> -Original Message-
> From: Richard Cochran
> Sent: Thursday, May 23, 2019 9:49 PM
> To: Y.b. Lu
> Cc: net...@vger.kernel.org; David Miller ; Claudiu
> Manoil ; Shawn Guo ; Rob
> Herring ; Alexandru Marginean
> ; devicet...@vger.kernel.org;
> linux-arm-ker...@lists.infradead.org;
In function con_insert_unipair(), when allocation for p2 and p1[n]
fails, ENOMEM is returned, but previously allocated p1 is not freed,
remains as leaking memory. Thus we should free p1 as well when this
allocation fails.
Signed-off-by: Gen Zhang
Reviewed-by: Kees Cook
---
diff --git
On Fri, May 24, 2019 at 10:15 AM Ley Foon Tan wrote:
>
> Fix compilation warning caused by patch "PCI: altera: Allow building as
> module".
>
> drivers/pci/controller/pcie-altera.c: In function ‘altera_pcie_irq_teardown’:
> drivers/pci/controller/pcie-altera.c:723:1: warning: no return statement
Fix compilation warning caused by patch "PCI: altera: Allow building as module".
drivers/pci/controller/pcie-altera.c: In function ‘altera_pcie_irq_teardown’:
drivers/pci/controller/pcie-altera.c:723:1: warning: no return statement in
function returning non-void [-Wreturn-type]
}
Hello Wengang,
This patch is used to add a filter attribute(the default value is 0), the
kernel module can use this attribute value to filter the lock resources dumping.
By default(the value is 0), the kernel module does not filter any lock
resources dumping, the behavior is like before.
If the
On Thu, May 23, 2019 at 09:54:18AM -0700, Kees Cook wrote:
> On Thu, May 23, 2019 at 08:34:52AM +0800, Gen Zhang wrote:
> > In function con_insert_unipair(), when allocation for p2 and p1[n]
> > fails, ENOMEM is returned, but previously allocated p1 is not freed,
> > remains as leaking memory.
In pcm030_fabric_probe(), 'pdata->codec_device' is allocated by
platform_device_alloc(). When this allocation fails, ENOMEM is returned.
However, 'pdata' is allocated by devm_kzalloc() before this site. We
should free 'pdata' before function ends to prevent memory leaking.
Similarly, we should
On Thu, 23 May 2019 14:45:35 +0200
Miguel Ojeda wrote:
> +/*
> + * Reset the state of the trace_iterator so that it can read consumed data.
> + * Normally, the trace_iterator is used for reading the data when it is not
> + * consumed, and must retain state.
> + */
> +static __always_inline void
On Sat, 4 May 2019 19:33:02 -0400 Michal Hocko wrote:
> CCing Oleg.
>
> On Fri 03-05-19 22:28:00, Alexey Dobriyan wrote:
> [...]
> > add/remove: 0/0 grow/shrink: 8/68 up/down: 49/-1147 (-1098)
> [...]
> > --- a/include/linux/sched/signal.h
> > +++ b/include/linux/sched/signal.h
> > @@ -15,10
>
> To ensure that we do not regress, when adding options, please consider
> (just a general suggestion, not a strong request for this patch set)
> updating the kselftest to run a variant of the test with the new code
> coverage. In this case, make the code pass/fail instead of only user
>
1 - 100 of 1433 matches
Mail list logo