Re: [PATCH] sched: remove useless code in yield_to

2012-08-09 Thread Michael Wang
On 07/03/2012 02:34 PM, Michael Wang wrote: From: Michael Wang wang...@linux.vnet.ibm.com it's impossible to enter else branch if we have set skip_clock_update in task_yield_fair(), as yield_to_task_fair() will directly return true after invoke task_yield_fair(). Could I get some conclusion

Re: [RFC][PATCH] tracepoints: Move the work out of line from hotpath sections

2012-08-09 Thread Steven Rostedt
On Thu, 2012-08-09 at 22:05 -0400, Mathieu Desnoyers wrote: In your example, is the attribute attached to the if() or the following basic block ? Attaching it to the basic block allows a nice level of genericity: Yes, I meant the block, not the if(). Heck, any block could be done this way,

Re: [PATCH] sched: remove useless code in yield_to

2012-08-09 Thread Michael Wang
On 08/10/2012 11:05 AM, Michael Wang wrote: On 07/03/2012 02:34 PM, Michael Wang wrote: From: Michael Wang wang...@linux.vnet.ibm.com it's impossible to enter else branch if we have set skip_clock_update in task_yield_fair(), as yield_to_task_fair() will directly return true after invoke

Re: [PATCH 1/7] perf hists: Separate out hist print functions

2012-08-09 Thread Namhyung Kim
Hi, Arnaldo On Thu, 9 Aug 2012 16:18:27 -0300, Arnaldo Carvalho de Melo wrote: Em Mon, Aug 06, 2012 at 05:57:36PM +0900, Namhyung Kim escreveu: From: Namhyung Kim namhyung@lge.com Separate out those functions into ui/hist.c. This is required for upcoming changes. Isn't it better to

3.5.1 ext4_ sleeping while atomic bug.

2012-08-09 Thread Dave Jones
BUG: sleeping function called from invalid context at include/linux/buffer_head.h:333 in_atomic(): 1, irqs_disabled(): 0, pid: 9894, name: fstest 3 locks held by fstest/9894: #0: (type-i_mutex_dir_key#4/1){+.+.+.}, at: [811d5dae] kern_path_create+0x7e/0x140 #1:

[PATCH v6 0/3] KVM: perf: kvm events analysis tool

2012-08-09 Thread Dong Hao
From: Xiao Guangrongxiaoguangr...@linux.vnet.ibm.com This patchset introduces a perf-based tool (perf kvm stat record/report) which can analysis kvm events more smartly. This is a presentation on 2012 Japan LinuxCon: http://events.linuxfoundation.org/images/stories/pdf/lcjp2012_guangrong.pdf You

[PATCH v6 2/3] KVM: x86: tracemmio begin and complete

2012-08-09 Thread Dong Hao
From: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com 'perf kvm stat record/report' will use kvm_exit and kvm_mmio(read...) to calculate mmio read emulated time for the old kernel, in order to trace mmio read event more exactly, we add kvm_mmio_begin to trace the time when mmio read begins, also,

[PATCH v6 1/3] KVM: x86: export svm/vmx exit code and vector code to userspace

2012-08-09 Thread Dong Hao
From: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com They will be needed by 'perf kvm stat' [ Dong Hao haod...@linux.vnet.ibm.com: rebase it on current kvm/tip tree] Signed-off-by: Xiao Guangrongxiaoguangr...@linux.vnet.ibm.com Signed-off-by: Dong Hao haod...@linux.vnet.ibm.com ---

[PATCH v6 3/3] KVM: perf kvm events analysis tool

2012-08-09 Thread Dong Hao
From: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com Add 'perf kvm stat' support to analyze kvm vmexit/mmio/ioport smartly. Usage: - kvm stat run a command and gather performance counter statistics, it is the alias of perf stat - trace kvm events: perf kvm stat record, or, if other

Re: [PATCH v5 03/12] KVM: introduce gfn_to_pfn_memslot_atomic

2012-08-09 Thread Xiao Guangrong
On 08/10/2012 02:50 AM, Marcelo Tosatti wrote: On Tue, Aug 07, 2012 at 05:49:36PM +0800, Xiao Guangrong wrote: It can instead of hva_to_pfn_atomic Signed-off-by: Xiao Guangrong xiaoguangr...@linux.vnet.ibm.com --- arch/x86/kvm/mmu.c |5 + include/linux/kvm_host.h |3 ++-

[PATCH][SCSI] remove the queue unlock in scsi_requset_fn

2012-08-09 Thread Chanho Min
We don't need to unlock the queue before put_device in scsi_request_fn() If we trigger the -remove() function, It occur a oops from the caller. So sdev reference count should not be dropped to zero here. Also It was added before scsi_device_dev_release() was moved to user context, so it is

Re: [Fwd: [PATCH] x86/smp: Fix cpuN startup panic]

2012-08-09 Thread H. Peter Anvin
On 08/09/2012 05:35 PM, Yanmin Zhang wrote: Peter, What's your opinion about the patch? We hit it when enabling Medfield Android mobile. This patch would put AP to a loop. Another method to fix it is just to enlarge the wait time, for example, from 2HZ to 10HZ. Hmm... and I presume

[RFC][PATCH 0/4] tracing/perf: Use helper functions to help shrink kernel size

2012-08-09 Thread Steven Rostedt
This patch set is another effort to bring down the size of trace events. Helper functions are used to remove duplicate code that is created for each event by the TRACE_EVENT() macros. With a config that enables 707 events in the core kernel, the size is brought down 91,774 bytes! Steven Rostedt

[RFC][PATCH 1/4] tracing: Move print code from macro to standalone function

2012-08-09 Thread Steven Rostedt
From: Steven Rostedt srost...@redhat.com The code for trace events to format the raw recorded event data into human readable format in the 'trace' file is repeated for every event in the system. When you have over 700 events, this can add up quite a bit. By making helper functions in the core

[RFC][PATCH 3/4] tracing: Use helper functions in event assignment to shrink macro size

2012-08-09 Thread Steven Rostedt
From: Steven Rostedt srost...@redhat.com The functions that assign the contents for the ftrace events are defined by the TRACE_EVENT() macros. Each event has its own unique way to assign data to its buffer. When you have over 700 events, that means there's 700 functions assigning data uniquely

[RFC][PATCH 2/4] tracing: Move event storage for array from macro to standalone function

2012-08-09 Thread Steven Rostedt
From: Steven Rostedt srost...@redhat.com The code that shows array fields for events is defined for all events. This can add up quite a bit when you have over 700 events. By making helper functions in the core kernel to do the work instead, we can shrink the size of the kernel down a little.

[RFC][PATCH 4/4] perf/events: Use helper functions in event assignment to shrink macro size

2012-08-09 Thread Steven Rostedt
From: Steven Rostedt srost...@redhat.com The functions that assign the contents for the perf software events are defined by the TRACE_EVENT() macros. Each event has its own unique way to assign data to its buffer. When you have over 700 events, that means there's 700 functions assigning data

Re: [Regression] x86-64/efi: Use EFI to deal with platform wall clock prevents my machine from booting

2012-08-09 Thread Jérôme Carretero
FYI I tried to contact support, here's their answer: Dear Valued Customer, Thank you for contacting ASUS Customer Service. My name is Carter and it's my pleasure to help you with your problem. I am afraid to say this board doesn't support linux,so we can't asure the compability. It is

[tip:x86/fpu] x86, fpu: fix build issues with CONFIG_IA32_EMULATION, CONFIG_X86_X32_ABI

2012-08-09 Thread tip-bot for Suresh Siddha
Commit-ID: b2104ba4a037e296901e73a0d396826c13625bb0 Gitweb: http://git.kernel.org/tip/b2104ba4a037e296901e73a0d396826c13625bb0 Author: Suresh Siddha suresh.b.sid...@intel.com AuthorDate: Thu, 9 Aug 2012 13:38:56 -0700 Committer: H. Peter Anvin h...@linux.intel.com CommitDate: Thu, 9 Aug

Re: [PATCH] ipv4: tcp: unicast_sock should not land outside of TCP stack

2012-08-09 Thread David Miller
From: Eric Dumazet eric.duma...@gmail.com Date: Fri, 10 Aug 2012 01:56:06 +0200 From: Eric Dumazet eduma...@google.com commit be9f4a44e7d41cee (ipv4: tcp: remove per net tcp_sock) added a selinux regression, reported and bisected by John Stultz selinux_ip_postroute_compat() expect to find

Re: [PATCH v2] staging: gdm72xx: fix reference counting in gdm_wimax_event_init

2012-08-09 Thread devendra.aaru
On Wed, Jul 25, 2012 at 7:23 PM, Ben Chan benc...@chromium.org wrote: Hi Devendra, Thanks for cleaning up the driver. If I understand the code correctly, the original author wanted to initialize wm_event once and reuse it for multiple devices, and thus reference counted it with ref_cnt.

Re: [RFC PATCH v2 10/16] ACPIHP: system device hotplug driver skeleton

2012-08-09 Thread Tang Chen
On 08/09/2012 05:36 PM, Jiang Liu wrote: And I just tried it some more times. It just hung up, but dmesg had no output. Like this: # modprobe acpihp_enum (OK, and sysfs interfaces have been created) # modprobe acpihp_drv (hang up) # dmesg (nothing) The modprobe acpihp_drv process's

Re: [PATCH] [PATCH V4]Extcon: adc_jack: adc-jack driver to support 3.5 pi or simliar devices

2012-08-09 Thread MyungJoo Ham
From: anish kumar anish198519851...@gmail.com External connector devices that decides connection information based on ADC values may use adc-jack device driver. The user simply needs to provide a table of adc range and connection states. Then, extcon framework will automatically notify

[PATCH 1/2] pwm: vt8500: Fix coding style issue

2012-08-09 Thread Sachin Kamat
Fixes the following: WARNING: Prefer pr_warn(... to pr_warning(... pr_warning(Waiting for status bits 0x%x to clear timed out\n, Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/pwm/pwm-vt8500.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

[PATCH 2/2] pwm: core: Fix coding style issues

2012-08-09 Thread Sachin Kamat
Fixes the following: WARNING: line over 80 characters ERROR: spaces required around that ':' (ctx:VxW) WARNING: Prefer pr_warn(... to pr_warning(... Signed-off-by: Sachin Kamat sachin.ka...@linaro.org --- drivers/pwm/core.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff

Re: [PATCH] [PATCH V4]Extcon: adc_jack: adc-jack driver to support 3.5 pi or simliar devices

2012-08-09 Thread anish kumar
On Fri, 2012-08-10 at 04:39 +, MyungJoo Ham wrote: From: anish kumar anish198519851...@gmail.com External connector devices that decides connection information based on ADC values may use adc-jack device driver. The user simply needs to provide a table of adc range and connection

Re: [PATCH 1/2] lp8727_charger: free_irq when lp8727_register_psy fail

2012-08-09 Thread Devendra Naga
Sorry to ask this again, But will anybody please ACK , NACK or comment on this patch? Thanks, Devendra. On Sun, Jul 29, 2012 at 11:16 PM, Devendra Naga develkernel412...@gmail.com wrote: actually the driver does a request_threaded_irq and after this it calls lp8727_register_psy, and if it

Re: thermal patches in linux-next

2012-08-09 Thread Amit Kachhap
On 10 August 2012 08:14, Zhang Rui rui.zh...@intel.com wrote: On 五, 2012-08-10 at 12:23 +1000, Stephen Rothwell wrote: Hi Rui, On Fri, 10 Aug 2012 09:41:06 +0800 Zhang Rui rui.zh...@intel.com wrote: And could you please drop these commits ef25a0fe0087963c1611c1c8903886fbea053f76

Re: [PATCH] Intel xhci: Only switch the switchable ports

2012-08-09 Thread Keng-Yu Lin
On Fri, Aug 10, 2012 at 3:38 AM, Sarah Sharp sarah.a.sh...@linux.intel.com wrote: On Fri, Aug 10, 2012 at 12:13:19AM +0800, Keng-Yu Lin wrote: On Thu, Aug 9, 2012 at 10:24 PM, Sarah Sharp sarah.a.sh...@linux.intel.com wrote: On Thu, Aug 09, 2012 at 05:31:51PM +0800, Keng-Yu Lin wrote: With

[PATCH] [PATCH V5]Extcon: adc_jack: adc-jack driver to support 3.5 pi or simliar devices

2012-08-09 Thread anish kumar
From: anish kumar anish198519851...@gmail.com External connector devices that decides connection information based on ADC values may use adc-jack device driver. The user simply needs to provide a table of adc range and connection states. Then, extcon framework will automatically notify others.

Re: [PATCH 3/5] uprobes: remove check for uprobe variable in handle_swbp()

2012-08-09 Thread Suzuki K. Poulose
On 08/08/2012 03:05 PM, Sebastian Andrzej Siewior wrote: On 08/08/2012 11:10 AM, Suzuki K. Poulose wrote: --- a/kernel/events/uprobes.c +++ b/kernel/events/uprobes.c @@ -1528,17 +1528,15 @@ cleanup_ret: utask-active_uprobe = NULL; utask-state = UTASK_RUNNING; } - if (uprobe) { - if

[3.5.1] tg3 waitqueue hang on hotplug remove...

2012-08-09 Thread Daniel J Blueman
Hi Matt, Michael, On my Macbook Retina with 3.5.1, I see the tg3 external adapter (via Thunderbolt) get logically disconnected after a while despite remaining connected (Thunderbolt issues). The problem is though, that the pciehp_wq workqueue fails to complete flushing from the call to

[no subject]

2012-08-09 Thread devendra.aaru
Hi, In function tegra_pinctrl_dt_node_to_map the num_maps the num_maps counter must be incremented for each child node? Actually we are doing free until num_maps if tegra_pinctrl_dt_subnode_to_map, not only that if num_maps == 0, we wont free up the maps, and also i think the for_each_of_node

Q: pinctrl: freeing out the allocated map in tegra_pinctrl_dt_node_to_map

2012-08-09 Thread devendra.aaru
On Fri, Aug 10, 2012 at 11:02 AM, devendra.aaru devendra.a...@gmail.com wrote: Hi, In function tegra_pinctrl_dt_node_to_map the num_maps the num_maps counter must be incremented for each child node? Actually we are doing free until num_maps if tegra_pinctrl_dt_subnode_to_map, not only

Re: thermal patches in linux-next

2012-08-09 Thread Zhang Rui
On 五, 2012-08-10 at 10:37 +0530, Amit Kachhap wrote: On 10 August 2012 08:14, Zhang Rui rui.zh...@intel.com wrote: On 五, 2012-08-10 at 12:23 +1000, Stephen Rothwell wrote: Hi Rui, On Fri, 10 Aug 2012 09:41:06 +0800 Zhang Rui rui.zh...@intel.com wrote: And could you please drop

Re: [PATCH] sched: remove useless code in yield_to

2012-08-09 Thread Mike Galbraith
On Fri, 2012-08-10 at 11:10 +0800, Michael Wang wrote: On 08/10/2012 11:05 AM, Michael Wang wrote: On 07/03/2012 02:34 PM, Michael Wang wrote: From: Michael Wang wang...@linux.vnet.ibm.com it's impossible to enter else branch if we have set skip_clock_update in task_yield_fair(), as

<    9   10   11   12   13   14