On Fri, 14 Oct 2016 11:53:18 +1100
Suraj Jitindar Singh wrote:
> This patch series makes some updates and bug fixes to the powerpc kvm-hv
> halt polling code.
>
> The first two patches are concerned with exporting the generic kvm module
> parameter variables and accessing these from the powerpc
Balbir Singh writes:
> On 12/10/16 17:57, Nicholas Piggin wrote:
>> diff --git a/arch/powerpc/kernel/head_64.S b/arch/powerpc/kernel/head_64.S
>> index 79da0641..bc9ceac 100644
>> --- a/arch/powerpc/kernel/head_64.S
>> +++ b/arch/powerpc/kernel/head_64.S
>> @@ -111,8 +111,12 @@ __secondary_hold_ac
On 13/10/16 22:08, Vaibhav Jain wrote:
This patch prevents resetting the cxl adapter via sysfs in presence of
one or more active cxl_context on it. This protects against an
unrecoverable error caused by PSL owning a dirty cache line even after
reset and host tries to touch the same cache line. In
On 14/10/16 14:16, Nicholas Piggin wrote:
> Thanks Balbir and Gautham for testing and reviewing.
>
> On Thu, 13 Oct 2016 22:54:32 +1100
> Balbir Singh wrote:
>
>> On 13/10/16 13:17, Nicholas Piggin wrote:
>>> This patch does a couple of things. First of all, powernv immediately
>>> explodes wh
On Fri, 14 Oct 2016 13:35:54 +1100
Balbir Singh wrote:
> On 12/10/16 18:00, Nicholas Piggin wrote:
> > Add --orphan-handling=error to final link flags. This ensures we have to
> > handle all sections. This would have caught subtle breakage such as
> > 7de3b27bac47da9de08409df1d69664acbb72197 at b
Cc Alan Modra
On Fri, 14 Oct 2016 14:27:45 +1100
Michael Ellerman wrote:
> With DEBUG_INFO=y I'm seeing:
>
> /opt/toolchains/6/powerpc/bin/ld: error: unplaced orphan section
> `.debug_addr' from `arch/powerpc/kernel/prom_init.o'.
> /opt/toolchains/6/powerpc/bin/ld: error: unplaced orphan s
On Fri, Oct 14, 2016 at 11:53:18AM +1100, Suraj Jitindar Singh wrote:
> This patch series makes some updates and bug fixes to the powerpc kvm-hv
> halt polling code.
>
> The first two patches are concerned with exporting the generic kvm module
> parameter variables and accessing these from the pow
Nicholas Piggin writes:
> diff --git a/arch/powerpc/kernel/vmlinux.lds.S
> b/arch/powerpc/kernel/vmlinux.lds.S
> index 8295f51..9f4d85e 100644
> --- a/arch/powerpc/kernel/vmlinux.lds.S
> +++ b/arch/powerpc/kernel/vmlinux.lds.S
> @@ -317,6 +319,16 @@ SECTIONS
> _end = . ;
> PROVIDE32
On Fri, 14 Oct 2016 11:48:35 +1100
Michael Ellerman wrote:
> Nicholas Piggin writes:
> > Signed-off-by: Nicholas Piggin
> > ---
> > The linker orphan sections error patch caught this
>
> You mean with the orphan sections patch applied this breaks the build
> when KPROBES=n right?
Yes. The l
Thanks Balbir and Gautham for testing and reviewing.
On Thu, 13 Oct 2016 22:54:32 +1100
Balbir Singh wrote:
> On 13/10/16 13:17, Nicholas Piggin wrote:
> > This patch does a couple of things. First of all, powernv immediately
> > explodes when running a relocated kernel, because the system reset
On 12/10/16 18:00, Nicholas Piggin wrote:
> Add --orphan-handling=error to final link flags. This ensures we have to
> handle all sections. This would have caught subtle breakage such as
> 7de3b27bac47da9de08409df1d69664acbb72197 at build-time.
>
> Also bring some wayward sections into the fold:
On Fri, 2016-10-14 at 09:16 +0800, Wanpeng Li wrote:
> 2016-10-14 8:53 GMT+08:00 Suraj Jitindar Singh com>:
> >
> > There is currently no documentation about the halt polling
> > capabilities
> > of the kvm module. Add some documentation describing the mechanism
> > as well
> > as the module para
2016-10-14 8:53 GMT+08:00 Suraj Jitindar Singh :
> There is currently no documentation about the halt polling capabilities
> of the kvm module. Add some documentation describing the mechanism as well
> as the module parameters to all better understanding of how halt polling
> should be used and the
There is currently no documentation about the halt polling capabilities
of the kvm module. Add some documentation describing the mechanism as well
as the module parameters to all better understanding of how halt polling
should be used and the effect of tuning the module parameters.
Signed-off-by:
Fix comment block to match kernel comment style.
Fix print format from signed to unsigned.
Signed-off-by: Suraj Jitindar Singh
---
arch/powerpc/kvm/book3s_hv.c | 3 ++-
arch/powerpc/kvm/trace_hv.h | 2 +-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/arch/powerpc/kvm/book3s_hv
KVM_HALT_POLL_NS_DEFAULT is an arch specific constant which sets the
default value of the halt_poll_ns kvm module parameter which determines
the global maximum halt polling interval.
The current value for powerpc is 50 (500us) which means that any
repetitive workload with a period of less than
The kvm module parameter halt_poll_ns defines the global maximum halt
polling interval and can be dynamically changed by writing to the
/sys/module/kvm/parameters/halt_poll_ns sysfs file. However in kvm-hv
this module parameter value is only ever checked when we grow the current
polling interval fo
The previous patch exported the variables which back the module parameters
of the generic kvm module. Now use these variables in the kvm-hv module
so that any change to the generic module parameters will also have the
same effect for the kvm-hv module. This removes the duplication of the
kvm module
The kvm module has the parameters halt_poll_ns, halt_poll_ns_grow, and
halt_poll_ns_shrink. Halt polling was recently added to the powerpc kvm-hv
module and these parameters were essentially duplicated for that. There is
no benefit to this duplication and it can lead to confusion when trying to
tun
This patch series makes some updates and bug fixes to the powerpc kvm-hv
halt polling code.
The first two patches are concerned with exporting the generic kvm module
parameter variables and accessing these from the powerpc specific code.
The third patch fixes a bug where changing the global max h
Nicholas Piggin writes:
> Signed-off-by: Nicholas Piggin
> ---
> The linker orphan sections error patch caught this
You mean with the orphan sections patch applied this breaks the build
when KPROBES=n right?
cheers
Nicholas Piggin writes:
> Add --orphan-handling=error to final link flags. This ensures we have to
> handle all sections. This would have caught subtle breakage such as
> 7de3b27bac47da9de08409df1d69664acbb72197 at build-time.
>
> Also bring some wayward sections into the fold:
> - .text.hot and
On 13/10/16 04:48, Mahesh Jagannath Salgaonkar wrote:
> On 10/10/2016 04:22 PM, Michael Ellerman wrote:
>> Mahesh J Salgaonkar writes:
>>
>>> From: Mahesh Salgaonkar
>>>
>>> There are chances that multiple CPUs can call crash_fadump() simultaneously
>>> and would start duplicating same info to
Frederic Barrat writes:
> Le 13/10/2016 à 12:51, Michael Ellerman a écrit :
>> Frederic Barrat writes:
>>
>>> ping? The patch still applies cleanly on recent trees.
>>
>> Fell through the cracks :/
>>
>> Fixes: ?
>
> Nothing obvious. Current code was introduced by
> 73d16a6e0e51990cbe13f8d8f43b
On 13/10/16 13:25, Nicholas Piggin wrote:
> On Wed, 12 Oct 2016 18:35:21 +1100
> Balbir Singh wrote:
>
>> On 12/10/16 17:57, Nicholas Piggin wrote:
>>> This adds a config option that can help exercise the case when
>>> the kernel is not running at PAGE_OFFSET.
>>>
>>> Signed-off-by: Nicholas Pi
Balbir Singh writes:
> On 13/10/16 16:27, Michael Ellerman wrote:
>> At boot we dump the NUMA memory topology in dump_numa_memory_topology(),
>> at KERN_DEBUG level, resulting in output like:
>>
>> Node 0 Memory: 0x0-0x1
>> Node 1 Memory: 0x1-0x2
>>
>> Which is nice
Vaibhav Jain writes:
> This patch prevents resetting the cxl adapter via sysfs in presence of
> one or more active cxl_context on it. This protects against an
> unrecoverable error caused by PSL owning a dirty cache line even after
> reset and host tries to touch the same cache line. In case a fo
Reza Arbab writes:
> The flow of the main loop in parse_numa_properties() is overly
> complicated. Simplify it to be less confusing and easier to read.
> No functional change.
I think you're right, but it's not obvious from the diff. Please explain
in more detail why the old and new code is equi
Hm, still behaves for me like I reported for v2:
http://marc.info/?l=linux-scsi&m=147637177902937&w=2
On 10/13/2016 05:00 PM, Johannes Thumshirn wrote:
Don't use fc_bsg_job::request and fc_bsg_job::reply directly, but use
helper variables bsg_request and bsg_reply. This will be helpfull when
tr
I'm puzzled.
$ git bisect start fc_bsg master
Bisecting: 8 revisions left to test after this (roughly 3 steps)
[005d51510eee6102636d5dbb06310531c5d46151] scsi: fc: implement kref
backed reference counting
$ git bisect bad
Bisecting: 3 revisions left to test after this (roughly 2 steps)
[bef6da2
Am 13.10.2016 um 02:20 schrieb Lorenzo Stoakes:
This patch series adjusts functions in the get_user_pages* family such that
desired FOLL_* flags are passed as an argument rather than implied by flags.
The purpose of this change is to make the use of FOLL_FORCE explicit so it is
easier to grep fo
The flow of the main loop in parse_numa_properties() is overly
complicated. Simplify it to be less confusing and easier to read.
No functional change.
Signed-off-by: Reza Arbab
---
arch/powerpc/mm/numa.c | 11 +++
1 file changed, 3 insertions(+), 8 deletions(-)
diff --git a/arch/powerpc
Hi Vaibhav,
Le 13/10/2016 à 13:08, Vaibhav Jain a écrit :
This patch prevents resetting the cxl adapter via sysfs in presence of
one or more active cxl_context on it. This protects against an
unrecoverable error caused by PSL owning a dirty cache line even after
reset and host tries to touch th
On Thu, Oct 13, 2016 at 05:15:25PM +0200, Steffen Maier wrote:
> I'm puzzled.
>
> $ git bisect start fc_bsg master
> Bisecting: 8 revisions left to test after this (roughly 3 steps)
> [005d51510eee6102636d5dbb06310531c5d46151] scsi: fc: implement kref backed
> reference counting
> $ git bisect bad
On Thu, Oct 13, 2016 at 05:55:11PM +0200, Steffen Maier wrote:
> Hm, still behaves for me like I reported for v2:
> http://marc.info/?l=linux-scsi&m=147637177902937&w=2
>
Well given what you've wrote for v2 it's kinda expected.
Byte,
Johannes
--
Johannes Thumshirn
Change FC drivers to use 'struct bsg_job' from bsg-lib.h instead of 'struct
fc_bsg_job' from scsi_transport_fc.h and remove 'struct fc_bsg_job'.
Signed-off-by: Johannes Thumshirn
Reviewed-by: Hannes Reinecke
---
drivers/s390/scsi/zfcp_ext.h | 4 +--
drivers/s390/scsi/zfcp_fc.c | 15 ++
fc_bsg_jobdone() and bsg_job_done() are 1:1 copies now so use the bsg-lib one
instead of the FC private implementation.
Signed-off-by: Johannes Thumshirn
Reviewed-by: Hannes Reinecke
---
drivers/s390/scsi/zfcp_fc.c | 2 +-
drivers/scsi/bfa/bfad_bsg.c | 4 ++--
drivers/scsi/ibmvscsi/
Don't use fc_bsg_job::request and fc_bsg_job::reply directly, but use
helper variables bsg_request and bsg_reply. This will be helpfull when
transitioning to bsg-lib.
Signed-off-by: Johannes Thumshirn
Reviewed-by: Hannes Reinecke
---
drivers/s390/scsi/zfcp_fc.c | 9 +-
drivers/scsi/bfa/
Provide fc_bsg_to_shost() helper that will become handy when we're moving from
struct fc_bsg_job to a plain struct bsg_job. Also use this little helper in
the LLDDs.
Signed-off-by: Johannes Thumshirn
Reviewed-by: Hannes Reinecke
---
drivers/s390/scsi/zfcp_fc.c | 4 +--
drivers/scsi/bfa/bf
Provide fc_bsg_to_rport() helper that will become handy when we're moving
from struct fc_bsg_job to a plain struct bsg_job. Also move all LLDDs to use
the new helper.
Signed-off-by: Johannes Thumshirn
Reviewed-by: Hannes Reinecke
---
drivers/s390/scsi/zfcp_fc.c | 5 +++--
drivers/scsi/ibmv
Unify the interfaces of fc_bsg_jobdone and bsg_job_done. This will reduce the
diff when moving from 'struct fc_bsg_job' to a plain 'struct bsg_job' later
on.
Signed-off-by: Johannes Thumshirn
Reviewed-by: Hannes Reinecke
---
drivers/s390/scsi/zfcp_fc.c | 2 +-
drivers/scsi/bfa/bfad_bsg.c
Export fc_bsg_jobdone so drivers can use it directly instead of doing
the round-trip via struct fc_bsg_job::job_done() and use it in the LLDDs.
As we've converted all LLDDs over to use fc_bsg_jobdone() directly,
we can remove the function pointer from struct fc_bsg_job as well.
Signed-off-by: Joh
On 13/10/16 16:27, Michael Ellerman wrote:
> At boot we dump the NUMA memory topology in dump_numa_memory_topology(),
> at KERN_DEBUG level, resulting in output like:
>
> Node 0 Memory: 0x0-0x1
> Node 1 Memory: 0x1-0x2
>
> Which is nice enough, but immediately after
On 13/10/16 13:17, Nicholas Piggin wrote:
> This patch does a couple of things. First of all, powernv immediately
> explodes when running a relocated kernel, because the system reset
> exception for handling sleeps does not do correct relocated branches.
>
> Secondly, the sleep handling code tra
Le 13/10/2016 à 12:51, Michael Ellerman a écrit :
Frederic Barrat writes:
ping? The patch still applies cleanly on recent trees.
Fell through the cracks :/
Fixes: ?
Nothing obvious. Current code was introduced by
73d16a6e0e51990cbe13f8d8f43bd5329bbab30a
but it was apparently moved from
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote:
> fc_bsg_jobdone() and bsg_job_done() are 1:1 copies now so use the bsg-lib one
> instead of the FC private implementation.
>
> Signed-off-by: Johannes Thumshirn
> ---
> drivers/s390/scsi/zfcp_fc.c | 2 +-
> drivers/scsi/bfa/bfad_bsg.c
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote:
> Change FC drivers to use 'struct bsg_job' from bsg-lib.h instead of 'struct
> fc_bsg_job' from scsi_transport_fc.h and remove 'struct fc_bsg_job'.
>
> Signed-off-by: Johannes Thumshirn
> ---
> drivers/s390/scsi/zfcp_ext.h | 4 +--
> driver
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote:
> Provide fc_bsg_to_rport() helper that will become handy when we're moving
> from struct fc_bsg_job to a plain struct bsg_job. Also move all LLDDs to use
> the new helper.
>
> Signed-off-by: Johannes Thumshirn
> ---
> drivers/s390/scsi/zfcp_fc.c
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote:
> Provide fc_bsg_to_shost() helper that will become handy when we're moving from
> struct fc_bsg_job to a plain struct bsg_job. Also use this little helper in
> the LLDDs.
>
> Signed-off-by: Johannes Thumshirn
> ---
> drivers/s390/scsi/zfcp_fc.c
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote:
> Unify the interfaces of fc_bsg_jobdone and bsg_job_done. This will reduce the
> diff when moving from 'struct fc_bsg_job' to a plain 'struct bsg_job' later
> on.
>
> Signed-off-by: Johannes Thumshirn
> ---
> drivers/s390/scsi/zfcp_fc.c |
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote:
> Export fc_bsg_jobdone so drivers can use it directly instead of doing
> the round-trip via struct fc_bsg_job::job_done() and use it in the LLDDs.
>
> As we've converted all LLDDs over to use fc_bsg_jobdone() directly,
> we can remove the function
On 10/12/2016 03:06 PM, Johannes Thumshirn wrote:
> Don't use fc_bsg_job::request and fc_bsg_job::reply directly, but use
> helper variables bsg_request and bsg_reply. This will be helpfull when
> transitioning to bsg-lib.
>
> Signed-off-by: Johannes Thumshirn
> ---
> drivers/s390/scsi/zfcp_fc.
Hi Stewart,
On Wed, Oct 12, 2016 at 04:35:35PM +1100, Stewart Smith wrote:
>
> What if we just treat the 0xF state from firmware as special and set it
> to DEFAULT_PSSCR_MASK in that case? That deals with old skiboot, new
> kernel, and sets a pretty small special case that's easy to track into
>
This patch prevents resetting the cxl adapter via sysfs in presence of
one or more active cxl_context on it. This protects against an
unrecoverable error caused by PSL owning a dirty cache line even after
reset and host tries to touch the same cache line. In case a force reset
of the card is requir
Hi Nick,
[Updated Shreyas's current e-mail address:
"Shreyas B. Prabhu" ]
On Thu, Oct 13, 2016 at 01:17:14PM +1100, Nicholas Piggin wrote:
> This patch does a couple of things. First of all, powernv immediately
> explodes when running a relocated kernel, because the system reset
> except
Frederic Barrat writes:
> ping? The patch still applies cleanly on recent trees.
Fell through the cracks :/
Fixes: ?
cheers
Mahesh Jagannath Salgaonkar writes:
> On 10/10/2016 04:22 PM, Michael Ellerman wrote:
>> Mahesh J Salgaonkar writes:
>>
>>> From: Mahesh Salgaonkar
>>>
>>> There are chances that multiple CPUs can call crash_fadump() simultaneously
>>> and would start duplicating same info to vmcoreinfo ELF no
On 13/10/16 13:07, Nicholas Piggin wrote:
> Signed-off-by: Nicholas Piggin
> ---
> The linker orphan sections error patch caught this
Seems reasonable
Acked-by: Balbir Singh
Hi Linus,
Please pull some more powerpc updates for 4.9:
The following changes since commit b7b7013cac55d794940bd9cb7b7c55c9dececac4:
powerpc/bpf: Add support for bpf constant blinding (2016-10-04 20:33:20 +1100)
are available in the git repository at:
git://git.kernel.org/pub/scm/linux/ke
Marcin Nowakowski writes:
> On 12.10.2016 10:50, Michael Ellerman wrote:
>> <...>
>> It's annoying that we have to duplicate all that just to do a + 1.
>>
>> How about this as a precursor?
> > <...>
>
> Thanks for the suggestion - unless anyone sees a reason to keep the
> current solution I'll
60 matches
Mail list logo
