Hi Linus,
Here's a PR with a couple of MMC fixes intended for v4.14-rc4. Details about the
highlights are as usual found in the signed tag.
Please pull this in!
Kind regards
Ulf Hansson
The following changes since commit 9e66317d3c92ddaab330c125dfe9d06eee268aff:
Linux 4.14-rc3 (2017-10-01 1
On Fri, Oct 06, 2017 at 11:59:57PM -0500, Mario Limonciello wrote:
> For WMI operations that are only Set or Query read or write sysfs
> attributes created by WMI vendor drivers make sense.
>
> For other WMI operations that are run on Method, there needs to be a
> way to guarantee to userspace tha
On Fri, Oct 06, 2017 at 11:59:58PM -0500, Mario Limonciello wrote:
> It's important for the driver to provide a R/W ioctl to ensure that
> two competing userspace processes don't race to provide or read each
> others data.
>
> This userspace character device will be used to perform SMBIOS calls
>
On Fri, Oct 06, 2017 at 11:59:54PM -0500, Mario Limonciello wrote:
> There are some categories of tokens and SMBIOS calls that it makes
> sense to protect userspace from accessing. These are calls that
> may write to one time use fields or activate hardware debugging
> capabilities. They are not
On Sat, Oct 07, 2017 at 09:41:32AM +0200, Greg KH wrote:
> And I still didn't see where you were verifying the list of commands in
> this structure, was that in some other patch?
Oh nevermind, that was a different patch, found it...
Hi,
On Fri, Oct 06, 2017 at 05:51:32PM +0200, srinivas.kandaga...@linaro.org wrote:
> From: Sagar Dharia
>
> This controller driver programs manager, interface, and framer
> devices for Qualcomm's slimbus HW block.
> Manager component currently implements logical address setting,
> and messaging
-CPU-ID-and-export-as-system-serial-number/20171007-122234
config: i386-allmodconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All errors (new ones prefixed by >>):
>> d
On Sat, Oct 7, 2017 at 6:05 AM, Herbert Xu wrote:
> On Tue, Sep 05, 2017 at 03:38:40PM +0300, Gilad Ben-Yossef wrote:
>>
>> diff --git a/crypto/algif_hash.c b/crypto/algif_hash.c
>> index 5e92bd2..3b3c154 100644
>> --- a/crypto/algif_hash.c
>> +++ b/crypto/algif_hash.c
>> @@ -39,6 +39,20 @@ struct
On Sat 07-10-17 13:05:24, Tetsuo Handa wrote:
> Johannes Weiner wrote:
> > On Sat, Oct 07, 2017 at 11:21:26AM +0900, Tetsuo Handa wrote:
> > > On 2017/10/05 19:36, Tetsuo Handa wrote:
> > > > I don't want this patch backported. If you want to backport,
> > > > "s/fatal_signal_pending/tsk_is_oom_vic
Hi, some trivial comments below.
On Fri, Oct 06, 2017 at 05:51:33PM +0200, srinivas.kandaga...@linaro.org wrote:
> From: Sagar Dharia
>
> Per slimbus specification, a reconfiguration sequence known as
> 'clock pause' needs to be broadcast over the bus while entering low-
> power mode. Clock-paus
> Thank you! Is it safe to asume it will be backported to 4.4 stable
> branch and upwards as the problematic commit did (once in mainline)?
Yes, it has a Fixes: and a stable tag. That should do.
> > Might have been good to CC the patch author of the problematic commit,
> > too.
>
> Yes, of cour
Hi, some more trivial comments below.
On Fri, Oct 06, 2017 at 05:51:34PM +0200, srinivas.kandaga...@linaro.org wrote:
> From: Sagar Dharia
>
> Slimbus HW mandates that clock-pause sequence has to be executed
> before disabling relevant interface and core clocks.
> Runtime-PM's autosuspend featur
Linus,
I2C has three driver fixes for the newly introduced drivers and one ID
addition for the i801 driver.
Please pull.
Thanks,
Wolfram
The following changes since commit 9e66317d3c92ddaab330c125dfe9d06eee268aff:
Linux 4.14-rc3 (2017-10-01 14:54:54 -0700)
are available in the git repo
On Fri, Oct 06, 2017 at 07:13:22PM +0100, Mark Rutland wrote:
> Hi Greg,
>
> On Fri, Oct 06, 2017 at 10:52:04AM +0200, Greg Kroah-Hartman wrote:
> > 4.9-stable review patch. If anyone has any objections, please let me know.
>
> I'm a little confused as to why this is being backported, given it
>
* Steven Rostedt wrote:
> On Fri, 6 Oct 2017 13:49:59 +0900
> Masami Hiramatsu wrote:
>
> > Steve, could you write a documentation how to use ftrace callback?
> > I think I should update the Documentation/kprobes.txt so that jprobe
> > user can easily migrate on that.
>
> I decided to do this
Hi
2017-10-04 18:01 GMT+09:00 Johan Hovold :
> A clean-up patch removing removing two redundant NULL-checks from the
^^
The word 'removing' was written twice. :)
> console disconnect handler inadvertently also removed a third check.
> This could lead
On Fri, Oct 06, 2017 at 11:30:44AM -0600, Shuah Khan wrote:
> On 10/06/2017 03:24 AM, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 3.18.74 release.
> > There are 35 patches in this series, all will be posted as a response
> > to this one. If anyone has any iss
On Sat, Oct 07, 2017 at 02:07:26AM +, Levin, Alexander (Sasha Levin) wrote:
> And quite a few lines of your added trace (lmk if you need more, or all):
Yeah, could you please upload all of it somewhere? That chunk didn't
include any hotplug bits at all.
On Fri, Oct 06, 2017 at 08:31:05PM -0700, Paul E. McKenney wrote:
> > > OK, I will bite... What do the smp_store_release() and the
> > > smp_load_acquire() correspond to? I see just plain locking in
> > > wait_for_completion() and complete().
> >
> > They reflect the concept of complete() / wai
On Fri, 6 Oct 2017 11:34:30 -0400
Steven Rostedt wrote:
> On Fri, 6 Oct 2017 13:49:59 +0900
> Masami Hiramatsu wrote:
>
> > Steve, could you write a documentation how to use ftrace callback?
> > I think I should update the Documentation/kprobes.txt so that jprobe
> > user can easily migrate on
On Fri, Oct 06, 2017 at 07:03:27AM -0700, Guenter Roeck wrote:
> On 10/06/2017 02:24 AM, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 3.18.74 release.
> > There are 35 patches in this series, all will be posted as a response
> > to this one. If anyone has any
On Fri, Oct 06, 2017 at 07:08:27AM -0700, Guenter Roeck wrote:
> On 10/06/2017 01:50 AM, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 4.9.54 release.
> > There are 104 patches in this series, all will be posted as a response
> > to this one. If anyone has any
On Fri, Oct 06, 2017 at 10:50:38AM +0200, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 4.9.54 release.
> There are 104 patches in this series, all will be posted as a response
> to this one. If anyone has any issues with these being applied, please
> let me know
On Fri, 06 Oct 2017 19:17:27 +0200,
Mark Salyzyn wrote:
>
> Can not replicate, issue discovered in fuzzing. Stack trace below.
> No functional or performance testing done regarding the fix.
>
> Trap at (reformatted):
>
> snd_seq_oss_readq_puts(struct seq_oss_readq *q, int dev,
>
On Fri, Oct 06, 2017 at 11:24:40AM +0200, Greg Kroah-Hartman wrote:
> This is the start of the stable review cycle for the 3.18.74 release.
> There are 35 patches in this series, all will be posted as a response
> to this one. If anyone has any issues with these being applied, please
> let me know
The kernel may sleep under a spinlock, and the function call paths are:
afs_do_unlk (acquire the spinlock)
posix_lock_file
posix_lock_inode (fs/locks.c)
locks_get_lock_context
kmem_cache_alloc(GFP_KERNEL) --> may sleep
afs_do_setlk (acquire the spinlock)
posix_lock_file
p
Michal Hocko wrote:
> On Sat 07-10-17 13:05:24, Tetsuo Handa wrote:
> > Johannes Weiner wrote:
> > > On Sat, Oct 07, 2017 at 11:21:26AM +0900, Tetsuo Handa wrote:
> > > > On 2017/10/05 19:36, Tetsuo Handa wrote:
> > > > > I don't want this patch backported. If you want to backport,
> > > > > "s/fat
On Sat, Oct 07 2017 at 12:13:24 am BST, Shanker Donthineni
wrote:
> The current ITS driver works fine as long as normal memory and GICR
> regions are located within the lower 48bit (>=0 && <2^48) physical
> address space. Some of the registers GICR_PEND/PROP, GICR_VPEND/VPROP
> and GITS_CBASER ar
On Fri, Oct 06, 2017 at 07:04:55AM -0700, Guenter Roeck wrote:
> On 10/06/2017 01:52 AM, Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 4.4.91 release.
> > There are 50 patches in this series, all will be posted as a response
> > to this one. If anyone has any i
-in-oom-message/20171007-173639
base: git://git.cmpxchg.org/linux-mmotm.git master
config: i386-tinyconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All errors (new ones prefixed
According to fs/afs/flock.c, the kernel may sleep under a spinlock,
and the function call paths are:
afs_do_setlk (acquire the spinlock: inode->i_lock)
afs_vnode_fetch_status
schedule --> may sleep
afs_do_setlk (acquire the spinlock: inode->i_lock)
wait_event_interruptible --> may sleep
Hi Rob,
> On Oct 6, 2017, at 16:55 , Rob Herring wrote:
>
> On Tue, Oct 3, 2017 at 12:39 PM, Pantelis Antoniou
> wrote:
>> Hi Rob,
>>
>> On Tue, 2017-10-03 at 12:13 -0500, Rob Herring wrote:
>>> On Tue, Oct 3, 2017 at 9:13 AM, Pantelis Antoniou
>>> wrote:
Hi Rob,
On Tue, 2017-
Thanks for the comments.
On 07/10/17 07:42, Jonathan Neuschäfer wrote:
Hi,
On Fri, Oct 06, 2017 at 05:51:31PM +0200, srinivas.kandaga...@linaro.org wrote:
From: Sagar Dharia
Slimbus devices use value-element, and information elements to
control device parameters (e.g. value element is used t
Thanks for the comments.
On 07/10/17 05:14, Jonathan Neuschäfer wrote:
Hi, I have some more or less trivial comments below.
On Fri, Oct 06, 2017 at 05:51:30PM +0200, srinivas.kandaga...@linaro.org wrote:
From: Sagar Dharia
SLIMbus (Serial Low Power Interchip Media Bus) is a specification
dev
Thanks for the review comments
On 07/10/17 09:06, Jonathan Neuschäfer wrote:
Hi, some trivial comments below.
On Fri, Oct 06, 2017 at 05:51:33PM +0200, srinivas.kandaga...@linaro.org wrote:
From: Sagar Dharia
Per slimbus specification, a reconfiguration sequence known as
'clock pause' need
Thanks for the review comments
On 07/10/17 08:45, Jonathan Neuschäfer wrote:
Hi,
On Fri, Oct 06, 2017 at 05:51:32PM +0200, srinivas.kandaga...@linaro.org wrote:
From: Sagar Dharia
This controller driver programs manager, interface, and framer
devices for Qualcomm's slimbus HW block.
Manager
Thanks for the review comments,
On 07/10/17 06:02, Jonathan Neuschäfer wrote:
Hi,
On Fri, Oct 06, 2017 at 05:51:35PM +0200, srinivas.kandaga...@linaro.org wrote:
From: Srinivas Kandagatla
This patch adds support to read/write slimbus value elements.
Currently it only supports byte read/write
Thanks for the review comments
On 07/10/17 09:22, Jonathan Neuschäfer wrote:
Hi, some more trivial comments below.
On Fri, Oct 06, 2017 at 05:51:34PM +0200, srinivas.kandaga...@linaro.org wrote:
From: Sagar Dharia
Slimbus HW mandates that clock-pause sequence has to be executed
before disabl
On Sat, Sep 30, 2017 at 5:07 PM, Heiko Stuebner wrote:
> Am Samstag, 30. September 2017, 20:13:21 CEST schrieb David Wu:
>> If the gmac-m1 optimization(bit10) is selected, the gpio function
>> of gmac pins is not valid. We may use the rmii mode for gmac interface,
>> the pins such as rx_d2, rx_d3,
I have seen some drivers or file systems calls "cond_resched_lock" when
holding a spinlock.
An example is in fs/ocfs2/dlm/dlmdomain.c:
dlm_migrate_all_locks (acquire the spinlock)
cond_resched_lock
I find that "cond_resched_lock" has two functions: "___might_sleep" and
"__cond_resched_lock".
On Sat, 2017-10-07 at 17:55 +0800, Jia-Ju Bai wrote:
> The kernel may sleep under a spinlock, and the function call paths are:
> afs_do_unlk (acquire the spinlock)
> posix_lock_file
> posix_lock_inode (fs/locks.c)
> locks_get_lock_context
> kmem_cache_alloc(GFP_KERNEL) --> may s
On Mon, Oct 2, 2017 at 2:08 PM, Quentin Schulz
wrote:
> The AXP209 and AXP813 PMICs have several pins (respectively 3 and 2) that can
> be used either as GPIOs or for other purposes (ADC or LDO here).
>
> We already have a GPIO driver for the GPIO use of those pins on the AXP209.
> Let's "upgrade
+++ Masami Hiramatsu [05/10/17 06:23 +]:
Hi Jessica,
On Wed, 4 Oct 2017 21:14:13 +0200
Jessica Yu wrote:
Improve error handling when arming ftrace-based kprobes. Specifically, if
we fail to arm a ftrace-based kprobe, register_kprobe()/enable_kprobe()
should report an error instead of suc
On Sat, Sep 30, 2017 at 5:40 AM, Doug Berger wrote:
> This commit allows a wakeup parent interrupt to be shared between
> instances.
>
> It also removes the redundant can_wake member of the private data
> structure by using whether the parent_wake_irq has been defined to
> indicate that the GPIO
On Sat, Sep 30, 2017 at 5:40 AM, Doug Berger wrote:
> This patch set collects a number of improvements to the GPIO driver
> used by Broadcom Set-Top-Box devices.
>
> Primarily they are aimed at correcting problems with the interrupt
> controller implementation, but they also extend the functional
2017-10-07 9:17 GMT+09:00 Randy Dunlap :
> From: Randy Dunlap
>
> Correct typo in kselftest help text.
>
> Signed-off-by: Randy Dunlap
> Cc: Shuah Khan
> Cc: linux-kselft...@vger.kernel.org
> Cc: Masahiro Yamada
> Cc: Jiri Kosina
> ---
Applied to linux-kbuild/fixes. Thanks.
On Tue, Oct 3, 2017 at 6:17 PM, Grygorii Strashko
wrote:
> Now acking of edge irqs happens the following way:
> - omap_gpio_irq_handler
> - "isr" = read irq status
> - omap_clear_gpio_irqbank(bank, isr_saved & ~level_mask);
> ^ clear edge status, so irq can be accepted
> - loop whil
2017-10-03 8:44 GMT+09:00 Shuah Khan :
> Change to enable dochelp run from main make level to make it easier to
> use it.
>
> Signed-off-by: Shuah Khan
> ---
> Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Makefile b/Makefile
> index cf007a31d575..f99d1b36c437
-in-oom-message/20171007-173639
base: git://git.cmpxchg.org/linux-mmotm.git master
config: sh-allnoconfig (attached as .config)
compiler: sh4-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin
Hello,
On Fri, Oct 06, 2017 at 11:06:04AM +0200, Michal Hocko wrote:
> On Fri 06-10-17 16:59:18, Jia-Ju Bai wrote:
> > According to fs/super.c, the kernel may sleep under a spinlock.
> > The function call path is:
> > put_super (acquire the spinlock)
> > __put_super
> > destroy_super
> >
> -Original Message-
> From: Greg KH [mailto:g...@kroah.com]
> Sent: Saturday, October 7, 2017 1:54 AM
> To: Limonciello, Mario
> Cc: dvh...@infradead.org; Andy Shevchenko ;
> LKML ; platform-driver-...@vger.kernel.org;
> Andy Lutomirski ; quasi...@google.com;
> pali.ro...@gmail.com; r...@
> -Original Message-
> From: Greg KH [mailto:g...@kroah.com]
> Sent: Saturday, October 7, 2017 2:35 AM
> To: Limonciello, Mario
> Cc: dvh...@infradead.org; Andy Shevchenko ;
> LKML ; platform-driver-...@vger.kernel.org;
> Andy Lutomirski ; quasi...@google.com;
> pali.ro...@gmail.com; r..
-in-oom-message/20171007-173639
base: git://git.cmpxchg.org/linux-mmotm.git master
config: openrisc-or1ksim_defconfig (attached as .config)
compiler: or1k-linux-gcc (GCC) 5.4.0
reproduce:
wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
From: Sandor Bodo-Merle
Add support for allocating multiple MSIs at the same time, so that the
MSI_FLAG_MULTI_PCI_MSI flag can be added to the msi_domain_info
structure.
Avoid storing the hwirq in the low 5 bits of the message data, as it is
used by the device. Also fix an endianness problem by
On Mon, Sep 04, 2017 at 02:52:13PM +0800, Leo Yan wrote:
> After applied Stefan Wahren patch ("ARM: cpuidle: Avoid memleak if init
> fail") there have no memleak issue, but the code is not consistent to
> handle initialization failure between driver registration and device
> registration. And when
On Mon, Sep 04, 2017 at 02:52:14PM +0800, Leo Yan wrote:
> commit d50a7d8acd78 ("ARM: cpuidle: Support asymmetric idle definition")
> supports multiple CPU idle driver so every CPU has its own driver. When
> the initialization fails, the failure handling releases the resources
> for every previous
> -Original Message-
> From: Greg KH [mailto:g...@kroah.com]
> Sent: Saturday, October 7, 2017 2:42 AM
> To: Limonciello, Mario
> Cc: dvh...@infradead.org; Andy Shevchenko ;
> LKML ; platform-driver-...@vger.kernel.org;
> Andy Lutomirski ; quasi...@google.com;
> pali.ro...@gmail.com; r...@
Hi Stephen, Wei,
On Thu, Aug 31, 2017 at 06:33:01PM -0700, Stephen Boyd wrote:
> On 09/01, Leo Yan wrote:
> > This patch series adds support for coresight on Hi6220; the first patch
> > is to fix coresight PLL so can avoid system hang after we enable
> > coresight, the second patch is to add DT bi
On Sat, Oct 07, 2017 at 11:24:33AM +0100, Srinivas Kandagatla wrote:
> Thanks for the comments.
>
> On 07/10/17 07:42, Jonathan Neuschäfer wrote:
> > Hi,
> >
> > On Fri, Oct 06, 2017 at 05:51:31PM +0200, srinivas.kandaga...@linaro.org
> > wrote:
> > > From: Sagar Dharia
[...]
> > > +int slim_xf
于 2017年10月5日 GMT+08:00 下午2:58:01, Kalle Valo 写到:
>Icenowy Zheng writes:
>
>> 于 2017年10月4日 GMT+08:00 下午6:11:45, Maxime Ripard
>> 写到:
>>>On Wed, Oct 04, 2017 at 10:02:48AM +, Arend van Spriel wrote:
On 10/4/2017 11:03 AM, Icenowy Zheng wrote:
>
>
> 于 2017年10月4日 GMT+08:0
On Sat, Oct 07, 2017 at 12:15:18PM +, mario.limoncie...@dell.com wrote:
> > > + struct wmi_smbios_priv *priv;
> > > + int ret = 0;
> > > + size_t size;
> > > +
> > > + switch (cmd) {
> > > + case DELL_WMI_SMBIOS_CMD:
> > > + priv = dev_get_drvdata(&wdev->dev);
> > > + if (!priv)
On Sat, Oct 07, 2017 at 11:59:52AM +, mario.limoncie...@dell.com wrote:
>
>
> > -Original Message-
> > From: Greg KH [mailto:g...@kroah.com]
> > Sent: Saturday, October 7, 2017 2:35 AM
> > To: Limonciello, Mario
> > Cc: dvh...@infradead.org; Andy Shevchenko ;
> > LKML ; platform-driv
On Sat, Oct 07, 2017 at 11:56:04AM +, mario.limoncie...@dell.com wrote:
> > -Original Message-
> > From: Greg KH [mailto:g...@kroah.com]
> > Sent: Saturday, October 7, 2017 1:54 AM
> > To: Limonciello, Mario
> > Cc: dvh...@infradead.org; Andy Shevchenko ;
> > LKML ; platform-driver-...
The cast to pointer types in kfree is not needed and can be dropped.
Done using the following semantic by coccinelle.
@r@
type T,P;
T* x;
@@
kfree(
-(P *)
x )
Signed-off-by: Srishti Sharma
---
drivers/staging/rtl8723bs/core/rtw_cmd.c | 58
1 file changed, 29 in
Hi Marc,
Thanks for your quick review comments.
On 10/07/2017 04:58 AM, Marc Zyngier wrote:
> On Sat, Oct 07 2017 at 12:13:24 am BST, Shanker Donthineni
> wrote:
>> The current ITS driver works fine as long as normal memory and GICR
>> regions are located within the lower 48bit (>=0 && <2^48)
在 2017-10-05 14:58，Kalle Valo 写道：
Icenowy Zheng writes:
于 2017年10月4日 GMT+08:00 下午6:11:45, Maxime Ripard
写到:
On Wed, Oct 04, 2017 at 10:02:48AM +, Arend van Spriel wrote:
On 10/4/2017 11:03 AM, Icenowy Zheng wrote:
>
>
> 于 2017年10月4日 GMT+08:00 下午5:02:17, Kalle Valo
写到:
> > Icenowy Zhe
-in-oom-message/20171007-173639
base: git://git.cmpxchg.org/linux-mmotm.git master
config: h8300-h8300h-sim_defconfig (attached as .config)
compiler: h8300-linux-gcc (GCC) 6.2.0
reproduce:
wget
https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O
~/bin/make.cross
From: Markus Elfring
Date: Sat, 7 Oct 2017 14:54:06 +0200
Omit extra messages for a memory allocation failure in these functions.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
kernel/power/swap.c | 5 -
1 file changed, 5 deletions(-)
diff --g
> -Original Message-
> From: Greg KH [mailto:g...@kroah.com]
> Sent: Saturday, October 7, 2017 7:37 AM
> To: Limonciello, Mario
> Cc: dvh...@infradead.org; andy.shevche...@gmail.com; linux-
> ker...@vger.kernel.org; platform-driver-...@vger.kernel.org; l...@kernel.org;
> quasi...@google.co
On Fri, 6 Oct 2017 23:41:25 -0700
"Joel Fernandes (Google)" wrote:
> Hi Steve,
>
> On Fri, Oct 6, 2017 at 11:07 AM, Steven Rostedt wrote:
> > From: "Steven Rostedt (VMware)"
> >
> > The ftrace_mod_map is a descriptor to save module init function names in
> > case they were traced, and the trac
[ Replying from an actual computer this time ]
On Fri, 6 Oct 2017 19:21:26 -0700
Nadav Amit wrote:
> IIRC, there was a problem in rc1, which should be resolved in newer rcs.
> If you need to run rc1, you can try to use the kernel parameter “nopcid".
I noticed the bug when I rebased my work on
-list-for-SB-s-s_inodes-list/20171007-210724
config: i386-tinyconfig (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All error/warnings (new ones prefixed by >>):
lib/dlock-list
The use of typedefs for struct is discouraged, and hence can be
eliminated. Done using the following semantic patch by coccinelle.
@r1@
type T;
@@
typedef struct {...} T;
@script: python p@
T << r1.T;
T1;
@@
if T[-2:] == "_t" or T[-2:] == "_T":
coccinelle.T1 = T[:-2]
else:
cocci
On Thu, Oct 5, 2017 at 1:42 PM, Colin King wrote:
> From: Colin Ian King
>
> The structures mlxplat_dev and mlxplat_hotplug are local to the source
> and do not need to be in global scope, so make them static.
>
> Cleans up sparse warnings:
> symbol 'mlxplat_dev' was not declared. Should it be st
On Fri, Oct 06, 2017 at 06:17:23PM -0500, Shiraz Saleem wrote:
> On Wed, Oct 04, 2017 at 05:45:41PM -0700, Kees Cook wrote:
> > In preparation for unconditionally passing the struct timer_list pointer to
> > all timer callbacks, switch to using the new timer_setup() and from_timer()
> > to pass the
From: Markus Elfring
Date: Sat, 7 Oct 2017 16:10:06 +0200
Omit extra messages for a memory allocation failure in these functions.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring
---
kernel/trace/trace_kprobe.c | 1 -
kernel/trace/trace_probe.c| 5
On Fri, Oct 06, 2017 at 08:06:00PM -0500, Brijesh Singh wrote:
> Add a include file which defines the ioctl and command id used for
> issuing SEV platform management specific commands.
>
> Cc: Paolo Bonzini
> Cc: "Radim Krčmář"
> Cc: Borislav Petkov
> Cc: Herbert Xu
> Cc: Gary Hook
> Cc: Tom
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/fs/cifs/cifsacl.c b/fs/cifs/cifsacl.c
--- a/fs/cifs/cifsacl.c
+++ b/fs/cifs/cifsacl.c
@@ -370,7 +370,7 @@ sid_to_id(struct cifs_sb_info *cifs_sb,
else
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/fs/afs/cache.c b/fs/afs/cache.c
--- a/fs/afs/cache.c
+++ b/fs/afs/cache.c
@@ -195,7 +195,7 @@ enum fscache_checkaux afs_vlocation_cach
* VL record from the cac
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/security/selinux/ss/mls.c b/security/selinux/ss/mls.c
--- a/security/selinux/ss/mls.c
+++ b/security/selinux/ss/mls.c
@@ -550,7 +550,7 @@ int mls_compute_sid(struct con
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/fs/ext4/extents.c b/fs/ext4/extents.c
--- a/fs/ext4/extents.c
+++ b/fs/ext4/extents.c
@@ -5242,7 +5242,7 @@ ext4_ext_shift_path_extents(struct ext4_
{
int dept
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c
--- a/fs/xfs/libxfs/xfs_bmap.c
+++ b/fs/xfs/libxfs/xfs_bmap.c
@@ -1490,14 +1490,14 @@ xfs_bmap_isaeof(
int
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/block/bfq-iosched.c b/block/bfq-iosched.c
--- a/block/bfq-iosched.c
+++ b/block/bfq-iosched.c
@@ -4986,7 +4986,7 @@ static ssize_t bfq_low_latency_store(str
i
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/security/integrity/ima/ima_fs.c b/security/integrity/ima/ima_fs.c
--- a/security/integrity/ima/ima_fs.c
+++ b/security/integrity/ima/ima_fs.c
@@ -32,7 +32,7 @@ bool ima
Bool initializations should use true and false. Bool tests don't need
comparisons.
Found by coccinelle spatch "misc/boolinit.cocci"
Run against version v4.14-rc1
P.S. If you find this email unwanted, set up a procmail rule junking on
the header:
X-Patch: Cocci
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/fs/f2fs/data.c b/fs/f2fs/data.c
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -419,7 +419,7 @@ next:
bio_page = fio->encrypted_page ? fio->encrypted_page : fio->
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/fs/configfs/file.c b/fs/configfs/file.c
--- a/fs/configfs/file.c
+++ b/fs/configfs/file.c
@@ -166,7 +166,7 @@ configfs_read_bin_file(struct file *file
r
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/fs/pstore/ftrace.c b/fs/pstore/ftrace.c
--- a/fs/pstore/ftrace.c
+++ b/fs/pstore/ftrace.c
@@ -148,7 +148,7 @@ void pstore_unregister_ftrace(void)
mutex_lock(&ps
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/fs/nfs/callback_proc.c b/fs/nfs/callback_proc.c
--- a/fs/nfs/callback_proc.c
+++ b/fs/nfs/callback_proc.c
@@ -439,7 +439,7 @@ static bool referring_call_exists(struct
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/fs/proc/generic.c b/fs/proc/generic.c
--- a/fs/proc/generic.c
+++ b/fs/proc/generic.c
@@ -325,7 +325,7 @@ static int proc_register(struct proc_dir
write_lock(
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/fs/nfsd/nfs4proc.c b/fs/nfsd/nfs4proc.c
--- a/fs/nfsd/nfs4proc.c
+++ b/fs/nfsd/nfs4proc.c
@@ -230,7 +230,7 @@ do_open_lookup(struct svc_rqst *rqstp, s
if (!*res
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/fs/exofs/super.c b/fs/exofs/super.c
--- a/fs/exofs/super.c
+++ b/fs/exofs/super.c
@@ -116,7 +116,7 @@ static int parse_options(char *options,
Isn't this bogus?
"bool" type in Linux kernel is a typedef to "_Bool"
and true/false is defined as 1 and 0 by enum at include/linux/stddef.h.
On Sat, Oct 7, 2017 at 11:02 PM, Thomas Meyer wrote:
> Bool initializations should use true and false. Bool tests don't need
> comparisons.
>
> Signed-off
On Tue, Oct 3, 2017 at 11:04 AM, Borislav Petkov wrote:
> On Mon, Oct 02, 2017 at 05:42:56PM +0200, Thomas Gleixner wrote:
>> On Mon, 2 Oct 2017, Borislav Petkov wrote:
>> > From: Nicolas Iooss
>> >
>> > parse_cec_param() compares a string with "cec_disable" using only 7
>> > characters of the 11
-list-for-SB-s-s_inodes-list/20171007-210724
config: i386-randconfig-i0-201740 (attached as .config)
compiler: gcc-4.8 (Debian 4.8.4-1) 4.8.4
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All errors (new ones prefixed by >>):
lib/dlock-list
On 10/07/2017 02:40 AM, Greg Kroah-Hartman wrote:
On Fri, Oct 06, 2017 at 11:24:40AM +0200, Greg Kroah-Hartman wrote:
This is the start of the stable review cycle for the 3.18.74 release.
There are 35 patches in this series, all will be posted as a response
to this one. If anyone has any issues
On 10/07/2017 02:39 AM, Greg Kroah-Hartman wrote:
On Fri, Oct 06, 2017 at 10:50:38AM +0200, Greg Kroah-Hartman wrote:
This is the start of the stable review cycle for the 4.9.54 release.
There are 104 patches in this series, all will be posted as a response
to this one. If anyone has any issues
Hello Prakash,
On Fri, Oct 06, 2017 at 07:52:20PM -0700, Prakash Sangappa wrote:
> Userfaultfd feature UFFD_FEATURE_SIGBUS was merged recently and should
> be available in Linux 4.14 release. This patch is for the manpage
> changes documenting this API.
>
> Documents the following commit:
>
> co
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/fs/btrfs/extent-tree.c b/fs/btrfs/extent-tree.c
--- a/fs/btrfs/extent-tree.c
+++ b/fs/btrfs/extent-tree.c
@@ -6958,7 +6958,7 @@ static int __btrfs_free_extent(struct bt
Bool initializations should use true and false. Bool tests don't need
comparisons.
Signed-off-by: Thomas Meyer
---
diff -u -p a/fs/ceph/caps.c b/fs/ceph/caps.c
--- a/fs/ceph/caps.c
+++ b/fs/ceph/caps.c
@@ -1711,7 +1711,7 @@ void ceph_check_caps(struct ceph_inode_i
/* if we are unmounti
1 - 100 of 278 matches
Mail list logo
