On 24/11/16 16:14, Andrew Donnellan wrote:
> On 24/11/16 13:05, Balbir Singh wrote:
>> 9. The license for these files is now GPL v3 or later
>
> As much as I love the GPLv3, isn't this an instant NAK?
>
Thats why I called it out, my bad though I should have done
a stronger check than just "git
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 is gone (@current or @current->mm is NULL).
This referen
There is already a helper to create a DMA window which does allocate
a table and programs it to the IOMMU group. However
tce_iommu_take_ownership_ddw() did not use it and did these 2 calls
itself to simplify error path.
Since we are going to delay the default window creation till
the default windo
At the moment the userspace tool is expected to request pinning of
the entire guest RAM when VFIO IOMMU SPAPR v2 driver is present.
When the userspace process finishes, all the pinned pages need to
be put; this is done as a part of the userspace memory context (MM)
destruction which happens on the
We are going to allow the userspace to configure container in
one memory context and pass container fd to another so
we are postponing memory allocations accounted against
the locked memory limit. One of previous patches took care of
it_userspace.
At the moment we create the default DMA window whe
The iommu_table struct manages a hardware TCE table and a vmalloc'd
table with corresponding userspace addresses. Both are allocated when
the default DMA window is created and this happens when the very first
group is attached to a container.
As we are going to allow the userspace to configure con
This changes mm_iommu_xxx helpers to take mm_struct as a parameter
instead of getting it from @current which in some situations may
not have a valid reference to mm.
This changes helpers to receive @mm and moves all references to @current
to the caller, including checks for !current and !current->
We are going to get rid of @current references in mmu_context_boos3s64.c
and cache mm_struct in the VFIO container. Since mm_context_t does not
have reference counting, we will be using mm_struct which does have
the reference counter.
This changes mm_iommu_init/mm_iommu_cleanup to receive mm_struc
These patches are to fix a bug when pages stay pinned hours
after QEMU which requested pinning exited.
Main change to v5 that it is now 7 patches.
Please comment. Thanks.
Alexey Kardashevskiy (7):
powerpc/iommu: Pass mm_struct to init/cleanup helpers
powerpc/iommu: Stop using @current in mm_
On Thu, Nov 24, 2016 at 1:38 AM, Segher Boessenkool
wrote:
> On Thu, Nov 24, 2016 at 12:08:40AM +1100, Nicholas Piggin wrote:
>> Question, are there any fundamental reasons we shouldn't use the ELFv2
>> ABI to build big endian kernels if the compiler supports it?
>
> No one uses ELFv2 for BE in pr
On Wed, Sep 21, 2016 at 10:11:54AM +0200, Christophe Leroy wrote:
> Today there are two implementations of hugetlbpages which are managed
> by exclusive #ifdefs:
> * FSL_BOOKE: several directory entries points to the same single hugepage
> * BOOK3S: one upper level directory entry points to a table
On 24/11/16 13:05, Balbir Singh wrote:
9. The license for these files is now GPL v3 or later
As much as I love the GPLv3, isn't this an instant NAK?
--
Andrew Donnellan OzLabs, ADL Canberra
andrew.donnel...@au1.ibm.com IBM Australia Limited
On Wed, Nov 23, 2016 at 04:06:30PM +1100, Alexey Kardashevskiy wrote:
> On 23/11/16 12:35, David Gibson wrote:
> > On Tue, Nov 22, 2016 at 06:29:39PM +1100, Alexey Kardashevskiy wrote:
> >> On 22/11/16 13:50, David Gibson wrote:
> >>> On Fri, Nov 11, 2016 at 11:32:15PM +1100, Alexey Kardashevskiy w
Hi Andrew,
After merging the akpm-current tree, today's linux-next build (powerpc
ppc64_defconfig) failed like this:
powerpc-linux-ld: unrecognized option '--no-dynamic-linker'
Caused by patch
"powerpc: add purgatory for kexec_file_load implementation"
I have disabled KEXEC_FILE for now:
Fr
On Wed, 23 Nov 2016 08:38:34 -0600
Segher Boessenkool wrote:
> On Thu, Nov 24, 2016 at 12:08:40AM +1100, Nicholas Piggin wrote:
> > Question, are there any fundamental reasons we shouldn't use the ELFv2
> > ABI to build big endian kernels if the compiler supports it?
>
> No one uses ELFv2 for
Hi Aneesh,
On 11/23/2016 02:41 AM, Aneesh Kumar K.V wrote:
Can you try this patch ?
commit 43e05fa840330f0f2deae1e8cc2effd5df68079f
Author: Aneesh Kumar K.V
Date: Wed Nov 23 15:23:05 2016 +0530
powerpc/mm: Kernel RO fixup for cell
I tested your patch with v4.7, v4.8 and v4.9-rc6, and
On Wed, Nov 23, 2016 at 04:14:05PM +1100, David Gibson wrote:
> Hi Paul,
>
> I'm still chasing this confusion about the CAS bit to send the real
> HPT resizing patches. However, in the meantime, here are some
> preliminary cleanups.
>
> These cleanups stand on their own, although I wrote them in
Acked-by: Ian Munsie
On Wed, 23 Nov 2016, Michael Ellerman wrote:
> That's nothing powerpc specific AFAICS, does this fix it?
Hm, so s/printk/pr_cont/ - but not in all places? But yeah, this fixes it
for me, at least on x86.
Tested-by: Christian Kujau
Thank you!
Christian.
>
> cheers
>
> diff --git a/lib/locki
Hi Paul,
Today's linux-next merge of the kvm-ppc-paulus tree got a conflict in:
arch/powerpc/mm/pgtable-radix.c
between commit:
555c16328ae6 ("powerpc/mm: Correct process and partition table max size")
from the powerpc tree and commit:
9d66195807ac ("powerpc/64: Provide functions for ac
Hi Paul,
Today's linux-next merge of the kvm-ppc-paulus tree got a conflict in:
arch/powerpc/include/asm/reg.h
between commit:
29a969b76481 ("powerpc: Revert Load Monitor Register Support")
from the powerpc tree and commits:
7fd317f8c330 ("powerpc/64: Add some more SPRs and SPR bits for
Hi Paul,
Today's linux-next merge of the kvm-ppc-paulus tree got a conflict in:
arch/powerpc/include/asm/asm-prototypes.h
between commit:
82de5797a260 ("powerpc: Remove extraneous header from asm-prototypes.h")
from the powerpc-fixes tree and commit:
ebe4535fbe7a ("KVM: PPC: Book3S HV:
Stephen Rothwell writes:
> Hi all,
>
> Today's linux-next merge of the powerpc tree got a conflict in:
>
> arch/powerpc/include/asm/asm-prototypes.h
>
> between commit:
>
> 9e5f68842276 ("powerpc: Fix missing CRCs, add more asm-prototypes.h
> declarations")
>
> from the powerpc-fixes tree an
On Tue, 22 Nov 2016 16:23:13 +0530
"Naveen N. Rao" wrote:
> On 2016/11/22 09:43PM, Michael Ellerman wrote:
> > "Naveen N. Rao" writes:
> > > On 2016/11/22 02:25PM, Masami Hiramatsu wrote:
> > >> On Mon, 21 Nov 2016 22:36:41 +0530
> > >> "Naveen N. Rao" wrote:
> > >> > diff --git a/arch/powerpc/
QE was supported on PowerPC, and dependent on PPC,
Now it is supported on other platforms. so remove PPCisms.
Signed-off-by: Zhao Qiang
---
Changes for v2:
- na
Changes for v3:
- add NO_IRQ
Changes for v4:
- modify spin_event_timeout to opencoded timeout loop
- rem
On Wed, Nov 23, 2016 at 11:31:54AM +1100, Paul Mackerras wrote:
> This series of patches adds support to HV KVM for running KVM guests
> on POWER9 systems. This allows us to run KVM guests that use HPT
> (hashed page table) address translation and know about the POWER9
> processor. With suitable
On Thu, 24 Nov 2016 09:33:20 +1030
Alan Modra wrote:
> On Thu, Nov 24, 2016 at 12:02:06AM +1100, Nicholas Piggin wrote:
> > I was building BookE and big endian with a little endian cross
> > compiler and it stopped working. My BookS BE tests must have been
> > building using the ELFv2 ABI. After
On Wed, 2016-11-23 at 22:58 +0800, Geliang Tang wrote:
> Use builtin_platform_driver() helper to simplify the code.
>
> Signed-off-by: Geliang Tang
> ---
Acked-by: Russell Currey
LSIs must be ack'ed with an MMIO otherwise they remain asserted
forever. This is controlled by the "clear_isr" flag.
While we set that flag properly when deciding initially whether
to use LSIs or MSIs, we fail to set it if we first chose MSIs,
the test fails, then fallback to LSIs.
Signed-off-by:
On 24/11/16 02:27, Geliang Tang wrote:
Drop duplicate header sched.h from native.c.
Signed-off-by: Geliang Tang
Good catch!
Reviewed-by: Andrew Donnellan
--
Andrew Donnellan OzLabs, ADL Canberra
andrew.donnel...@au1.ibm.com IBM Australia Limited
On Wed, Oct 26, 2016 at 12:15:34PM +1100, Gavin Shan wrote:
> This moves pcibios_sriov_enable() to the point before VF and VF BARs
> are enabled on PowerNV platform. Also, pci_update_resource() is used
> to update IOV BARs on PowerNV platform, the PF might have been functional
> when the function i
Hi all,
Today's linux-next merge of the powerpc tree got a conflict in:
arch/powerpc/include/asm/asm-prototypes.h
between commit:
9e5f68842276 ("powerpc: Fix missing CRCs, add more asm-prototypes.h
declarations")
from the powerpc-fixes tree and commit:
82de5797a260 ("powerpc: Remove ex
On Thu, Nov 24, 2016 at 12:02:06AM +1100, Nicholas Piggin wrote:
> I was building BookE and big endian with a little endian cross
> compiler and it stopped working. My BookS BE tests must have been
> building using the ELFv2 ABI. After this, the build sometimes still
> strangely fails with dot symb
On Tue, 2016-11-22 at 08:43 +, Jun Yang wrote:
> Hello Scott,
> Do you know the IOMMU maintainer's mail address?
>From MAINTAINERS:
IOMMU DRIVERS
M: Joerg Roedel
L: io...@lists.linux-foundation.org
T: git git://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu.git
S: Mai
tree: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git next
head: 3382a6220ff3bac886d9d90766f3fe18cf25b468
commit: d0563a1297e234ed37f6b51c2e9321accebd1839 [81/84] powerpc: Implement
{cmp}xchg for u8 and u16
config: powerpc-allyesconfig (attached as .config)
compiler: powerpc6
Drop duplicate header sched.h from native.c.
Signed-off-by: Geliang Tang
---
drivers/misc/cxl/native.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/misc/cxl/native.c b/drivers/misc/cxl/native.c
index c336350..aeefa53 100644
--- a/drivers/misc/cxl/native.c
+++ b/drivers/misc/cxl/nat
On 24/11/16 01:36, Aneesh Kumar K.V wrote:
> Balbir Singh writes:
>
>> On 23/11/16 22:53, Aneesh Kumar K.V wrote:
>>> Balbir Singh writes:
>>>
On 23/11/16 22:09, Aneesh Kumar K.V wrote:
> When we are updating pte, we just need to flush the tlb mapping for
> that pte. Right now we
Use builtin_platform_driver() helper to simplify the code.
Signed-off-by: Geliang Tang
---
drivers/soc/fsl/qe/qe.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/drivers/soc/fsl/qe/qe.c b/drivers/soc/fsl/qe/qe.c
index 2707a82..ade168f 100644
--- a/drivers/soc/fsl/qe/qe.
Use builtin_platform_driver() helper to simplify the code.
Signed-off-by: Geliang Tang
---
arch/powerpc/sysdev/fsl_pmc.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/arch/powerpc/sysdev/fsl_pmc.c b/arch/powerpc/sysdev/fsl_pmc.c
index 1d6fd7c..232225e 100644
--- a/arch
Use builtin_platform_driver() helper to simplify the code.
Signed-off-by: Geliang Tang
---
arch/powerpc/platforms/83xx/suspend.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/arch/powerpc/platforms/83xx/suspend.c
b/arch/powerpc/platforms/83xx/suspend.c
index 24717d0..
Use builtin_platform_driver() helper to simplify the code.
Signed-off-by: Geliang Tang
---
arch/powerpc/kernel/of_platform.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/arch/powerpc/kernel/of_platform.c
b/arch/powerpc/kernel/of_platform.c
index b60a67d..34aeac5 100
On Thu, Nov 24, 2016 at 12:08:40AM +1100, Nicholas Piggin wrote:
> Question, are there any fundamental reasons we shouldn't use the ELFv2
> ABI to build big endian kernels if the compiler supports it?
No one uses ELFv2 for BE in production, and it isn't thoroughly tested
at all, not even regularly
Balbir Singh writes:
> On 23/11/16 22:53, Aneesh Kumar K.V wrote:
>> Balbir Singh writes:
>>
>>> On 23/11/16 22:09, Aneesh Kumar K.V wrote:
When we are updating pte, we just need to flush the tlb mapping for
that pte. Right now we do a full mm flush because we don't track page
si
Balbir Singh writes:
> On 23/11/16 22:09, Aneesh Kumar K.V wrote:
>> W.r.t hash page table config, we support 16MB and 16GB as the hugepage
>> size. Update the hstate_get_psize to handle 16M and 16G.
>>
>> Signed-off-by: Aneesh Kumar K.V
>> ---
>> arch/powerpc/include/asm/book3s/64/hugetlb.h |
On Thu, Nov 24, 2016 at 12:08 AM, Nicholas Piggin wrote:
> Question, are there any fundamental reasons we shouldn't use the ELFv2
> ABI to build big endian kernels if the compiler supports it?
>
Does this have implications w.r.t interfaces to
1. openfirmware/skiboot
2. glibc/vdso
Keen to find o
On 23/11/16 22:09, Aneesh Kumar K.V wrote:
> W.r.t hash page table config, we support 16MB and 16GB as the hugepage
> size. Update the hstate_get_psize to handle 16M and 16G.
>
> Signed-off-by: Aneesh Kumar K.V
> ---
> arch/powerpc/include/asm/book3s/64/hugetlb.h | 4
> 1 file changed, 4
On 23/11/16 22:53, Aneesh Kumar K.V wrote:
> Balbir Singh writes:
>
>> On 23/11/16 22:09, Aneesh Kumar K.V wrote:
>>> When we are updating pte, we just need to flush the tlb mapping for
>>> that pte. Right now we do a full mm flush because we don't track page
>>> size. Update the interface to t
On Thu 24-11-16 00:05:12, Balbir Singh wrote:
>
>
> On 23/11/16 20:28, Michal Hocko wrote:
[...]
> > I am more worried about synchronization with the hotplug which tends to
> > be a PITA in places were we were simply safe by definition until now. We
> > do not have all that many users of memcg->n
Question, are there any fundamental reasons we shouldn't use the ELFv2
ABI to build big endian kernels if the compiler supports it?
Thanks,
Nick
---
arch/powerpc/Makefile | 15 +--
arch/powerpc/boot/Makefile | 2 +-
2 files changed, 6 insertions(+), 11 deletions(-)
diff --git
On 23/11/16 20:28, Michal Hocko wrote:
> On Wed 23-11-16 19:37:16, Balbir Singh wrote:
>>
>>
>> On 23/11/16 19:07, Michal Hocko wrote:
>>> On Wed 23-11-16 18:50:42, Balbir Singh wrote:
On 23/11/16 18:25, Michal Hocko wrote:
> On Wed 23-11-16 15:36:51, Balbir Singh wrote:
>>
This converts one that was missed by b1576fec7f4d ("powerpc: No need
to use dot symbols when branching to a function").
Signed-off-by: Nicholas Piggin
---
arch/powerpc/kernel/exceptions-64e.S | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/powerpc/kernel/exceptions
GCC can compile with either endian, but the ABI version always
defaults to the default endian. Alan Modra says:
you need both -mbig and -mabi=elfv1 to make a powerpc64le gcc
generate powerpc64 code
The opposite is true for powerpc64 when generating -mlittle it
requires -mabi=elfv2 to generate
>From 80f23935cadb ("powerpc: Convert cmp to cmpd in idle enter sequence"):
PowerPC's "cmp" instruction has four operands. Normally people write
"cmpw" or "cmpd" for the second cmp operand 0 or 1. But, frequently
people forget, and write "cmp" with just three operands.
With older
I was building BookE and big endian with a little endian cross
compiler and it stopped working. My BookS BE tests must have been
building using the ELFv2 ABI. After this, the build sometimes still
strangely fails with dot symbols in syscall table unable to be found,
but that's looking like it may b
"Naveen N. Rao" writes:
> entry_*.S now includes a lot more than just kernel entry/exit code. As a
> first step at cleaning this up, let's split out the ftrace bits into
> separate files.
>
> No functional changes.
>
> Suggested-by: Michael Ellerman
> Signed-off-by: Naveen N. Rao
> ---
> v2: up
Balbir Singh writes:
> On 23/11/16 22:09, Aneesh Kumar K.V wrote:
>> When we are updating pte, we just need to flush the tlb mapping for
>> that pte. Right now we do a full mm flush because we don't track page
>> size. Update the interface to track the page size and use that to
>> do the right tl
On 23/11/16 21:15, Michael Ellerman wrote:
> Balbir Singh writes:
>
>> Just a quick patch to trace tlbie(l)'s. The idea being that it can be
>> enabled when we suspect corruption or when we need to see if we are doing
>> the right thing during flush. I think the format can be enhanced to
>> mak
On 23/11/16 22:09, Aneesh Kumar K.V wrote:
> When we are updating pte, we just need to flush the tlb mapping for
> that pte. Right now we do a full mm flush because we don't track page
> size. Update the interface to track the page size and use that to
> do the right tlb flush.
>
Could you also
This will improve the task exit case, by batching tlb invalidates.
Signed-off-by: Aneesh Kumar K.V
---
arch/powerpc/include/asm/book3s/64/radix.h | 16 +++-
1 file changed, 11 insertions(+), 5 deletions(-)
diff --git a/arch/powerpc/include/asm/book3s/64/radix.h
b/arch/powerpc/inclu
When we are updating pte, we just need to flush the tlb mapping for
that pte. Right now we do a full mm flush because we don't track page
size. Update the interface to track the page size and use that to
do the right tlb flush.
Signed-off-by: Aneesh Kumar K.V
---
arch/powerpc/include/asm/book3s/
We want to switch pte_update to use va based tlb flush. In order to do that we
need to track the page size. With hugetlb we currently don't have page size
available in these functions. Hence switch hugetlb to use seperate functions
for update. In later patch we will update hugetlb functions to take
In the subsequent patch we will change the implementation of book3s 64. This
also avoid #ifdef in the code.
Signed-off-by: Aneesh Kumar K.V
---
arch/powerpc/include/asm/book3s/32/pgtable.h | 5 +
arch/powerpc/include/asm/book3s/64/hugetlb.h | 6 ++
arch/powerpc/include/asm/hugetlb.h
W.r.t hash page table config, we support 16MB and 16GB as the hugepage
size. Update the hstate_get_psize to handle 16M and 16G.
Signed-off-by: Aneesh Kumar K.V
---
arch/powerpc/include/asm/book3s/64/hugetlb.h | 4
1 file changed, 4 insertions(+)
diff --git a/arch/powerpc/include/asm/book3s
We will start moving some book3s specific hugetlb functions there.
Signed-off-by: Aneesh Kumar K.V
---
arch/powerpc/include/asm/book3s/64/{hugetlb-radix.h => hugetlb.h} | 4 ++--
arch/powerpc/include/asm/hugetlb.h| 2 +-
2 files changed, 3 insertions(+), 3 deletio
When we are updating pte, we just need to flush the tlb mapping for
that pte. Right now we do a full mm flush because we don't track page
size. Update the interface to track the page size and use that to
do the right tlb flush.
Signed-off-by: Aneesh Kumar K.V
---
arch/powerpc/include/asm/book3s/
Geoff Levand writes:
> Hi Aneesh,
>
>> --- a/arch/powerpc/platforms/ps3/spu.c
>> +++ b/arch/powerpc/platforms/ps3/spu.c
>> @@ -205,7 +205,7 @@ static void spu_unmap(struct spu *spu)
>> static int __init setup_areas(struct spu *spu)
>> {
>> struct table {char* name; unsigned long addr; unsi
"Aneesh Kumar K.V" writes:
> diff --git a/arch/powerpc/include/asm/book3s/32/pgtable.h
> b/arch/powerpc/include/asm/book3s/32/pgtable.h
> index 6b8b2d57fdc8..0713626e9189 100644
> --- a/arch/powerpc/include/asm/book3s/32/pgtable.h
> +++ b/arch/powerpc/include/asm/book3s/32/pgtable.h
> @@ -224,7 +
Balbir Singh writes:
> Just a quick patch to trace tlbie(l)'s. The idea being that it can be
> enabled when we suspect corruption or when we need to see if we are doing
> the right thing during flush. I think the format can be enhanced to
> make it nicer (expand the RB/RS/IS/L cases in more detai
"Gautham R. Shenoy" writes:
> From: "Gautham R. Shenoy"
>
> The power9_idle_stop method currently takes only the requested stop
> level as a parameter and picks up the rest of the PSSCR bits from a
> hand-coded macro. This is not a very flexible design, especially when
> the firmware has the cap
"Gautham R. Shenoy" writes:
> diff --git a/drivers/cpuidle/cpuidle-powernv.c
> b/drivers/cpuidle/cpuidle-powernv.c
> index 7fe442c..9240e08 100644
> --- a/drivers/cpuidle/cpuidle-powernv.c
> +++ b/drivers/cpuidle/cpuidle-powernv.c
> @@ -243,28 +262,31 @@ static int powernv_add_idle_states(void)
On Wed 23-11-16 19:37:16, Balbir Singh wrote:
>
>
> On 23/11/16 19:07, Michal Hocko wrote:
> > On Wed 23-11-16 18:50:42, Balbir Singh wrote:
> >>
> >>
> >> On 23/11/16 18:25, Michal Hocko wrote:
> >>> On Wed 23-11-16 15:36:51, Balbir Singh wrote:
> In the absence of hotplug we use extra memo
On 11/21/16 at 09:49pm, Thiago Jung Bauermann wrote:
> Hello Dave,
>
> Thanks for your review.
>
> Am Sonntag, 20. November 2016, 10:45:46 BRST schrieb Dave Young:
> > On 11/10/16 at 01:27am, Thiago Jung Bauermann wrote:
> > > powerpc's purgatory.ro has 12 relocation types when built as
> > > a r
On 23/11/16 19:07, Michal Hocko wrote:
> On Wed 23-11-16 18:50:42, Balbir Singh wrote:
>>
>>
>> On 23/11/16 18:25, Michal Hocko wrote:
>>> On Wed 23-11-16 15:36:51, Balbir Singh wrote:
In the absence of hotplug we use extra memory proportional to
(possible_nodes - online_nodes) * number
On Wed 23-11-16 18:50:42, Balbir Singh wrote:
>
>
> On 23/11/16 18:25, Michal Hocko wrote:
> > On Wed 23-11-16 15:36:51, Balbir Singh wrote:
> >> In the absence of hotplug we use extra memory proportional to
> >> (possible_nodes - online_nodes) * number_of_cgroups. PPC64 has a patch
> >> to disab
Just a quick patch to trace tlbie(l)'s. The idea being that it can be
enabled when we suspect corruption or when we need to see if we are doing
the right thing during flush. I think the format can be enhanced to
make it nicer (expand the RB/RS/IS/L cases in more detail). For now I am
sharing the i
75 matches
Mail list logo
