On 11/21/2013 07:06 PM, Arnaldo Carvalho de Melo wrote:
This changes machines__find to always return the machine for the
host (pid == 0). This function is also used for VM lookups.
Nope:
/* Native host kernel uses -1 as pid index in machine */
#define HOST_KERNEL_ID (-1)
-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/perf/util/evlist.c | 8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/tools/perf/util/evlist.c b/tools/perf/util/evlist.c
index bbc746a..2258eb4 100644
--- a/tools/perf/util/evlist.c
+++ b/tools/perf/util/evlist.c
@@ -819,11
On 11/27/2013 04:43 AM, Adrian Hunter wrote:
On 28/11/13 00:23, Dongsheng Yang wrote:
There are four condition branchs in perf_evlist__create_maps, but they
only do two kinds of thing. 1st and 4th call cpu_map__new(), and the others
call cpu_map__dummy_new(). This patch joins them in two
On 11/27/2013 05:49 AM, Adrian Hunter wrote:
On 28/11/13 00:52, Dongsheng Yang wrote:
On 11/27/2013 04:43 AM, Adrian Hunter wrote:
On 28/11/13 00:23, Dongsheng Yang wrote:
There are four condition branchs in perf_evlist__create_maps, but they
only do two kinds of thing. 1st and 4th call
Check for cpu_map__dummy_new() or cpu_map__new() to be called in
perf_evlist__create_maps() is more complicated. This patch moves
the checking work into target.h, combining two conditions and making
perf_evlist__create_maps() more readable.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
In machine__get_kernel_start_addr, the code, which is
using machine-root_dir to build filename, works for both
host and guests initialized from guestmount. So this patch
remove the branch for machine__is_host.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/perf/util/machine.c
Sorry for the error patch :(, Please ignore it.
On 11/28/2013 04:47 PM, Dongsheng Yang wrote:
In machine__get_kernel_start_addr, the code, which is
using machine-root_dir to build filename, works for both
host and guests initialized from guestmount. So this patch
remove the branch
I will resend the correct one. Sorry for the bother.
On 11/28/2013 02:32 PM, Dongsheng Yang wrote:
Check for cpu_map__dummy_new() or cpu_map__new() to be called in
perf_evlist__create_maps() is more complicated. This patch moves
the checking work into target.h, combining two conditions
In machine__get_kernel_start_addr, the code, which is
using machine-root_dir to build filename, works for both
host and guests initialized from guestmount. So this patch
remove the branch for machine__is_host.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/perf/util/machine.c
In machine__get_kernel_start_addr, the code, which is
using machine-root_dir to build filename, works for both
host and guests initialized from guestmount. So this patch
remove the branch for machine__is_host.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/perf/util/machine.c
ccing linux-kernel@vger.kernel.org
On 11/29/2013 01:14 PM, Arnaldo Carvalho de Melo wrote:
Em Fri, Nov 29, 2013 at 04:05:52PM -0500, Dongsheng Yang escreveu:
The package required for numa is named numactl-devel in Fedora and RHEL and
libnuma-devel in OpenSuSE. This patch correct the package
On 12/01/2013 08:33 PM, Peter Hurley wrote:
On 12/02/2013 09:11 AM, Dongsheng Yang wrote:
ccing linux-kernel@vger.kernel.org
On 11/29/2013 01:14 PM, Arnaldo Carvalho de Melo wrote:
Em Fri, Nov 29, 2013 at 04:05:52PM -0500, Dongsheng Yang escreveu:
The package required for numa is named
The package required for numa is named numactl-devel in Fedora or RHEL,
and libnuma-devel in OpenSuSE, and libnuma-dev in Ubuntu. This patch
corrects the package name in warning message in feature-libnuma checking.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
Changes since v1
- perf.data.host
* --guest- perf.data.guest
* --host --guest - perf.data.kvm
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/perf/builtin-kvm.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/tools/perf/builtin-kvm.c b/tools/perf/builtin
- perf.data.host
* --guest- perf.data.guest
* --host --guest - perf.data.kvm
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/perf/builtin-kvm.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/tools/perf/builtin-kvm.c b/tools/perf/builtin
Hi David:
On 11/14/2013 03:38 PM, David Ahern wrote:
On 11/14/13, 2:51 PM, Dongsheng Yang wrote:
Currently, we can not record the event counters of host to
perf.data.host.
Example:
perf kvm --host record -a sleep 1
[ perf record: Woken up 1 times to write data ]
[ perf record: Captured
. If you think it is not necessary, I am
ok to squash them into one commit.
Thanx
Yang
Dongsheng Yang (5):
perf tools: Remove condition in machine__get_kernel_start_addr.
perf tools: Move the checking of which map function to call into
function.
perf tools: Update the Document for perf
-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/perf/Documentation/perf-kvm.txt | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/tools/perf/Documentation/perf-kvm.txt
b/tools/perf/Documentation/perf-kvm.txt
index be1d263..96a9a1d 100644
--- a/tools/perf
.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/perf/Documentation/perf-kvm.txt | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/tools/perf/Documentation/perf-kvm.txt
b/tools/perf/Documentation/perf-kvm.txt
index 830613a..be1d263 100644
--- a/tools/perf
In machine__get_kernel_start_addr, the code, which is
using machine-root_dir to build filename, works for both
host and guests initialized from guestmount. So this patch
remove the branch for machine__is_host.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/perf/util/machine.c
As we have changed the default behavior of perf kvm to --guest enabled,
the document about perf kvm record is outdated. This patch update it to
show the correct output with --host/--guest/neither/both of them.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/perf/Documentation
Check for cpu_map__dummy_new() or cpu_map__new() to be called in
perf_evlist__create_maps() is more complicated. This patch moves
the checking work into target.h, combining two conditions and making
perf_evlist__create_maps() more readable.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
On 12/04/2013 09:27 AM, Arnaldo Carvalho de Melo wrote:
Em Wed, Dec 04, 2013 at 05:56:41PM -0500, Dongsheng Yang escreveu:
As we have changed the default behavior of perf kvm to --guest enabled,
the document about perf kvm record is outdated. This patch update it to
show the correct output
] scheduler_tick
1.75%:6471 [guest.kernel.kallsyms] [g] native_apic_mem_write
0.21%:6471 [guest.kernel.kallsyms] [g] apic_timer_interrupt
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/perf/util/session.c | 7 ++-
1 file changed, 6 insertions(+), 1
.
This patch remove the hard coding of default filenames in builtin-diff.c,
and generate the suitable filename from current options in perf kvm diff
command. It makes the default behavior of perf kvm diff be more
valuable.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/perf/builtin
In manpage of perf-kvm, --guestmount is supported by diff command,
but it does not work well.
This patch change the extend the checking in buildid-diff from
guestkallsyms or guestmodules to perf_guest. Then this checking can
cover the all cases perf kvm is used for.
Signed-off-by: Dongsheng Yang
The code in builtin-kvm.c to generate filename for perf-kvm is useful
to other command such as builtin-diff.
This patch move the related code form builtin-kvm.c to util/util.c and
wrap them in a function named get_filename_for_perf_kvm.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
The package required for dwarf is named libdwarf-devel, rather
than elfutils-devel.
This patch corrects the package name in warning message in
feature-dwarf checking.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/perf/config/Makefile | 4 ++--
1 file changed, 2 insertions
Hi Arnaldo,
Sorry for the noise, please ignore this patch.
On 03/18/2014 05:12 PM, Dongsheng Yang wrote:
The package required for dwarf is named libdwarf-devel, rather
than elfutils-devel.
This patch corrects the package name in warning message in
feature-dwarf checking.
Signed-off
Hi Peter.
It seems the all other patches in this set were all applied to tip
except this one.
What is the problem with the [9/9]? Is there any thing I can do?
Thanx
On 02/11/2014 03:34 PM, Dongsheng Yang wrote:
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: Steven Rostedt
On 02/24/2014 06:46 PM, Peter Zijlstra wrote:
On Sat, Feb 22, 2014 at 05:05:36PM +0800, Dongsheng Yang wrote:
Hi Peter.
It seems the all other patches in this set were all applied to tip
except this one.
What is the problem with the [9/9]? Is there any thing I can do?
The problem
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: Steven Rostedt rost...@goodmis.org
cc: Frederic Weisbecker fweis...@gmail.com
cc: Ingo Molnar mi...@redhat.com
cc: Peter Zijlstra pet...@infradead.org
---
kernel/trace/ring_buffer_benchmark.c | 6 +++---
1 file changed, 3 insertions
() function.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: Peter Zijlstra a.p.zijls...@chello.nl
cc: Paul Mackerras pau...@samba.org
cc: Ingo Molnar mi...@redhat.com
cc: Arnaldo Carvalho de Melo a...@ghostprotocols.net
cc: Thomas Gleixner t...@linutronix.de
cc: H. Peter Anvin h
As Kees suggested, I use clamp() function to replace the if and
else branch, making it more readable and modular.
Suggested-by: Kees Cook keesc...@chromium.org
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
kernel/sched/core.c | 11 ++-
1 file changed, 2 insertions(+), 9
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
kernel/sched/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index 5273037..7ed08ad 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -3070,7 +3070,7
Function task_nice() was reimplemented as inline function, we can use it here
to replace the open coded implementation.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: Steven Rostedt rost...@goodmis.org
---
kernel/trace/trace.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
Hi Joe,
On 03/11/2014 06:23 AM, Joe Perches wrote:
Convert the use of 20 to NICE_MAX + 1.
What about adding a macro in prio.h to convert nice value [19,-20] to
rlimit
style value [1,40]?
It seems that it will be used in several places.
- Dongsheng
Reduce the indent the switch case
On 03/11/2014 10:21 AM, Joe Perches wrote:
On Tue, 2014-03-11 at 09:17 +0800, Dongsheng Yang wrote:
Hi Joe,
On 03/11/2014 06:23 AM, Joe Perches wrote:
Convert the use of 20 to NICE_MAX + 1.
What about adding a macro in prio.h to convert nice value [19,-20] to
rlimit
style value [1,40
[-20, 19] to io priority [0, 7].
3. Others are all about replace hardcoding value about nice to MIN_NICE or
MAX_NICE.
Dongsheng Yang (14):
sched/prio: Add a macro named NICE_TO_RLIMIT in prio.h.
workqueue: Replace hardcoding of -20 with MIN_NICE.
locktorture: Replace hardcoding of 19
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
kernel/locking/locktorture.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/locking/locktorture.c b/kernel/locking/locktorture.c
index f26b1a1..23343be 100644
--- a/kernel/locking/locktorture.c
+++ b/kernel
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: nbd-gene...@lists.sourceforge.net
cc: Wei Yongjun yongjun_...@trendmicro.com.cn
cc: Tejun Heo t...@kernel.org
---
drivers/block/loop.c| 2 +-
drivers/block/nbd.c | 2 +-
drivers/block/pktcdvd.c | 2 +-
3 files changed, 3
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: fcoe-de...@open-fcoe.org
cc: James Smart james.sm...@emulex.com
cc: Robert Jennings r...@linux.vnet.ibm.com
cc: Robert Love robert.w.l...@intel.com
---
drivers/scsi/bnx2fc/bnx2fc_fcoe.c | 4 ++--
drivers/scsi/bnx2i/bnx2i_hwi.c| 2
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: linux-s...@vger.kernel.org
cc: Heiko Carstens heiko.carst...@de.ibm.com
cc: Martin Schwidefsky schwidef...@de.ibm.com
cc: Ingo Tuchscherer ingo.tuchsche...@de.ibm.com
---
drivers/s390/crypto/ap_bus.c | 2 +-
1 file changed, 1 insertion
void k_getrusage(struct task_struct *p, int who,
struct rusage *r)
-
+ /* fall-through */
Signed-off-by: Joe Perches j...@perches.com
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: Eric W. Biederman ebied...@xmission.com
cc: Robin Holt h...@sgi.com
cc: Oleg Nesterov o
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: linux...@kvack.org
cc: Bob Liu lliu...@gmail.com
cc: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
cc: Kirill A. Shutemov kirill.shute...@linux.intel.com
cc: Mel Gorman mgor...@suse.de
cc: Rik van Riel r...@redhat.com
cc: Andrew
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: de...@driverdev.osuosl.org
cc: Xiong Zhou jencce.ker...@gmail.com
cc: Andreas Dilger andreas.dil...@intel.com
cc: Cyril Roelandt tipec...@gmail.com
cc: Kent Overstreet k...@daterainc.com
cc: Peng Tao bergw...@gmail.com
---
drivers
There is a macro RLINIT_TO_NICE in linux/sched/prio.h to convert priority in
rlimit
to nice value.
This patch replace the opened implementation with RLINIT_TO_NICE.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: de...@driverdev.osuosl.org
cc: Masanari Iida standby2...@gmail.com
cc
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: openipmi-develo...@lists.sourceforge.net
cc: Corey Minyard miny...@acm.org
---
drivers/char/ipmi/ipmi_si_intf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char/ipmi
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/testing/selftests/mqueue/mq_perf_tests.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/testing/selftests/mqueue/mq_perf_tests.c
b/tools/testing/selftests/mqueue/mq_perf_tests.c
index 2fadd4b
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
include/linux/sched/prio.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/linux/sched/prio.h b/include/linux/sched/prio.h
index 8e78c47..e34bf0a 100644
--- a/include/linux/sched/prio.h
+++ b/include/linux/sched/prio.h
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: ocfs2-de...@oss.oracle.com
cc: Dong Fang yp.fangd...@gmail.com
---
fs/ocfs2/cluster/heartbeat.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/ocfs2/cluster/heartbeat.c b/fs/ocfs2/cluster/heartbeat.c
index
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: Tejun Heo t...@kernel.org
---
kernel/workqueue.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index 3fa5b8f..7528bec 100644
--- a/kernel/workqueue.c
+++ b/kernel
-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
include/linux/ioprio.h | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/include/linux/ioprio.h b/include/linux/ioprio.h
index beb9ce1..0164584 100644
--- a/include/linux/ioprio.h
+++ b/include/linux/ioprio.h
@@ -18,6 +18,11
This patch add a macro named NICE_TO_RLIMIT in prio.h to
convert nice value [19,-20] to rlimit style value [1,40].
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
include/linux/sched/prio.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/include/linux/sched/prio.h b/include
Hi steve,
Could you take these three patches if they looks fine to you??
Thanx
On 03/05/2014 08:36 PM, Dongsheng Yang wrote:
Function task_nice() was reimplemented as inline function, we can use it here
to replace the open coded implementation.
Signed-off-by: Dongsheng Yang yangds.f
() is only called by __init intel_uncore_init(),
then we can
free it after system initialization.
This patch add __init for uncore_cpumask_init() function.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
Acked-by: Stephane Eranian eran...@google.com
---
Changlogs:
-v1:
Add
On 03/11/2014 04:17 PM, Heiko Carstens wrote:
On Tue, Mar 11, 2014 at 12:59:16PM +0800, Dongsheng Yang wrote:
This patch add a macro named NICE_TO_RLIMIT in prio.h to
convert nice value [19,-20] to rlimit style value [1,40].
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
On 03/11/2014 04:36 PM, Heiko Carstens wrote:
On Tue, Mar 11, 2014 at 04:20:24PM +0800, Dongsheng Yang wrote:
On 03/11/2014 04:17 PM, Heiko Carstens wrote:
On Tue, Mar 11, 2014 at 12:59:16PM +0800, Dongsheng Yang wrote:
+#define NICE_TO_RLIMIT(nice) (MAX_NICE - nice + 1)
Where is MAX_NICE
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: de...@driverdev.osuosl.org
cc: Xiong Zhou jencce.ker...@gmail.com
cc: Andreas Dilger andreas.dil...@intel.com
cc: Cyril Roelandt tipec...@gmail.com
cc: Kent Overstreet k...@daterainc.com
cc: Peng Tao bergw...@gmail.com
---
drivers
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: Tejun Heo t...@kernel.org
---
kernel/workqueue.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index 3fa5b8f..7528bec 100644
--- a/kernel/workqueue.c
+++ b/kernel
-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
include/linux/ioprio.h | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/include/linux/ioprio.h b/include/linux/ioprio.h
index beb9ce1..c0faa0b 100644
--- a/include/linux/ioprio.h
+++ b/include/linux/ioprio.h
@@ -18,6 +18,11
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: ocfs2-de...@oss.oracle.com
cc: Dong Fang yp.fangd...@gmail.com
---
fs/ocfs2/cluster/heartbeat.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/ocfs2/cluster/heartbeat.c b/fs/ocfs2/cluster/heartbeat.c
index
There is a macro rlimit_to_nice in linux/sched/prio.h to convert priority in
rlimit
to nice value.
This patch replace the opened implementation with rlimit_to_nice.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: de...@driverdev.osuosl.org
cc: Masanari Iida standby2...@gmail.com
cc
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: linux-s...@vger.kernel.org
cc: Heiko Carstens heiko.carst...@de.ibm.com
cc: Martin Schwidefsky schwidef...@de.ibm.com
cc: Ingo Tuchscherer ingo.tuchsche...@de.ibm.com
---
drivers/s390/crypto/ap_bus.c | 2 +-
1 file changed, 1 insertion
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: linux...@kvack.org
cc: Bob Liu lliu...@gmail.com
cc: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
cc: Kirill A. Shutemov kirill.shute...@linux.intel.com
cc: Mel Gorman mgor...@suse.de
cc: Rik van Riel r...@redhat.com
cc: Andrew
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: openipmi-develo...@lists.sourceforge.net
cc: Corey Minyard miny...@acm.org
---
drivers/char/ipmi/ipmi_si_intf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char/ipmi
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: fcoe-de...@open-fcoe.org
cc: James Smart james.sm...@emulex.com
cc: Robert Jennings r...@linux.vnet.ibm.com
cc: Robert Love robert.w.l...@intel.com
---
drivers/scsi/bnx2fc/bnx2fc_fcoe.c | 4 ++--
drivers/scsi/bnx2i/bnx2i_hwi.c| 2
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
kernel/sched/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index ee8004c..d2735eb 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -3046,7 +3046,7
This patch add an inline function named rlimit_to_nice to convert
rlimit style value in [1, 40] to nice value in [-20, 19].
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
include/linux/sched/prio.h | 8
1 file changed, 8 insertions(+)
diff --git a/include/linux/sched
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: nbd-gene...@lists.sourceforge.net
cc: Wei Yongjun yongjun_...@trendmicro.com.cn
cc: Tejun Heo t...@kernel.org
---
drivers/block/loop.c| 2 +-
drivers/block/nbd.c | 2 +-
drivers/block/pktcdvd.c | 2 +-
3 files changed, 3
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
kernel/locking/locktorture.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/locking/locktorture.c b/kernel/locking/locktorture.c
index f26b1a1..23343be 100644
--- a/kernel/locking/locktorture.c
+++ b/kernel
This patch add an inline function named nice_to_rlimit() in prio.h to
convert nice value [19,-20] to rlimit style value [1,40].
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
include/linux/sched/prio.h | 8
1 file changed, 8 insertions(+)
diff --git a/include/linux/sched
void k_getrusage(struct task_struct *p, int who,
struct rusage *r)
-
+ /* fall-through */
Signed-off-by: Joe Perches j...@perches.com
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: Eric W. Biederman ebied...@xmission.com
cc: Robin Holt h...@sgi.com
cc: Oleg Nesterov o
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/testing/selftests/mqueue/mq_perf_tests.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/testing/selftests/mqueue/mq_perf_tests.c
b/tools/testing/selftests/mqueue/mq_perf_tests.c
index 2fadd4b
opened code implement with
nice_to_rlimit().
Dongsheng Yang (15):
sched/prio: Add a inline function named nice_to_rlimit() in prio.h.
workqueue: Replace hardcoding of -20 with MIN_NICE.
locktorture: Replace hardcoding of 19 with MAX_NICE.
tools/mq_perf_tests: Replace hardcoding of -20
On 03/11/2014 05:26 PM, Peter Zijlstra wrote:
On Tue, Mar 11, 2014 at 05:20:23PM +0800, Dongsheng Yang wrote:
From: Joe Perches j...@perches.com
Convert 20 - task_nice(p) to nice_to_rlimit(task_nice(p)).
Reduce the indent the switch case labels while there.
git diff -w shows 3 lines changed
On 03/11/2014 05:28 PM, Peter Zijlstra wrote:
On Tue, Mar 11, 2014 at 05:20:28PM +0800, Dongsheng Yang wrote:
As the task nice value is in [-20, 19] and the io priority is in [0, 7],
and the convert method from niceval to ioprio is implemented with an
opened code in task_nice_ioprio
From: Joe Perches j...@perches.com
Fix the indent issue and add a comment line for /* fall-through */.
Signed-off-by: Joe Perches j...@perches.com
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
kernel/sys.c | 204 +--
1 file
Peter,
Sorry for the noise, I should add this patch into my thread 'A
series patchse about sched priority', as some patch depends on this
patch. Otherwise, there
will be a conflict.
Thanx.,
On 03/11/2014 05:44 PM, Dongsheng Yang wrote:
From: Joe Perches j...@perches.com
Fix
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: de...@driverdev.osuosl.org
cc: Xiong Zhou jencce.ker...@gmail.com
cc: Andreas Dilger andreas.dil...@intel.com
cc: Cyril Roelandt tipec...@gmail.com
cc: Kent Overstreet k...@daterainc.com
cc: Peng Tao bergw...@gmail.com
---
drivers
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: fcoe-de...@open-fcoe.org
cc: James Smart james.sm...@emulex.com
cc: Robert Jennings r...@linux.vnet.ibm.com
cc: Robert Love robert.w.l...@intel.com
---
drivers/scsi/bnx2fc/bnx2fc_fcoe.c | 4 ++--
drivers/scsi/bnx2i/bnx2i_hwi.c| 2
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: linux-s...@vger.kernel.org
cc: Heiko Carstens heiko.carst...@de.ibm.com
cc: Martin Schwidefsky schwidef...@de.ibm.com
cc: Ingo Tuchscherer ingo.tuchsche...@de.ibm.com
---
drivers/s390/crypto/ap_bus.c | 2 +-
1 file changed, 1 insertion
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: linux...@kvack.org
cc: Bob Liu lliu...@gmail.com
cc: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
cc: Kirill A. Shutemov kirill.shute...@linux.intel.com
cc: Mel Gorman mgor...@suse.de
cc: Rik van Riel r...@redhat.com
cc: Andrew
There is a macro rlimit_to_nice in linux/sched/prio.h to convert priority in
rlimit
to nice value.
This patch replace the opened implementation with rlimit_to_nice.
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: de...@driverdev.osuosl.org
cc: Masanari Iida standby2...@gmail.com
cc
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: nbd-gene...@lists.sourceforge.net
cc: Wei Yongjun yongjun_...@trendmicro.com.cn
cc: Tejun Heo t...@kernel.org
---
drivers/block/loop.c| 2 +-
drivers/block/nbd.c | 2 +-
drivers/block/pktcdvd.c | 2 +-
3 files changed, 3
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
kernel/sched/core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/sched/core.c b/kernel/sched/core.c
index ee8004c..d2735eb 100644
--- a/kernel/sched/core.c
+++ b/kernel/sched/core.c
@@ -3046,7 +3046,7
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: openipmi-develo...@lists.sourceforge.net
cc: Corey Minyard miny...@acm.org
---
drivers/char/ipmi/ipmi_si_intf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char/ipmi
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: ocfs2-de...@oss.oracle.com
cc: Dong Fang yp.fangd...@gmail.com
---
fs/ocfs2/cluster/heartbeat.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/ocfs2/cluster/heartbeat.c b/fs/ocfs2/cluster/heartbeat.c
index
This patch add an inline function named rlimit_to_nice to convert
rlimit style value in [1, 40] to nice value in [-20, 19].
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
include/linux/sched/prio.h | 8
1 file changed, 8 insertions(+)
diff --git a/include/linux/sched
From: Joe Perches j...@perches.com
Fix the indent issue and add a comment line for /* fall-through */.
Signed-off-by: Joe Perches j...@perches.com
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
kernel/sys.c | 204 +--
1 file
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
kernel/sys.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/kernel/sys.c b/kernel/sys.c
index 61a2ae9..1bfe79a 100644
--- a/kernel/sys.c
+++ b/kernel/sys.c
@@ -250,7 +250,7 @@ SYSCALL_DEFINE2(getpriority, int
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
kernel/locking/locktorture.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/locking/locktorture.c b/kernel/locking/locktorture.c
index f26b1a1..23343be 100644
--- a/kernel/locking/locktorture.c
+++ b/kernel
-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
include/linux/ioprio.h | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/include/linux/ioprio.h b/include/linux/ioprio.h
index beb9ce1..c0faa0b 100644
--- a/include/linux/ioprio.h
+++ b/include/linux/ioprio.h
@@ -18,6 +18,11
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: Tejun Heo t...@kernel.org
---
kernel/workqueue.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/kernel/workqueue.c b/kernel/workqueue.c
index 3fa5b8f..7528bec 100644
--- a/kernel/workqueue.c
+++ b/kernel
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
tools/testing/selftests/mqueue/mq_perf_tests.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tools/testing/selftests/mqueue/mq_perf_tests.c
b/tools/testing/selftests/mqueue/mq_perf_tests.c
index 2fadd4b
:
*Implement nice_to_rlimit and rlimit_to_nice with inline functions
rather than macro.
*Add a patch[16/16] to replace opened code implement with
nice_to_rlimit().
Dongsheng Yang (16):
sched/prio: Add a inline function named nice_to_rlimit() in prio.h.
kernel/sys
This patch add an inline function named nice_to_rlimit() in prio.h to
convert nice value [19,-20] to rlimit style value [1,40].
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
---
include/linux/sched/prio.h | 8
1 file changed, 8 insertions(+)
diff --git a/include/linux/sched
Hi Tejun,
On 03/11/2014 08:55 PM, Tejun Heo wrote:
On Tue, Mar 11, 2014 at 08:22:25AM -0400, Tejun Heo wrote:
On Tue, Mar 11, 2014 at 06:09:12PM +0800, Dongsheng Yang wrote:
Signed-off-by: Dongsheng Yang yangds.f...@cn.fujitsu.com
cc: Tejun Heo t...@kernel.org
Applied to wq/for-3.15
, please take a look at the other patches
in this thread. Thank you.
Dongsheng Yang (3):
tracing: Use inline task_nice() to get rid of an open coded
implementation of it.
sched/prio: Replace hardcoding of 40 with NICE_WIDTH.
sched: Use clamp() and clamp_val() to make it more readable
On 03/11/2014 06:09 PM, Dongsheng Yang wrote:
As the task nice value is in [-20, 19] and the io priority is in [0, 7],
and the convert method from niceval to ioprio is implemented with an
opened code in task_nice_ioprio().
This patch move the implementation to a macro NICE_TO_IOPRIO, making
1 - 100 of 733 matches
Mail list logo