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 > >> > >> it's impossible to enter else branch if we have set skip_clock_update > >> in task_yield_fair(), as

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 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 wrote: > >> > > >> > > > And could you please drop these commits > >>

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 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 that if num_maps

[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

[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

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

[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 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. Changes in V1: added

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 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 >> wrote: >> > On Thu, Aug 09, 2012 at 05:31:51PM +0800, Keng-Yu Lin wrote: >> >> With a previous patch to enable the EHCI/XHCI port

Re: thermal patches in linux-next

2012-08-09 Thread Amit Kachhap
On 10 August 2012 08:14, Zhang Rui 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 wrote: >> > >> > > > And could you please drop these commits >> > > > ef25a0fe0087963c1611c1c8903886fbea053f76 >> > > >

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 wrote: > actually the driver does a request_threaded_irq and after this it calls > lp8727_register_psy, and if it fails it doesn't free the

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 > > > > 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,

[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 --- drivers/pwm/pwm-vt8500.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/pwm/pwm-vt8500.c

[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 --- drivers/pwm/core.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git

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 > > 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. 1. Missing

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

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 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. > > For instance,

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

2012-08-09 Thread David Miller
From: Eric Dumazet Date: Fri, 10 Aug 2012 01:56:06 +0200 > From: Eric Dumazet > > 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 a valid sk->sk_security > pointer,

[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 AuthorDate: Thu, 9 Aug 2012 13:38:56 -0700 Committer: H. Peter Anvin CommitDate: Thu, 9 Aug 2012 17:13:57 -0700 x86, fpu: fix build

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

[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 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 uniquely for each

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

2012-08-09 Thread Steven Rostedt
From: Steven Rostedt 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. With a kernel

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

2012-08-09 Thread Steven Rostedt
From: Steven Rostedt 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 for each event (not

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

2012-08-09 Thread Steven Rostedt
From: Steven Rostedt 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 kernel to do the work

[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

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

[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: [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 >> --- >> arch/x86/kvm/mmu.c |5 + >> include/linux/kvm_host.h |3 ++- >>

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

2012-08-09 Thread Dong Hao
From: Xiao Guangrong 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 tracepoints are interesting as well,

[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 They will be needed by 'perf kvm stat' [ Dong Hao : rebase it on current kvm/tip tree] Signed-off-by: Xiao Guangrong Signed-off-by: Dong Hao --- arch/x86/include/asm/kvm_host.h | 36 --- arch/x86/include/asm/svm.h | 205 +--

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

2012-08-09 Thread Dong Hao
From: Xiao Guangrong '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, add kvm_io_done to trace the

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

2012-08-09 Thread Dong Hao
From: Xiao Guangrong 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 can get more detail from it. Any

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: (>i_mutex_dir_key#4/1){+.+.+.}, at: [] kern_path_create+0x7e/0x140 #1: (>i_data_sem){..}, at: []

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 >> >> Separate out those functions into ui/hist.c. This is required for >> upcoming changes. > > Isn't it better to further

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 >> >> 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().

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

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 > > 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 on this patch?

Re: [PATCH 6/6] cpupower tools: Fix warning and a bug with the cpu package count

2012-08-09 Thread Palmer Cox
On Thu, Aug 09, 2012 at 12:07:36PM +0200, Thomas Renninger wrote: > On Tuesday 07 August 2012 04:24:48 Palmer Cox wrote: > > The pkgs member of cpupower_topology is being used as the number of > > cpu packages. As the comment in get_cpu_topology notes, the package ids > > are not guaranteed to be

compile a kernel for kgdb with less "optimized out"

2012-08-09 Thread gaoqiang
I think those who use kgdb must hate this sentence "optimized out". I have tried many times to build a kernel with less optimization but failed. today,I found a trick method ,just get rid of the -O2 and -Os on the top level of the kernel source and add -O2 for the arch/x86 directory,it

Re: [PATCH] SubmittingPatches: clarify SOB tag usage when evolving submissions

2012-08-09 Thread Ryan Mallon
On 10/08/12 06:51, Luis R. Rodriguez wrote: > From: "Luis R. Rodriguez" > > Initial large code submissions typically are not accepted > on their first patch submission. The developers are > typically given feedback and at times some developers may > even submit changes to the original authors

[PATCH] dlm: remove redundant variable assignments

2012-08-09 Thread Ying Xue
From: Xue Ying Once the tcp_create_listen_sock() is returned successfully, we will invoke add_sock() immediately. In add_sock(), the 'con' variable is assigned to 'sk_user_data', meanwhile, the 'sock' is also set to 'con->sock'. So it's unnecessary to do the same thing in

Re: [PATCH 02/16] perf symbol: remove unused 'end' arg in kallsyms parse cb

2012-08-09 Thread Namhyung Kim
Hi, Cody On Thu, 9 Aug 2012 15:18:27 -0700, Cody P. Schafer wrote: > kallsyms__parse() takes a callback that is called on every discovered > symbol. As /proc/kallsyms does not supply symbol sizes, the callback was > simply called with end=start, faking the symbol size to 1. > > All of the

Re: thermal patches in linux-next

2012-08-09 Thread Zhang Rui
On 五, 2012-08-10 at 12:23 +1000, Stephen Rothwell wrote: > Hi Rui, > > On Fri, 10 Aug 2012 09:41:06 +0800 Zhang Rui wrote: > > > > > > And could you please drop these commits > > > > ef25a0fe0087963c1611c1c8903886fbea053f76 > > > > 09ec52fca274ba88d68df3198de92abdaaff417b > > > >

Re: [BUGFIX -v3 3/4] PCI/PM: Fix config reg access for D3cold and bridge suspending

2012-08-09 Thread Huang Ying
On Wed, 2012-08-08 at 09:07 +0800, Huang Ying wrote: > This patch fixes the following bug: > > http://marc.info/?l=linux-pci=134338059022620=2 > > Where lspci does not work properly if a device and the corresponding > parent bridge (such as PCIe port) is suspended. This is because the > device

[PATCH resend] Extcon: renamed files to comply with the standard naming.

2012-08-09 Thread MyungJoo Ham
Replaced '_' with '-' in the extcon file names, which has been bogging since new drivers have been using the standard naming. Signed-off-by: MyungJoo Ham --- drivers/extcon/Makefile |4 ++-- drivers/extcon/{extcon_class.c => extcon-class.c} |0

Re: [dm-devel] [PATCH v5 12/12] block: Only clone bio vecs that are in use

2012-08-09 Thread Muthu Kumar
Tejun, On Thu, Aug 9, 2012 at 12:01 AM, Tejun Heo wrote: > Hello, > > On Wed, Aug 08, 2012 at 04:47:46PM -0700, Muthu Kumar wrote: >> You are changing the meaning of __bio_clone() here. In old code, the >> number of io_vecs, bi_idx, bi_vcnt are preserved. But in this modified >> code, you are

Re: thermal patches in linux-next

2012-08-09 Thread Stephen Rothwell
Hi Rui, On Fri, 10 Aug 2012 09:41:06 +0800 Zhang Rui wrote: > > > > And could you please drop these commits > > > ef25a0fe0087963c1611c1c8903886fbea053f76 > > > 09ec52fca274ba88d68df3198de92abdaaff417b > > > ab6d2f029358c917508bf88bbd6a9193a8e39fc8 > > > 66447fa993cbce56b4076f169a56f62350f6c7b8

Re: [ 055/109] tg3: Fix Read DMA workaround for 5719 A0.

2012-08-09 Thread Ben Hutchings
On Tue, 2012-08-07 at 15:35 -0700, Greg Kroah-Hartman wrote: > From: Greg KH > > 3.4-stable review patch. If anyone has any objections, please let me know. > > -- > > From: Michael Chan > > commit 10ce95d6ef36c65df7dcd3b8fcf86913f8b298bd upstream. > > The workaround was

Re: [PATCH] leds: wm8350: Convert to devm_regulator_get()

2012-08-09 Thread Bryan Wu
Thanks, it's a good change. I've applied it to my for-next branch. -Bryan On Fri, Aug 10, 2012 at 10:08 AM, Axel Lin wrote: > Signed-off-by: Axel Lin > --- > drivers/leds/leds-wm8350.c | 29 +++-- > 1 file changed, 7 insertions(+), 22 deletions(-) > > diff --git

Re: [PATCH v5 02/12] KVM: hide KVM_MEMSLOT_INVALID from userspace

2012-08-09 Thread Xiao Guangrong
On 08/10/2012 02:48 AM, Marcelo Tosatti wrote: >> +#define KVM_MEMSLOT_INVALID (1UL << 31) >> + >> /* >> * If we support unaligned MMIO, at most one fragment will be split into >> two: >> */ > > Please document which range is for userspace visible flags, which range > is reserved. Mention

linux-next: Tree for Aug 10

2012-08-09 Thread Stephen Rothwell
Hi all, Changes since 20120809: New trees: xtensa and thermal The tty tree still has its build failures for which I have disabled 2 staging drivers and applied a patch. It lost its conflicts. I have still reverted 3 commits from the signal tree at the request of the arm maintainer. The akpm

[PATCH] leds: wm8350: Convert to devm_regulator_get()

2012-08-09 Thread Axel Lin
Signed-off-by: Axel Lin --- drivers/leds/leds-wm8350.c | 29 +++-- 1 file changed, 7 insertions(+), 22 deletions(-) diff --git a/drivers/leds/leds-wm8350.c b/drivers/leds/leds-wm8350.c index 918d4ba..f5d9ac3 100644 --- a/drivers/leds/leds-wm8350.c +++

[PATCH] Extcon: renamed files to comply with the standard naming.

2012-08-09 Thread MyungJoo Ham
Replaced '_' with '-' in the extcon file names, which has been bogging me since new drivers have been using the standard naming. Signed-off-by: MyungJoo Ham --- drivers/extcon/Makefile |4 +- drivers/extcon/extcon-class.c | 832 +

[PATCH] ACPI: power: Use KERN_DEBUG when no power resources are found

2012-08-09 Thread Aaron Lu
commit a606dac368eed5696fb38e16b1394f1d049c09e9 adds support to link devices which have _PRx, if a device does not have _PRx, a warning message will be printed. This commit is for ZPODD on Intel's platform, on AMD's platform, there is no _PRx to support ZPODD, we use _PSx. So instead of printing

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

2012-08-09 Thread Mathieu Desnoyers
* Steven Rostedt (rost...@goodmis.org) wrote: > On Thu, 2012-08-09 at 16:54 -0700, David Daney wrote: > > On 08/09/2012 04:16 PM, H. Peter Anvin wrote: > > > On 08/09/2012 03:25 PM, Steven Rostedt wrote: > > >>> > > >>> It might be better to improve gcc to move really cold branches out of > > >>>

[PATCH] Yama: higher restrictions should block PTRACE_TRACEME

2012-08-09 Thread Kees Cook
The higher ptrace restriction levels should be blocking even PTRACE_TRACEME requests. The comments in the LSM documentation are misleading about when the checks happen (the parent does not go through security_ptrace_access_check() on a PTRACE_TRACEME call). Signed-off-by: Kees Cook Cc:

Re: yama_ptrace_access_check(): possible recursive locking detected

2012-08-09 Thread Fengguang Wu
On Thu, Aug 09, 2012 at 06:39:34PM -0700, Kees Cook wrote: > Hi, > > So, after taking a closer look at this, I cannot understand how it's > possible. Yama's task_lock call is against "current", not "child", > which is what ptrace_may_access() is locking. And the same code makes > sure that

Re: [PATCH] extcon: fixing typos

2012-08-09 Thread 함명주
> Signed-off-by: Peter Meerwald The three "typo" patches: Signed-off-by: MyungJoo Ham I will apply the patches as soon as the branch is cleared up (will be using http://git.infradead.org/users/kmpark/linux-samsung/shortlog/refs/heads/extcon-for-next until I get a decent repository

Re: [dm-devel] [PATCH v5 12/12] block: Only clone bio vecs that are in use

2012-08-09 Thread Muthu Kumar
Kent, >> -- >> You are changing the meaning of __bio_clone() here. In old code, the >> number of io_vecs, bi_idx, bi_vcnt are preserved. But in this modified >> code, you are mapping bio_src's bi_iovec[bi_idx] to bio_dests >> bi_iovec[0] and also restricting the number of allocated io_vecs of >>

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 16:54 -0700, David Daney wrote: > On 08/09/2012 04:16 PM, H. Peter Anvin wrote: > > On 08/09/2012 03:25 PM, Steven Rostedt wrote: > >>> > >>> It might be better to improve gcc to move really cold branches out of > >>> line (really, really far away), and use the compiler to do

Re: [PATCH] efikamx: reintroduce Genesi Efika MX Smarttop via device tree

2012-08-09 Thread Shawn Guo
On Thu, Aug 09, 2012 at 09:29:39AM -0500, Matt Sealey wrote: > The reason the new kernel depends on the new U-Boot is we're trying to > do all pinmux configuration in U-Boot (and we do in-house, and it > works). No pinctrl stuff in the kernel or device tree is required at > this point. The Old

Re: thermal patches in linux-next

2012-08-09 Thread Zhang Rui
Hi, Stephen, On 五, 2012-08-10 at 09:08 +1000, Stephen Rothwell wrote: > Hi Rui, > > On Thu, 09 Aug 2012 14:45:46 +0800 Zhang Rui wrote: > > > > On 二, 2012-08-07 at 10:53 +0800, Zhang Rui wrote: > > > Hi, all, > > > > > > I just created a git tree for catching all thermal changes. > > >

Re: yama_ptrace_access_check(): possible recursive locking detected

2012-08-09 Thread Kees Cook
Hi, So, after taking a closer look at this, I cannot understand how it's possible. Yama's task_lock call is against "current", not "child", which is what ptrace_may_access() is locking. And the same code makes sure that current != child. Yama would never get called if current == child. How did

[PATCH 6/6] regulator: mc13xxx: Remove get_voltage implementation for single voltage regulators

2012-08-09 Thread Axel Lin
This is not required after commit f7df20ec "regulator: core: Use list_voltage() to read single voltage regulators" Signed-off-by: Axel Lin --- drivers/regulator/mc13783-regulator.c |1 - drivers/regulator/mc13892-regulator.c |1 - drivers/regulator/mc13xxx-regulator-core.c |

[PATCH 5/6] regulator: twl: Remove get_voltage implementation for single voltage regulators

2012-08-09 Thread Axel Lin
This is not required after commit f7df20ec "regulator: core: Use list_voltage() to read single voltage regulators" Signed-off-by: Axel Lin --- drivers/regulator/twl-regulator.c | 11 --- 1 file changed, 11 deletions(-) diff --git a/drivers/regulator/twl-regulator.c

[PATCH 4/6] regulator: isl6271a: Remove get_voltage implementation for isl_fixed_ops

2012-08-09 Thread Axel Lin
This is not required after commit f7df20ec "regulator: core: Use list_voltage() to read single voltage regulators" Signed-off-by: Axel Lin --- drivers/regulator/isl6271a-regulator.c |6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/regulator/isl6271a-regulator.c

[PATCH 3/6] regulator: ab8500: Remove get_voltage implementation for ab8500_regulator_fixed_ops

2012-08-09 Thread Axel Lin
This is not required after commit f7df20ec "regulator: core: Use list_voltage() to read single voltage regulators" Signed-off-by: Axel Lin --- drivers/regulator/ab8500.c |6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/regulator/ab8500.c b/drivers/regulator/ab8500.c index

[PATCH 2/6] regulator: ab3100: Remove get_voltage implementation for regulator_ops_fixed

2012-08-09 Thread Axel Lin
This is not required after commit f7df20ec "regulator: core: Use list_voltage() to read single voltage regulators" Signed-off-by: Axel Lin --- drivers/regulator/ab3100.c |6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/regulator/ab3100.c b/drivers/regulator/ab3100.c index

[PATCH 1/6] regulator: core: Add checking n_voltages if using list_voltage() to read voltage regulators

2012-08-09 Thread Axel Lin
Use list_voltage() to read single voltage regulators should be only applied to single voltage regulators, thus add checking n_voltages for this case. Signed-off-by: Axel Lin --- drivers/regulator/core.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

Re: [patch] mmap: feed back correct prev vma when finding vma

2012-08-09 Thread Hugh Dickins
On Thu, 9 Aug 2012, Hillf Danton wrote: > After walking rb tree, if vma is determined, prev vma has to be determined > based on vma; and rb_prev should be considered only if no vma determined. Why? Because you think more code is better code? I disagree. If you have seen a bug here, please tell

Re: [PATCH 0/7] zram/zsmalloc promotion

2012-08-09 Thread Konrad Rzeszutek Wilk
On Wed, Aug 08, 2012 at 10:35:37AM -0700, Nitin Gupta wrote: > On 08/07/2012 11:12 PM, Minchan Kim wrote: > > This patchset promotes zram/zsmalloc from staging. > > Both are very clean and zram is used by many embedded product > > for a long time. > > > > [1-3] are patches not merged into

Re: Re: [PATCH] PM QoS: Add a metric : Bus Throughput.

2012-08-09 Thread 함명주
> + Myungjoo Ham, > > It used at devfreq. Mr. Ham can you explain it in detail? > > Thank you, > Kyungmin Park > , > On 8/9/12, Rafael J. Wysocki wrote: > > On Wednesday, August 08, 2012, Jonghwa Lee wrote: > >> Bus throughput metric is added to PM QoS in order to control the > >> frequency of

Re: [PATCH v2] SubmittingPatches: clarify SOB tag usage when evolving submissions

2012-08-09 Thread Joe Perches
On Thu, 2012-08-09 at 14:48 -0700, Luis R. Rodriguez wrote: > From: "Luis R. Rodriguez" > > Initial large code submissions typically are not accepted > on their first patch submission. The developers are > typically given feedback and at times some developers may > even submit changes to the

[PATCH] wm831x-dcdc: fix coccinelle warnings of missing IRQF_ONESHOT

2012-08-09 Thread Feng Tang
From: Fengguang Wu /c/kernel-tests/src/linux/drivers/regulator/wm831x-dcdc.c:829:7-27: ERROR: Threaded IRQ with no primary handler requested without IRQF_ONESHOT /c/kernel-tests/src/linux/drivers/regulator/wm831x-dcdc.c:695:7-27: ERROR: Threaded IRQ with no primary handler requested without

Re: [dm-devel] [PATCH v2 1/2] dm: verity support data device offset (Linux 3.4.7)

2012-08-09 Thread Wesley Miaw
On Aug 8, 2012, at 7:04 PM, Alasdair G Kergon wrote: > > However, first you need to address the second part of Mikulas's email, > namely to make the case for these changes rather than making no kernel > changes and instead stacking the verity target over a linear target. > > To put this another

Re: [PATCH 02/19] mm/mpol: Remove NUMA_INTERLEAVE_HIT

2012-08-09 Thread Andi Kleen
Andrea Arcangeli writes: > Hi, > > On Tue, Jul 31, 2012 at 09:12:06PM +0200, Peter Zijlstra wrote: >> Since the NUMA_INTERLEAVE_HIT statistic is useless on its own; it wants >> to be compared to either a total of interleave allocations or to a miss >> count, remove it. >> >> Fixing it would be

Re: [PATCH] Smack: remove task_wait() hook.

2012-08-09 Thread Casey Schaufler
On 12/20/2011 11:20 PM, Jarkko Sakkinen wrote: > Allow SIGCHLD to be passed to child process without > explicit policy. This will help to keep the access > control policy simple and easily maintainable with > complex applications that require use of multiple > security contexts. It will also help

[PATCH] debugfs: make __create_file static

2012-08-09 Thread Chris Wright
It's only used locally, no need to pollute global namespace. Signed-off-by: Chris Wright --- fs/debugfs/inode.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/fs/debugfs/inode.c b/fs/debugfs/inode.c index 4733eab..2c9fafb 100644 --- a/fs/debugfs/inode.c +++

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

2012-08-09 Thread Yanmin Zhang
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. Yanmin Forwarded Message > From: Chen, LinX Z > To:

RE: [PATCH] net: add new QCA alx ethernet driver

2012-08-09 Thread Huang, Xiong
> The alterations to the description of atl1c ought to be broken out as a > separate patch, though. > Yes, we will separate it. thanks ! -Xiong

Re: BUG: RDMA/ocrdma calls invalid vlan_dev_real_dev()

2012-08-09 Thread Fengguang Wu
On Thu, Aug 09, 2012 at 04:54:37PM -0700, Roland Dreier wrote: > thanks for the report. I assume the system doesn't actually have ocrdma hw? Yeah, it's a test boot inside KVM. Thanks, Fengguang > - R. > On Aug 9, 2012 3:00 AM, "Fengguang Wu" wrote: > > > Hi Parav, > > > > commit fe2caefcdf

Re: [PATCH] [trivial] infiniband: Fix typo in infiniband driver

2012-08-09 Thread Masanari Iida
Thanks for the review. I just posted version 2. Masanari On Fri, Aug 10, 2012 at 8:51 AM, Hefty, Sean wrote: >> diff --git a/drivers/infiniband/hw/amso1100/c2_rnic.c >> b/drivers/infiniband/hw/amso1100/c2_rnic.c >> index 8c81992..b80867e 100644 >> --- a/drivers/infiniband/hw/amso1100/c2_rnic.c

[PATCH/v2] [trivial] infiniband: Fix typo in infiniband driver

2012-08-09 Thread Masanari Iida
Correct spelling typo in infiniband/hw and infiniband/ulp Signed-off-by: Masanari Iida --- drivers/infiniband/hw/amso1100/c2_rnic.c | 2 +- drivers/infiniband/hw/cxgb3/iwch_cm.c| 2 +- drivers/infiniband/hw/qib/qib_sd7220.c | 2 +- drivers/infiniband/ulp/srpt/ib_srpt.c| 2 +- 4 files

Re: "Inconsistent kallsyms data" error

2012-08-09 Thread Jan Engelhardt
On Saturday 2012-07-07 23:40, Michal Marek wrote: >index cd9c6c6..4629038 100644 >--- a/scripts/link-vmlinux.sh >+++ b/scripts/link-vmlinux.sh >@@ -210,8 +210,8 @@ if [ -n "${CONFIG_KALLSYMS}" ]; then > mksysmap ${kallsyms_vmlinux} .tmp_System.map > > if ! cmp -s System.map

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

2012-08-09 Thread Eric Dumazet
From: Eric Dumazet 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 a valid sk->sk_security pointer, but this field is NULL for unicast_sock It turns out that unicast_sock

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

2012-08-09 Thread David Daney
On 08/09/2012 04:16 PM, H. Peter Anvin wrote: On 08/09/2012 03:25 PM, Steven Rostedt wrote: It might be better to improve gcc to move really cold branches out of line (really, really far away), and use the compiler to do this, rather than to use an extra indirection that adds bloat and

RE: [PATCH] [trivial] infiniband: Fix typo in infiniband driver

2012-08-09 Thread Hefty, Sean
> diff --git a/drivers/infiniband/hw/amso1100/c2_rnic.c > b/drivers/infiniband/hw/amso1100/c2_rnic.c > index 8c81992..b80867e 100644 > --- a/drivers/infiniband/hw/amso1100/c2_rnic.c > +++ b/drivers/infiniband/hw/amso1100/c2_rnic.c > @@ -439,7 +439,7 @@ static int c2_rnic_close(struct c2_dev

Re: [PATCH net-next,1/1] hyperv: Add comments for the extended buffer after RNDIS message

2012-08-09 Thread David Miller
From: Haiyang Zhang Date: Thu, 9 Aug 2012 11:04:18 -0700 > Reported-by: Olaf Hering > Signed-off-by: Haiyang Zhang Applied, thanks. -- 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

[PATCH] x86, pci: Fix all early PCI scans to check the vendor ID first v2

2012-08-09 Thread Andi Kleen
From: Andi Kleen According to the Intel PCI experts it's not safe to check any other field than vendor ID for 0x when doing PCI scans to see if the device exists. Several of the early PCI scans violated this. I changed them all to always check the vendor ID first. v2: Change some continues

Re: [PATCH] alarmtimer: implement minimum alarm interval for allowing suspend

2012-08-09 Thread Arve Hjønnevåg
On Thu, Aug 9, 2012 at 2:27 PM, John Stultz wrote: > On 08/09/2012 12:37 AM, Todd Poynor wrote: >> >> alarmtimer suspend return -EBUSY if the next alarm will fire in less >> than 2 seconds. This allows one RTC seconds tick to occur subsequent >> to this check before the alarm wakeup time is set,

Re: [PATCH] x86, pci: Fix all early PCI scans to check the vendor ID first

2012-08-09 Thread Andi Kleen
Thanks Betty. All fixed. I'll post a v2. -Andi -- a...@linux.intel.com -- Speaking for myself only. -- 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 http://vger.kernel.org/majordomo-info.html

Re: [PATCH] ipv4: tcp: security_sk_alloc() needed for unicast_sock

2012-08-09 Thread David Miller
From: Eric Dumazet Date: Thu, 09 Aug 2012 23:29:03 +0200 > diff --git a/net/ipv4/ip_output.c b/net/ipv4/ip_output.c > index 76dde25..ec410e0 100644 > --- a/net/ipv4/ip_output.c > +++ b/net/ipv4/ip_output.c > @@ -1536,6 +1536,7 @@ void ip_send_unicast_reply(struct net *net, struct > sk_buff

Re: [PATCH 3/5] mm: compaction: Capture a suitable high-order page immediately when it is made available

2012-08-09 Thread Minchan Kim
On Thu, Aug 09, 2012 at 02:49:23PM +0100, Mel Gorman wrote: > While compaction is migrating pages to free up large contiguous blocks for > allocation it races with other allocation requests that may steal these > blocks or break them up. This patch alters direct compaction to capture a > suitable

[PATCH] [trivial] infiniband: Fix typo in infiniband driver

2012-08-09 Thread Masanari Iida
Correct spelling typo in infiniband/hw and infiniband/ulp Signed-off-by: Masanari Iida --- drivers/infiniband/hw/amso1100/c2_rnic.c | 2 +- drivers/infiniband/hw/cxgb3/iwch_cm.c| 2 +- drivers/infiniband/hw/qib/qib_sd7220.c | 2 +- drivers/infiniband/ulp/srpt/ib_srpt.c| 2 +- 4 files

Re: [PATCH 2/6] mm: vmscan: Scale number of pages reclaimed by reclaim/compaction based on failures

2012-08-09 Thread Minchan Kim
On Thu, Aug 09, 2012 at 10:20:35AM +0100, Mel Gorman wrote: > On Thu, Aug 09, 2012 at 05:27:15PM +0900, Minchan Kim wrote: > > > > > + * pages reclaimed based on the number of consecutive allocation > > > > > + * failures > > > > > + */ > > > > > + zone = lruvec_zone(lruvec); >

Re: [PATCH net-next] time: jiffies_delta_to_clock_t() helper to the rescue

2012-08-09 Thread David Miller
From: Eric Dumazet Date: Thu, 09 Aug 2012 09:13:53 +0200 > From: Eric Dumazet > > Various /proc/net files sometimes report crazy timer values, expressed > in clock_t units. > > This happens when an expired timer delta (expires - jiffies) is passed > to jiffies_to_clock_t(). > > This function

Re: [PATCH] tun: don't zeroize sock->file on detach

2012-08-09 Thread David Miller
From: Stanislav Kinsbursky Date: Thu, 09 Aug 2012 16:50:40 +0400 > This is a fix for bug, introduced in 3.4 kernel by commit > 1ab5ecb90cb6a3df1476e052f76a6e8f6511cb3d, which, among other things, replaced > simple sock_put() by sk_release_kernel(). Below is sequence, which leads to > oops for

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

2012-08-09 Thread H. Peter Anvin
On 08/09/2012 03:25 PM, Steven Rostedt wrote: >> >> It might be better to improve gcc to move really cold branches out of >> line (really, really far away), and use the compiler to do this, rather >> than to use an extra indirection that adds bloat and complexity to the >> kernel. > > I think

[PATCH V2 2/3] PPC64: Add support for instantiating SML from Open Firmware

2012-08-09 Thread Ashley Lai
This patch instantiate Stored Measurement Log (SML) and put the log address and size in the device tree. Signed-off-by: Ashley Lai --- arch/powerpc/kernel/prom_init.c | 64 +++ 1 files changed, 64 insertions(+), 0 deletions(-) diff --git

[PATCH V2 3/3] drivers/char/tpm: Add securityfs support for event log

2012-08-09 Thread Ashley Lai
This patch retrieves the event log data from the device tree during file open. The event log data will then displayed through securityfs. Signed-off-by: Ashley Lai --- drivers/char/tpm/Makefile |8 - drivers/char/tpm/tpm.h| 12 --

[PATCH V2 1/3] drivers/char/tpm: Add new device driver to support IBM vTPM

2012-08-09 Thread Ashley Lai
This patch adds a new device driver to support IBM virtual TPM (vTPM) for PPC64. IBM vTPM is supported through the adjunct partition with firmware release 740 or higher. With vTPM support, each lpar is able to have its own vTPM without the physical TPM hardware. This driver provides TPM

  1   2   3   4   5   6   7   8   9   10   >