Re: [PATCH kernel v5 5/6] vfio/spapr: Reference mm in tce_container

On 11/11/16 23:32, Alexey Kardashevskiy wrote: > In some situations the userspace memory context may live longer than > the userspace process itself so if we need to do proper memory context > cleanup, we better have tce_container take a reference to mm_struct and > use it later when the process

Re: [PATCH v2] Fix loading of module radeonfb on PowerMac

Tomi, On Tue, Nov 15, 2016 at 12:46 PM, Tomi Valkeinen wrote: > Hi, > > On 14/11/16 21:59, Mathieu Malaterre wrote: >> When the linux kernel is build with (typical kernel ship with Debian >> installer): >> >> CONFIG_FB_OF=y >> CONFIG_VT_HW_CONSOLE_BINDING=y >>

Re: [PATCH v7 06/11] x86, paravirt: Add interface to support kvm/xen vcpu preempted check

在 2016/11/16 18:23, Peter Zijlstra 写道: On Wed, Nov 16, 2016 at 12:19:09PM +0800, Pan Xinhui wrote: Hi, Peter. I think we can avoid a function call in a simpler way. How about below static inline bool vcpu_is_preempted(int cpu) { /* only set in pv case*/ if

[PATCH] powerpc/eeh: Refactor EEH PE reset functions

eeh_pe_reset and eeh_reset_pe are two different functions in the same file which do mostly the same thing. Not only is this confusing, but potentially causes disrepancies in functionality, notably eeh_reset_pe as it does not check return values for failure. Refactor this into the following: -

Re: powerpc/64: Fix setting of AIL in hypervisor mode

On Tue, 2016-15-11 at 04:28:33 UTC, Benjamin Herrenschmidt wrote: > Commit d3cbff1b5 "powerpc: Put exception configuration in a common place" > broke the setting of the AIL bit (which enables taking exceptions with > the MMU still on) on all processors, moving it incorrectly to a function > called

Re: [PATCH] powerpc: crypto/vmx: various build fixes

"Naveen N. Rao" writes: > First up, clean up the generated .S files properly on a 'make clean'. > Secondly, force re-generation of these files when building for different > endian-ness than what was built previously. Finally, generate the new > files in the build

Re: [RESEND] [PATCH v1 1/3] Add basic infrastructure for memcg hotplug support

On 16/11/16 20:01, Vladimir Davydov wrote: > Hello, > > On Wed, Nov 16, 2016 at 10:44:59AM +1100, Balbir Singh wrote: >> The lack of hotplug support makes us allocate all memory >> upfront for per node data structures. With large number >> of cgroups this can be an overhead. PPC64 actually

Re: [powerpc v6 1/3] Setup AMOR in HV mode

On 16/11/16 18:58, Aneesh Kumar K.V wrote: > Balbir Singh writes: > >> AMOR should be setup in HV mode, we set it up once >> and let the generic kernel handle IAMR. This patch is >> used to enable storage keys in a following patch as >> defined in ISA 3. We don't setup

Re: [PATCH] cxl: Fix potential build failure - select ANON_INODES

On Thu, 2016-11-17 at 09:30 +1100, Andrew Donnellan wrote: > On 17/11/16 05:29, Ben Hutchings wrote: > > cxl calls anon_inode_get() so must select ANON_INODES. > > > > Fixes: 6f7f0b3df6d4 ("cxl: Add AFU virtual PHB and kernel API") > > Signed-off-by: Ben Hutchings > >

Re: [PATCH] cxl: Fix potential build failure - select ANON_INODES

On 17/11/16 05:29, Ben Hutchings wrote: cxl calls anon_inode_get() so must select ANON_INODES. Fixes: 6f7f0b3df6d4 ("cxl: Add AFU virtual PHB and kernel API") Signed-off-by: Ben Hutchings Thanks for catching this. Is this patch still necessary once

Re: [PATCH 1/1] sched: provide common cpu_relax_yield definition

Am 16.11.2016 um 13:23 schrieb Christian Borntraeger: No need to duplicate the same define everywhere. Since the only user is stop-machine and the only provider is s390, we can use a default implementation of cpu_relax_yield in sched.h. Suggested-by: Russell King

Re: [RESEND] [PATCH v1 1/3] Add basic infrastructure for memcg hotplug support

Hello, On Wed, Nov 16, 2016 at 10:44:59AM +1100, Balbir Singh wrote: > The lack of hotplug support makes us allocate all memory > upfront for per node data structures. With large number > of cgroups this can be an overhead. PPC64 actually limits > n_possible nodes to n_online to avoid some of

[PATCH] cxl: Fix potential build failure - select ANON_INODES

cxl calls anon_inode_get() so must select ANON_INODES. Fixes: 6f7f0b3df6d4 ("cxl: Add AFU virtual PHB and kernel API") Signed-off-by: Ben Hutchings --- drivers/misc/cxl/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/misc/cxl/Kconfig

[PATCH] powerpc/boot: Fix build failure in 32-bit boot wrapper

OPAL is not callable from 32-bit mode and the assembly code for it may not even build (depending on how binutils was configured). References: https://buildd.debian.org/status/fetch.php?pkg=linux=powerpcspe=4.8.7-1=1479203712 Fixes: 656ad58ef19e ("powerpc/boot: Add OPAL console to epapr

Re: [PATCHC v7 00/10] ima: carry the measurement list across kexec

On Thu, Nov 10, 2016 at 4:56 PM, Mimi Zohar wrote: > [Posting with abbreviated Cc list.] > > The TPM PCRs are only reset on a hard reboot. In order to validate a > TPM's quote after a soft reboot (eg. kexec -e), the IMA measurement list > of the running kernel must be

Re: [PATCHC v7 02/10] ima: on soft reboot, restore the measurement list

On Thu, Nov 10, 2016 at 4:56 PM, Mimi Zohar wrote: > The TPM PCRs are only reset on a hard reboot. In order to validate a > TPM's quote after a soft reboot (eg. kexec -e), the IMA measurement list > of the running kernel must be saved and restored on boot. This patch >

Re: [PATCH 2/2] soc: fsl: fix section mismatch build warnings

On 16/11/16 17:07, Arnd Bergmann wrote: On Wednesday, November 16, 2016 4:39:27 PM CET Sudeep Holla wrote: @@ -223,6 +222,7 @@ static struct platform_driver fsl_guts_driver = { static int __init fsl_guts_init(void) { + machine = of_flat_dt_get_machine_name(); return

Re: [PATCH 2/2] soc: fsl: fix section mismatch build warnings

On Wednesday, November 16, 2016 4:39:27 PM CET Sudeep Holla wrote: > @@ -223,6 +222,7 @@ static struct platform_driver fsl_guts_driver = { > > static int __init fsl_guts_init(void) > { > + machine = of_flat_dt_get_machine_name(); > return platform_driver_register(_guts_driver); >

[PATCH] powerpc/mm: allow memory hotplug into an offline node

Relax the check preventing us from hotplugging into an offline node. This limitation was added in commit 482ec7c403d2 ("[PATCH] powerpc numa: Support sparse online node map") to prevent adding resources to an uninitialized node. These days, there is no harm in doing so. The addition will

Re: [PATCH 1/2] soc: fsl: make it explicitly non-modular

On 16/11/16 16:39, Sudeep Holla wrote: The Kconfig currently controlling compilation of this code is: drivers/soc/fsl/Kconfig:config FSL_GUTS drivers/soc/fsl/Kconfig: bool ...meaning that it currently is not being built as a module by anyone. Lets remove the modular code that is

Re: [RESEND] [PATCH v1 3/3] powerpc: fix node_possible_map limitations

On Wed, Nov 16, 2016 at 10:45:01AM +1100, Balbir Singh wrote: Reverts: commit 3af229f2071f ("powerpc/numa: Reset node_possible_map to only node_online_map") Nice! With this limitation going away, I have a small patch to enable onlining new nodes via memory hotplug. Incoming. -- Reza Arbab

[PATCH 1/2] soc: fsl: make it explicitly non-modular

The Kconfig currently controlling compilation of this code is: drivers/soc/fsl/Kconfig:config FSL_GUTS drivers/soc/fsl/Kconfig: bool ...meaning that it currently is not being built as a module by anyone. Lets remove the modular code that is essentially orphaned, so that when reading the

[PATCH 2/2] soc: fsl: fix section mismatch build warnings

We get the following warning with the driver is compiled in: WARNING: modpost: Found 1 section mismatch(es). To see full details build your kernel with: 'make CONFIG_DEBUG_SECTION_MISMATCH=y' With CONFIG_DEBUG_SECTION_MISMATCH enabled, the details are reported: WARNING:

[PATCH] powerpc: crypto/vmx: various build fixes

First up, clean up the generated .S files properly on a 'make clean'. Secondly, force re-generation of these files when building for different endian-ness than what was built previously. Finally, generate the new files in the build tree, rather than the source tree. Signed-off-by: Michael

[PATCH 16/17] soc/qman: Clean up CGR CSCN target update operations

Signed-off-by: Claudiu Manoil --- drivers/soc/fsl/qbman/qman.c | 37 +++-- include/soc/fsl/qman.h | 4 ++-- 2 files changed, 25 insertions(+), 16 deletions(-) diff --git a/drivers/soc/fsl/qbman/qman.c b/drivers/soc/fsl/qbman/qman.c

[PATCH 17/17] soc/qman: Handle endianness of h/w descriptors

The hardware descriptors have big endian (BE) format. Provide proper endianness handling for the remaining descriptor fields, to ensure they are correctly accessed by non-BE CPUs too. Signed-off-by: Claudiu Manoil --- drivers/soc/fsl/qbman/qman.c| 65

[PATCH 15/17] soc/qman: Change remaining contextB into context_b

There are multiple occurences of both contextB and context_b in different h/w descriptors, referring to the same descriptor field known as "Context B". Stick with the "context_b" naming, for obvious reasons including consistency (see also context_a). Signed-off-by: Claudiu Manoil

[PATCH 14/17] soc/qbman: Handle endianness of qm/bm_in/out()

Signed-off-by: Claudiu Manoil --- drivers/soc/fsl/qbman/bman.c | 6 +++--- drivers/soc/fsl/qbman/qman.c | 6 +++--- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/soc/fsl/qbman/bman.c b/drivers/soc/fsl/qbman/bman.c index ffa48fd..345e50c 100644 ---

[PATCH 06/17] soc/qman: Fix direct access to fd's addr_lo, use proper accesor

Use the proper accessor to get the FD address. Accessing the internal field "addr_lo" directly is not portable and error prone. Signed-off-by: Claudiu Manoil --- drivers/soc/fsl/qbman/qman.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 08/17] soc/qman: Don't add a new platform device for dma mapping

The qman portals are platform devices themselves, so they should handle dma mappings. Creating a dummy platform device in order to support dma mapping operations is not justified (and not portable). Instead, do the mapping against the first portal that has been initialised. Signed-off-by:

[PATCH 13/17] soc/qman: Drop unused field from eqcr/dqrr descriptors

ORP ("Order Restoration Point") mechanism not supported. Signed-off-by: Claudiu Manoil --- drivers/soc/fsl/qbman/qman.c | 3 +-- include/soc/fsl/qman.h | 3 +-- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/soc/fsl/qbman/qman.c

[PATCH 12/17] soc/qman: Fix accesses to fqid, cleanup

Preventively mask every access to the 'fqid' h/w field, since it is defined as a 24-bit field, for every h/w descriptor. Add generic accessors for this field to ensure correct access. Signed-off-by: Claudiu Manoil --- drivers/soc/fsl/qbman/qman.c | 32

[PATCH 11/17] soc/qman: Remove unused struct qm_mcc* layouts

1. qm_mcc_querywq layout not used for now, so drop it; 2. queryfq, queryfq_np and alterfq are used only for accesses to the 'fqid' field, so replace these with a generic 'fq' layout. As a consequence, 'querycgr' turns into 'cgr' following the same reasoning above and for consistent

[PATCH 10/17] soc/qman: Remove redundant checks from qman_create_cgr()

opts is checked redundantly. Move local_opts declaration inside its usage scope. Signed-off-by: Claudiu Manoil --- drivers/soc/fsl/qbman/qman.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/soc/fsl/qbman/qman.c

[PATCH 09/17] soc/qman: test: Don't use dummy platform device for dma mapping

Replace dummy platform device hack with a reference to a portal's platform device, in order to dma map the test frame for this small unit test. The 2 qman symbols need to be exported because this self test is a kernel module. Signed-off-by: Claudiu Manoil ---

[PATCH 04/17] soc/qbman: Fix resource leak on portal probing error path

In case init_pcfg() returns with error the CI region must be unmapped too. Signed-off-by: Claudiu Manoil --- drivers/soc/fsl/qbman/bman_portal.c | 17 - drivers/soc/fsl/qbman/qman_portal.c | 17 - 2 files changed, 24 insertions(+), 10

[PATCH 07/17] soc/qman: test: Fix implementation of fd_cmp()

This function must only return the truth value of whether two frame descriptors are different or not. It does NOT have to compute some obscure difference between fd fields and return it as an int, making sparse complain about type conversions in the process. Signed-off-by: Claudiu Manoil

[PATCH 05/17] soc/qman: Fix struct qm_fqd set accessor for context_a

context_a.hi is 32bit Signed-off-by: Claudiu Manoil --- include/soc/fsl/qman.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/soc/fsl/qman.h b/include/soc/fsl/qman.h index 37f3eb0..1405810 100644 --- a/include/soc/fsl/qman.h +++

[PATCH 03/17] soc/qman: Fix h/w resource cleanup error path handling

qman_query_fq*() may return other error codes apart from -ERANGE, in which cases the error handling done by the resource cleanup callers would be wrong. The patch fixes the handling of those cases, and cleans up related code inside the resource cleanup & release handlers (i.e. replace hardcoded

[PATCH 02/17] soc/qman: Replace of_get_property() with portable equivalent

From: Madalin Bucur Use arch portable of_property_read_u32() instead, which takes care of endianness conversions. Signed-off-by: Madalin Bucur Signed-off-by: Claudiu Manoil --- drivers/soc/fsl/qbman/dpaa_sys.h| 1 +

[PATCH 01/17] soc/qman: Check ioremap return value

From: Madalin Bucur Signed-off-by: Madalin Bucur Signed-off-by: Claudiu Manoil --- drivers/soc/fsl/qbman/qman_ccsr.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/soc/fsl/qbman/qman_ccsr.c

[PATCH 00/17] soc/qman: Fixes, endianness handling and related cleanup

* endianness fixes to ensure correct handling on little endian CPUs, as the qman h/w is big endian; * removal of a platform device hack that may fail, used for dma mapping; * various fixes and code cleanup; Claudiu Manoil (15): soc/qman: Fix h/w resource cleanup error path handling

Re: [bug report] Driver for IBM System i/p VNIC protocol

On 11/16/2016 06:25 AM, Dan Carpenter wrote: > Hello Thomas Falcon, > > The patch 032c5e82847a: "Driver for IBM System i/p VNIC protocol" > from Dec 21, 2015, leads to the following static checker warning: > > drivers/net/ethernet/ibm/ibmvnic.c:2957 error_level_write() > why cast

[PATCH] cxl: Do adapter fence check before handling afu interrupt

If an afu interrupt is in flight when an eeh error is triggered the control still reaches the function native_irq_multiplexed and the PE-Handle read from the CXL_PSL_PEHandle_An register is 0x. The function then erroneously assumes that the interrupt belonged to a detached context and

Re: [PATCH 1/1] sched: provide common cpu_relax_yield definition

On Wed, Nov 16, 2016 at 01:23:05PM +0100, Christian Borntraeger wrote: > No need to duplicate the same define everywhere. Since > the only user is stop-machine and the only provider is > s390, we can use a default implementation of cpu_relax_yield > in sched.h. > > Suggested-by: Russell King

[bug report] Driver for IBM System i/p VNIC protocol

Hello Thomas Falcon, The patch 032c5e82847a: "Driver for IBM System i/p VNIC protocol" from Dec 21, 2015, leads to the following static checker warning: drivers/net/ethernet/ibm/ibmvnic.c:2957 error_level_write() why cast 'kstrtoul()?' drivers/net/ethernet/ibm/ibmvnic.c 2946

[PATCH 1/1] sched: provide common cpu_relax_yield definition

No need to duplicate the same define everywhere. Since the only user is stop-machine and the only provider is s390, we can use a default implementation of cpu_relax_yield in sched.h. Suggested-by: Russell King Signed-off-by: Christian Borntraeger

Re: [PATCH v7 06/11] x86, paravirt: Add interface to support kvm/xen vcpu preempted check

On Wed, Nov 16, 2016 at 12:29:44PM +0100, Christian Borntraeger wrote: > On 11/16/2016 11:23 AM, Peter Zijlstra wrote: > > On Wed, Nov 16, 2016 at 12:19:09PM +0800, Pan Xinhui wrote: > >> Hi, Peter. > >>I think we can avoid a function call in a simpler way. How about below > >> > >> static

Re: [PATCH v3 02/16] scsi: don't use fc_bsg_job::request and fc_bsg_job::reply directly

On Tue, Nov 15, 2016 at 04:39:33PM +0100, Johannes Thumshirn wrote: > On Tue, Nov 15, 2016 at 03:31:27PM +0100, Steffen Maier wrote: > > Hi Johannes, > > > > On 11/15/2016 12:56 PM, Johannes Thumshirn wrote: > > > On Tue, Oct 25, 2016 at 09:43:14AM +0200, Johannes Thumshirn wrote: > > > > On Fri,

Re: [PATCH v7 06/11] x86, paravirt: Add interface to support kvm/xen vcpu preempted check

On 11/16/2016 11:23 AM, Peter Zijlstra wrote: > On Wed, Nov 16, 2016 at 12:19:09PM +0800, Pan Xinhui wrote: >> Hi, Peter. >> I think we can avoid a function call in a simpler way. How about below >> >> static inline bool vcpu_is_preempted(int cpu) >> { >> /* only set in pv case*/ >>

Re: [PATCH v2] powerpc/mm/radix: Invalidate ERAT on tlbiel for POWER9 DD1

Balbir Singh writes: > On 16/11/16 21:51, Michael Ellerman wrote: >> Michael Neuling writes: >>> diff --git a/arch/powerpc/mm/tlb-radix.c b/arch/powerpc/mm/tlb-radix.c >>> index bda8c43be7..4a19cdd8a0 100644 >>> --- a/arch/powerpc/mm/tlb-radix.c >>> +++

Re: [RFC PATCH] powerpc: crypto/vmx: clean up generated files

On 2016/11/16 09:02PM, Michael Ellerman wrote: > "Naveen N. Rao" writes: > > > ..as stray .S files result in build errors, especially when using > > cross-compilers. > > They should be cleaned on make clean, so adding them to clean-files > seems correct. > > >

Re: [PATCH v2] powerpc/mm/radix: Invalidate ERAT on tlbiel for POWER9 DD1

On 16/11/16 21:51, Michael Ellerman wrote: > Michael Neuling writes: >> diff --git a/arch/powerpc/mm/tlb-radix.c b/arch/powerpc/mm/tlb-radix.c >> index bda8c43be7..4a19cdd8a0 100644 >> --- a/arch/powerpc/mm/tlb-radix.c >> +++ b/arch/powerpc/mm/tlb-radix.c >> @@ -50,6 +50,9 @@

Re: [PATCH v2] powerpc/mm/radix: Invalidate ERAT on tlbiel for POWER9 DD1

Michael Neuling writes: > diff --git a/arch/powerpc/mm/tlb-radix.c b/arch/powerpc/mm/tlb-radix.c > index bda8c43be7..4a19cdd8a0 100644 > --- a/arch/powerpc/mm/tlb-radix.c > +++ b/arch/powerpc/mm/tlb-radix.c > @@ -50,6 +50,9 @@ static inline void _tlbiel_pid(unsigned long pid,

Re: [PATCH v7 06/11] x86, paravirt: Add interface to support kvm/xen vcpu preempted check

On Wed, Nov 16, 2016 at 12:19:09PM +0800, Pan Xinhui wrote: > Hi, Peter. > I think we can avoid a function call in a simpler way. How about below > > static inline bool vcpu_is_preempted(int cpu) > { > /* only set in pv case*/ > if (pv_lock_ops.vcpu_is_preempted) >

Re: [RFC PATCH] powerpc: crypto/vmx: clean up generated files

"Naveen N. Rao" writes: > ..as stray .S files result in build errors, especially when using > cross-compilers. They should be cleaned on make clean, so adding them to clean-files seems correct. > More specifically, the generated .S files are endian-specific and

Re: [PATCH 1/2] usb: dwc2: add amcc,dwc-otg support

Hi, John Youn writes: > On 11/14/2016 3:00 PM, John Youn wrote: >> On 11/11/2016 3:12 PM, Christian Lamparter wrote: >>> On Friday, November 11, 2016 2:20:42 PM CET John Youn wrote: On 11/11/2016 2:05 PM, Christian Lamparter wrote: > On Friday, November 11,

Re: [Patch v7 3/7] irqchip: xilinx: restructure and use jump label api

On 15.11.2016 17:03, Zubair Lutfullah Kakakhel wrote: > Hi, > > On 11/15/2016 12:49 PM, Michal Simek wrote: >> On 14.11.2016 13:13, Zubair Lutfullah Kakakhel wrote: >>> Add a global structure to house various variables. >>> And cleanup read/write handling by using jump label api. >>> >>>

Re: [PATCH v7 0/6] perf annotate: Cross arch support + few fixes

On 2016/09/21 09:17PM, Ravi Bangoria wrote: > Currently Perf annotate support code navigation (branches and calls) > only when run on the same architecture where perf.data was recorded. > But, for example, record on powerpc server and annotate on client's > x86 desktop is not supported. > > This

Re: [powerpc v6 2/3] Detect instruction fetch denied and report

Balbir Singh writes: > ISA 3 allows for prevention of instruction fetch and execution > of user mode pages. If such an error occurs, SRR1 bit 35 > reports the error. We catch and report the error in do_page_fault() > > Signed-off-by: Balbir Singh >