I split the code of stopping/waking tx queue into the another patch.
Hayes Wang (4):
r8152: fix tx/rx memory overflow
r8152: modify the tx flow
r8152: support stopping/waking tx queue
r8152: fix incorrect type in assignment
drivers/net/usb/r8152.c | 109
The tx/rx would access the memory which is out of the desired range.
Modify the method of checking the end of the memory to avoid it.
For r8152_tx_agg_fill(), the variable remain may become negative.
However, the declaration is unsigned, so the while loop wouldn't
break when reaching the end of
Hello,
Shawn, Hugh, can you please verify whether the attached patch makes
the deadlock go away?
Thanks.
diff --git a/kernel/cgroup.c b/kernel/cgroup.c
index e0839bc..dc9dc06 100644
--- a/kernel/cgroup.c
+++ b/kernel/cgroup.c
@@ -90,6 +90,14 @@ static DEFINE_MUTEX(cgroup_mutex);
static
Hi Huajun,
[snip]
> +static int __f2fs_convert_inline_data(struct inode *inode, struct page *page)
> +{
> + int err;
> + struct page *ipage;
> + struct dnode_of_data dn;
> + void *src_addr, *dst_addr;
> + block_t old_blk_addr, new_blk_addr;
> + struct f2fs_sb_info *sbi =
Commit-ID: cc08e04c3fdcaab767b8db27527002b2b4d758cb
Gitweb: http://git.kernel.org/tip/cc08e04c3fdcaab767b8db27527002b2b4d758cb
Author: David Rientjes
AuthorDate: Thu, 14 Nov 2013 15:05:32 -0800
Committer: Ingo Molnar
CommitDate: Fri, 15 Nov 2013 08:38:30 +0100
x86: Export
Hi Eduardo,
Sorry for joining the discussion a little late, I could never find the
time to look into this patch series so far.
On Tue, 12 Nov 2013 15:46:08 -0400, Eduardo Valentin wrote:
> This patch adds to lm75 temperature sensor the possibility
> to expose itself as thermal zone device,
> -Original Message-
> From: Mark Rutland [mailto:mark.rutl...@arm.com]
> Sent: Thursday, November 14, 2013 6:46 PM
> To: Lu Jingchang-B35083
> Cc: vinod.k...@intel.com; devicet...@vger.kernel.org; Wang Huan-B18965;
> linux-kernel@vger.kernel.org; shawn@linaro.org; linux-arm-
>
Commit-ID: 539e6bb71e350541105e67e3d6c31392d9da25ef
Gitweb: http://git.kernel.org/tip/539e6bb71e350541105e67e3d6c31392d9da25ef
Author: Adrian Hunter
AuthorDate: Fri, 1 Nov 2013 15:51:34 +0200
Committer: Arnaldo Carvalho de Melo
CommitDate: Thu, 14 Nov 2013 16:10:27 -0300
perf record:
Drop EPOLLWAKEUP from epoll events mask if CONFIG_PM_SLEEP is disabled.
Signed-off-by: Amit Pundir
---
Changes in v3:
Renamed ep_epollwakeup_check() to ep_take_care_of_epollwakeup().
Didn't update ep_create_wakeup_source() to return -ENOSYS if PM_SLEEP is unset.
---
fs/eventpoll.c
Commit-ID: 48d038fcd09fa231e254965c3b69f8f640c9e62d
Gitweb: http://git.kernel.org/tip/48d038fcd09fa231e254965c3b69f8f640c9e62d
Author: Arnaldo Carvalho de Melo
AuthorDate: Thu, 14 Nov 2013 15:30:41 -0300
Committer: Arnaldo Carvalho de Melo
CommitDate: Thu, 14 Nov 2013 16:00:31 -0300
Commit-ID: 7486d6da09d4d76d49c68826c97578246936092b
Gitweb: http://git.kernel.org/tip/7486d6da09d4d76d49c68826c97578246936092b
Author: Joe Perches
AuthorDate: Thu, 14 Nov 2013 14:59:45 -0800
Committer: Ingo Molnar
CommitDate: Fri, 15 Nov 2013 07:15:26 +0100
MAINTAINERS: Update file
Commit-ID: 37676af15c8d5a9689c9d1220d2a27d510cbe238
Gitweb: http://git.kernel.org/tip/37676af15c8d5a9689c9d1220d2a27d510cbe238
Author: Arnaldo Carvalho de Melo
AuthorDate: Wed, 13 Nov 2013 17:40:36 -0300
Committer: Arnaldo Carvalho de Melo
CommitDate: Thu, 14 Nov 2013 16:00:23 -0300
Commit-ID: ea432a8bb940e6bea2aaeca3c0ff3d931ad81f2e
Gitweb: http://git.kernel.org/tip/ea432a8bb940e6bea2aaeca3c0ff3d931ad81f2e
Author: Ingo Molnar
AuthorDate: Wed, 13 Nov 2013 00:26:09 +0100
Committer: Arnaldo Carvalho de Melo
CommitDate: Thu, 14 Nov 2013 15:59:50 -0300
perf top: Add
Commit-ID: 9a354cdc2f40344a177d369fb4987a8270dd94df
Gitweb: http://git.kernel.org/tip/9a354cdc2f40344a177d369fb4987a8270dd94df
Author: Arnaldo Carvalho de Melo
AuthorDate: Wed, 13 Nov 2013 15:54:30 -0300
Committer: Arnaldo Carvalho de Melo
CommitDate: Thu, 14 Nov 2013 16:00:10 -0300
Commit-ID: 35e17b2450e09968f9702d4048c228199af171bc
Gitweb: http://git.kernel.org/tip/35e17b2450e09968f9702d4048c228199af171bc
Author: Azat Khuzhin
AuthorDate: Mon, 28 Oct 2013 12:04:24 +0400
Committer: Arnaldo Carvalho de Melo
CommitDate: Thu, 14 Nov 2013 16:06:28 -0300
perf probe:
Commit-ID: d87fcb4a2d990ba2de9284ede84a816c5066d54b
Gitweb: http://git.kernel.org/tip/d87fcb4a2d990ba2de9284ede84a816c5066d54b
Author: Arnaldo Carvalho de Melo
AuthorDate: Wed, 13 Nov 2013 15:56:40 -0300
Committer: Arnaldo Carvalho de Melo
CommitDate: Thu, 14 Nov 2013 16:00:16 -0300
Commit-ID: 9d4ecc8893832337daf241236841db966fa53489
Gitweb: http://git.kernel.org/tip/9d4ecc8893832337daf241236841db966fa53489
Author: Don Zickus
AuthorDate: Wed, 13 Nov 2013 15:32:06 -0300
Committer: Arnaldo Carvalho de Melo
CommitDate: Thu, 14 Nov 2013 16:00:01 -0300
perf tools:
Commit-ID: b13936ef7d48908be2fab7639dd535c88045
Gitweb: http://git.kernel.org/tip/b13936ef7d48908be2fab7639dd535c88045
Author: Davidlohr Bueso
AuthorDate: Tue, 12 Nov 2013 22:24:24 -0800
Committer: Arnaldo Carvalho de Melo
CommitDate: Thu, 14 Nov 2013 15:59:38 -0300
perf
Commit-ID: 27a778b512e002d856952b4f01842ba4d34bc3d1
Gitweb: http://git.kernel.org/tip/27a778b512e002d856952b4f01842ba4d34bc3d1
Author: Pekka Enberg
AuthorDate: Wed, 13 Nov 2013 14:21:48 +0200
Committer: Arnaldo Carvalho de Melo
CommitDate: Thu, 14 Nov 2013 15:59:20 -0300
perf trace:
On 11/15/2013 02:39 AM, Marcelo Tosatti wrote:
> On Thu, Nov 14, 2013 at 01:15:24PM +0800, Xiao Guangrong wrote:
>>
>> Hi Marcelo,
>>
>> On 11/14/2013 08:36 AM, Marcelo Tosatti wrote:
>>
>>>
>>> Any code location which reads the writable bit in the spte and assumes if
>>> its not
>>> set, that
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
-- Forwarded message --
From: 韩磊
Date: 2013/11/15
Subject: A problem about IO scheduler in kernel
To: Linux Kernel Mailing List
These days I was programming about IO scheduler called
"Simple_Deadline" in kernel.
"Simple_Deadline" is based on "deadline-iosched".The algorithm is
On Wed, Nov 13, 2013 at 5:57 PM, Ethan Zhao wrote:
> On Wed, Nov 6, 2013 at 10:49 PM, ethan.zhao wrote:
>> Though no specification about NumVFs register initial value after POST, to
>> void the confusion
>> lspci output as following before VF was enabled, we should clear the NumVFs
>> value
On 11/14/2013 10:55 PM, Yinghai Lu wrote:
>
> Why just asking distros to append ",high" in their installation
> program for 64bit by default?
>
[...]
>
> What is hpa's suggestion?
>
Pretty much what you just said ;)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel"
On Thu, Nov 14, 2013 at 10:24 PM, Ingo Molnar wrote:
>
> * jerry.hoem...@hp.com wrote:
>
>> On Thu, Nov 14, 2013 at 08:44:04PM +0200, Pekka Enberg wrote:
>> > On Thu, Nov 14, 2013 at 8:04 PM, wrote:
>> > > Making this issue a quirk will be a lot more practical. Its a small,
>> > > focused
>>
On Fri, Nov 15, 2013 at 02:42:01PM +0800, Shawn Guo wrote:
> On Thu, Nov 14, 2013 at 07:07:09PM +0800, Nicolin Chen wrote:
> > We must specify the value of audmux pinctrl if we want to use pinctrl_pm().
> > Thus change bypass value 0x8000 to what we exactly need.
> >
> > This patch also
Avoids wasting cycles at boot specially on slower simulators
Signed-off-by: Vineet Gupta
Cc: David Daney
Cc: Michal Marek
Cc: Francois Bedard
Cc: linux-kernel@vger.kernel.org
---
arch/arc/Kconfig | 1 +
scripts/sortextable.c | 5 +
2 files changed, 6 insertions(+)
diff --git
On Thursday 14 November 2013 23:34:19 Sebastian Reichel wrote:
> On Thu, Nov 14, 2013 at 07:05:06PM +0100, Pali Rohár wrote:
> > On Thursday 14 November 2013 13:38:53 Sebastian Reichel
wrote:
> > > diff --git a/drivers/power/isp1704_charger.c
> > > b/drivers/power/isp1704_charger.c index
On Thu, Nov 14, 2013 at 07:07:09PM +0800, Nicolin Chen wrote:
> We must specify the value of audmux pinctrl if we want to use pinctrl_pm().
> Thus change bypass value 0x8000 to what we exactly need.
>
> This patch also seperately unset PUE bit for TXD so that IOMUX won't pull
> up/down the
* Arnaldo Carvalho de Melo wrote:
> From: Arnaldo Carvalho de Melo
>
> Hi Ingo,
>
> Please consider pulling, done on top of tip/perf/urgent.
>
> - Arnaldo
>
> The following changes since commit e310718d0e83aeb9969264dc577c45db16d9104d:
>
> tools/perf/build: Fix
* Stephane Eranian wrote:
> Jiri,
>
> I was trying the grouping support in perf stat and I was surprised
> to see that if I create a group that is too big to be scheduled, and
> where only N out of P events can fit, perf stat still yields counts
> for the N events. I was expecting 0 counts
Greetings,
I am contacting you regarding our company urgent need for (DINOSILE OIL) a raw
material I found in India recently.
It is the major liquid material our company has been using for the production
of animal injections and vaccines and also for research since 2001. Our company
is
Hello,
On Thu, Nov 14, 2013 at 04:56:49PM -0600, Shawn Bohrer wrote:
> After running both concurrently on 40 machines for about 12 hours I've
> managed to reproduce the issue at least once, possibly more. One
> machine looked identical to this reported issue. It has a bunch of
> stuck
* jerry.hoem...@hp.com wrote:
> On Thu, Nov 14, 2013 at 08:44:04PM +0200, Pekka Enberg wrote:
> > On Thu, Nov 14, 2013 at 8:04 PM, wrote:
> > > Making this issue a quirk will be a lot more practical. Its a small,
> > > focused
> > > change whose implications are limited and more easily
Per the ePAPR v1.1 specification, 'phy-connection-type' is the canonical
property name for describing an Ethernet to PHY connection type. Make
sure that of_get_phy_mode() also attempts to parse that property and
update the comments mentioning 'phy-mode' to also include
'phy-connection-type'.
On 11/15/2013 12:54 PM, Eric W. Biederman wrote:
> Gao feng writes:
>
>> On 11/15/2013 12:54 AM, Andy Lutomirski wrote:
>>> On Thu, Nov 14, 2013 at 3:10 AM, Gao feng wrote:
On 11/13/2013 03:26 PM, Gao feng wrote:
> On 11/09/2013 01:42 PM, Eric W. Biederman wrote:
>> Right now I
* Arnaldo Carvalho de Melo wrote:
> +--force-per-cpu::
> + Force the use of per-cpu mmaps. By default, when tasks are specified (i.e.
> -p,
> + -t or -u options) per-thread mmaps are created. This option overrides that
> and
> + forces per-cpu mmaps. A side-effect of that is that
The related code has been changed and the comment is out of data.
So remove it.
Signed-off-by: Lan Tianyu
---
drivers/cpufreq/cpufreq_governor.c | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/cpufreq/cpufreq_governor.c
b/drivers/cpufreq/cpufreq_governor.c
index 0806c31..e6be635
Hi Arnaldo,
On Tue, 12 Nov 2013 11:41:01 -0300, Arnaldo Carvalho de Melo wrote:
> Em Tue, Nov 12, 2013 at 09:07:36AM +0200, Adrian Hunter escreveu:
>> -#define NR_LOOPS 100
>> +#define NR_LOOPS 1000
>
> Lower frequency, need to generate more noise, ugh. Adding that, but I
> think this
Currently, governor of nonboot cpus will be put to EXIT when system syspend.
Since all these cpus will be unplugged and the governor usage_count decreases
to zero. The governor data and its sysfs interfaces will be freed or released.
This makes user config of these governors loss during suspend
On 11/15/2013 03:44 AM, Geert Uytterhoeven wrote:
> If CONFIG_INITRAMFS_SOURCE contains relative paths inside the source tree
> (e.g. in a defconfig pointing to arch-specific files), the corresponding
> file system entries are not found when building outside the source tree.
>
> Prefix all simple
Hi Tony,
Can you please host a ARC cross compiler on
https://www.kernel.org/pub/tools/crosstool
Mainline buildroot supports ARC (alas that build system doesn't generate a
relocatable toolchain).
Or you can simply use our homegrown build-system to checkout and build tools
(which can even do a
On Fri, Nov 15, 2013 at 12:13:07AM +0800, Peng Tao wrote:
> From: "John L. Hammond"
>
> Add a const void *h_owner member to struct portals_handle. Add a const
> void *owner parameter to class_handle2object() which must be matched
> by the h_owner member of the handle in addition to the cookie.
On Fri, Nov 15, 2013 at 12:13:02AM +0800, Peng Tao wrote:
> [sadly git-send-email died sending the patchset... so resending]
>
> Hi Greg,
>
> Here are 40 patches. Three cleanup patches and 37 patches ported
> from Lustre master to commit
> LU-2940 llite: Fix for oops in vvp_pgcache_show()
>
>
On Fri, Nov 15, 2013 at 12:13:06AM +0800, Peng Tao wrote:
> From: JC Lafoucriere
>
> When a client accesses data in a released file,
> or truncate it, client must trig a restore request.
> During this restore, the client must not glimpse and
> must use size from MDT. To bring the "restore is
btw., here's some 'perf top' call graph performance and profiling
quality feedback, with the latest perf code:
'perf top --call-graph fp' now works very well, using just 0.2%
of CPU time on a fast system:
4676 mingo 20 0 612m 56m 9948 S 1 0.2 0:00.68 perf
On Fri, Nov 15, 2013 at 12:13:09AM +0800, Peng Tao wrote:
> From: Jinshan Xiong
>
> In this patch, a framework of lease is implemented. However,
> only exclusive lease is supported right now.
Which is a problem, why?
Why do you want to add this new functionality? What is it good for?
Who will
On Fri, Nov 15, 2013 at 12:32:24AM +0800, Peng Tao wrote:
> From: "John L. Hammond"
>
> Remove the file /proc/fs/lustre/mdc/*/hsm_nl which was introduced "for
> testing purposes."
What does this mean? Why is this allowed? How do you know no one uses
it? I need a much better changelog comment
On Fri, Nov 15, 2013 at 12:28:20AM +0800, Peng Tao wrote:
> On Fri, Nov 15, 2013 at 12:22 AM, Cheng Shao wrote:
> > Hi Peng,
> >
> > This patch was eventually reverted as it was causing interoperability
> > issues with 2.1 Lustre server. We are working on the new patch, so please
> > don't
These days I was programming about IO scheduler called
"Simple_Deadline" in kernel.
"Simple_Deadline" is based on "deadline-iosched".The algorithm is very
simple.It have two lists,one is read,the other is write.A request
enter lists based its weight which count accord to the request's
size,read or
From: "H. Peter Anvin"
This reverts commit 69f0554ec261fd686ac7fa1c598cc9eb27b83a80.
This patch breaks randconfig on at least the x86-64 architecture, and
most likely on others. There is work underway to support uncompressed
kernels in a generic way, but it looks like it will amount to
* David Ahern wrote:
> --- a/tools/perf/Documentation/perf-top.txt
> +++ b/tools/perf/Documentation/perf-top.txt
> @@ -143,12 +142,12 @@ Default is to monitor all CPUS.
> --asm-raw::
> Show raw instruction encoding of assembly instructions.
>
> --G::
> +-g::
> Enables call-graph
Hi all,
Please do *not* add any v3.14 material to linux-next until after
v3.13-rc1 is released.
Changes since 20131114:
The idle tree gained a conflict against the pm tree.
The dmaengine tree gained a conflict against the slave-dma tree.
The akpm tree gained a conflict against the sparc-next
From: Stephen Rothwell
Date: Fri, 15 Nov 2013 16:00:00 +1100
> [Forgot to cc Dave]
Yes, I told Linus about this conflict when I sent in my sparc pull
request this afternoon.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
This patch adds f2fs_issue_discard() to clean up blkdev_issue_discard() flows.
Signed-off-by: Jaegeuk Kim
---
fs/f2fs/segment.c | 23 +--
1 file changed, 13 insertions(+), 10 deletions(-)
diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c
index 505a889..d021cf3 100644
---
If frequent small discards are issued to the device, the performance would
be degraded significantly.
So, this patch adds a sysfs entry to control the number of discards to be
issued during a checkpoint procedure.
By default, f2fs does not issue any small discards, which means max_discards
is
This patch adds a tracepoint for f2fs_issue_discard.
Signed-off-by: Jaegeuk Kim
---
fs/f2fs/segment.c | 1 +
include/trace/events/f2fs.h | 23 +++
2 files changed, 24 insertions(+)
diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c
index d021cf3..1f83999 100644
This patch adds key functions to activate the small discard feature.
Note that this procedure is conducted during the checkpoint only.
In flush_sit_entries(), when a new dirty sit entry is flushed, f2fs calls
add_discard_addrs() which searches candidates to be discarded.
The candidates should be
This patch adds a slab cache entry for small discards.
Each entry consists of:
struct discard_entry {
struct list_head list; /* list head */
block_t blkaddr;/* block address to be discarded */
int len;/* # of consecutive blocks of the discard */
On 10/28/2013 08:05:11 AM, Henrik Austad wrote:
Hi Rob, Jiri
Hacking away at python shows me that of the 254 subfolders 57 has
outdated
00-INDEX, either with missing files, or files that has been removed.
(see
list below)
I'm a bit behind on my email just now. (Started a new job last
Jeff Layton writes:
> On Tue, 12 Nov 2013 17:02:36 +0400
> Stanislav Kinsbursky wrote:
>
>> 12.11.2013 15:12, Jeff Layton пишет:
>> > On Mon, 11 Nov 2013 16:47:03 -0800
>> > Greg KH wrote:
>> >
>> >> On Mon, Nov 11, 2013 at 07:18:25AM -0500, Jeff Layton wrote:
>> >>> We have a bit of a problem
[Forgot to cc Dave]
On Fri, 15 Nov 2013 15:57:33 +1100 Stephen Rothwell
wrote:
>
> Hi Andrew,
>
> Today's linux-next merge of the akpm tree got a conflict in
> arch/sparc/mm/init_64.c between commit 37b3a8ff3e08 ("sparc64: Move from
> 4MB to 8MB huge pages") from the sparc-next tree and commit
Gao feng writes:
> On 11/15/2013 12:54 AM, Andy Lutomirski wrote:
>> On Thu, Nov 14, 2013 at 3:10 AM, Gao feng wrote:
>>> On 11/13/2013 03:26 PM, Gao feng wrote:
On 11/09/2013 01:42 PM, Eric W. Biederman wrote:
> Right now I would rather not have the empty directory exception than
Use NOKPROBE_SYMBOL() to protect handlers from kprobes
in sample modules.
Signed-off-by: Masami Hiramatsu
Ananth N Mavinakayanahalli
---
samples/kprobes/jprobe_example.c|1 +
samples/kprobes/kprobe_example.c|3 +++
samples/kprobes/kretprobe_example.c |2 ++
3 files changed,
To blacklist the functions in a module (e.g. user-defined
kprobe handler and the functions invoked from it), expand
blacklist support for modules.
With this change, users can use NOKPROBE_SYMBOL() macro in
their own modules.
Signed-off-by: Masami Hiramatsu
Cc: Ananth N Mavinakayanahalli
Cc:
There is no need to prohibit probing on the functions
used in preparation phase. Those are safely probed because
those are not invoked from breakpoint/fault/debug handlers,
there is no chance to cause recursive exceptions.
Following functions are now removed from the kprobes blacklist.
can_boost
Use NOKPROBE_SYMBOL macro for protecting functions
from kprobes instead of __kprobes annotation in
dumpstack.c.
Signed-off-by: Masami Hiramatsu
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Peter Anvin"
Cc: Andrew Morton
Cc: Jiri Slaby
Cc: Tejun Heo
Cc: Vineet Gupta
---
Use NOKPROBE_SYMBOL macro to protect functions from
kprobes instead of __kprobes annotation in sched/core.c.
Signed-off-by: Masami Hiramatsu
Cc: Ingo Molnar
Cc: Peter Zijlstra
---
kernel/sched/core.c |6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git
Use NOKPROBE_SYMBOL macro to protect functions from
kprobes instead of __kprobes annotation for nmi handlers.
Signed-off-by: Masami Hiramatsu
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Peter Anvin"
Cc: Peter Zijlstra
Cc: Paul Mackerras
Cc: Arnaldo Carvalho de Melo
Cc: Michel Lespinasse
Hi Andrew,
Today's linux-next merge of the akpm tree got a conflict in
arch/sparc/mm/init_64.c between commit 37b3a8ff3e08 ("sparc64: Move from
4MB to 8MB huge pages") from the sparc-next tree and commit "sparc:
handle pgtable_page_ctor() fail" from the akpm tree.
I fixed it up (see below) and
Use NOKPROBE_SYMBOL macro to protect functions from kprobes
instead of __kprobes annotation in alternative.c.
Signed-off-by: Masami Hiramatsu
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Peter Anvin"
Cc: Jiri Kosina
Cc: Borislav Petkov
---
arch/x86/kernel/alternative.c |3 ++-
1 file
Use NOKPROBE_SYMBOL macro to protect functions from kprobes
instead of __kprobes annotation in fault.c.
This applies __always_inline annotation for some cases,
because NOKPROBE_SYMBOL() will inhibit inlining by
referring the symbol address.
Signed-off-by: Masami Hiramatsu
Cc: Thomas Gleixner
Prohibit probing on func_ptr_is_kernel_text() by adding
it to the kprobe_blacklist.
Since the func_ptr_is_kernel_text() is called from
notifier_call_chain() which is called from int3 handler,
probing it may cause double int3 fault and kernel will
reboot.
This happenes when the kernel built with
There is no need to prohibit probing on the functions
used for preparation. Those are safely probed because
those are not invoked from breakpoint/fault/debug handlers,
there is no chance to cause recursive exceptions.
Following functions are now removed from the kprobes blacklist.
There is no need to prohibit probing on the functions
used for preparation, registeration, optimization,
controll etc. Those are safely probed because those are
not invoked from breakpoint/fault/debug handlers,
there is no chance to cause recursive exceptions.
Following functions are now removed
Prohibit probing on debug_stack_reset and debug_stack_set_zero.
Since the both functions are called from TRACE_IRQS_ON/OFF_DEBUG
macros which run in int3 ist entry, probing it may cause a soft
lockup.
This happens when the kernel built with CONFIG_DYNAMIC_FTRACE=y
and CONFIG_TRACE_IRQFLAGS=y.
Use NOKPROBE_SYMBOL macro for protecting functions
from kprobes instead of __kprobes annotation in x86
kprobes code.
This applies __always_inline annotation for some cases,
because NOKPROBE_SYMBOL() will inhibit inlining by
referring the symbol address.
Signed-off-by: Masami Hiramatsu
Cc: Thomas
Show blacklist entries (function names with the address
range) via /sys/kernel/debug/kprobes/blacklist.
Signed-off-by: Masami Hiramatsu
Cc: Ananth N Mavinakayanahalli
Cc: "David S. Miller"
---
kernel/kprobes.c | 61 +++---
1 file changed, 53
Use NOKPROBE_SYMBOL macro to protect functions from
kprobes instead of __kprobes annotation in notifier.
Signed-off-by: Masami Hiramatsu
---
kernel/notifier.c | 22 +-
1 file changed, 13 insertions(+), 9 deletions(-)
diff --git a/kernel/notifier.c b/kernel/notifier.c
Use NOKPROBE_SYMBOL macro to protect functions from
kprobes instead of __kprobes annotation in ftrace.
This applies __always_inline annotation for some cases,
because NOKPROBE_SYMBOL() will inhibit inlining by
referring the symbol address.
Signed-off-by: Masami Hiramatsu
Cc: Steven Rostedt
Cc:
Use NOKPROBE_SYMBOL macro for protecting functions
from kprobes instead of __kprobes annotation in kvm.c.
This also adds kvm_read_and_reset_pf_reason in
the blacklist because it can be called before
do_page_fault.
Signed-off-by: Masami Hiramatsu
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H.
Use NOKPROBE_SYMBOL macro to protect functions from kprobes
instead of __kprobes annotation in trap.c.
This also applies __always_inline annotation for some cases,
because NOKPROBE_SYMBOL() will inhibit inlining by referring
the symbol address.
Signed-off-by: Masami Hiramatsu
Cc: Thomas Gleixner
Use NOKPROBE_SYMBOL macro to protect functions from kprobes
instead of __kprobe annotation in hw_breakpoint.
Signed-off-by: Masami Hiramatsu
Cc: Thomas Gleixner
Cc: Ingo Molnar
Cc: "H. Peter Anvin"
Cc: Andrew Morton
Cc: Oleg Nesterov
---
arch/x86/kernel/hw_breakpoint.c |6 --
1
Introduce NOKPROBE_SYMBOL() macro which builds a kprobe
blacklist in build time. The usage of this macro is similar
to the EXPORT_SYMBOL, put the NOKPROBE_SYMBOL(function); just
after the function definition.
If CONFIG_KPROBES=y, the macro is expanded to the definition
of a static data structure
Use NOKPROBE_SYMBOL macro to protect functions from
kprobes instead of __kprobes annotation.
Signed-off-by: Masami Hiramatsu
Cc: Ananth N Mavinakayanahalli
Cc: "David S. Miller"
---
kernel/kprobes.c | 67 +-
1 file changed, 41
.entry.text is a code area which is used for interrupt/syscall
entries, and there are many sensitive codes.
Thus, it is better to prohibit probing on all of such codes
instead of a part of that.
Since some symbols are already registered on kprobe blacklist,
this also removes them from the
Currently the blacklist is maintained by hand in kprobes.c
which is separated from the function definition and is hard
to catch up the kernel update.
To solve this issue, I've tried to implement new
NOKPROBE_SYMBOL() macro for making kprobe blacklist at
build time. Since the NOKPROBE_SYMBOL()
Hi Dave and all,
Please ignore this patch from this series.
On 11/15 2013 12:35 PM, Jeff Liu wrote:
> From: Jie Liu
>
> Simplify the code in lmLogSync() via list_last_entry_or_null().
>
> Signed-off-by: Jie Liu
> ---
> fs/jfs/jfs_logmgr.c | 10 +++---
> 1 file changed, 3 insertions(+),
I've been reviewing different users of tracepoints and I stumbled
across this:
drivers/gpu/host1x/cdma.c: host1x_cdma_push()
if (host1x_debug_trace_cmdbuf)
trace_host1x_cdma_push(dev_name(cdma_to_channel(cdma)->dev),
op1, op2);
That
On Wednesday 13 November 2013 01:16 AM, Eduardo Valentin wrote:
> This patch changes the cpufreq-cpu0 driver to consider if
> a cpu needs cooling (with cpufreq). In case the cooling is needed,
> the cpu0 device tree node needs to be properly configured
> with cooling device properties.
>
> In
On Friday 15 November 2013 03:30 AM, Rafael J. Wysocki wrote:
> I'm not going to apply anything like this. If I have already, that's been a
> mistake.
>
> Do not mix assignments with logical operators in such outrageous ways, please.
> That's completely unreadable and confusing.
Okay... Will
From: Jie Liu
Simplify the code in lmLogSync() via list_last_entry_or_null().
Signed-off-by: Jie Liu
---
fs/jfs/jfs_logmgr.c | 10 +++---
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/fs/jfs/jfs_logmgr.c b/fs/jfs/jfs_logmgr.c
index 7b565d0..2f90779 100644
---
From: Jie Liu
Simplify the code in lmNextPage via list_last_entry_or_null().
Signed-off-by: Jie Liu
---
fs/jfs/jfs_logmgr.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/fs/jfs/jfs_logmgr.c b/fs/jfs/jfs_logmgr.c
index 360d27c..7b565d0 100644
--- a/fs/jfs/jfs_logmgr.c
From: Jie Liu
Simplify the code in drop_last_node() via list_last_entry_or_null().
Signed-off-by: Jie Liu
---
fs/ubifs/recovery.c | 8 +++-
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/fs/ubifs/recovery.c b/fs/ubifs/recovery.c
index 065096e..e46c394 100644
---
From: Jie Liu
Simplify the code in current_tail() via list_last_entry_or_null().
Signed-off-by: Jie Liu
---
fs/gfs2/log.c | 10 +++---
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/fs/gfs2/log.c b/fs/gfs2/log.c
index 610613f..555f767 100644
--- a/fs/gfs2/log.c
+++
From: Jie Liu
Simplify xfs_ail_max() with list_last_entry_or_null(), and move it to
xfs_trans_priv.h as an inline function as now it is one line.
Signed-off-by: Jie Liu
---
fs/xfs/xfs_trans_ail.c | 14 --
fs/xfs/xfs_trans_priv.h | 12
2 files changed, 12
From: Jie Liu
Introduce a trivial helper list_last_entry_or_null() to fetch the
last entry from a list, return NULL if the list is empty.
Signed-off-by: Jie Liu
---
include/linux/list.h | 11 +++
1 file changed, 11 insertions(+)
diff --git a/include/linux/list.h
Hi Folks,
This patch is trying to introduce a new list helper to retrieve the
last entry or return NULL if the list is empty corresponding to it,
which is inspired by Jiri Pirko's list_first_entry_or_null().
With this trivial helper, we could get a little benefit to simplify
the code logic of
2013/11/7 Joe Perches :
> On Thu, 2013-11-07 at 11:11 +0800, Changbin Du wrote:
>> 2013/11/1 Joe Perches :
>> match_ functions in lib/parser.c just do simple match, they
>> doesn't support wildcards.
>> So it's not useful for us.
>
> It's not meant to be useful so much as be a possible
> generic
Doing an if statement to test some condition to know if we should
trigger a tracepoint is pointless when tracing is disabled. This just
adds overhead and wastes a branch prediction. This is why the
TRACE_EVENT_CONDITION() was created. It places the check inside the jump
label so that the branch
1 - 100 of 1412 matches
Mail list logo
