Re: [PATCH v2 0/7] net: mvneta: Switch to per-CPU irq and make rxq_def useful

2015-09-29 Thread David Miller
From: Gregory CLEMENT Date: Fri, 25 Sep 2015 18:09:31 +0200 > As stated in the first version: "this patchset reworks the Marvell > neta driver in order to really support its per-CPU interrupts, instead > of faking them as SPI, and allow the use of any RX queue instead of > the hardcoded RX queue

Re: [PATCH 3/3] kernel/audit: audit_tree_match can be boolean

2015-09-29 Thread Paul Moore
On Tuesday, September 29, 2015 10:15:41 PM Yaowei Bai wrote: > This patch makes audit_tree_match return bool to improve readability > due to this particular function only using either one or zero as its > return value. > > No functional change. > > Signed-off-by: Yaowei Bai > --- >

Re: [PATCH] lockd: create NSM handles per net namespace

2015-09-29 Thread J. Bruce Fields
On Wed, Sep 23, 2015 at 03:49:29PM +0300, Andrey Ryabinin wrote: > Commit cb7323fffa85 ("lockd: create and use per-net NSM > RPC clients on MON/UNMON requests") introduced per-net > NSM RPC clients. Unfortunately this doesn't make any sense > without per-net nsm_handle. Makes sense to me. Is

[tip:irq/core] irqchip/renesas-irqc: Move over to nested generic chip

2015-09-29 Thread tip-bot for Magnus Damm
Commit-ID: 99c221df33fbfa1b30e15dee879eb0a9ae1be353 Gitweb: http://git.kernel.org/tip/99c221df33fbfa1b30e15dee879eb0a9ae1be353 Author: Magnus Damm AuthorDate: Mon, 28 Sep 2015 18:42:37 +0900 Committer: Thomas Gleixner CommitDate: Tue, 29 Sep 2015 20:41:00 +0200 irqchip/renesas-irqc:

[tip:irq/core] irqchip/renesas-irqc: Add r8a7795 INTC-EX DT documentation

2015-09-29 Thread tip-bot for Magnus Damm
Commit-ID: e7dbe2da5e8e247c6e62a0894935d94d7153ed16 Gitweb: http://git.kernel.org/tip/e7dbe2da5e8e247c6e62a0894935d94d7153ed16 Author: Magnus Damm AuthorDate: Mon, 28 Sep 2015 18:42:28 +0900 Committer: Thomas Gleixner CommitDate: Tue, 29 Sep 2015 20:41:00 +0200 irqchip/renesas-irqc:

Re: [PATCH 0/1] ns: introduce proc_get_ns_by_fd()

2015-09-29 Thread Oleg Nesterov
On 09/29, Eric W. Biederman wrote: > > Oleg Nesterov writes: > > > OK, I won't insist, this too looks better to me than proc_ns_fdget(_ref). > > > > And in any case fcheck_files() makes more sense than fdget(), somehow I did > > not think about this when I sent 1/1. > > > > Hmm. and after the

Re: [PATCH] arch/x86: fix out-of-bounds in get_wchan()

2015-09-29 Thread Borislav Petkov
On Tue, Sep 29, 2015 at 11:30:33AM -0700, Andy Lutomirski wrote: > Also, I like Borislav's READ_ONCE suggestion. Let's avoid TOCTOU due > to optimization. Dmitry's original patch did READ_ONCE already. > Re: a CVE: if anyone wants a CVE, ask oss-security. It's unclear to me > exactly how one

Re: [PATCH v3 4/5] regulators: tps65912: Add regulator driver for the TPS65912 PMIC

2015-09-29 Thread Mark Brown
On Tue, Sep 29, 2015 at 01:08:50PM -0500, Andrew F. Davis wrote: > On 09/29/2015 10:13 AM, Mark Brown wrote: > >>sure that will save me anything as my probe function is called with a DT > >>match already, so no searching is needed. > >You've not understood what that change is replacing, the code

Re: [PATCH 2/3] kernel/audit: audit_string_contains_control can be boolean

2015-09-29 Thread Paul Moore
On Tuesday, September 29, 2015 10:15:40 PM Yaowei Bai wrote: > This patch makes audit_string_contains_control return bool to improve > readability due to this particular function only using either one or > zero as its return value. > > Signed-off-by: Yaowei Bai > --- > include/linux/audit.h | 2

[PATCH net] net: dsa: fix preparation of a port STP update

2015-09-29 Thread Vivien Didelot
Because of the default 0 value of ret in dsa_slave_port_attr_set, a driver may return -EOPNOTSUPP from the commit phase of a STP state, which triggers a WARN() from switchdev. This happened on a 6185 switch which does not support hardware bridging. Fixes: 3563606258cf ("switchdev: convert STP

Re: [PATCH 04/38] IB/ehca: fix handling idr_alloc result

2015-09-29 Thread Doug Ledford
On 09/21/2015 09:33 AM, Andrzej Hajda wrote: > The function can return negative value. > > The problem has been detected using proposed semantic patch > scripts/coccinelle/tests/unsigned_lesser_than_zero.cocci [1]. > > [1]: http://permalink.gmane.org/gmane.linux.kernel/2038576 > >

Re: [PATCH RFC 3/5] powerpc:numa create 1:1 mappaing between chipid and nid

2015-09-29 Thread Raghavendra K T
On 09/28/2015 10:58 PM, Nishanth Aravamudan wrote: On 27.09.2015 [23:59:11 +0530], Raghavendra K T wrote: Once we have made the distinction between nid and chipid create a 1:1 mapping between them. This makes compacting the nids easy later. Didn't the previous patch just do the opposite

Re: [PATCH] mtd: nand: denali: max_banks calculation changed in revision 5.1

2015-09-29 Thread Brian Norris
On Tue, Sep 29, 2015 at 10:01:10AM -0500, Graham Moore wrote: > Howdy folks, > > Any comment on this patch? A tiny comment below. > Is anyone else using Denali NAND rev 5.1 > controller? I have no devices to test, but CC'ing Enrico, who sent a denali patch recently. In general, MTD has a lot

Re: [PATCH] x86: Use entire page for the per-cpu GDT only if paravirt-enabled

2015-09-29 Thread Andy Lutomirski
On Tue, Sep 29, 2015 at 11:18 AM, H. Peter Anvin wrote: > SGDT would be easy to use, and it is logical that it is faster since it reads > an internal register. SIDT does too but unlike the GDT has a secondary limit > (it can never be larger than 4096 bytes) and so all limits in the range >

Re: [PATCH RFC 2/5] powerpc:numa Rename functions referring to nid as chipid

2015-09-29 Thread Raghavendra K T
On 09/28/2015 10:57 PM, Nishanth Aravamudan wrote: On 27.09.2015 [23:59:10 +0530], Raghavendra K T wrote: There is no change in the fuctionality Signed-off-by: Raghavendra K T --- arch/powerpc/mm/numa.c | 42 +- 1 file changed, 21 insertions(+), 21

Re: Trivial clocksource driver

2015-09-29 Thread Thomas Gleixner
On Tue, 29 Sep 2015, Mason wrote: > Hello everyone, > > I am trying to submit a new ARM port, and Arnd pointed out that the > clocksource code could not live in arch/arm/$PLATFORM, but had to > move to drivers/clocksource (and it had to support DT). > > Did I understand correctly? Is this the

Re: [PATCH] x86: Use entire page for the per-cpu GDT only if paravirt-enabled

2015-09-29 Thread H. Peter Anvin
Ugh. Didn't realize that. On September 29, 2015 11:22:04 AM PDT, Andy Lutomirski wrote: >On Tue, Sep 29, 2015 at 11:18 AM, H. Peter Anvin wrote: >> SGDT would be easy to use, and it is logical that it is faster since >it reads an internal register. SIDT does too but unlike the GDT has a

[PATCH 0/2] mm: fix the racy mm->locked_vm change in acct_stack_growth()

2015-09-29 Thread Oleg Nesterov
Kirill, Hugh, could you please review and ack/nack? Looks simple, but I'm afraid I could miss something and I have no idea how to test this. Oleg. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info

[PATCH 1/2] mm: fix the racy mm->locked_vm change in

2015-09-29 Thread Oleg Nesterov
"mm->locked_vm += grow" and vm_stat_account() in acct_stack_growth() are not safe; multiple threads using the same ->mm can do this at the same time trying to expans different vma's under down_read(mmap_sem). This means that one of the "locked_vm += grow" changes can be lost and we can miss

Re: [PATCH] arch/x86: fix out-of-bounds in get_wchan()

2015-09-29 Thread Andy Lutomirski
On Tue, Sep 29, 2015 at 11:15 AM, Andy Lutomirski wrote: > On Mon, Sep 28, 2015 at 9:32 AM, Thomas Gleixner wrote: >> On Mon, 28 Sep 2015, Dmitry Vyukov wrote: >> >>> On Mon, Sep 28, 2015 at 5:40 PM, Andrey Ryabinin >>> wrote: >>> > 2015-09-28 12:00 GMT+03:00 Dmitry Vyukov : >>> >>

Re: [PATCH 2/2] regmap: debugfs: improve regmap_reg_ranges_read_file()

2015-09-29 Thread Mark Brown
On Tue, Sep 29, 2015 at 12:29:02AM +0200, Rasmus Villemoes wrote: This patch is an example of why SubmittingPatches recommends splitting things up into one change per patch, it would be much easier to read and review as a series (especially given that there's very few collisions). > * A page is

[PATCH 2/2] mm: add the "struct mm_struct *mm" local into

2015-09-29 Thread Oleg Nesterov
Cosmetic, but expand_upwards() and expand_downwards() overuse vma->vm_mm, a local variable makes sense imho. Signed-off-by: Oleg Nesterov --- mm/mmap.c | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/mm/mmap.c b/mm/mmap.c index 4efdc37..7edf9ed

Re: [PATCH 1/3] kernel/audit: audit_dummy_context can be boolean

2015-09-29 Thread Paul Moore
On Tuesday, September 29, 2015 10:15:39 PM Yaowei Bai wrote: > This patch makes audit_dummy_context return bool due to this > particular function only using either one or zero as its return > value. > > No functional change. > > Signed-off-by: Yaowei Bai > --- > include/linux/audit.h | 6

Linux 4.2.2

2015-09-29 Thread Greg KH
I'm announcing the release of the 4.2.2 kernel. All users of the 4.2 kernel series must upgrade. The updated 4.2.y git tree can be found at: git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git linux-4.2.y and can be browsed at the normal kernel.org git web browser:

Re: [PATCH v4 1/5] Documentation: add DT bindings for ARM SCPI sensors

2015-09-29 Thread Rob Herring
On Tue, Sep 15, 2015 at 11:50 AM, Punit Agrawal wrote: > The System Control Processor (SCP) provides access to SoC sensors via > the System Control and Power Interface (SCPI) Message Protocol. Add > bindings to allow probing of these sensors. Also support referencing > of the sensors for setting

Re: [PATCH] x86: Use entire page for the per-cpu GDT only if paravirt-enabled

2015-09-29 Thread H. Peter Anvin
SGDT would be easy to use, and it is logical that it is faster since it reads an internal register. SIDT does too but unlike the GDT has a secondary limit (it can never be larger than 4096 bytes) and so all limits in the range 4095-65535 are exactly equivalent. Anything that causes a write to

Re: [PATCH RFC 0/5] powerpc:numa Add serial nid support

2015-09-29 Thread Raghavendra K T
On 09/28/2015 10:34 PM, Nishanth Aravamudan wrote: On 28.09.2015 [13:44:42 +0300], Denis Kirjanov wrote: On 9/27/15, Raghavendra K T wrote: Problem description: Powerpc has sparse node numbering, i.e. on a 4 node system nodes are numbered (possibly) as 0,1,16,17. At a lower level, we map the

Re: [PATCH v3 3/5] mfd: tps65912: Add driver for the TPS65912 PMIC

2015-09-29 Thread Andrew F. Davis
On 09/28/2015 11:01 AM, Grygorii Strashko wrote: On 09/24/2015 09:52 AM, Andrew F. Davis wrote: This patch adds support for TPS65912 mfd device. It provides communication through the I2C and SPI interfaces. It contains the following components: - Regulators - GPIO controller

Re: [PATCH 5/5] selinux: use sprintf return value

2015-09-29 Thread Stephen Smalley
On 09/25/2015 06:34 PM, Rasmus Villemoes wrote: sprintf returns the number of characters printed (excluding '\0'), so we can use that and avoid duplicating the length computation. Signed-off-by: Rasmus Villemoes Acked-by: Stephen Smalley --- security/selinux/ss/services.c | 5 + 1

Linux 4.1.9

2015-09-29 Thread Greg KH
I'm announcing the release of the 4.1.9 kernel. All users of the 4.1 kernel series must upgrade. The updated 4.1.y git tree can be found at: git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git linux-4.1.y and can be browsed at the normal kernel.org git web browser:

Re: [PATCH] arch/x86: fix out-of-bounds in get_wchan()

2015-09-29 Thread Andy Lutomirski
On Mon, Sep 28, 2015 at 9:32 AM, Thomas Gleixner wrote: > On Mon, 28 Sep 2015, Dmitry Vyukov wrote: > >> On Mon, Sep 28, 2015 at 5:40 PM, Andrey Ryabinin >> wrote: >> > 2015-09-28 12:00 GMT+03:00 Dmitry Vyukov : >> >> stack = (unsigned long)task_stack_page(p); >> >> - if

Re: [PATCH 1/2] regmap: debugfs: remove bogus check

2015-09-29 Thread Mark Brown
On Tue, Sep 29, 2015 at 12:29:01AM +0200, Rasmus Villemoes wrote: > snprintf cannot return a negative value (unless map->dev->driver->name > happens to be over 2G long). So remove the bogus check. > ret = snprintf(buf, PAGE_SIZE, "%s\n", map->dev->driver->name); > - if (ret < 0) { > -

Re: [PATCH 4/5] selinux: use kstrdup() in security_get_bools()

2015-09-29 Thread Stephen Smalley
On 09/25/2015 06:34 PM, Rasmus Villemoes wrote: This is much simpler. Signed-off-by: Rasmus Villemoes Acked-by: Stephen Smalley --- security/selinux/ss/services.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/security/selinux/ss/services.c

[PATCH v2 0/2] Fix tpm_tis ACPI issue with TPM 2.0

2015-09-29 Thread Jarkko Sakkinen
Both for FIFO and CRB interface TCG has decided to use the same HID MSFT0101. They can be differentiated by looking at the start method from TPM2 ACPI table. These patches make necessary fixes to tpm_tis and tpm_crb modules in order to correctly detect, which module should be used. v2: * One

[PATCH v2 2/2] tpm, tpm_tis: detect TPM2 FIFO devices based on HID

2015-09-29 Thread Jarkko Sakkinen
It turned out that the root cause in b371616b8 was not correct for https://bugzilla.kernel.org/show_bug.cgi?id=98181. All TPM 2.0 FIFO and CRB devices have the same HID. For FIFO devices the start method is always 6. This patch changes FIFO and CRB drivers so that they check start method value

Re: List corruption on epoll_ctl(EPOLL_CTL_DEL) an AF_UNIX socket

2015-09-29 Thread Mathias Krause
On 14 September 2015 at 04:39, Eric Wong wrote: > +cc Jason Baron since he might be able to provide more insight into > epoll. > > Mathias Krause wrote: >> Hi, >> >> this is an attempt to resurrect the thread initially started here: >> >> http://thread.gmane.org/gmane.linux.network/353003 >>

Re: [PATCH 3/5] selinux: use kmemdup in security_sid_to_context_core()

2015-09-29 Thread Stephen Smalley
On 09/25/2015 06:34 PM, Rasmus Villemoes wrote: Signed-off-by: Rasmus Villemoes Acked-by: Stephen Smalley --- security/selinux/ss/services.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/security/selinux/ss/services.c b/security/selinux/ss/services.c index

[PATCH v2 1/2] tpm, tpm_tis: use acpi_driver instead of pnp_driver

2015-09-29 Thread Jarkko Sakkinen
Migrate to struct acpi_driver in order to get out of 7 character limitation for the HID. Reported-by: Matthew Garrett Signed-off-by: Jarkko Sakkinen --- drivers/char/tpm/tpm_tis.c | 124 ++--- 1 file changed, 73 insertions(+), 51 deletions(-) diff --git

Re: [PATCH 2/5] selinux: remove pointless cast in selinux_inode_setsecurity()

2015-09-29 Thread Stephen Smalley
On 09/25/2015 06:34 PM, Rasmus Villemoes wrote: security_context_to_sid() expects a const char* argument, so there's no point in casting away the const qualifier of value. Signed-off-by: Rasmus Villemoes Acked-by: Stephen Smalley --- security/selinux/hooks.c | 2 +- 1 file changed, 1

Re: [PATCH 1/2] tpm, tpm_tis: use acpi_driver instead of pnp_driver

2015-09-29 Thread Matthew Garrett
On Tue, Sep 29, 2015 at 08:07:10PM +0300, Jarkko Sakkinen wrote: > Migrate to struct acpi_driver in order to get out of 7 character > limitation for the HID. Are we guaranteed that there are no old systems reporting these devices via pnpbios rather than acpi? -- Matthew Garrett |

[PATCH] ver_linux: uniform output across various linux distros

2015-09-29 Thread Alexander Kapshuk
On Tue, Sep 29, 2015 at 7:43 PM, Greg KH wrote: > On Tue, Sep 29, 2015 at 06:36:12PM +0300, Alexander Kapshuk wrote: >> I've reread 'Documentation/SubmittingPatches', and opted for method 1 >> to create the patch. >> >> When you mentioned about the patch changing the formatting, were you >>

Re: [PATCH v3 4/5] regulators: tps65912: Add regulator driver for the TPS65912 PMIC

2015-09-29 Thread Andrew F. Davis
On 09/29/2015 10:13 AM, Mark Brown wrote: On Fri, Sep 25, 2015 at 03:10:04PM -0500, Andrew F. Davis wrote: On 09/25/2015 01:05 PM, Mark Brown wrote: On Thu, Sep 24, 2015 at 09:52:53AM -0500, Andrew F. Davis wrote: + match = of_match_device(tps65912_regulator_of_match_table, >dev); +

Applied "ASoC: sgtl5000: fix error message output for MicBias voltage" to the asoc tree

2015-09-29 Thread Mark Brown
The patch ASoC: sgtl5000: fix error message output for MicBias voltage has been applied to the asoc tree at git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24

Applied "regulator: axp20x: Fix enable bit indexes for DCDC4 and DCDC5" to the regulator tree

2015-09-29 Thread Mark Brown
The patch regulator: axp20x: Fix enable bit indexes for DCDC4 and DCDC5 has been applied to the regulator tree at git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git All being well this means that it will be integrated into the linux-next tree (usually sometime in the

Re: [PATCH 1/5] selinux: introduce security_context_str_to_sid

2015-09-29 Thread Stephen Smalley
On 09/25/2015 06:34 PM, Rasmus Villemoes wrote: There seems to be a little confusion as to whether the scontext_len parameter of security_context_to_sid() includes the nul-byte or not. Reading security_context_to_sid_core(), it seems that the expectation is that it does not (both the string

Re: [PATCH] x86: Use entire page for the per-cpu GDT only if paravirt-enabled

2015-09-29 Thread Andy Lutomirski
On Tue, Sep 29, 2015 at 10:50 AM, Linus Torvalds wrote: > On Tue, Sep 29, 2015 at 1:35 PM, Andy Lutomirski wrote: >> >> Does anyone know what happens if you stick a non-accessed segment in >> the GDT, map the GDT RO, and access it? > > You should get a #PF, as you guess, but go ahead and test it

Re: [PATCH 0/5] selinux: minor cleanup suggestions

2015-09-29 Thread Stephen Smalley
On 09/25/2015 06:34 PM, Rasmus Villemoes wrote: A few random things I stumbled on. While I'm pretty sure of the change in 1/5, I'm also confused, because the doc for the reverse security_sid_to_context state that @scontext_len is set to "the length of the string", which one would normally

Re: [PATCH v7 03/11] task_isolation: support PR_TASK_ISOLATION_STRICT mode

2015-09-29 Thread Andy Lutomirski
On Tue, Sep 29, 2015 at 10:57 AM, Chris Metcalf wrote: > On 09/29/2015 01:46 PM, Andy Lutomirski wrote: >> >> On Tue, Sep 29, 2015 at 10:35 AM, Chris Metcalf >> wrote: >>> >>> Well, the most interesting category is things that don't actually >>> trigger a signal (e.g. minor page fault) since

Re: [PATCH v2] capabilities.7, prctl.2: Document ambient capabilities

2015-09-29 Thread Kees Cook
On Tue, Sep 29, 2015 at 10:49 AM, Andy Lutomirski wrote: > On Tue, Sep 29, 2015 at 10:47 AM, Kees Cook wrote: >> On Mon, Sep 28, 2015 at 6:03 PM, Andy Lutomirski wrote: >>> Signed-off-by: Andy Lutomirski >>> --- >>> man2/prctl.2| 12 >>> man7/capabilities.7 | 40

Re: [PATCH v7 03/11] task_isolation: support PR_TASK_ISOLATION_STRICT mode

2015-09-29 Thread Chris Metcalf
On 09/29/2015 01:46 PM, Andy Lutomirski wrote: On Tue, Sep 29, 2015 at 10:35 AM, Chris Metcalf wrote: Well, the most interesting category is things that don't actually trigger a signal (e.g. minor page fault) since those are things that cause significant issues with task isolation processes

Re: [PATCH v7 07/11] arch/x86: enable task isolation functionality

2015-09-29 Thread Andy Lutomirski
On Tue, Sep 29, 2015 at 10:42 AM, Chris Metcalf wrote: > On 09/28/2015 06:43 PM, Andy Lutomirski wrote: >> >> Why are we treating alarms as something that should defer entry to >> userspace? I think it would be entirely reasonable to set an alarm >> for ten minutes, ask for isolation, and then

Re: [PATCH 3.14 79/84] x86/nmi/64: Switch stacks on userspace NMI entry

2015-09-29 Thread Greg Kroah-Hartman
On Tue, Sep 29, 2015 at 10:25:52AM -0700, Andy Lutomirski wrote: > On Tue, Sep 29, 2015 at 8:19 AM, Greg Kroah-Hartman > wrote: > > 3.14-stable review patch. If anyone has any objections, please let me know. > > > > -- > > > > From: Andy Lutomirski > > > > commit

Re: [PATCH] x86: Use entire page for the per-cpu GDT only if paravirt-enabled

2015-09-29 Thread Linus Torvalds
On Tue, Sep 29, 2015 at 1:35 PM, Andy Lutomirski wrote: > > Does anyone know what happens if you stick a non-accessed segment in > the GDT, map the GDT RO, and access it? You should get a #PF, as you guess, but go ahead and test it if you want to make sure. We do something very similar for the

Re: [PATCH v2] capabilities.7, prctl.2: Document ambient capabilities

2015-09-29 Thread Andy Lutomirski
On Tue, Sep 29, 2015 at 10:47 AM, Kees Cook wrote: > On Mon, Sep 28, 2015 at 6:03 PM, Andy Lutomirski wrote: >> Signed-off-by: Andy Lutomirski >> --- >> man2/prctl.2| 12 >> man7/capabilities.7 | 40 ++-- >> 2 files changed, 46

Re: [PATCH v2] capabilities.7, prctl.2: Document ambient capabilities

2015-09-29 Thread Kees Cook
On Mon, Sep 28, 2015 at 6:03 PM, Andy Lutomirski wrote: > Signed-off-by: Andy Lutomirski > --- > man2/prctl.2| 12 > man7/capabilities.7 | 40 ++-- > 2 files changed, 46 insertions(+), 6 deletions(-) > > diff --git a/man2/prctl.2

RE: [PATCH v3 1/5] Input: goodix - reset device at init

2015-09-29 Thread Tirdea, Irina
> -Original Message- > From: Bastien Nocera [mailto:had...@hadess.net] > Sent: 29 September, 2015 5:04 > To: Tirdea, Irina; linux-in...@vger.kernel.org > Cc: linux-kernel@vger.kernel.org; Rob Herring; Pawel Moll; Ian Campbell; > Kumar Gala; Purdila, Octavian; Dmitry Torokhov; Mark >

Re: [PATCH v7 03/11] task_isolation: support PR_TASK_ISOLATION_STRICT mode

2015-09-29 Thread Andy Lutomirski
On Tue, Sep 29, 2015 at 10:35 AM, Chris Metcalf wrote: > On 09/28/2015 06:38 PM, Andy Lutomirski wrote: >> >> On Mon, Sep 28, 2015 at 2:54 PM, Chris Metcalf >> wrote: >>> >>> On 09/28/2015 04:51 PM, Andy Lutomirski wrote: On Mon, Sep 28, 2015 at 11:17 AM, Chris Metcalf > > @@

Re: [PATCH 2/2] tpm, tpm_tis: detect TPM2 FIFO devices based on HID

2015-09-29 Thread kbuild test robot
Hi Jarkko, [auto build test results on v4.3-rc3 -- if it's inappropriate base, please ignore] reproduce: # apt-get install sparse make ARCH=x86_64 allmodconfig make C=1 CF=-D__CHECK_ENDIAN__ sparse warnings: (new ones prefixed by >>) >> drivers/char/tpm/tpm_tis.c:136:13: sparse: cast

Re: [PATCH] kselftest: replace $(RM) with rm -f command

2015-09-29 Thread Kees Cook
On Tue, Sep 29, 2015 at 2:57 AM, Michael Ellerman wrote: > On Mon, 2015-09-28 at 03:16 +, Mathieu Desnoyers wrote: >> - On Sep 27, 2015, at 10:10 PM, Wang Long long.wangl...@huawei.com wrote: >> >> > Some test's Makefile using "$(RM)" while the other's >> > using "rm -f". It is better to

Re: [PATCH v7 07/11] arch/x86: enable task isolation functionality

2015-09-29 Thread Chris Metcalf
On 09/28/2015 06:43 PM, Andy Lutomirski wrote: Why are we treating alarms as something that should defer entry to userspace? I think it would be entirely reasonable to set an alarm for ten minutes, ask for isolation, and then think hard for ten minutes. A bigger issue would be if there's an RT

Re: [RFC][PATCH] sched: Fix TASK_DEAD race in finish_task_switch()

2015-09-29 Thread Linus Torvalds
On Tue, Sep 29, 2015 at 12:50 PM, Peter Zijlstra wrote: > > The control dependency creates a LOAD->STORE order, that is, no STOREs > can happen until we observe !p->on_cpu. Fair enough. > Right, but wmb isn't sufficient as it doesn't order the prev->state LOAD > vs the prev->on_cpu = 0 STORE.

Re: [PATCH 0/2] memory: Fix module autoload for OF platform driver

2015-09-29 Thread santosh shilimkar
+Greg, arm-sock folks On 9/17/2015 10:56 AM, santosh shilimkar wrote: On 9/17/2015 10:41 AM, Luis de Bethencourt wrote: Hello, These patches add the missing MODULE_DEVICE_TABLE() for OF to export the information so modules have the correct aliases built-in and autoloading works correctly. A

Re: [PATCH 0/1] ns: introduce proc_get_ns_by_fd()

2015-09-29 Thread Eric W. Biederman
Oleg Nesterov writes: > On 09/28, Eric W. Biederman wrote: >> >> Oleg Nesterov writes: >> >> > Honestly, I do not really like the new helper... I understand this >> > is subjective, so I won't insist. But how about 1/1? We do not need >> > fd/file at all. With this patch your sys_getvpid() can

[PATCH] selftests: run test_printf module

2015-09-29 Thread Kees Cook
This runs to test_printf module to make sure printf is operating sanely. Signed-off-by: Kees Cook --- Designed for us with "test_printf: test printf family at runtime": http://www.spinics.net/lists/kernel/msg2083336.html http://www.spinics.net/lists/kernel/msg2085894.html ---

Re: [PATCH v7 03/11] task_isolation: support PR_TASK_ISOLATION_STRICT mode

2015-09-29 Thread Chris Metcalf
On 09/28/2015 06:38 PM, Andy Lutomirski wrote: On Mon, Sep 28, 2015 at 2:54 PM, Chris Metcalf wrote: On 09/28/2015 04:51 PM, Andy Lutomirski wrote: On Mon, Sep 28, 2015 at 11:17 AM, Chris Metcalf @@ -35,8 +36,12 @@ static inline enum ctx_state exception_enter(void) return

Re: [PATCH] x86: Use entire page for the per-cpu GDT only if paravirt-enabled

2015-09-29 Thread Andy Lutomirski
On Sep 29, 2015 2:01 AM, "Ingo Molnar" wrote: > > > * Denys Vlasenko wrote: > > > On 09/28/2015 09:58 AM, Ingo Molnar wrote: > > > > > > * Denys Vlasenko wrote: > > > > > >> On 09/26/2015 09:50 PM, H. Peter Anvin wrote: > > >>> NAK. We really should map the GDT read-only on all 64 bit systems,

Re: [PATCH v7 05/11] task_isolation: add debug boot flag

2015-09-29 Thread Chris Metcalf
On 09/28/2015 06:40 PM, Andy Lutomirski wrote: If I were writing a program that used this feature, I think I'd want to know early that it's not going to work so I can tell the admin very loudly to fix it. Maybe just failing the prctl would be enough. If someone turns on the prctl and then

Re: [PATCH 4/4] test_printf: test printf family at runtime

2015-09-29 Thread Kees Cook
On Tue, Sep 29, 2015 at 12:10 AM, Rasmus Villemoes wrote: > On Tue, Sep 29 2015, Kees Cook wrote: > >>> +static void __init >>> +test_string(void) >>> +{ >>> + test("", "%s%.0s", "", "123"); >>> + test("ABCD|abc|123", "%s|%.3s|%.*s", "ABCD", "abcdef", 3, "123456"); >>> +

Re: 4.3-rc3 Regression: NFS access stall by commit 6ae459bdaaee

2015-09-29 Thread Pravin Shelar
On Tue, Sep 29, 2015 at 3:33 AM, Takashi Iwai wrote: > On Tue, 29 Sep 2015 02:35:04 +0200, > Pravin Shelar wrote: >> >> On Mon, Sep 28, 2015 at 6:12 AM, Takashi Iwai wrote: >> > [I resent this since the previous mail didn't go out properly, as it >> > seems; apologies if you already read it,

Re: [PATCH v4 0/7] blk-mq: fix race conditions on cpu hotplug handling

2015-09-29 Thread Jens Axboe
On 09/26/2015 11:09 AM, Akinobu Mita wrote: This patchset addresses several race conditions on cpu hotplug handling for blk-mq. All problems can be reproducible by the following script. while true; do echo 0 > /sys/devices/system/cpu/cpu1/online echo 1 >

Re: [PATCH] x86: include: asm: Fix build warning

2015-09-29 Thread Borislav Petkov
On Tue, Sep 29, 2015 at 07:19:32PM +0200, Ksenija Stanojević wrote: > On Tue, Sep 29, 2015 at 7:16 PM, Borislav Petkov wrote: > > On Tue, Sep 29, 2015 at 07:12:07PM +0200, Ksenija Stanojević wrote: > >> when building with: > >> make drivers/staging/oplc_dcon/olpc_dcon.o > >> I get this warnings.

Re: [PATCH 1/4] of: Add of_parse_phandle_with_opt_args() helper function

2015-09-29 Thread Rob Herring
On Tue, Sep 22, 2015 at 12:52 PM, Marc Zyngier wrote: > of_parse_phandle_with_args() is slightly inflexible as it doesn't > allow the (unusual) case where the #*-cells property is not defined. > In order to support this, introduce of_parse_phandle_with_opt_args() > which assumes that #*-cells is

Re: [PATCH] sched/fair: Skip wake_affine() for core siblings

2015-09-29 Thread Mike Galbraith
On Tue, 2015-09-29 at 19:00 +0300, Kirill Tkhai wrote: > > On 29.09.2015 17:55, Mike Galbraith wrote: > > On Mon, 2015-09-28 at 18:36 +0300, Kirill Tkhai wrote: > > > >> --- > >> diff --git a/kernel/sched/fair.c b/kernel/sched/fair.c > >> index 4df37a4..dfbe06b 100644 > >> ---

Re: [PATCH] ARM: dts: Add Exynos5250 Snow Rev5+ support

2015-09-29 Thread Doug Anderson
Javier, On Tue, Sep 29, 2015 at 4:57 AM, Javier Martinez Canillas wrote: > There are 2 revisions of the Exynos5250 Snow Chromebook that were shipped: > Rev4 and Rev5. The only difference between these 2 revisions is the codec, > Rev4 has a max98095 codec while Rev5 has a max98090. > > Mainline

Re: [PATCH 1/2] tpm, tpm_tis: use acpi_driver instead of pnp_driver

2015-09-29 Thread Jason Gunthorpe
On Tue, Sep 29, 2015 at 08:07:10PM +0300, Jarkko Sakkinen wrote: > -static struct pnp_device_id tpm_pnp_tbl[] = { > +static struct acpi_device_id tpm_acpi_tbl[] = { > {"PNP0C31", 0}, /* TPM */ > {"ATM1200", 0}, /* Atmel */ > {"IFX0102", 0}, /* Infineon */

Re: [PATCH 3.14 79/84] x86/nmi/64: Switch stacks on userspace NMI entry

2015-09-29 Thread Andy Lutomirski
On Tue, Sep 29, 2015 at 8:19 AM, Greg Kroah-Hartman wrote: > 3.14-stable review patch. If anyone has any objections, please let me know. > > -- > > From: Andy Lutomirski > > commit 9b6e6a8334d56354853f9c255d1395c2ba570e0a upstream. > > Returning to userspace is tricky: IRET can

Re: [PATCH] x86: include: asm: Fix build warning

2015-09-29 Thread Ksenija Stanojević
On Tue, Sep 29, 2015 at 7:16 PM, Borislav Petkov wrote: > On Tue, Sep 29, 2015 at 07:12:07PM +0200, Ksenija Stanojević wrote: >> when building with: >> make drivers/staging/oplc_dcon/olpc_dcon.o >> I get this warnings. > > Yeah, you can't pick out single files from the source tree and try to >

Re: [PATCH v4 0/9] Making the generic ACPI GSI layer irqdomain aware

2015-09-29 Thread Marc Zyngier
On 29/09/15 18:11, Rob Herring wrote: > On Mon, Sep 28, 2015 at 11:42 AM, Marc Zyngier wrote: >> The irqdomain code is not entierely ACPI friendly, as it has some >> built-in knowledge of the device-tree. Nothing too harmful, but enough >> to scare the ARM ACPI developpers which end up with their

Re: AMD-IOMMU and problem with __init(data)?

2015-09-29 Thread Alexander Holler
Am 29.09.2015 um 17:06 schrieb Joerg Roedel: As expected it is no bug in the AMD IOMMU driver, but in your code. On Wed, Sep 23, 2015 at 09:04:31PM +0200, Alexander Holler wrote: struct _annotated_initcall { initcall_t initcall; unsigned driver_id; unsigned

Re: [PATCH] x86: include: asm: Fix build warning

2015-09-29 Thread Borislav Petkov
On Tue, Sep 29, 2015 at 07:12:07PM +0200, Ksenija Stanojević wrote: > when building with: > make drivers/staging/oplc_dcon/olpc_dcon.o > I get this warnings. Yeah, you can't pick out single files from the source tree and try to build them and expect that to always work. That olpc_dcon think

Re: [PATCH] perf record: Limit --intr-regs to platforms supporting PERF_REGS

2015-09-29 Thread Jiri Olsa
On Tue, Sep 29, 2015 at 10:01:36PM +0530, Naveen N. Rao wrote: > On 2015/09/29 12:47PM, Jiri Olsa wrote: > > On Tue, Sep 29, 2015 at 01:30:10PM +0530, Naveen N. Rao wrote: > > > > SNIP > > > > > > > Suka has also posted a fix for this with a different approach [1]. > > > > > Can > > > > > you

Re: [RFC PATCH 0/3] iommu: Add range flush operation

2015-09-29 Thread Robin Murphy
On 29/09/15 17:40, Russell King - ARM Linux wrote: On Tue, Sep 29, 2015 at 05:27:12PM +0100, Robin Murphy wrote: Eh, swings and roundabouts. An argument denoting whether the flush is being called on the map or unmap path would be fine, Sorry, that statement is wrong. It's not about whether

Re: [PATCH] x86: include: asm: Fix build warning

2015-09-29 Thread Ksenija Stanojević
Hi, On Tue, Sep 29, 2015 at 7:08 PM, H. Peter Anvin wrote: > On 09/29/2015 10:06 AM, Borislav Petkov wrote: >> On Tue, Sep 29, 2015 at 06:25:46PM +0200, Ksenija Stanojevic wrote: >>> Fix build warnings: >>> >>> error: implicit declaration of function ‘olpc_board_at_least’ >>>

Re: [PATCH v4 0/9] Making the generic ACPI GSI layer irqdomain aware

2015-09-29 Thread Rob Herring
On Mon, Sep 28, 2015 at 11:42 AM, Marc Zyngier wrote: > The irqdomain code is not entierely ACPI friendly, as it has some > built-in knowledge of the device-tree. Nothing too harmful, but enough > to scare the ARM ACPI developpers which end up with their own version > of the square wheel. > >

Re: [PATCH] x86: include: asm: Fix build warning

2015-09-29 Thread H. Peter Anvin
On 09/29/2015 10:06 AM, Borislav Petkov wrote: > On Tue, Sep 29, 2015 at 06:25:46PM +0200, Ksenija Stanojevic wrote: >> Fix build warnings: >> >> error: implicit declaration of function ‘olpc_board_at_least’ >> [-Werror=implicit-function-declaration] >> >> error: implicit declaration of function

[PATCH] staging: vt6655: Fixed two lines over 80 characters

2015-09-29 Thread Arjun Krishna Babu
Two lines of code that were over 80 characters long is fixed by splitting them into multiple lines. This way, the lines of code are now easier to read. Issue found by checkpatch. Signed-off-by: Arjun Krishna Babu --- drivers/staging/vt6655/baseband.c | 13 +++-- 1 file changed, 11

[PATCH 1/2] tpm, tpm_tis: use acpi_driver instead of pnp_driver

2015-09-29 Thread Jarkko Sakkinen
Migrate to struct acpi_driver in order to get out of 7 character limitation for the HID. Reported-by: Matthew Garrett Signed-off-by: Jarkko Sakkinen --- drivers/char/tpm/tpm_tis.c | 124 ++--- 1 file changed, 73 insertions(+), 51 deletions(-) diff --git

[PATCH 2/2] tpm, tpm_tis: detect TPM2 FIFO devices based on HID

2015-09-29 Thread Jarkko Sakkinen
It turned out that the root cause in b371616b8 was not correct for https://bugzilla.kernel.org/show_bug.cgi?id=98181. All TPM 2.0 FIFO and CRB devices have the same HID. For FIFO devices the start method is always 6. This patch changes FIFO and CRB drivers so that they check start method value

[PATCH 0/2] Fix tpm_tis ACPI issue with TPM 2.0

2015-09-29 Thread Jarkko Sakkinen
Both for FIFO and CRB interface TCG has decided to use the same HID MSFT0101. They can be differentiated by looking at the start method from TPM2 ACPI table. These patches make necessary fixes to tpm_tis and tpm_crb modules in order to correctly detect, which module should be used. Jarkko

Re: [PATCH] x86: include: asm: Fix build warning

2015-09-29 Thread Borislav Petkov
On Tue, Sep 29, 2015 at 06:25:46PM +0200, Ksenija Stanojevic wrote: > Fix build warnings: > > error: implicit declaration of function ‘olpc_board_at_least’ > [-Werror=implicit-function-declaration] > > error: implicit declaration of function ‘olpc_board’ >

[PATCH v2 5/5] writeback: fix incorrect calculation of available memory for memcg domains

2015-09-29 Thread Tejun Heo
For memcg domains, the amount of available memory was calculated as min(the amount currently in use + headroom according to memcg, total clean memory) This isn't quite correct as what should be capped by the amount of clean memory is the headroom, not the sum of memory in use and headroom.

[PATCH] misc: mic: replace kfree with put_device

2015-09-29 Thread Geliang Tang
Handle a failed device_register(), replace kfree() with put_device(), which will call mbus_release_dev() or scif_release_dev(). Signed-off-by: Geliang Tang --- drivers/misc/mic/bus/mic_bus.c | 2 +- drivers/misc/mic/bus/scif_bus.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff

Re: [RFC PATCH 0/3] iommu: Add range flush operation

2015-09-29 Thread Russell King - ARM Linux
On Tue, Sep 29, 2015 at 05:27:12PM +0100, Robin Murphy wrote: > Eh, swings and roundabouts. An argument denoting whether the flush is being > called on the map or unmap path would be fine, Sorry, that statement is wrong. It's not about whether you flush before or after the DMA operation. I'm

Re: [RFC][PATCH] sched: Fix TASK_DEAD race in finish_task_switch()

2015-09-29 Thread Peter Zijlstra
On Tue, Sep 29, 2015 at 12:40:22PM -0400, Linus Torvalds wrote: > On Tue, Sep 29, 2015 at 8:45 AM, Peter Zijlstra wrote: > > > > +* > > +* Pairs with the control dependency and rmb in try_to_wake_up(). > > */ > > So this comment makes me nervous. A control dependency

[PATCH 2/5] writeback: fix bdi_writeback iteration in wakeup_dirtytime_writeback()

2015-09-29 Thread Tejun Heo
wakeup_dirtytime_writeback() walks and wakes up all wb's of all bdi's; unfortunately, it was always waking up bdi->wb instead of the wb being walked. Fix it. Signed-off-by: Tejun Heo Fixes: 001fe6f617b1 ("writeback: make wakeup_dirtytime_writeback() handle multiple bdi_writeback's") ---

Re: [PATCH v6 20/22] driver core: Allow deferring probes until late init

2015-09-29 Thread Rob Herring
On Tue, Sep 29, 2015 at 3:05 AM, Tomeu Vizoso wrote: > On 26 September 2015 at 20:15, Rob Herring wrote: >> On 09/21/2015 09:03 AM, Tomeu Vizoso wrote: >>> Add a field to struct device that instructs the device-driver core to >>> defer the probe of this device until the late_initcall level. >>

[PATCH 4/5] writeback: memcg dirty_throttle_control should be initialized with wb->memcg_completions

2015-09-29 Thread Tejun Heo
MDTC_INIT() is used to initialize dirty_throttle_control for memcg domains. It used DTC_INIT_COMMON() to initialized mdtc->wb and ->wb_completions which is incorrect as DTC_INIT_COMMON() sets the latter to wb->completions instead of wb->memcg_completions. This can lead to wildly incorrect

Re: [PATCH 3.10 00/56] 3.10.90-stable review

2015-09-29 Thread Shuah Khan
On 09/29/2015 07:46 AM, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 3.10.90 release. > There are 56 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. > > Responses

[PATCH 5/5] writeback: fix incorrect calculation of available memory for memcg domains

2015-09-29 Thread Tejun Heo
For memcg domains, the amount of available memory was calculated as min(the amount currently in use + headroom according to memcg, total clean memory) This isn't quite correct as what should be capped by the amount of clean memory is the headroom, not the sum of memory in use and headroom.

[PATCH 2/5] ftrace: clarify code for mod command

2015-09-29 Thread Dmitry Safonov
"Not" is too abstract variable name - changed to clear_filter. Removed ftrace_match_module_records function: comparison with !* or * not does the general code in filter_parse_regex() as it works without mod command for sh# echo '!*' > /sys/kernel/debug/tracing/set_ftrace_filter Signed-off-by:

<    1   2   3   4   5   6   7   8   9   10   >