On Tue, Oct 30, 2018 at 11:00:51AM +0800, Peter Xu wrote:
> On Mon, Oct 29, 2018 at 12:29:22PM -0600, Alex Williamson wrote:
> > On Mon, 29 Oct 2018 17:14:46 +0800
> > Jason Wang wrote:
> >
> > > On 2018/10/29 上午10:42, Simon Guo wrote:
> > > > Hi,
>
On Tue, Oct 30, 2018 at 11:00:51AM +0800, Peter Xu wrote:
> On Mon, Oct 29, 2018 at 12:29:22PM -0600, Alex Williamson wrote:
> > On Mon, 29 Oct 2018 17:14:46 +0800
> > Jason Wang wrote:
> >
> > > On 2018/10/29 上午10:42, Simon Guo wrote:
> > > > Hi,
>
Hi,
I am using network device pass through mode with qemu x86(-device
vfio-pci,host=:xx:yy.z)
and “intel_iommu=on” in host kernel command line, and it shows the whole guest
memory
were pinned(vfio_pin_pages()), viewed by the “top” RES memory output. I
understand it is due
to device can DMA
Hi,
I am using network device pass through mode with qemu x86(-device
vfio-pci,host=:xx:yy.z)
and “intel_iommu=on” in host kernel command line, and it shows the whole guest
memory
were pinned(vfio_pin_pages()), viewed by the “top” RES memory output. I
understand it is due
to device can DMA
On Fri, Oct 07, 2016 at 08:44:48AM +1100, Michael Ellerman wrote:
> wei.guo.si...@gmail.com writes:
>
> > From: Anshuman Khandual
> >
> > This patch adds ptrace interface test for EBB/PMU specific
> > registers. This also adds some generic ptrace interface
> > based
On Fri, Oct 07, 2016 at 08:44:48AM +1100, Michael Ellerman wrote:
> wei.guo.si...@gmail.com writes:
>
> > From: Anshuman Khandual
> >
> > This patch adds ptrace interface test for EBB/PMU specific
> > registers. This also adds some generic ptrace interface
> > based helper functions to be used
system-ppc64 --enable-kvm -machine pseries,accel=kvm,usb=off -m
> 4096 -realtime mlock=off -smp 4,sockets=1,cores=2,threads=2 -nographic
> -vga none
>
>
> > Signed-off-by: Anshuman Khandual <khand...@linux.vnet.ibm.com>
> > Signed-off-by: Simon Guo <wei.guo.si...@gmail.
es,accel=kvm,usb=off -m
> 4096 -realtime mlock=off -smp 4,sockets=1,cores=2,threads=2 -nographic
> -vga none
>
>
> > Signed-off-by: Anshuman Khandual
> > Signed-off-by: Simon Guo
> > ---
> > tools/testing/selftests/powerpc/ptrace/Makefile| 3 +-
>
On Wed, Sep 14, 2016 at 03:04:12PM +1000, Cyril Bur wrote:
> On Mon, 2016-09-12 at 15:33 +0800, wei.guo.si...@gmail.com wrote:
> > From: Anshuman Khandual
> >
> > This patch adds ptrace interface test for TM SPR registers. This
> > also adds ptrace interface based
On Wed, Sep 14, 2016 at 03:04:12PM +1000, Cyril Bur wrote:
> On Mon, 2016-09-12 at 15:33 +0800, wei.guo.si...@gmail.com wrote:
> > From: Anshuman Khandual
> >
> > This patch adds ptrace interface test for TM SPR registers. This
> > also adds ptrace interface based helper functions related to TM
On Tue, Jul 26, 2016 at 04:06:01PM +0800, wei.guo.si...@gmail.com wrote:
> From: Simon Guo <wei.guo.si...@gmail.com>
>
> Normally, when MSR[VSX/VR/SPE] bits = 1, the used_vsr/used_vr/used_spe
> bit have already been set. However signal frame locates at user space
> and it
On Tue, Jul 26, 2016 at 04:06:01PM +0800, wei.guo.si...@gmail.com wrote:
> From: Simon Guo
>
> Normally, when MSR[VSX/VR/SPE] bits = 1, the used_vsr/used_vr/used_spe
> bit have already been set. However signal frame locates at user space
> and it is controlled by user applic
Hi David,
On Wed, Aug 31, 2016 at 04:14:14PM -0700, David Rientjes wrote:
> On Tue, 30 Aug 2016, wei.guo.si...@gmail.com wrote:
>
> > From: Simon Guo <wei.guo.si...@gmail.com>
> >
> > This patch adds mlock() test for multiple invocation on
> > the same addr
Hi David,
On Wed, Aug 31, 2016 at 04:14:14PM -0700, David Rientjes wrote:
> On Tue, 30 Aug 2016, wei.guo.si...@gmail.com wrote:
>
> > From: Simon Guo
> >
> > This patch adds mlock() test for multiple invocation on
> > the same address area, and verify it doe
On Thu, Jul 21, 2016 at 08:57:29PM +1000, Michael Ellerman wrote:
> Can one of you send a properly formatted and signed-off patch.
I will work on that.
Thanks,
Simon
On Thu, Jul 21, 2016 at 08:57:29PM +1000, Michael Ellerman wrote:
> Can one of you send a properly formatted and signed-off patch.
I will work on that.
Thanks,
Simon
Michael, Ben,
On Fri, Jul 08, 2016 at 08:02:42PM +1000, Michael Ellerman wrote:
> Benjamin Herrenschmidt writes:
>
> > On Thu, 2016-07-07 at 23:21 +1000, Benjamin Herrenschmidt wrote:
> >>
> >> I think the right fix is that if a restore_sigcontext() has the MSR
> >>
Michael, Ben,
On Fri, Jul 08, 2016 at 08:02:42PM +1000, Michael Ellerman wrote:
> Benjamin Herrenschmidt writes:
>
> > On Thu, 2016-07-07 at 23:21 +1000, Benjamin Herrenschmidt wrote:
> >>
> >> I think the right fix is that if a restore_sigcontext() has the MSR
> >> bits set,
> >> it should set
On Fri, Jul 08, 2016 at 08:02:42PM +1000, Michael Ellerman wrote:
> Benjamin Herrenschmidt writes:
>
> > On Thu, 2016-07-07 at 23:21 +1000, Benjamin Herrenschmidt wrote:
> >>
> >> I think the right fix is that if a restore_sigcontext() has the MSR
> >> bits set,
> >>
On Fri, Jul 08, 2016 at 08:02:42PM +1000, Michael Ellerman wrote:
> Benjamin Herrenschmidt writes:
>
> > On Thu, 2016-07-07 at 23:21 +1000, Benjamin Herrenschmidt wrote:
> >>
> >> I think the right fix is that if a restore_sigcontext() has the MSR
> >> bits set,
> >> it should set the
On Thu, Jul 07, 2016 at 11:21:18PM +1000, Benjamin Herrenschmidt wrote:
> I think the right fix is that if a restore_sigcontext() has the MSR bits set,
> it should set the corresponding used_* flag.
>
> Or is there a reason why that won't work ?
That sounds reaonable to me.
I will prepare a
On Thu, Jul 07, 2016 at 11:21:18PM +1000, Benjamin Herrenschmidt wrote:
> I think the right fix is that if a restore_sigcontext() has the MSR bits set,
> it should set the corresponding used_* flag.
>
> Or is there a reason why that won't work ?
That sounds reaonable to me.
I will prepare a
Hi Michael,
On Tue, Jul 05, 2016 at 03:40:40PM +1000, Michael Ellerman wrote:
> On Wed, 2016-06-04 at 07:00:12 UTC, Simon Guo wrote:
> > These 2 fields track whether user process has used Altivec/VSX
> > registers or not. They are used by kernel to setup signal frame
> > on
Hi Michael,
On Tue, Jul 05, 2016 at 03:40:40PM +1000, Michael Ellerman wrote:
> On Wed, 2016-06-04 at 07:00:12 UTC, Simon Guo wrote:
> > These 2 fields track whether user process has used Altivec/VSX
> > registers or not. They are used by kernel to setup signal frame
> > on
Hi,
On Mon, May 09, 2016 at 05:22:15PM -0700, Shaohua Li wrote:
> if trace isn't enabled, parsing cgroup path just wastes cpu
>
> Signed-off-by: Shaohua Li
> ---
> block/blk-throttle.c | 5 ++---
> include/linux/blktrace_api.h | 9 +
> 2 files changed, 11
Hi,
On Mon, May 09, 2016 at 05:22:15PM -0700, Shaohua Li wrote:
> if trace isn't enabled, parsing cgroup path just wastes cpu
>
> Signed-off-by: Shaohua Li
> ---
> block/blk-throttle.c | 5 ++---
> include/linux/blktrace_api.h | 9 +
> 2 files changed, 11 insertions(+), 3
Hi Michael,
On Wed, Apr 06, 2016 at 03:00:12PM +0800, Simon Guo wrote:
> These 2 fields track whether user process has used Altivec/VSX
> registers or not. They are used by kernel to setup signal frame
> on user stack correctly regarding vector part.
>
> CRIU(Checkpoint and Restor
Hi Michael,
On Wed, Apr 06, 2016 at 03:00:12PM +0800, Simon Guo wrote:
> These 2 fields track whether user process has used Altivec/VSX
> registers or not. They are used by kernel to setup signal frame
> on user stack correctly regarding vector part.
>
> CRIU(Checkpoint and Restor
to
setup signal frame correctly. And CRIU will need to restore these
2 fields for the restored process.
Signed-off-by: Simon Guo <wei.guo.si...@gmail.com>
Reviewed-by: Laurent Dufour <lduf...@linux.vnet.ibm.com>
---
arch/powerpc/include/uapi/asm/ptrace.h | 11 ++
arch/po
to
setup signal frame correctly. And CRIU will need to restore these
2 fields for the restored process.
Signed-off-by: Simon Guo
Reviewed-by: Laurent Dufour
---
arch/powerpc/include/uapi/asm/ptrace.h | 11 ++
arch/powerpc/kernel/ptrace.c | 39
to setup signal
frame correctly. And CRIU will need to restore these 2 fields for
the restored process.
Signed-off-by: Simon Guo <wei.guo.si...@gmail.com>
Reviewed-by: Laurent Dufour <lduf...@linux.vnet.ibm.com>
---
arch/powerpc/include/uapi/asm/ptrace.h | 11 ++
arch/po
to setup signal
frame correctly. And CRIU will need to restore these 2 fields for
the restored process.
Signed-off-by: Simon Guo
Reviewed-by: Laurent Dufour
---
arch/powerpc/include/uapi/asm/ptrace.h | 11 ++
arch/powerpc/kernel/ptrace.c | 39
to setup signal
frame correctly. And CRIU will need to restore these 2 fields for
the restored process.
Signed-off-by: Simon Guo <wei.guo.si...@gmail.com>
Reviewed-by: Laurent Dufour <lduf...@linux.vnet.ibm.com>
---
arch/powerpc/include/uapi/asm/ptrace.h | 11 ++
arch/po
to setup signal
frame correctly. And CRIU will need to restore these 2 fields for
the restored process.
Signed-off-by: Simon Guo
Reviewed-by: Laurent Dufour
---
arch/powerpc/include/uapi/asm/ptrace.h | 11 ++
arch/powerpc/kernel/ptrace.c | 39
used_vsr flag is set if process has used VSX register,
instead of Altivec register.
This patch corrects the wrong comment.
Signed-off-by: Simon Guo <wei.guo.si...@gmail.com>
---
arch/powerpc/include/asm/processor.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
used_vsr flag is set if process has used VSX register,
instead of Altivec register.
This patch corrects the wrong comment.
Signed-off-by: Simon Guo
---
arch/powerpc/include/asm/processor.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/powerpc/include/asm/processor.h
Declare request_cache variable as static. It is only used by
drivers/staging/lustre/lustre/ptlrpc/client.c, and its naming is common
which will lead to namespace pollution.
Signed-off-by: Simon Guo
---
drivers/staging/lustre/lustre/ptlrpc/client.c | 2 +-
1 file changed, 1 insertion(+), 1
Declare request_cache variable as static. It is only used by
drivers/staging/lustre/lustre/ptlrpc/client.c, and its naming is common
which will lead to namespace pollution.
Signed-off-by: Simon Guo wei.guo.si...@gmail.com
---
drivers/staging/lustre/lustre/ptlrpc/client.c | 2 +-
1 file changed
Delare request_cache variable as static. It is only used by
drivers/staging/lustre/lustre/ptlrpc/client.c, and its naming is common
which will lead to namespace pollution.
Signed-off-by: Simon Guo
---
drivers/staging/lustre/lustre/ptlrpc/client.c | 2 +-
1 file changed, 1 insertion(+), 1
Delare request_cache variable as static. It is only used by
drivers/staging/lustre/lustre/ptlrpc/client.c, and its naming is common
which will lead to namespace pollution.
Signed-off-by: Simon Guo wei.guo.si...@gmail.com
---
drivers/staging/lustre/lustre/ptlrpc/client.c | 2 +-
1 file changed, 1
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- line over 80 characters
Signed-off-by: Simon Guo
---
drivers/staging/comedi/drivers/pcl818.c | 15 ++-
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/comedi/drivers/pcl818.c
b
Signed-off-by: Simon Guo
---
drivers/staging/comedi/drivers/pcl818.c | 189 +++-
1 file changed, 91 insertions(+), 98 deletions(-)
diff --git a/drivers/staging/comedi/drivers/pcl818.c
b/drivers/staging/comedi/drivers/pcl818.c
index ba2e137..5d9050c 100644
--- a/drivers
Signed-off-by: Simon Guo wei.guo.si...@gmail.com
---
drivers/staging/comedi/drivers/pcl818.c | 189 +++-
1 file changed, 91 insertions(+), 98 deletions(-)
diff --git a/drivers/staging/comedi/drivers/pcl818.c
b/drivers/staging/comedi/drivers/pcl818.c
index ba2e137..5d9050c
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- line over 80 characters
Signed-off-by: Simon Guo wei.guo.si...@gmail.com
---
drivers/staging/comedi/drivers/pcl818.c | 15 ++-
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/drivers/staging
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- line over 80 characters
Signed-off-by: Simon Guo
---
drivers/staging/comedi/drivers/pcl818.c | 15 ++-
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/comedi/drivers/pcl818.c
b
Signed-off-by: Simon Guo
---
drivers/staging/comedi/drivers/pcl818.c | 189 +++-
1 file changed, 91 insertions(+), 98 deletions(-)
diff --git a/drivers/staging/comedi/drivers/pcl818.c
b/drivers/staging/comedi/drivers/pcl818.c
index ba2e137..5d9050c 100644
--- a/drivers
Signed-off-by: Simon Guo wei.guo.si...@gmail.com
---
drivers/staging/comedi/drivers/pcl818.c | 189 +++-
1 file changed, 91 insertions(+), 98 deletions(-)
diff --git a/drivers/staging/comedi/drivers/pcl818.c
b/drivers/staging/comedi/drivers/pcl818.c
index ba2e137..5d9050c
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- line over 80 characters
Signed-off-by: Simon Guo wei.guo.si...@gmail.com
---
drivers/staging/comedi/drivers/pcl818.c | 15 ++-
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/drivers/staging
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- please, no space before tabs
Signed-off-by: Simon Guo
---
drivers/staging/comedi/drivers/pcl818.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/comedi/drivers/pcl818.c
b/drivers
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- line over 80 characters
Signed-off-by: Simon Guo
---
drivers/staging/comedi/drivers/pcl818.c | 15 ++-
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/comedi/drivers/pcl818.c
b
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- code indent should use tabs where possible
It is fixed by reformatting the comment block to usual comment style.
Signed-off-by: Simon Guo
---
drivers/staging/comedi/drivers/pcl818.c | 195
Ian,
At 2015/1/26 19:31, Ian Abbott wrote:
Okay, but this comment block really needs reformatting to use the usual
block comment style, and with spaces to line up the columns instead of
tabs so it still looks okay when the " * " is removed from the start of
each line of the block comment.
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- please, no space before tabs
Signed-off-by: Simon Guo wei.guo.si...@gmail.com
---
drivers/staging/comedi/drivers/pcl818.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/comedi/drivers
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- code indent should use tabs where possible
It is fixed by reformatting the comment block to usual comment style.
Signed-off-by: Simon Guo wei.guo.si...@gmail.com
---
drivers/staging/comedi/drivers/pcl818.c | 195
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- line over 80 characters
Signed-off-by: Simon Guo wei.guo.si...@gmail.com
---
drivers/staging/comedi/drivers/pcl818.c | 15 ++-
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/drivers/staging
Ian,
At 2015/1/26 19:31, Ian Abbott wrote:
Okay, but this comment block really needs reformatting to use the usual
block comment style, and with spaces to line up the columns instead of
tabs so it still looks okay when the * is removed from the start of
each line of the block comment.
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- please, no space before tabs
Signed-off-by: Simon Guo
---
drivers/staging/comedi/drivers/pcl818.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/comedi/drivers/pcl818.c
b/drivers
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- line over 80 characters
Signed-off-by: Simon Guo
---
drivers/staging/comedi/drivers/pcl818.c | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/comedi/drivers/pcl818.c
b
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- code indent should use tabs where possible
Signed-off-by: Simon Guo
---
drivers/staging/comedi/drivers/pcl818.c | 26 +-
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/drivers
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- line over 80 characters
Signed-off-by: Simon Guo wei.guo.si...@gmail.com
---
drivers/staging/comedi/drivers/pcl818.c | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/drivers/staging
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- code indent should use tabs where possible
Signed-off-by: Simon Guo wei.guo.si...@gmail.com
---
drivers/staging/comedi/drivers/pcl818.c | 26 +-
1 file changed, 13 insertions(+), 13 deletions
Correct one coding style problem(detected by checkpatch.pl) in pcl818.c.
- please, no space before tabs
Signed-off-by: Simon Guo wei.guo.si...@gmail.com
---
drivers/staging/comedi/drivers/pcl818.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/comedi/drivers
Correct coding style problem in pcl818.c. The coding style problems(29
warnings detected by checkpatch.pl) includes:
- line over 80 characters
- There is space before tabs
The changed file has been verified with checkpatch.pl to be clean for
warnings/errors.
Signed-off-by: Simon Guo
Correct coding style problem in pcl818.c. The coding style problems(29
warnings detected by checkpatch.pl) includes:
- line over 80 characters
- There is space before tabs
The changed file has been verified with checkpatch.pl to be clean for
warnings/errors.
Signed-off-by: Simon Guo wei.guo.si
64 matches
Mail list logo