Commit-ID: 3900dea4cda7c28d7921370bc4d22b08463ed94c
Gitweb: http://git.kernel.org/tip/3900dea4cda7c28d7921370bc4d22b08463ed94c
Author: Christophe JAILLET
AuthorDate: Sat, 18 Feb 2017 09:34:34 +0100
Committer: Thomas Gleixner
CommitDate: Sun, 19 Feb 2017 08:17:25 +0100
irqchip/qcom: Fix
On Fri, Feb 17, 2017 at 12:43:21PM +1100, Stephen Rothwell wrote:
> Hi all,
>
> Today's linux-next merge of the tip tree got a conflict in:
>
> arch/openrisc/include/asm/Kbuild
>
> between commit:
>
> 157e82f58007 ("openrisc: add cmpxchg and xchg implementations")
>
> from the openrisc tre
On Tue, Jan 31, 2017 at 07:41:51AM -0800, James Bottomley wrote:
> > Please-please-please, let's not use WARN for something that is not a
> > kernel bug and is user-triggerable.
>
> It is a kernel bug and it should not be user triggerable, so it should
> have a warn_on or bug_on. It means someth
Hi Deepa,
[auto build test WARNING on linus/master]
[also build test WARNING on v4.10-rc8 next-20170217]
[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/Deepa-Dinamani/vfs-Add-timestamp-range-che
From: "Michael S. Tsirkin"
Date: Sun, 19 Feb 2017 07:17:17 +0200
> Dave, could you merge this before 4.10? If not - I can try.
I just sent my last pull request to Linus, please merge it to
him directly.
Thanks.
Hi Byungchul,
On Tue, 2017-02-14 at 16:26 +0900, Byungchul Park wrote:
> Although llist provides proper APIs, they are not used. Make them used.
>
> Signed-off-by: Byungchul Park
> ---
> drivers/vhost/scsi.c | 11 +++
> 1 file changed, 3 insertions(+), 8 deletions(-)
Acked-by: Nicholas
Fix possible resource leak in tools/vm/page_owner_sort.c
Signed-off-by: Saurabh Badhwar
---
tools/vm/page_owner_sort.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/tools/vm/page_owner_sort.c b/tools/vm/page_owner_sort.c
index f1c055f..8346261 100644
--- a/tools/vm/page_
Hi Dmitry,
On Wed, 2017-02-15 at 23:04 +0300, Dmitry V. Levin wrote:
> Consistently use types from linux/types.h to fix the following
> linux/target_core_user.h userspace compilation errors:
>
> /usr/include/linux/target_core_user.h:108:4: error: unknown type name
> 'uint32_t'
> uint32_t iov
On Sat, Feb 18, 2017 at 06:28:39PM +0100, Dmitry Vyukov wrote:
> On Fri, Feb 10, 2017 at 6:17 AM, Jason Wang wrote:
> >
> >
> > On 2017年02月10日 02:10, Michael S. Tsirkin wrote:
> >>
> >> On Thu, Feb 09, 2017 at 05:02:31AM -0500, Jason Wang wrote:
> >>>
> >>> - Original Message -
>
> >>
Resizing currently drops consumer lock. This can cause entries to be
reordered, which isn't good in itself. More importantly, consumer can
detect a false ring empty condition and block forever.
Further, nesting of consumer within producer lock is problematic for
tun, since it produces entries in
On Fri, Feb 17, 2017 at 01:10:08PM +0800, Jason Wang wrote:
>
>
> On 2017年02月17日 12:53, John Fastabend wrote:
> > On 17-02-15 01:08 AM, Jason Wang wrote:
> > > We set queues before reset which will cause a crash[1]. This is
> > > because is_xdp_raw_buffer_queue() depends on the old xdp queue pair
On Fri, 17 Feb 2017 17:42:54 -0300
Arnaldo Carvalho de Melo wrote:
> Em Fri, Feb 17, 2017 at 07:44:33PM +0900, Masami Hiramatsu escreveu:
> > On Thu, 16 Feb 2017 13:47:37 +0530
> > "Naveen N. Rao" wrote:
> >
> > > I am posting the powerpc bits in the same thread so as to keep these
> > > change
Hi,
On Sat, Feb 18, 2017 at 10:02:07AM +0100, Pavel Machek wrote:
> On Fri 2017-02-17 16:27:30, Chen Yu wrote:
> > Previously a bug was reported that on certain Broadwell
> > platform, after resumed from S3, the CPU is running at
> > an anomalously low speed, due to the BIOS has enabled the
> > MSR
One last brown-paper-bag fix for the release. If we fail the ipv4
mapped source address check, we have to release the route. From
Willem de Bruijn.
Please pull, thanks a lot!
The following changes since commit 2763f92f858f7c4c3198335c0542726eaed07ba3:
Merge tag 'fixes-for-linus' of
git://g
On Fri, 2017-02-17 at 15:35 -0500, Vivek Goyal wrote:
> On Fri, Feb 17, 2017 at 09:34:07AM -0800, James Bottomley wrote:
> > On Fri, 2017-02-17 at 02:55 +, Al Viro wrote:
> > > On Thu, Feb 16, 2017 at 07:56:30AM -0800, James Bottomley wrote:
> > >
> > > > > Hi James,
> > > > >
> > > > > Shoul
Use a fixed clock to enable ttyS0 output.
Signed-off-by: Andreas Färber
---
arch/arm64/boot/dts/marvell/iap140-andromeda-box-edge.dts | 7 +++
1 file changed, 7 insertions(+)
diff --git a/arch/arm64/boot/dts/marvell/iap140-andromeda-box-edge.dts
b/arch/arm64/boot/dts/marvell/iap140-androme
Signed-off-by: Andreas Färber
---
Documentation/devicetree/bindings/arm/marvell/iap140.txt | 7 +++
1 file changed, 7 insertions(+)
create mode 100644 Documentation/devicetree/bindings/arm/marvell/iap140.txt
diff --git a/Documentation/devicetree/bindings/arm/marvell/iap140.txt
b/Documentat
Suppress COMMON_CLK_PXA and MMC_PXA for now as they don't build on arm64.
Also suppress the deprecated SERIAL_PXA in favor of SERIAL_8250_PXA.
Signed-off-by: Andreas Färber
---
arch/arm64/Kconfig.platforms | 5 +
drivers/clk/Kconfig | 2 +-
drivers/mmc/host/Kconfig | 2 +-
drive
PPI interrupts 11, 10, 9 are guesses, and so are the second two memory
regions of the GIC.
Signed-off-by: Andreas Färber
---
arch/arm64/boot/dts/marvell/Makefile | 2 +
.../boot/dts/marvell/iap140-andromeda-box-edge.dts | 70
arch/arm64/boot/dts/marvell/iap140.dtsi
Hello,
This mini-series adds initial support for the Marvell IAP140 SoC (aka PXA1908)
and the Andromeda Box Edge development board.
In order to enable the 8250 UART driver, it reuses ARCH_PXA; but some drivers
enabled with ARCH_PXA don't build due to arm assembly or arm64's lack of mach-,
so thei
Signed-off-by: Matthew Giassa
---
drivers/staging/ks7010/ks_wlan_ioctl.h | 80 +-
1 file changed, 41 insertions(+), 39 deletions(-)
diff --git a/drivers/staging/ks7010/ks_wlan_ioctl.h
b/drivers/staging/ks7010/ks_wlan_ioctl.h
index 84554b6..00eec18 100644
--- a/dr
On Fri, Feb 17, 2017 at 08:51:12AM -0800, Stephen Boyd wrote:
> Sparse complains a bit on this file about endian issues and
> __user casting:
>
> arch/arm64/kernel/traps.c:87:37: warning: incorrect type in argument 1
> (different address spaces)
> arch/arm64/kernel/traps.c:87:37:expected void
On Sat, Feb 18, 2017 at 03:30:04PM -0800, Deepa Dinamani wrote:
> Add fields to the superblock to track the min and max
> timestamps supported by filesystems.
>
> Initially, when a superblock is allocated, initialize
> it to the max and min values the fields can hold.
> Individual filesystems over
Hello Rui,
Please pull the following changes to get the Thermal SoC updates
for 4.11-rc1. Here we have:
- thermal support for zx2967. thanks to Baoyou Xie.
- new driver support for rcar-gen3.
- a couple of fixes on TI, exynos, and IMX drivers.
I might also send a second pull, depending if we get
Steve,
On Thu, Feb 02, 2017 at 09:03:47AM +, Steve Twiss wrote:
> From: Steve Twiss
>
> Add junction temperature monitoring supervisor device driver, compatible
> with the DA9062 and DA9061 PMICs. A MODULE_DEVICE_TABLE() macro is added.
>
> If the PMIC's internal junction temperature rises
On Tue, Feb 07, 2017 at 11:30:13AM +0800, Zhang Rui wrote:
> Just one comment,
> the subject should be
> [PATCH] Thermal: imx_thermal : Fix possible NULL derefrence
I fixed the commit title, but I am taking this one
On Mon, Feb 13, 2017 at 05:01:58PM +0800, Dawei Chien wrote:
> Hi, Matthias,
>
> On Thu, 2017-02-09 at 22:25 +0100, Matthias Brugger wrote:
> >
> > On 02/08/2017 06:31 AM, Dawei Chien wrote:
> > > Thermal driver should read TEMP_MSR3 if thermal bank with 4 sensors.
> > > However, Currently therma
On Sun, Feb 19, 2017 at 01:10:47AM +0100, Borislav Petkov wrote:
Hey Fengguang,
On Sun, Feb 19, 2017 at 07:29:50AM +0800, Fengguang Wu wrote:
Good point! I noticed it too while sending out the report. It'll be
showed as this in future:
https://github.com/0day-ci/linux/commits/Borislav-
On Sat, Feb 11, 2017 at 10:14:56PM +0200, Krzysztof Kozlowski wrote:
> In Odroid XU3 Lite board, the temperature levels reported for thermal
> zone 0 were weird. In warm room:
> /sys/class/thermal/thermal_zone0/temp:32000
> /sys/class/thermal/thermal_zone1/temp:51000
> /sys/class/
I originally reported this issue on bugzilla.kernel.org : bug # 194609 :
https://bugzilla.kernel.org/show_bug.cgi?id=194609
, but it was not posted to the list .
My CPU reports 'model name' as
"Intel(R) Core(TM) i7-4910MQ CPU @ 2.90GHz" ,
has 4 physical & 8 hyperthreading cores with a frequency sc
Hi Jiri,
On Fri, Feb 17, 2017 at 12:09:35PM +0100, Jiri Olsa wrote:
> On Fri, Feb 17, 2017 at 05:17:39PM +0900, Namhyung Kim wrote:
> > The -q/--quiet option is to suppress any message. Sometimes users just
> > want to see the numbers and it can be used for that case.
>
> could you put in some e
Hey Fengguang,
On Sun, Feb 19, 2017 at 07:29:50AM +0800, Fengguang Wu wrote:
> Good point! I noticed it too while sending out the report. It'll be
> showed as this in future:
>
>
> https://github.com/0day-ci/linux/commits/Borislav-Petkov/x86-Optimize-clear_page/20170210-053052
How about
Hi,
Am 18.02.2017 um 15:12 schrieb Jason Cooper:
> On Fri, Feb 17, 2017 at 09:42:17PM +0100, Ansuel Smith wrote:
>> In others board we have the sata led set to funcion with the sata led
>> trigger by default.
>> This patch makes the same for these board that have sata led but disabled by
>> not
system]
>
> url:
> https://github.com/0day-ci/linux/commits/Eddie-Kovsky/provide-check-for-ro_after_init-memory-sections/20170218-141040
> :: branch date: 2 hours ago
> :: commit date: 2 hours ago
>
> >> drivers/hv/vmbus_drv.c:1058:2-45: code aligned with foll
From: Colin Ian King
trivial fix to spelling mistake in error message
Signed-off-by: Colin Ian King
---
scripts/gcc-plugins/initify_plugin.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/gcc-plugins/initify_plugin.c
b/scripts/gcc-plugins/initify_plugin.c
index 56
Hi,
On Sat, Feb 18, 2017 at 12:33:34PM +0100, H. Nikolaus Schaller wrote:
> Hi Sebastian,
>
> > Am 18.02.2017 um 04:22 schrieb Sebastian Reichel :
> >
> > Hi,
> >
> > On Fri, Feb 17, 2017 at 12:40:41PM -0800, Dmitry Torokhov wrote:
> >>> AFAIK there is no mainline board using the DT except ours
From: Colin Ian King
trivial fix to spelling mistake, en_tsu_err_exeption should
be en_tsu_err_exception
Signed-off-by: Colin Ian King
---
drivers/net/ethernet/freescale/fman/fman_dtsec.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ethernet/freescale
Hi Alexey,
[auto build test WARNING on linus/master]
[also build test WARNING on v4.10-rc8 next-20170217]
[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/Alexey-Gladkov/Add-pidfs-filesystem/20170
Hi Alexey,
[auto build test ERROR on linus/master]
[also build test ERROR on v4.10-rc8 next-20170217]
[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/Alexey-Gladkov/Add-pidfs-filesystem/20170219-
Allow read only mounts for filesystems that do not
have maximum timestamps beyond the y2038 expiry
timestamp.
Also, allow a sysctl override to all such filesystems
to be mounted with write permissions.
A boot param supports initial override of these
checks from the early boot without recompilation
Add fields to the superblock to track the min and max
timestamps supported by filesystems.
Initially, when a superblock is allocated, initialize
it to the max and min values the fields can hold.
Individual filesystems override these to match their
actual limits.
Pseudo filesystems are assumed to
ext4 has different overflow limits for max filesystem
timestamps based on the extra bytes available.
Signed-off-by: Deepa Dinamani
---
fs/ext4/ext4.h | 4
fs/ext4/super.c | 7 ++-
2 files changed, 10 insertions(+), 1 deletion(-)
diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h
index 2bef0
POSIX.1 section for futimens, utimensat and utimes says:
The file's relevant timestamp shall be set to the
greatest value supported by the file system that is
not greater than the specified time.
Clamp the timestamps accordingly before assignment.
Note that the clamp_t macro is used for clamping
The series is aimed at adding timestamp checking and policy
related to it to vfs.
The series was developed with discussions and guidance from
Arnd Bergmann.
The original thread is at https://lkml.org/lkml/2016/11/2/294
Associated test: xfstests generic/402
Note that the above test will be run an
Hi Borislav,
On Sat, Feb 18, 2017 at 03:48:00PM +0100, Borislav Petkov wrote:
Guys,
please fix the 0day bot reporting. See below for more info.
On Sat, Feb 18, 2017 at 01:01:53PM +0800, Fengguang Wu wrote:
Greetings,
0day kernel testing robot got the below dmesg and the first bad commit is
timespec_trunc() function is used to truncate a
filesystem timestamp to the right granularity.
But, the function does not clamp tv_sec part of the
timestamps according to the filesystem timestamp limits.
Also, timespec_trunc() is exclusively used for filesystem
timestamps. Move the api to be part
Hi Colin,
> trivial fix to spelling mistake in error message
>
> Signed-off-by: Colin Ian King
> ---
> drivers/bluetooth/btmrvl_sdio.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
patch has been applied to bluetooth-next tree.
Regards
Marcel
On Sat, 18 Feb 2017, Ondrej Zary wrote:
> On Friday 17 February 2017 23:38:12 Finn Thain wrote:
> > On Thu, 16 Feb 2017, Ondrej Zary wrote:
> > > On Tuesday 31 January 2017 02:31:45 Finn Thain wrote:
> > > [...]
> > >
> > > > Are you trying to figure out which commands are going to
> > > > disco
Hi Jiancheng,
Am 09.02.2017 um 08:07 schrieb Jiancheng Xue:
> Add basic dts files for hi3798cv200-poplar board. Poplar is the
> first development board compliant with the 96Boards Enterprise
> Edition TV Platform specification. The board features the
> Hi3798CV200 with an integrated quad-core 64-b
The ethtool api {get|set}_settings is deprecated.
We move this driver to new api {get|set}_link_ksettings.
As I don't have the hardware, I'd be very pleased if
someone may test this patch.
Signed-off-by: Philippe Reynes
---
drivers/net/ethernet/qlogic/qla3xxx.c | 29 ++
On Sat 2017-02-18 20:17:09, H. Nikolaus Schaller wrote:
>
> > Am 18.02.2017 um 19:08 schrieb Pavel Machek :
> >
> >>> And I agree that kernel should _not_ attempt rescaling itself, as it
> >>> would lose precision.
> >>
> >> With an almost 1:1 mapping you won't loose precision.
> >
> > How do y
The pidfs filesystem contains a subset of the /proc file system which
contains only information about the processes.
Some of the container virtualization systems are mounted /proc inside
the container. This is done in most cases to operate with information
about the processes. Knowing that /proc f
Hi!
I guess I'll need some help here.
> @@ -160,6 +163,33 @@ static int ccp2_if_enable(struct isp_ccp2_device *ccp2,
> u8 enable)
> return ret;
> }
>
> + if (isp->revision == ISP_REVISION_2_0) {
> + struct media_pad *pad;
> + struct v4l2_
From: Colin Ian King
trivial fix to spelling mistake in error message
Signed-off-by: Colin Ian King
---
drivers/bluetooth/btmrvl_sdio.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/bluetooth/btmrvl_sdio.c b/drivers/bluetooth/btmrvl_sdio.c
index 23711fe..08e01f0 1
From: Stefan Seyfried
The USB control messages require DMA to work. We cannot pass
a stack-allocated buffer, as it is not warranted that the
stack would be into a DMA enabled area.
Signed-off-by: Stefan Seyfried
---
This fixes at least dvb-usb-technisat-usb2 for me, but probably
the other driv
This patch series is a batch of fixes from the lustre 2.8
release that is missing from the upstream client. These
patches are dependent on the order applied.
Alex Zhuravlev (1):
staging: lustre: llog: limit file size of plain logs
Andreas Dilger (1):
staging: lustre: llite: remove extraneous
On 16 February 2017 at 07:29, Jan Kiszka wrote:
> On 2017-02-16 04:00, Kweh, Hock Leong wrote:
>>> -Original Message-
>>> From: Jan Kiszka [mailto:jan.kis...@siemens.com]
>>> Sent: Thursday, February 16, 2017 3:00 AM
>>> To: Andy Shevchenko
>>> Cc: Matt Fleming ; Ard Biesheuvel
>>> ; linu
From: Bobi Jam
Truncate and write can happen at the same time, so that a file can
be set modified even though the file is not restored from released
state, and ll_hsm_state_set() is not applicable for the file, and
it will return error in this case, we'd lower the error message level
in this case
From: Bobi Jam
In ll_setattr_raw(), after op_data->op_attr has been copied, the attr
is updated and op_data->op_attr does not get updated afterward.
Signed-off-by: Bobi Jam
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6813
Reviewed-on: http://review.whamcloud.com/16462
Reviewed-by: Jinsh
From: Jinshan Xiong
Remove the no longer needed assert in the function
osc_cache_truncate_start(). The assertion in
osc_object_prune() will become faulty with upcoming
changes. The reason this will become a problem is
that there may exist freeing pages in object's
radix tree at the time of osc_ob
From: Yang Sheng
Reduce struct lov_io_sub to smaller memory usage
on wide-stripe file systems.
Signed-off-by: Yang Sheng
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-7085
Reviewed-on: http://review.whamcloud.com/17476
Reviewed-by: Bob Glossman
Reviewed-by: Jian Yu
Reviewed-by: Andreas
From: Vitaly Fertman
disconnect takes too long time if there are many locks to cancel.
besides the amount of time spent on each lock cancel, there is a
resched() in cfs_hash_for_each_relax(), i.e. disconnect or eviction
may take unexpectedly long time. While this patch only contains
the client si
From: Alex Zhuravlev
on small filesystems plain log can grow dramatically. especially
given large record sizes produced by DNE and extended chunksize.
I saw >50% of space consumed by a single llog file which was still
in use. this leads to test failures (sanityn, etc).
the patch introduces additi
From: wang di
Change lgh_hdr_lock from spinlock to mutex because if
the llog object is a remote object it can be stalled
while being fetched.
Signed-off-by: wang di
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6602
Reviewed-on: http://review.whamcloud.com/15274
Reviewed-by: James Simmons
From: Andreas Dilger
The ll_close_inode_openhandle() and ll_md_close() functions passed an
extra "obd_export *md_exp" parameter, but it turns out that all of the
callers already pass inode->i_sb->s_fs_info->lsi_llsbi->ll_md_exp in
one form or another, so it can just be extracted from "inode" dire
From: Vitaly Fertman
Reduce the sleep period from 50 seconds down to
LDLM_POOL_CLI_DEF_RECALC_PERIOD which is 10 seconds.
Signed-off-by: Vitaly Fertman
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3031
Xyratex-bug-id: MRP-395 MRP-1366 MRP-1366
Reviewed-by: Andriy Skulysh
Reviewed-by: Al
From: Niu Yawei
There is race in the code of starting bl threads which leads to
thread number exceeds the maximum number when race happened, it
can also lead to duplicated thread name. This patch fixes the
race and cleanup the code a bit.
Signed-off-by: Niu Yawei
Intel-bug-id: https://jira.hpdd
When compiling the kernel without optimization, when using GCOV,
the lprocfs_stats_alloc_one() symbol is not properly exported to
other modules and causes the ptlrpc module to fail loading with
an unknown symbol. There is no reason to export the function
lprocfs_stats_alloc_one. The reason is due t
From: Jinshan Xiong
A ldlm lock could be canceled simutaneously by ldlm bl thread and
cleanup_resource(). In this case, only one side will win the race
and the other side should wait for the work to complete. Eviction
on group lock is now well supported.
Signed-off-by: Jinshan Xiong
Intel-bug-i
From: Jinshan Xiong
Define osc_lru_reserve() and osc_lru_unreserve() to reserve LRU
slots in osc_io_write_iter_init() and unreserve them in fini();
Signed-off-by: Jinshan Xiong
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6271
Reviewed-on: http://review.whamcloud.com/16456
Reviewed-by: B
From: Jinshan Xiong
In lov_io_iter_init(), if cl_io_iter_init() against sub io fails,
it should call cl_io_iter_fini() to cleanup leftover information;
Signed-off-by: Jinshan Xiong
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6271
Reviewed-on: http://review.whamcloud.com/16456
Reviewed-b
From: Markus Elfring
Date: Sat, 18 Feb 2017 18:52:11 +0100
Use space characters at some source code places according to
the Linux coding style convention.
Signed-off-by: Markus Elfring
---
drivers/infiniband/hw/mlx4/qp.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/d
The following changes since commit 7089db84e356562f8ba737c29e472cc42d530dbc:
Linux 4.10-rc8 (2017-02-12 13:03:20 -0800)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/kernel/git/arm/arm-soc.git
tags/fixes-for-linus
for you to fetch changes up to 69e05170ef0d0c00
Simplify function returns by merging assignment and return into
one command line.
Found with Coccinelle
@@
local idexpression ret;
expression e;
@@
-ret =
+return
e;
-return ret;
Signed-off-by: simran singhal
---
v2:
-Removed assignment
v3:
-Removed declaration of qwTSFOffset
From: Markus Elfring
Date: Fri, 17 Feb 2017 22:06:24 +0100
The script "checkpatch.pl" pointed information out like the following.
WARNING: kfree(NULL) is safe and this check is probably not required
Thus fix the affected source code place.
Signed-off-by: Markus Elfring
---
drivers/infiniband
Hi Colin,
> trivial fix to spelling mistake in error message
>
> Signed-off-by: Colin Ian King
> ---
> drivers/bluetooth/hci_qca.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
patch has been applied to bluetooth-next tree.
Regards
Marcel
From: Markus Elfring
Date: Sat, 18 Feb 2017 19:30:41 +0100
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
The script "checkpatch.pl" pointed information out like the following.
WARNING: sizeof … should be sizeof(…)
Thus fix the affected source code pla
From: Markus Elfring
Date: Sat, 18 Feb 2017 21:00:50 +0100
The kfree() function was called in up to five cases
by the create_kernel_qp() function during error handling
even if the passed data structure member contained a null pointer.
* Adjust jump targets according to the Linux coding style con
From: Markus Elfring
Date: Sat, 18 Feb 2017 21:18:15 +0100
* A multiplication for the size determination of a memory allocation
indicated that an array data structure should be processed.
Thus use the corresponding function "kmalloc_array".
This issue was detected by using the Coccinelle s
From: Markus Elfring
Date: Sat, 18 Feb 2017 20:12:44 +0100
Multiplications for the size determination of memory allocations
indicated that array data structures should be processed.
Thus use the corresponding function "kmalloc_array".
This issue was detected by using the Coccinelle software.
Si
From: Markus Elfring
Date: Sat, 18 Feb 2017 18:33:00 +0100
* Replace the source code "sizeof (" by "sizeof("
according to the Linux coding style convention.
* Adjust indentation at a few places.
Signed-off-by: Markus Elfring
---
drivers/infiniband/hw/mlx4/qp.c | 128
From: Markus Elfring
Date: Sat, 18 Feb 2017 18:29:30 +0100
Replace the specification of two data structures by pointer dereferences
as the parameter for the operator "sizeof" to make the corresponding size
determinations a bit safer according to the Linux coding style convention.
Signed-off-by:
From: Markus Elfring
Date: Sat, 18 Feb 2017 16:56:52 +0100
* A multiplication for the size determination of a memory allocation
indicated that an array data structure should be processed.
Thus use the corresponding function "kmalloc_array".
This issue was detected by using the Coccinelle s
On Sun, 19 Feb 2017, simran singhal wrote:
> gre...@linuxfoundation.org,de...@driverdev.osuosl.org,linux-kernel@vger.kernel.org,outreachy-ker...@googlegroups.com
> Bcc:
> Subject: [PATCH v2 1/6] staging: vt6655: Compress return logic
> Reply-To:
This didn't come out well.
>
> Simplify function
From: Markus Elfring
Date: Sat, 18 Feb 2017 15:25:19 +0100
Replace the specification of a data structure by a pointer dereference
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: Mar
From: Markus Elfring
Date: Sat, 18 Feb 2017 16:32:41 +0100
Do not use curly brackets at one source code place
where a single statement should be sufficient.
Signed-off-by: Markus Elfring
---
drivers/infiniband/hw/mlx4/main.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a
From: Markus Elfring
Date: Sat, 18 Feb 2017 16:15:20 +0100
Omit an extra message for a memory allocation failure in this function.
Link:
http://events.linuxfoundation.org/sites/events/files/slides/LCJ16-Refactor_Strings-WSang_0.pdf
Signed-off-by: Markus Elfring
---
drivers/infiniband/hw/mlx4/
From: Markus Elfring
Date: Sat, 18 Feb 2017 15:04:50 +0100
* Call the function "memset" directly in an if branch without using
an intermediate variable.
* Delete the local variable "clear" which became unnecessary with
this refactoring.
Signed-off-by: Markus Elfring
---
drivers/infiniband
From: Markus Elfring
Date: Sat, 18 Feb 2017 16:00:28 +0100
Delete an assignment for the local variable "num_ports" at the beginning
because it was initialised with the same value.
Signed-off-by: Markus Elfring
---
drivers/infiniband/hw/mlx4/main.c | 2 --
1 file changed, 2 deletions(-)
diff -
From: Markus Elfring
Date: Sat, 18 Feb 2017 14:43:34 +0100
The kfree() function was called in up to two cases during error handling
even if the passed variable contained a null pointer.
* Split a condition check for memory allocation failures.
* Adjust jump targets according to the Linux coding
gre...@linuxfoundation.org,de...@driverdev.osuosl.org,linux-kernel@vger.kernel.org,outreachy-ker...@googlegroups.com
Bcc:
Subject: [PATCH v2 1/6] staging: vt6655: Compress return logic
Reply-To:
Simplify function returns by merging assignment and return into
one command line.
Found with Coccinel
From: Markus Elfring
Date: Sat, 18 Feb 2017 10:32:32 +0100
Multiplications for the size determination of memory allocations
indicated that array data structures should be processed.
Thus use the corresponding function "kmalloc_array".
This issue was detected by using the Coccinelle software.
Si
From: Markus Elfring
Date: Sat, 18 Feb 2017 15:43:54 +0100
Replace the specification of a data structure by a pointer dereference
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: Mar
From: Markus Elfring
Date: Sat, 18 Feb 2017 15:52:31 +0100
The local variable "new_counter_index" will be set to an appropriate pointer
a bit later. Thus omit the explicit initialisation at the beginning.
Signed-off-by: Markus Elfring
---
drivers/infiniband/hw/mlx4/main.c | 2 +-
1 file change
From: Markus Elfring
Date: Sat, 18 Feb 2017 15:16:54 +0100
The script "checkpatch.pl" pointed information out like the following.
WARNING: void function return statements are not generally useful
Thus remove such a statement here.
Signed-off-by: Markus Elfring
---
drivers/infiniband/hw/mlx4/
From: Markus Elfring
Date: Sat, 18 Feb 2017 11:28:41 +0100
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
The script "checkpatch.pl" pointed information out like the following.
WARNING: sizeof … should be sizeof(…)
Thus fix the affected source code pla
From: Markus Elfring
Date: Sat, 18 Feb 2017 09:54:15 +0100
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
The script "checkpatch.pl" pointed information out like the following.
WARNING: sizeof … should be sizeof(…)
Thus fix the affected source code pla
From: Markus Elfring
Date: Fri, 17 Feb 2017 22:22:53 +0100
The script "checkpatch.pl" pointed information out like the following.
ERROR: do not use assignment in if condition
Thus fix the affected source code place.
Signed-off-by: Markus Elfring
---
drivers/infiniband/hw/mlx4/mad.c | 3 ++-
From: Markus Elfring
Date: Fri, 17 Feb 2017 21:41:25 +0100
The kfree() function was called in up to two cases by the
handle_slaves_guid_change() function during error handling even if
the passed variable contained a null pointer.
* Split a condition check for memory allocation failures.
* Adjus
From: Markus Elfring
Date: Fri, 17 Feb 2017 20:00:34 +0100
* A multiplication for the size determination of a memory allocation
indicated that an array data structure should be processed.
Thus use the corresponding function "kcalloc".
This issue was detected by using the Coccinelle softwar
1 - 100 of 225 matches
Mail list logo