On Wed, Dec 11, 2013 at 08:31:35AM +0100, Antonio Quartulli wrote:
> Joe,
>
> we have other places in the batman-adv code where we use seq_printf, but
> at the moment we don't check the return value and we always return 0 at
> the end of the function.
>
> I think we could use seq_overflow here
On Tue, Dec 10, 2013 at 09:12:43PM -0800, Joe Perches wrote:
> diff --git a/net/batman-adv/gateway_client.c b/net/batman-adv/gateway_client.c
> index 2449afa..dfa5d2d 100644
> --- a/net/batman-adv/gateway_client.c
> +++ b/net/batman-adv/gateway_client.c
> @@ -517,29 +517,28 @@ static int
On 12/10/2013 11:51 PM, Sascha Hauer wrote:
On Mon, Dec 09, 2013 at 06:00:12PM -0800, Paul Walmsley wrote:
Treat both negative and zero return values from clk_round_rate() as
errors. This is needed since subsequent patches will convert
clk_round_rate()'s return value to be an unsigned type,
On Mon, Dec 09, 2013 at 06:00:12PM -0800, Paul Walmsley wrote:
>
> Treat both negative and zero return values from clk_round_rate() as
> errors. This is needed since subsequent patches will convert
> clk_round_rate()'s return value to be an unsigned type, rather than a
> signed type, since some
On 08.12.2013 01:49, Steven Luo wrote:
This patch causes problems with DSP codecs on OMAP3 devices running
Android -- specifically, when the decoder is cleaning up after itself,
munmap() of the mapped area fails, leading to a memory leak which
eventually crashes the system.
As far as I can
On 11/12/13 06:12, Joe Perches wrote:
> Convert the uses of the return of seq_printf to
> instead check seq_overflow to determine if a buffer
> overflow has occurred.
>
> This will eventually allow seq_printf & seq_puts to
> be converted to a void return instead of the often
> misused return that
On 12/10/2013 01:43 AM, Eric Miao wrote:
Haojian, could you help take this via your tree to arm-soc?
Applied.
Thanks
Haojian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at
Joe,
we have other places in the batman-adv code where we use seq_printf, but
at the moment we don't check the return value and we always return 0 at
the end of the function.
I think we could use seq_overflow here as well?
Thanks,
--
Antonio Quartulli
signature.asc
Description: OpenPGP
On 12/11/2013 02:31 AM, Marek Vasut wrote:
On Tuesday, December 10, 2013 at 11:48:59 AM, Daniel Mack wrote:
On 12/10/2013 09:43 AM, Haojian Zhuang wrote:
On 12/10/2013 12:39 PM, Sergei Ianovich wrote:
Erratum 71 of PXA270M Processor Family Specification Update
(April 19, 2010) explains that
Le jeudi 14 novembre 2013, 15:05:56 Srinivas Kandagatla a écrit :
> According to Documentation/devicetree/bindings/net/phy.txt device_type
> property of PHY nodes is mandatory, which should be set to
> "ethernet-phy". This patch adds check in scanning phys and only scans
> node which have
From: "Yan, Zheng"
The new HSW call stack feature provides a facility such that
unfiltered call data will be collected as normal, but as return
instructions are executed the last captured branch record is
popped from the LBR stack. Thus, branch information relative to
leaf functions will not be
From: "Yan, Zheng"
Enable pmu context switch callback if LBR is used. Use the callback
to flush LBR stack when process is scheduled in.
Signed-off-by: Yan, Zheng
---
arch/x86/kernel/cpu/perf_event.c | 7 ---
arch/x86/kernel/cpu/perf_event.h | 2 -
From: "Yan, Zheng"
The callback is invoked when process is scheduled in/out. To avoid
unnecessary overhead, the callback can be enabled/disabled.
Signed-off-by: Yan, Zheng
---
arch/x86/kernel/cpu/perf_event.c | 7 +
arch/x86/kernel/cpu/perf_event.h | 4 +++
include/linux/perf_event.h
From: "Yan, Zheng"
Haswell has a new feature that utilizes the existing Last Branch Record
facility to record call chains. When the feature is enabled, function
call will be collected as normal, but as return instructions are executed
the last captured branch record is popped from the on-chip
From: "Yan, Zheng"
The index of lbr_sel_map is bit value of perf branch_sample_type.
PERF_SAMPLE_BRANCH_MAX is 1024 at present, so each lbr_sel_map
uses 4096 bytes. If we use bit shift as index, each lbr_sel_map
only uses 40 bytes.
Signed-off-by: Yan, Zheng
---
From: "Yan, Zheng"
allow allocating PMU specific data for perf task context
Signed-off-by: Yan, Zheng
---
include/linux/perf_event.h | 5 +
kernel/events/core.c | 19 ++-
2 files changed, 23 insertions(+), 1 deletion(-)
diff --git a/include/linux/perf_event.h
From: "Yan, Zheng"
Swtich pmu specific data even if switching perf task conext is
optimized out.
Signed-off-by: Yan, Zheng
---
kernel/events/core.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/kernel/events/core.c b/kernel/events/core.c
index 6499dae..974f7c7 100644
---
From: "Yan, Zheng"
When the LBR call stack is enabled, it is necessary to save/restore
the LBR stack on context switch. The solution is saving/restoring
the LBR stack to/from task's perf event context.
Don't save/restore the LBR stack if task has no perf event context
or there are system-wide
From: "Yan, Zheng"
So far only CPU pmu uses context switch callback and it is the
first registered pmu. So we can avoid iterating the whole pmu list.
Signed-off-by: Yan, Zheng
---
kernel/events/core.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git
Le mercredi 13 novembre 2013, 15:07:49 Jonas Jensen a écrit :
> Add MDIO bus node segment and update the example,
> allowing trivial bindings to break out boilerplate.
>
> Signed-off-by: Jonas Jensen
> ---
>
> Notes:
> Thanks Mark,
>
> This should have the changes from your comments.
Change log from v1:
o fix a mistake
>From 59df095b83e601b1b317ce97a5f767a8c3303902 Mon Sep 17 00:00:00 2001
From: Jaegeuk Kim
Date: Wed, 11 Dec 2013 14:29:39 +0900
Subject: [PATCH] f2fs: fix the location of tracepoint
Cc: linux-fsde...@vger.kernel.org, linux-kernel@vger.kernel.org,
From: "Yan, Zheng"
Allocate memory for perf task context, use the memory to save
LBR stack.
Signed-off-by: Yan, Zheng
---
arch/x86/kernel/cpu/perf_event.c | 1 +
arch/x86/kernel/cpu/perf_event.h | 7 +++
2 files changed, 8 insertions(+)
diff --git a/arch/x86/kernel/cpu/perf_event.c
From: "Yan, Zheng"
use event->attr.branch_sample_type to check if branch stack is needed.
Signed-off-by: Yan, Zheng
---
arch/x86/kernel/cpu/perf_event_intel.c | 20 +++-
include/linux/perf_event.h | 5 +
kernel/events/core.c | 11 +++
From: "Yan, Zheng"
New Intel CPU can record call chains by using existing last branch
record facility. perf_callchain_user() can make use of the call
chains recorded by hardware in case of there is no frame pointer.
Signed-off-by: Yan, Zheng
---
arch/arm/kernel/perf_event.c | 4 ++--
From: "Yan, Zheng"
Try utilizing the LBR call stack to get user callchain in case of
there is no frame pointer
Signed-off-by: Yan, Zheng
---
arch/x86/kernel/cpu/perf_event.c | 33 ++
arch/x86/kernel/cpu/perf_event_intel.c | 12 ++-
From: "Yan, Zheng"
"Zero length call" uses the attribute of the call instruction to push
the immediate instruction pointer on to the stack and then pops off
that address into a register. This is accomplished without any matching
return instruction. It confuses the hardware and make the recorded
From: "Yan, Zheng"
Try enabling the LBR call stack feature if event requests recording
callchain. Also adds a cpu pmu attribute to enable/disable this
feature (disabled by default).
Signed-off-by: Yan, Zheng
---
arch/x86/kernel/cpu/perf_event.c | 94 +++-
From: "Yan, Zheng"
Signed-off-by: Yan, Zheng
---
arch/x86/kernel/cpu/perf_event.h | 1 +
arch/x86/kernel/cpu/perf_event_intel_lbr.c | 25 -
2 files changed, 25 insertions(+), 1 deletion(-)
diff --git a/arch/x86/kernel/cpu/perf_event.h
Oops,
Please ignore the second patch.
Sorry for the noise.
2013-12-11 (수), 15:05 +0900, Jaegeuk Kim:
> We need to get a trace before submit_bio, since its bi_sector is remapped
> during
> the submit_bio.
>
> Signed-off-by: Jaegeuk Kim
> ---
> fs/f2fs/data.c | 4 ++--
> 1 file changed, 2
Op 10 dec. 2013, om 15:13 heeft Pantelis Antoniou
het volgende geschreven:
> When booting platforms that do very early OF initialization before
> core_initcalls are performed of_init is called too late.
>
> This results in a hard-hard without getting a chance to output anything.
I think that
On 12/11/2013 03:56 AM, David Miller wrote:
From: David Miller
Date: Tue, 10 Dec 2013 21:52:53 -0500 (EST)
This series looks good, applied to net-next, thanks.
Actually, I had to revert.
You cannot use late_initcall_sync() from code that is potentially
built modular, this caused my build
On 12/08/2013 07:36 PM, John Stultz wrote:
> On 12/08/2013 03:45 PM, Sasha Levin wrote:
>> Hi all,
>>
>> I've stumbled on this spew inside a KVM tools guest running latest
>> -next kernel:
>>
>>
>> [ 251.100221] ==
>> [ 251.100221] [ INFO:
On 2013-12-10 21:03, Grant Likely wrote:
[...]
>> +/* Parked Address in ACPI GIC structure can be used as cpu release addr */
>> +int acpi_get_parked_address_with_gic_id(u32 gic_id, u64 *parked_address)
>> +{
>> +struct acpi_table_header *table_header = NULL;
>> +struct
On Thu, Nov 14, 2013 at 10:50:29PM +0100, Rafael J. Wysocki wrote:
> On Thursday, November 14, 2013 08:43:27 AM Shuah Khan wrote:
> > On 11/07/2013 05:03 PM, Shuah Khan wrote:
> > > Add Legacy PM OPS usage checks to class, bus, and driver register
> > > functions.
> > > If Legacy PM OPS usage is
> -Original Message-
> From: Neil Zhang [mailto:zhan...@marvell.com]
> Sent: 2013年12月11日 14:18
> To: ba...@ti.com; gre...@linuxfoundation.org
> Cc: linux-...@vger.kernel.org; linux-kernel@vger.kernel.org; Neil Zhang
> Subject: [PATCH] usb: phy: initilize the notifier when add a new phy
>
We need to initilize the notifer before use it.
So lets initilize it when add a new phy device to reduce the code
redundance.
Signed-off-by: Neil Zhang
---
drivers/usb/phy/phy-ab8500-usb.c|2 --
drivers/usb/phy/phy-generic.c |1 -
drivers/usb/phy/phy-gpio-vbus-usb.c |2 --
On Tue, 2013-12-10 at 19:31 +0100, Daniel Lezcano wrote:
> I think I am a bit puzzled with the 'idle_avg' name. I am guessing the
> semantic of this variable is "how long this cpu has been idle".
Average distance between idles.
> The idle duration, with the no_hz, could be long, several
On Sun, Nov 03, 2013 at 08:33:08PM +0100, Sebastian Andrzej Siewior wrote:
> I run a couple times into the case where "put" was called on an already
> cleanup object. The results was either nothing because "0" went back to
> 0xff…ff and release was not called a second time or some thing like this
Hi,
On Tue, Dec 10, 2013 at 7:31 PM, Heikki Krogerus
wrote:
> Hi,
Thanks for reviewing this.
>
> On Tue, Dec 10, 2013 at 04:25:23PM +0530, Vivek Gautam wrote:
>> Some PHY controllers may need to tune PHY post-initialization,
>> so that the PHY consumers can call phy-tuning at appropriate
>>
Somethings I compiled the Linux network modules (especially bridge and
netfilter)
without optimization, the kernel always crashes because of exhausted kernel
stack.
The similar problem has been discussed in
http://lists.linuxfoundation.org/pipermail/bridge/2005-January/004402.html
Is it OK the
Hi Frederic,
Sorry for idiot of nohz_full. When we using this feature on my mobile
devices, we found this feature keep cpu0 in periodic tick mode. then the
timer interrupt on cpu0 is very higher than normal nohz mode.
that cause high power consuming cost.
I found you have mention this on commit:
On 2013年12月10日 09:01, David Miller wrote:
From: fre...@asix.com.tw
Date: Fri, 6 Dec 2013 17:58:18 +0800
From: Freddy Xin
The AX88179_178A has a hardware feature that it can insert a 2-bytes pseudo
header in front of each received frame by setting the AX_RX_CTL_IPE bit.
This feature is used
On 12/11/2013 03:13 AM, Glauber Costa wrote:
> On Tue, Dec 10, 2013 at 5:59 PM, Vladimir Davydov
> wrote:
>> Hi,
>>
>> Looking through the per-memcg kmem_cache initialization code, I have a
>> bad feeling that it is prone to a race. Before getting to fixing it, I'd
>> like to ensure this race is
> -Original Message-
> From: Neil Zhang [mailto:zhan...@marvell.com]
> Sent: 2013年12月11日 13:05
> To: ba...@ti.com; gre...@linuxfoundation.org
> Cc: linux-...@vger.kernel.org; linux-kernel@vger.kernel.org; Neil Zhang
> Subject: [PATCH] usb: phy: Initilize the spinlock in notifier
>
> We
We need to initilize the notifer before use it.
So lets initilize it when add a new phy device to reduce the code
redundance.
Signed-off-by: Neil Zhang
---
drivers/usb/phy/phy-ab8500-usb.c|2 --
drivers/usb/phy/phy-generic.c |1 -
drivers/usb/phy/phy-gpio-vbus-usb.c |2 --
Signed-off-by: Libo Chen
---
security/integrity/ima/ima_api.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/security/integrity/ima/ima_api.c b/security/integrity/ima/ima_api.c
index c38bbce..6d76d4a 100644
--- a/security/integrity/ima/ima_api.c
+++
Signed-off-by: Libo Chen
---
include/linux/fs.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/fs.h b/include/linux/fs.h
index 121f11f..a146e2a 100644
--- a/include/linux/fs.h
+++ b/include/linux/fs.h
@@ -2780,7 +2780,7 @@ static inline bool dir_emit(struct
We need to get a trace before submit_bio, since its bi_sector is remapped during
the submit_bio.
Signed-off-by: Jaegeuk Kim
---
fs/f2fs/data.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
index ebc9177..969df55 100644
---
This patch introduces f2fs_io_info to mitigate the complex parameter list.
struct f2fs_io_info {
enum page_type type;/* contains DATA/NODE/META/META_FLUSH */
int rw; /* contains R/RS/W/WS */
int rw_flag;/* contains
Howard Chu wrote:
Howard Chu wrote:
Li Zefan wrote:
On 2013/12/11 10:59, Howard Chu wrote:
I just upgraded a system from a 3.5 kernel to 3.12.3 and attempted to run some
new benchmarks on it. I see my test program ramps up in CPU usage for a few
seconds and then it gradually tails off.
Signed-off-by: Libo Chen
---
fs/btrfs/ioctl.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
- just change style
diff --git a/fs/btrfs/ioctl.c b/fs/btrfs/ioctl.c
index a111622..fdfc0d7 100644
--- a/fs/btrfs/ioctl.c
+++ b/fs/btrfs/ioctl.c
@@ -2690,7 +2690,7 @@ static long
enum dma_transfer_direction is currently used
in struct dma_slave_config, so update the comment
Signed-off-by: Alexander Popov
---
include/linux/dmaengine.h | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/include/linux/dmaengine.h b/include/linux/dmaengine.h
index
Signed-off-by: Libo Chen f_dentry->d_inode;
+ struct inode *src = file_inode(file);
struct file *dst_file = NULL;
struct inode *dst;
u64 off;
@@ -2779,7 +2779,7 @@ static long btrfs_ioctl_file_extent_same(struct file
*file,
if (file->f_path.mnt !=
On 2013/12/11 7:02, Andrew Morton wrote:
> On Mon, 9 Dec 2013 20:11:54 +0800 Rui Xiang wrote:
>
>> While getting proc task error, it shoule return -ESRCH.
>>
>> ...
>>
>> --- a/fs/proc/base.c
>> +++ b/fs/proc/base.c
>> @@ -174,9 +174,10 @@ static int get_task_root(struct task_struct *task,
>>
Applied, thanks!
On Wed, 11 Dec 2013, Libo Chen wrote:
>
> Signed-off-by: Libo Chen
> ---
> fs/ceph/file.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/ceph/file.c b/fs/ceph/file.c
> index 3de8982..7549bd6 100644
> --- a/fs/ceph/file.c
> +++ b/fs/ceph/file.c
>
Signed-off-by: Libo Chen
---
fs/ceph/file.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/ceph/file.c b/fs/ceph/file.c
index 3de8982..7549bd6 100644
--- a/fs/ceph/file.c
+++ b/fs/ceph/file.c
@@ -1018,7 +1018,7 @@ static long ceph_fallocate(struct file *file, int mode,
On Wed, 2013-12-11 at 06:11 +0100, Arnd Bergmann wrote:
> It depends: if the driver is for the entire FPGA and does
> the irqchip stuff in addition, it should probably live
> in drivers/mfd. If it's a pure irqchip driver, drivers/irqchip
> is better. You have to be careful in the second case
On Tue, Dec 10, 2013 at 09:50:04AM -0500, Tejun Heo wrote:
> Hello,
>
> Yeap, I was planning to send this out earlier but was completely
> passed out yesterday after the first snowboarding in over a decade. :)
>
> The offending commit a8b14744429f isn't applicable to
> driver-core-next. This
Is that an actual requirement of the API?
Xishi Qiu wrote:
>On 2013/12/11 12:02, H. Peter Anvin wrote:
>
>> On 12/10/2013 07:55 PM, Xishi Qiu wrote:
>>>
>>> I think there is a problem.
>>> e.g.
>>> [start, end)=[8, 12), and [A, B)=[0, 10), [B, C)=[10,20),
>>> then e820_all_mapped() will return
On Tuesday 10 December 2013, Grant Likely wrote:
> > --- a/drivers/acpi/plat/arm-core.c
> > +++ b/drivers/acpi/plat/arm-core.c
> > @@ -90,7 +90,7 @@ enum acpi_irq_model_id acpi_irq_model =
> > ACPI_IRQ_MODEL_GIC;
> >
> > static unsigned int gsi_to_irq(unsigned int gsi)
> > {
> > - int irq
From: Joe Perches
Date: Tue, 10 Dec 2013 21:12:41 -0800
> Many uses of the return value of seq_printf/seq_puts/seq_putc are
> incorrect. Many assume that the return value is the number of
> chars emitted into a buffer like printf/puts/putc.
>
> It would be better to make the return value of
Convert the uses of the return of seq_printf to
instead check seq_overflow to determine if a buffer
overflow has occurred.
This will eventually allow seq_printf & seq_puts to
be converted to a void return instead of the often
misused return that is often assumed to be an int for
the number of
Many uses of the return value of seq_printf/seq_puts/seq_putc are
incorrect. Many assume that the return value is the number of
chars emitted into a buffer like printf/puts/putc.
It would be better to make the return value of these functions void
to avoid these misuses.
Start to do so.
Convert
seq_printf and seq_puts returns are often misused.
Instead of checking the seq_printf or seq_puts return,
add a new seq_overflow function to test if a seq_file has
overflowed the available buffer space.
This will eventually allow seq_printf and seq_puts to be
converted to have a void return
Convert the uses of the return of seq_printf/seq_puts/seq_putc to
use check seq_overflow to determine if a buffer overflow has occurred.
This will eventually allow seq_printf & seq_puts to be converted to a
void return instead of the often misused return that is often assumed
to be an int for the
On Wednesday 11 December 2013, Sergei Ianovich wrote:
> > It probably makes sense to have a single driver file for the
> > FPGA device that does this, and only split out the other devices
> > from it that consume the irqs.
>
> Is drivers/irqchip/ the right place this driver?
>
> I am asking
Commit 6680ec68eff47d36f67b4351bc9836fd6cba9532
(tuntap: hardware vlan tx support) breaks the truncated packet signal by nev
return a length greater than iov length in tun_put_user(). This patch fixes
by always return the length of packet plus possible vlan header. Caller can
detect the truncated
macvtap_put_user() never return a value grater than iov length, this in fact
bypasses the truncated checking in macvtap_recvmsg(). Fix this by always
returning the size of packet plus the possible vlan header to let the trunca
checking work.
Cc: Vlad Yasevich
Cc: Zhi Yong Wu
Cc: Michael S.
We need to initilize every spinlock before use it.
So lets initilize the spinlock in notifier when add a new phy device.
Signed-off-by: Neil Zhang
---
drivers/usb/phy/phy.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/usb/phy/phy.c b/drivers/usb/phy/phy.c
index
Hi Daniel, Dinh,
On Tue, Dec 10, 2013 at 08:16:12PM +0100, Daniel Lezcano wrote:
> From: Dinh Nguyen
>
> In commit 620f5e1cbf (dts: Rename DW APB timer compatible strings), both
> "snps,dw-apb-timer-sp" and "snps,dw-apb-timer-osc" were deprecated in place
> of "snps,dw-apb-timer". But the
On Wed, 11 Dec 2013 15:56:48 +1100 Stephen Rothwell
wrote:
>
> Non-merge commits (relative to Linus' tree): 3141
> 3634 files changed, 143080 insertions(+), 83685 deletions(-)
Actually:
Non-merge commits (relative to Linus' tree): 3236
3756 files changed, 155884 insertions(+), 86331
When adjusting the link speed, the target frequency is determined by a
'swith (LINK_SPEED)' statement, that assigns the target rate only for
valid and expected LINK_SPEED values. This incomplete switch statement
leads to the following build warning:
drivers/net/ethernet/cadence/macb.c: In
Howard Chu wrote:
Li Zefan wrote:
On 2013/12/11 10:59, Howard Chu wrote:
I just upgraded a system from a 3.5 kernel to 3.12.3 and attempted to run some
new benchmarks on it. I see my test program ramps up in CPU usage for a few
seconds and then it gradually tails off. There's nothing obvious
Hi all,
Changes since 20131210:
Removed tree: arm-v7-cache-opt (merged into the arm tree)
The powerpc tree still had its build failure for which I applied a
supplied patch.
The sound-asoc tree lost its build failure.
The usb-gadget tree still has its build failure so I used the version from
As we need a const char * as a result of get_filename_for_perf_kvm(),
There is no need to use strdup() for the return value.
This patch drop the strdup() to save memory in get_filename_for_perf_kvm().
Signed-off-by: Dongsheng Yang
---
tools/perf/builtin-kvm.c | 8 +---
Command perf kvm diff is used to diff perf.data.host and
perf.data.guest by default currently. But it is not a good
default behavior.
Example:
# perf kvm --guestkallsyms /home/kallsyms --guestmodules /home/modules
record -a sleep 1
[ perf record: Woken up 1 times to write data ]
On 2013/12/11 12:02, H. Peter Anvin wrote:
> On 12/10/2013 07:55 PM, Xishi Qiu wrote:
>>
>> I think there is a problem.
>> e.g.
>> [start, end)=[8, 12), and [A, B)=[0, 10), [B, C)=[10,20),
>> then e820_all_mapped() will return 1, it spans two regions.
>>
>
> Why is that a problem?
>
[start,
Greetings
I am a US Army officer currently on military assignment in Iraq, I humbly ask
of your assistance secure and invest some money for me in your country. As a
matter of fact, I have the sum of US$5 Million which I would like you to help
me to invest.
I will appreciate it if you can
Commit 1902efe7f for the new comm infra added the wrong check for return
code on thread__set_comm. err == 0 is normal, so don't return at that
point unless err != 0.
Signed-off-by: David Ahern
Cc: Frederic Weisbecker
---
tools/perf/util/thread.c | 2 +-
1 file changed, 1 insertion(+), 1
On Tue, 2013-12-10 at 22:57 +0100, Arnd Bergmann wrote:
> It should be possible to make it a loadable module, with deferred
> probing etc. You wouldn't use IRQCHIP_DECLARE() for this though,
> but instead have a platform driver that sets up the irq domain.
Thanks for explaning.
> It probably
On Tue, Dec 10, 2013 at 06:45:54PM -0800, Kees Cook wrote:
> On Tue, Dec 10, 2013 at 6:00 PM, Dave Chinner wrote:
> > On Tue, Dec 10, 2013 at 08:10:51PM -0500, Josh Boyer wrote:
> >> On Tue, Dec 10, 2013 at 8:03 PM, Dave Chinner wrote:
> >> > Security processes are not something that should be
On Tue, Dec 10, 2013 at 02:48:52PM -0800, Linus Torvalds wrote:
> Dave, can you re-create that trinity run and test that patch?
Looks ok so far, but I'll leave it run overnight to be sure.
Dave
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of
On 12/10/2013 07:55 PM, Xishi Qiu wrote:
>
> I think there is a problem.
> e.g.
> [start, end)=[8, 12), and [A, B)=[0, 10), [B, C)=[10,20),
> then e820_all_mapped() will return 1, it spans two regions.
>
Why is that a problem?
-hpa
--
To unsubscribe from this list: send the line
On 2013/12/11 10:55, H. Peter Anvin wrote:
> On 12/10/2013 05:35 PM, Xishi Qiu wrote:
>>
>> In this case, old code is right, but I discuss in another one that
>> you wrote above.
>>
>
> So is there a problem or not? I have lost track...
>
I think there is a problem.
e.g.
[start, end)=[8, 12),
From: Soren Brinkmann
Date: Tue, 10 Dec 2013 16:07:18 -0800
> I'd really like to have Ethernet working for Zynq, so I want to at least
> revive this discussion regarding this patchset. And the first four
> patches should not even be too controversial.
> I didn't change anything compared to my
On 12/10/2013 10:38 PM, David Ahern wrote:
On 12/11/13, 9:30 AM, Dongsheng Yang wrote:
@@ -1001,8 +1002,28 @@ static int data_init(int argc, const char **argv)
use_default = false;
}
} else if (perf_guest) {
-defaults[0] = "perf.data.host";
-
On 12/11/13, 9:30 AM, Dongsheng Yang wrote:
@@ -1001,8 +1002,28 @@ static int data_init(int argc, const char **argv)
use_default = false;
}
} else if (perf_guest) {
- defaults[0] = "perf.data.host";
- defaults[1] =
(2013/12/11 11:32), Alexei Starovoitov wrote:
> On Tue, Dec 10, 2013 at 7:47 AM, Ingo Molnar wrote:
>>
>> * Alexei Starovoitov wrote:
>>
I'm fine if it becomes a requirement to have a vmlinux built with
DEBUG_INFO to use BPF and have a tool like perf to translate the
filters. But
Command perf kvm diff is used to diff perf.data.host and
perf.data.guest by default currently. But it is not a good
default behavior.
Example:
# perf kvm --guestkallsyms /home/kallsyms --guestmodules /home/modules
record -a sleep 1
[ perf record: Woken up 1 times to write data ]
On 12/10/2013 11:29 PM, Michael S. Tsirkin wrote:
> On Tue, Dec 10, 2013 at 01:49:46PM +0800, Jason Wang wrote:
>> > macvtap_put_user() never return a value grater than iov length, this in
>> > fact
>> > bypasses the truncated checking in macvtap_recvmsg(). Fix this by always
>> > returning the
On 12/10/2013 11:32 PM, Michael S. Tsirkin wrote:
> On Tue, Dec 10, 2013 at 01:49:45PM +0800, Jason Wang wrote:
>> > Commit 6680ec68eff47d36f67b4351bc9836fd6cba9532
>> > (tuntap: hardware vlan tx support) breaks the truncated packet signal by
>> > nev
>> > return a length greater than iov length
On 12/11/2013 11:11 AM, David Miller wrote:
> You and Michael are still discussing these changes it seems.
>
> I accidently commited the first version of these patches, but then
> immediately reverted that after I saw the followups.
>
> Let me know when you have something both of you are happy
From: Yann Droneaud
Date: Mon, 9 Dec 2013 22:42:20 +0100
> This patch makes socketpair() use error paths which do not
> rely on heavy-weight call to sys_close(): it's better to try
> to push the file descriptor to userspace before installing
> the socket file to the file descriptor, so that
On Wed, Dec 11, 2013 at 11:19 AM, David Miller wrote:
> From: Zhi Yong Wu
> Date: Wed, 11 Dec 2013 11:14:04 +0800
>
>> Only one reminder, since David has committed the two patches, you
>> maybe need to take their impact on your patches into account.
>
> I reverted these changes from net-next.
So
From: Michael Chan
Date: Tue, 10 Dec 2013 10:49:39 -0800
> On Tue, 2013-12-10 at 13:43 -0500, David Miller wrote:
>
>> What if the kernel is booted via kexec, and the driver in the kernel
>> we are kexec'ing from left indirect access enabled in MISC_HOST_CTRL?
>
> That should be ok. The
On Wed, Dec 11, 2013 at 10:32 AM, Greg Kroah-Hartman
wrote:
> On Mon, Dec 09, 2013 at 10:56:58PM +0800, Peng Tao wrote:
>> From: Lai Siyao
>>
>> Mnt root dentry will never be revalidated, but its d_op->d_compare
>> will be called for its children, to simplify code, we use the same
>> ll_d_ops as
Command perf kvm diff is used to diff perf.data.host and
perf.data.guest by default currently. But it is not a good
default behavior.
Example:
# perf kvm --guestkallsyms /home/kallsyms --guestmodules /home/modules
record -a sleep 1
[ perf record: Woken up 1 times to write data ]
Alakazam..
pk cor CPU%c0 GHz TSC SMI%c1%c3%c6 CTMP %pc3 %pc6
0.17 2.01 2.26 0 0.02 99.82 0.00 49 99.55 0.00
0 0 0 0.95 1.45 2.26 2 0.43 98.62 0.00 48 98.48 0.00
1 0 8 0.24 1.99 2.26 2 0.02 99.75 0.00 38 99.68
On Mon, Dec 9, 2013 at 7:01 PM, Octavian Purdila wrote:
> On Thu, Dec 5, 2013 at 4:02 AM, Arve Hjønnevåg wrote:
>> On Wed, Dec 4, 2013 at 2:02 PM, Greg KH wrote:
>>> On Wed, Dec 04, 2013 at 01:55:34PM -0800, Colin Cross wrote:
On Wed, Dec 4, 2013 at 1:43 PM, Greg KH wrote:
> On Wed,
From: Zhi Yong Wu
Date: Wed, 11 Dec 2013 11:14:04 +0800
> Only one reminder, since David has committed the two patches, you
> maybe need to take their impact on your patches into account.
I reverted these changes from net-next.
--
To unsubscribe from this list: send the line "unsubscribe
1 - 100 of 2256 matches
Mail list logo