[PATCH 2/2] ASoC: core: fix the memory leak in case of remove_aux_dev()

2012-12-24 Thread Chuansheng Liu
When probing aux_dev, initializing is as below: device_initialize() device_add() So when remove aux_dev, we need do as below: device_del() device_put() Otherwise, the rtd_release() will not be called. So here using device_unregister() to replace device_del(), like the action in soc_remove_link_d

[PATCH 1/2] ASoC: core: fix the memory leak in case of device_add() failure

2012-12-24 Thread Chuansheng Liu
After called device_initialize(), even device_add() returns error, we still need use the put_device() to release the reference to call rtd_release(), which will do the free() action. Signed-off-by: liu chuansheng --- sound/soc/soc-core.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(

Re: kernel panic when running /etc/init.d/iptables restart

2012-12-24 Thread canqun zhang
Hi Gao feng The stack information is as follows. The kenel will panic because the nf_ct_destroy is NULL. Reproduction: (1) starting a lxc container (2) iptables -t nat -A POSTROUTING -s 10.48.254.18 -o eth1 -j MASQUERADE (run it on host machine) (3) /etc/ini.d/iptables save (run it on host machine

Re: kernel panic when running /etc/init.d/iptables restart

2012-12-24 Thread Gao feng
cc netdev Hi canqun: On 2012/12/24 13:51, canqun zhang wrote: > Hi Patrick, > If i start one lxc container instance, and then in the system there > will be two net namespaces,one is init_net namespace, the other is > created by lxc.If running "/etc/init.d/iptables restart",the system > will be pa

Re: [PATCH] ARM: tegra: dts: add aliases and DMA requestor for serial controller

2012-12-24 Thread Stephen Warren
On 12/22/2012 01:15 PM, Laxman Dewangan wrote: > On Saturday 22 December 2012 02:20 AM, Stephen Warren wrote: >> On 12/20/2012 09:18 PM, Laxman Dewangan wrote: >>> On Thursday 20 December 2012 10:33 PM, Stephen Warren wrote: On 12/19/2012 11:15 PM, Laxman Dewangan wrote: > On Wednesday 19

Re: [PATCH 1/7] KVM: Write protect the updated slot only when we start dirty logging

2012-12-24 Thread Takuya Yoshikawa
On Tue, 25 Dec 2012 07:05:55 +0200 Gleb Natapov wrote: > > Other parts are doing "if (!slot->dirty_bitmap)" to see if the slot > > is in dirty logging mode. If you prefer, we can do the same here. > > > Does not matter to me. If you think it will be more consistent do it. No preference at the

Re: [PATCHv4 5/8] drm: tegra: Remove redundant host1x

2012-12-24 Thread Stephen Warren
On 12/21/2012 11:50 PM, Terje Bergström wrote: > On 21.12.2012 16:36, Thierry Reding wrote: >> On Fri, Dec 21, 2012 at 01:39:21PM +0200, Terje Bergstrom wrote: >>> +static struct platform_driver tegra_drm_platform_driver = { >>> + .driver = { >>> + .name = "tegradrm", >> >> This should

Re: [PATCH 1/7] KVM: Write protect the updated slot only when we start dirty logging

2012-12-24 Thread Gleb Natapov
On Tue, Dec 25, 2012 at 01:08:40PM +0900, Takuya Yoshikawa wrote: > On Mon, 24 Dec 2012 15:27:17 +0200 > Gleb Natapov wrote: > > > > @@ -6897,7 +6897,14 @@ void kvm_arch_commit_memory_region(struct kvm *kvm, > > > spin_lock(&kvm->mmu_lock); > > > if (nr_mmu_pages) > > > kvm_mmu_chan

Re: [PATCH] clk: debug clock tree

2012-12-24 Thread Prashant Gaikwad
On Monday 24 December 2012 11:07 PM, Mike Turquette wrote: On Sun, Dec 23, 2012 at 4:26 AM, Prashant Gaikwad wrote: On Saturday 22 December 2012 04:26 AM, Mike Turquette wrote: On Wed, Dec 19, 2012 at 11:53 PM, Prashant Gaikwad wrote: On Thursday 13 December 2012 11:31 PM, Stephen Warren wro

Re: radeon 0000:02:00.0: GPU lockup CP stall for more than 10000msec

2012-12-24 Thread Shuah Khan
On Sun, Dec 23, 2012 at 6:31 AM, Borislav Petkov wrote: > On Sun, Dec 23, 2012 at 01:22:12PM +0100, Borislav Petkov wrote: >> Right, let me try that and report back. > > Yep, looks like reverting the above commit fixes it - the boston.com > website loads just fine. > > Thanks. > > -- > Regards/Gru

kernel BUG at mm/huge_memory.c:1798!

2012-12-24 Thread Zhouping Liu
Hello all, I found the below kernel bug using latest mainline(637704cbc95), my hardware has 2 numa nodes, and it's easy to reproduce the issue using LTP test case: "# ./mmap10 -a -s -c 200": [root@localhost linux]# cat .config | grep NUMA CONFIG_ARCH_SUPPORTS_NUMA_BALANCING=y CONFIG_ARCH_WANTS_PR

Re: USB 2.0 (EHCI) keeps crapping out (3.2.29)

2012-12-24 Thread Logan Rathbone
On 12/24/2012 11:10 PM, Rob Landley wrote: On 12/24/2012 09:02:11 PM, Logan Rathbone wrote: *** NOTE: Kindly CC me directly if you reply to list since I'm not an LKML subscriber. *** Running vanilla 3.2.29 (Slackware 14.0). I also had this issue on 2.6.37.6 and *possibly* 2.6.33.x, but I do

Re: USB 2.0 (EHCI) keeps crapping out (3.2.29)

2012-12-24 Thread Rob Landley
On 12/24/2012 09:02:11 PM, Logan Rathbone wrote: *** NOTE: Kindly CC me directly if you reply to list since I'm not an LKML subscriber. *** Running vanilla 3.2.29 (Slackware 14.0). I also had this issue on 2.6.37.6 and *possibly* 2.6.33.x, but I don't recall. Every now and again (could b

Re: [PATCH 1/7] KVM: Write protect the updated slot only when we start dirty logging

2012-12-24 Thread Takuya Yoshikawa
On Mon, 24 Dec 2012 15:27:17 +0200 Gleb Natapov wrote: > > @@ -6897,7 +6897,14 @@ void kvm_arch_commit_memory_region(struct kvm *kvm, > > spin_lock(&kvm->mmu_lock); > > if (nr_mmu_pages) > > kvm_mmu_change_mmu_pages(kvm, nr_mmu_pages); > > - kvm_mmu_slot_remove_write_access(

Re: [PATCH v7 06/27] x86, 64bit: early #PF handler set page table

2012-12-24 Thread Yinghai Lu
On Mon, Dec 24, 2012 at 4:16 PM, H. Peter Anvin wrote: > On 12/20/2012 08:56 AM, Yinghai Lu wrote: >>> >>> >>> So in that case, kgdb is broken and will need to be fixed up. That >>> happens all the time with debugging tools. >> >> >> If there is a way that we can make all parties happy, we really

Re: [PATCH 10/25] kprobes: fix wait_for_kprobe_optimizer()

2012-12-24 Thread Masami Hiramatsu
(2012/12/22 10:57), Tejun Heo wrote: > wait_for_kprobe_optimizer() seems largely broken. It uses > optimizer_comp which is never re-initialized, so > wait_for_kprobe_optimizer() will never wait for anything once > kprobe_optimizer() finishes all pending jobs for the first time. Thank you for fixi

[PATCH] sched: Fix print format for u64

2012-12-24 Thread Fabio Estevam
From: Fabio Estevam commit c566e8e9 (sched: Aggregate total task_group load) introduced the following build warning: kernel/sched/debug.c:225:2: warning: format '%ld' expects argument of type 'long int', but argument 4 has type 'u64' [-Wformat] kernel/sched/debug.c:225:2: warning: format '%ld'

Re: [PATCH 25/25] ipc: don't use [delayed_]work_pending()

2012-12-24 Thread Tejun Heo
Hello, On Mon, Dec 24, 2012 at 08:41:01PM +0100, Borislav Petkov wrote: > On Mon, Dec 24, 2012 at 10:45:20AM -0800, Tejun Heo wrote: > > I was confused a bit there. We can't. Nothing guarantees that the > > queuer sees the cleared PENDING before the work item starts execution, > > and I think ipc

Regression in kernel 3.8-rc1 bisected to commit adfa79d: I now get many "unable to enumerate USB device" messages

2012-12-24 Thread Larry Finger
The problem has been bisected to commit adfa79d entitled "USB: EHCI: make ehci-pci a separate driver". The symptom is that my NVIDIA controller again reverts to unended logging of messages of the form "hub 2-0:1.0: unable to enumerate USB device on port 5". This behavior was previously fixed w

Re: [PATCH 25/25] ipc: don't use [delayed_]work_pending()

2012-12-24 Thread Tejun Heo
Hello, Borislav. On Mon, Dec 24, 2012 at 08:32:58PM +0100, Borislav Petkov wrote: > Ok, understood. I have only one question: how do you make sure > schedule_work() is used only in cold paths? Hot and cold are relative terms and unless someone is doing things like invoking queue_work() from high-

USB 2.0 (EHCI) keeps crapping out (3.2.29)

2012-12-24 Thread Logan Rathbone
*** NOTE: Kindly CC me directly if you reply to list since I'm not an LKML subscriber. *** Running vanilla 3.2.29 (Slackware 14.0). I also had this issue on 2.6.37.6 and *possibly* 2.6.33.x, but I don't recall. Every now and again (could be every few days, could take a couple of weeks... d

[PATCH] mfd: pcf50633: Init pcf->dev before using it

2012-12-24 Thread Axel Lin
Current code uses pcf->dev in the dev_err call before setting it to &client->dev. Fix it. Signed-off-by: Axel Lin --- drivers/mfd/pcf50633-core.c |5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/mfd/pcf50633-core.c b/drivers/mfd/pcf50633-core.c index 64803f1..d

[PATCH v2 4/8] MFD:rtsx: Add callback function switch_output_voltage

2012-12-24 Thread wei_wang
From: Wei WANG Different card reader has different method to switch output voltage, add this callback to let the card reader implement its individual switch function. Signed-off-by: Wei WANG --- drivers/mfd/rtl8411.c| 16 drivers/mfd/rts5209.c| 20 +

[PATCH v2 2/8] MFD:rtsx: Remove redundant code

2012-12-24 Thread wei_wang
From: Wei WANG In function rtsx_pci_add_sg_tbl, the statement "ptr++" is useless. Signed-off-by: Wei WANG --- drivers/mfd/rtsx_pcr.c |1 - 1 file changed, 1 deletion(-) diff --git a/drivers/mfd/rtsx_pcr.c b/drivers/mfd/rtsx_pcr.c index 3a44efa..fa2c2bc 100644 --- a/drivers/mfd/rtsx_pcr.c

[PATCH v2 1/8] MFD:rtsx: Fix typo in comment

2012-12-24 Thread wei_wang
From: Wei WANG Fix a misspelling word in comment Signed-off-by: Wei WANG --- include/linux/mfd/rtsx_pci.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/linux/mfd/rtsx_pci.h b/include/linux/mfd/rtsx_pci.h index 060b721..bebe9f9 100644 --- a/include/linux/mfd/rts

[PATCH v2 7/8] MFD:rtsx: Fix checkpatch warning

2012-12-24 Thread wei_wang
From: Wei WANG WARNING: Avoid CamelCase: + u8 N, min_N, max_N, clk_divider; WARNING: Avoid CamelCase: + u8 N, min_N, max_N, clk_divider; Signed-off-by: Wei WANG --- drivers/mfd/rtsx_pcr.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff

[PATCH v2 8/8] MFD:rtsx: Use macro defines to replace some variables

2012-12-24 Thread wei_wang
From: Wei WANG In function rtsx_pci_switch_clock, some variables, such as min_n, max_n, and max_div, are not necessary. And those assigned values look very obscure for others. It's more proper to use macro definitions here to replace these variables. Signed-off-by: Wei WANG --- drivers/mfd/rts

[PATCH v2 5/8] MMC:rtsx: Using callback function to switch output voltage

2012-12-24 Thread wei_wang
From: Wei WANG Different card reader has different method to switch output voltage, so we have to use the callback function provided by MFD driver to switch output pad voltage. Signed-off-by: Wei WANG --- drivers/mmc/host/rtsx_pci_sdmmc.c | 30 +- 1 file changed,

[PATCH v2 6/8] MFD:rtsx: Add callback function conv_clk_and_div_n

2012-12-24 Thread wei_wang
From: Wei WANG Add callback function conv_clk_and_div_n to convert between SSC clock and its divider N. For rtl8411, the formula to calculate SSC clock divider N is different with the other card reader models. Signed-off-by: Wei WANG --- drivers/mfd/rtl8411.c | 13 + dr

[PATCH v2 3/8] MFD:rtsx: Declare that the DMA address limitation is 32bit explicitly

2012-12-24 Thread wei_wang
From: Wei WANG Signed-off-by: Wei WANG --- drivers/mfd/rtsx_pcr.c |4 1 file changed, 4 insertions(+) diff --git a/drivers/mfd/rtsx_pcr.c b/drivers/mfd/rtsx_pcr.c index fa2c2bc..1dc64bb 100644 --- a/drivers/mfd/rtsx_pcr.c +++ b/drivers/mfd/rtsx_pcr.c @@ -1010,6 +1010,10 @@ static int

[PATCH v2 0/8] rtsx patchset for MFD and MMC tree

2012-12-24 Thread wei_wang
From: Wei WANG Wei WANG (8): MFD:rtsx: Fix typo in comment MFD:rtsx: Remove redundant code MFD:rtsx: Declare that the DMA address limitation is 32bit explicitly MFD:rtsx: Add callback function switch_output_voltage MMC:rtsx: Using callback function to switch output voltage MFD:rtsx: A

[PATCH v2] fadvise: perform WILLNEED readahead asynchronously

2012-12-24 Thread Eric Wong
Using fadvise with POSIX_FADV_WILLNEED can be very slow and cause user-visible latency. This hurts interactivity and encourages userspace to resort to background threads for readahead (or avoid POSIX_FADV_WILLNEED entirely). "strace -T" timing on an uncached, one gigabyte file: Before: fadvise6

[PATCH] regulator: lp8755: Fix lp8755_regulator_init unwind code

2012-12-24 Thread Axel Lin
It's safe to pass NULL argument to regulator_unregister(), so we can remove the NULL checking before calling regulator_unregister(). However pass a ERR_PTR to regulator_unregister() is wrong, so we need to explicitly set "pchip->rdev[buck_num] = NULL" before goto err_buck. This patch also includes

Re: [PATCH V2] output the cpu number when printking.

2012-12-24 Thread Yanmin Zhang
On Mon, 2012-12-24 at 09:55 -0800, Greg KH wrote: > On Mon, Dec 24, 2012 at 01:01:55PM +0800, he, bo wrote: > > From: "he, bo" > > > > We often hit kernel panic issues on SMP machines because processes race > > on multiple cpu. By adding a new parameter printk.cpu, kernel prints > > cpu number at

Re: [PATCH v7 00/27] x86, boot, 64bit: Add support for loading ramdisk and bzImage above 4G

2012-12-24 Thread H. Peter Anvin
On 12/22/2012 05:24 AM, Konrad Rzeszutek Wilk wrote: Interestingly enough I only see this on any AMD machine - different motherboards, different memory configurations. And also in a HVM guest running on an AMD machine. If it runs on Intel it works just fine. Could this be the ljmpq problem t

[PATCH] Bluetooth: fix the oops due to conn->hcon == NULL in shutdown case

2012-12-24 Thread Chuansheng Liu
Meet one panic issue as below stack: <1>[11340.226404] BUG: unable to handle kernel NULL pointer dereference at 0008 <4>[11340.226619] EIP is at __sco_sock_close+0xe8/0x1a0 <4>[11340.226629] EAX: f063a740 EBX: ECX: f58f4544 EDX: <4>[11340.226640] ESI: dec83e00 EDI: 5f9a081f

Re: [PATCH 1/4] regulator: core: Allow specify apply_[reg|bit] for regmap based voltage_sel operations

2012-12-24 Thread Haojian Zhuang
On Tue, Dec 25, 2012 at 12:34 AM, Mark Brown wrote: > On Tue, Dec 18, 2012 at 09:30:10AM +0800, Axel Lin wrote: >> Some DVM regulators needs to update apply_bit after setting vsel_reg to >> initiate voltage change on the output. This patch adds apply_reg and > > Applied all except patch 2 (where

Re: [PATCH v2 2/2] regulator: 88pm8607: Use apply_[reg|bit] with regmap based voltage_sel operations

2012-12-24 Thread Haojian Zhuang
On Mon, Dec 24, 2012 at 9:14 AM, Axel Lin wrote: > Signed-off-by: Axel Lin > --- > drivers/regulator/88pm8607.c | 38 ++ > 1 file changed, 6 insertions(+), 32 deletions(-) > > diff --git a/drivers/regulator/88pm8607.c b/drivers/regulator/88pm8607.c > index 5

Re: [PATCH 28/29] net/: rename net_random() to prandom_u32()

2012-12-24 Thread Neil Horman
On Mon, Dec 24, 2012 at 11:14:15AM +0900, Akinobu Mita wrote: > Use more preferable function name which implies using a pseudo-random > number generator. > > Signed-off-by: Akinobu Mita > Cc: Jesse Gross > Cc: Venkat Venkatsubra > Cc: Vlad Yasevich > Cc: Sridhar Samudrala > Cc: Neil Horman >

Re: [PATCH v7 06/27] x86, 64bit: early #PF handler set page table

2012-12-24 Thread H. Peter Anvin
On 12/20/2012 08:56 AM, Yinghai Lu wrote: So in that case, kgdb is broken and will need to be fixed up. That happens all the time with debugging tools. If there is a way that we can make all parties happy, we really should not break KGDB. Please reconsider to stop #PF handler in x86_64_start

Re: [PATCH v7 00/27] x86, boot, 64bit: Add support for loading ramdisk and bzImage above 4G

2012-12-24 Thread H. Peter Anvin
On 12/23/2012 10:00 AM, Yinghai Lu wrote: > On Sun, Dec 23, 2012 at 6:33 AM, H. Peter Anvin wrote: >> Explanation please? >> > > you have following change in the patch > > /* Finally jump to run C code and to be on real kernel address > * Since we are running on identity-mapped

Re: [PATCH v7 24/27] x86: Add swiotlb force off support

2012-12-24 Thread Yinghai Lu
On Mon, Dec 24, 2012 at 3:34 PM, H. Peter Anvin wrote: > On 12/21/2012 09:11 PM, Yinghai Lu wrote: >> On Fri, Dec 21, 2012 at 7:25 PM, H. Peter Anvin wrote: >>> On 12/21/2012 07:23 PM, Eric W. Biederman wrote: In this case YH has been working on the case of loading a kernel complet

Re: Porting problem: ndo_set_multicast_list removed

2012-12-24 Thread Woody Wu
On Mon, Dec 24, 2012 at 11:16:27AM -0800, Eric Dumazet wrote: > On Tue, 2012-12-25 at 00:30 +0800, Woody Wu wrote: > > Hi, list > > > > I am porting an ethernet driver from 2.6.x to 3.7.1. I found in the new > > kernel, the ndo_set_multicast_list method in the net_device_ops had been > > removed.

Re: [PATCH v7 24/27] x86: Add swiotlb force off support

2012-12-24 Thread H. Peter Anvin
On 12/21/2012 09:11 PM, Yinghai Lu wrote: > On Fri, Dec 21, 2012 at 7:25 PM, H. Peter Anvin wrote: >> On 12/21/2012 07:23 PM, Eric W. Biederman wrote: >>> >>> In this case YH has been working on the case of loading a kernel >>> completely above 4G, and apparently he has also been testing the case

[PATCH] nfsd: Don't overuse NFS?ERR_PERM error code

2012-12-24 Thread Pawel Dziepak
From: Pawel Dziepak Unlike EPERM, NFS?ERR_PERM error code applies only when the operation could not be completed because the caller is either not the owner or not a privileged user. Actually, only OPEN, CREATE and SETATTR are supposed to ever return such error code when a problem occurs when chan

Re: [PATCH 18/25] TMIO MMC: don't use [delayed_]work_pending()

2012-12-24 Thread Guennadi Liakhovetski
Hi Tejun On Fri, 21 Dec 2012, Tejun Heo wrote: > There's no need to test whether a (delayed) work item in pending > before queueing, flushing or cancelling it. Most uses are unnecessary > and quite a few of them are buggy. > > Remove unnecessary pending tests from tmio mmc. Only compile tested

[PATCH] drivers: video: fix warnings int-to-pointer-cast

2012-12-24 Thread Stefan Hasko
Fixed compile warnings cast to pointer from integer of different size Signed-off-by: Stefan Hasko --- drivers/video/arkfb.c|2 +- drivers/video/s3fb.c |2 +- drivers/video/vt8623fb.c |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/video/arkfb.c b

Re: [PATCH 2/4] signal: add a helper for dequeuing signals from a specified queue

2012-12-24 Thread Michael Kerrisk
[CC+=linux-api] On Mon, Dec 24, 2012 at 9:13 AM, Andrey Vagin wrote: > Cc: Ingo Molnar > Cc: Peter Zijlstra > Cc: Serge Hallyn > Cc: Oleg Nesterov > Cc: Andrew Morton > Cc: "Eric W. Biederman" > Cc: Al Viro > Cc: Pavel Emelyanov > CC: Cyrill Gorcunov > Signed-off-by: Andrey Vagin > ---

Re: [PATCH 0/4] signalfd: a kernel interface for dumping/restoring pending signals

2012-12-24 Thread Michael Kerrisk
[CC+=linux-api] Hi Andrey, On Mon, Dec 24, 2012 at 9:13 AM, Andrey Vagin wrote: > The idea is simple. We need to get the siginfo for each signal on dump, > and then return it back on restore. > > The first problem is that the kernel doesn’t report complete siginfo-s > in user-space. In a signal

Re: [PATCH 3/4] signalfd: add ability to choose a private or shared queue

2012-12-24 Thread Michael Kerrisk
[CC+=linux-api] Andrey, (Among other things, some suggested wording fixes below to improve the eventual commit log entry). On Mon, Dec 24, 2012 at 9:13 AM, Andrey Vagin wrote: > This patch is added two flags SFD_GROUP and SFD_SHARED. > SFD_SHARED - read signals from a shared queue > SFD_PRIVATE

usb issues with 3.6.X and 3.7.X

2012-12-24 Thread Fabio Coatti
Hi all, I'm experiencing some usb-related issues with recent kernels (3.6.7 and 3.7.1 at least.) Basically I have a V500 epson scanner that worked just fine with older kernels; since last time I used it I've changed several kernels and now the oldest that I have on my machine is 3.6.7. Recently I t

Re: [B.A.T.M.A.N.] [PATCH] net, batman: don't crash on zero length strings in routing_algo

2012-12-24 Thread Marek Lindner
On Monday, December 24, 2012 23:42:15 Pau Koning wrote: > On Mon, Dec 24, 2012 at 3:38 PM, Marek Lindner wrote: > > On Monday, December 24, 2012 22:18:52 Pau Koning wrote: > >> On Mon, Nov 19, 2012 at 9:08 PM, Sasha Levin wrote: > >> > The code that works with routing_algo assumes that the stri

Re: [PATCH 25/25] ipc: don't use [delayed_]work_pending()

2012-12-24 Thread Borislav Petkov
On Mon, Dec 24, 2012 at 10:45:20AM -0800, Tejun Heo wrote: > I was confused a bit there. We can't. Nothing guarantees that the > queuer sees the cleared PENDING before the work item starts execution, > and I think ipc memory hotplug could also be broken from that. Stupid question: why not clear PE

Re: [PATCH 25/25] ipc: don't use [delayed_]work_pending()

2012-12-24 Thread Borislav Petkov
On Mon, Dec 24, 2012 at 11:07:23AM -0800, Tejun Heo wrote: > And that's broken. It seems trivial but it really isn't and trying to > optimize things like that in cold paths is just a bad idea. Not enough > people will pay attention to them and they will stay subtly broken for > a very long time. So

Re: [lm-sensors] RFC: Intel QST driver

2012-12-24 Thread Guenter Roeck
On Mon, Dec 24, 2012 at 08:37:57AM +, Simon J. Rowe wrote: > I've written a driver for the Intel Quiet System Technology (QST) > function of the Management Engine Interface found on recent Intel > chipsets. > > The git repo can be found here: > > http://mose.dyndns.org/mei.git > > A few

Re: [Regression w/ patch] Media commit causes user space to misbahave (was: Re: Linux 3.8-rc1)

2012-12-24 Thread Mauro Carvalho Chehab
Em Sun, 23 Dec 2012 14:29:04 -0800 Linus Torvalds escreveu: > On Sun, Dec 23, 2012 at 12:21 PM, Mauro Carvalho Chehab > wrote: > > > > Agreed: ENOENT was a bad choice, and it should be reverted. > > Well, *any* other error value is likely a bad choice. Well, UVC should return the same error co

[PATCH v2] MIPS: MSP71xx: Move include files

2012-12-24 Thread Shane McDonald
Now that Yosemite's gone we can move the MSP71xx include files one level up. Signed-off-by: Shane McDonald --- Patch history: V1: Original patch V2: Use format-patch's -M option to indicate file renames .../cpu-feature-overrides.h|0 .../msp71xx => mach-pmcs-msp71xx}

Re: Porting problem: ndo_set_multicast_list removed

2012-12-24 Thread Eric Dumazet
On Tue, 2012-12-25 at 00:30 +0800, Woody Wu wrote: > Hi, list > > I am porting an ethernet driver from 2.6.x to 3.7.1. I found in the new > kernel, the ndo_set_multicast_list method in the net_device_ops had been > removed. What's the story behind? Can I simply ignore this method > defined in an

Re: [PATCH 25/25] ipc: don't use [delayed_]work_pending()

2012-12-24 Thread Tejun Heo
Hello, Borislav. On Mon, Dec 24, 2012 at 07:55:55PM +0100, Borislav Petkov wrote: > Basically, with the amount of bloat we're adding to the kernel, the > couple of instructions we're adding here and there and think they won't > harm us, tends to crop up with time until we're too damn slow to do >

Re: [PATCH 25/25] ipc: don't use [delayed_]work_pending()

2012-12-24 Thread Borislav Petkov
On Mon, Dec 24, 2012 at 10:33:34AM -0800, Tejun Heo wrote: > If one looks at something happening in a path as cold as memory > hotplug and thinks about optimizing a coupld memory accesses, the > person's priorities need serious reconsideration. I think approaches > like that are actively harmful. T

Re: [PATCH] resubmit w1_therm:Retries:remove old code add CRC

2012-12-24 Thread Evgeniy Polyakov
Hi Sorry for late reply - too much things to finish before the new year. On Tue, Dec 18, 2012 at 01:37:56AM +, david (da...@avoncliff.com) wrote: > w1_therm includes some obsolete code to detect bad_roms, this is no > longer relevant. > The retry code is only used for this bad_rom test, howev

Re: [PATCH 25/25] ipc: don't use [delayed_]work_pending()

2012-12-24 Thread Tejun Heo
Hello, again. On Mon, Dec 24, 2012 at 10:33:34AM -0800, Tejun Heo wrote: > I don't think we have cases where this actually matters but it could > be that we can add work_pending() tests to queue_work_on(). I *think* > that shouldn't break work scheduling semantics. Not completely sure > tho. Ne

Re: [PATCH v7 00/27] x86, boot, 64bit: Add support for loading ramdisk and bzImage above 4G

2012-12-24 Thread Yinghai Lu
On Mon, Dec 24, 2012 at 2:04 AM, Borislav Petkov wrote: > On Sun, Dec 23, 2012 at 08:54:37PM -0800, H. Peter Anvin wrote: >> Makes sense. Ljmpq it is. A comment might be useful. > > Yes, a LRETQ comment is definitely useful so that we know. Yinghai, > please add it the next time you're rebasing.

Re: [PATCH] virtio_console: correct error message on failure of debugfs_create_dir

2012-12-24 Thread Greg Kroah-Hartman
On Mon, Dec 24, 2012 at 01:24:52PM -0500, Sasha Levin wrote: > On Mon, Dec 24, 2012 at 12:39 PM, Greg Kroah-Hartman > wrote: > > On Mon, Dec 24, 2012 at 11:55:46AM +, Arnd Bergmann wrote: > >> On Friday 21 December 2012, Amit Shah wrote: > >> > On (Thu) 20 Dec 2012 [14:11:21], Sasha Levin wrot

Re: [PATCH 25/25] ipc: don't use [delayed_]work_pending()

2012-12-24 Thread Tejun Heo
Hello, Borislav. On Sat, Dec 22, 2012 at 12:09:29PM +0100, Borislav Petkov wrote: > Now you say those paths are not fast paths, but the reverse of > this optimization is also true: what happens if people start using > schedule_work() in fast paths without checking whether work is pending? > A usel

Re: [PATCH] virtio_console: correct error message on failure of debugfs_create_dir

2012-12-24 Thread Sasha Levin
On Mon, Dec 24, 2012 at 12:39 PM, Greg Kroah-Hartman wrote: > On Mon, Dec 24, 2012 at 11:55:46AM +, Arnd Bergmann wrote: >> On Friday 21 December 2012, Amit Shah wrote: >> > On (Thu) 20 Dec 2012 [14:11:21], Sasha Levin wrote: >> > > debugfs_create_dir() returns NULL if it fails, there's little

Re: [PATCH 17/25] wm97xx: don't use [delayed_]work_pending()

2012-12-24 Thread Tejun Heo
Hello, Dmitry. On Sun, Dec 23, 2012 at 01:54:50AM -0800, Dmitry Torokhov wrote: > This is not 100% equivalent transformation as now we schedule first and > disable IRQ later... Anyway, I think the driver shoudl be converted to > threaded IRQ instead. Mark, does the patch below make any sense to yo

[PATCH] namei.h: include errno.h

2012-12-24 Thread Stephen Warren
This solves: In file included from fs/ext3/symlink.c:20:0: include/linux/namei.h: In function 'retry_estale': include/linux/namei.h:114:19: error: 'ESTALE' undeclared (first use in this function) Signed-off-by: Stephen Warren --- include/linux/namei.h |1 + 1 file changed, 1 insertion(+)

Re: [PATCH V2] output the cpu number when printking.

2012-12-24 Thread Greg KH
On Mon, Dec 24, 2012 at 01:01:55PM +0800, he, bo wrote: > From: "he, bo" > > We often hit kernel panic issues on SMP machines because processes race > on multiple cpu. By adding a new parameter printk.cpu, kernel prints > cpu number at printk information line. It’s useful to debug what cpus > are

Re: [PATCH] clk: debug clock tree

2012-12-24 Thread Mike Turquette
On Sun, Dec 23, 2012 at 4:26 AM, Prashant Gaikwad wrote: > On Saturday 22 December 2012 04:26 AM, Mike Turquette wrote: >> >> On Wed, Dec 19, 2012 at 11:53 PM, Prashant Gaikwad >> wrote: >>> >>> On Thursday 13 December 2012 11:31 PM, Stephen Warren wrote: On 12/13/2012 09:27 AM, Mike Tu

Re: [PATCH] virtio_console: correct error message on failure of debugfs_create_dir

2012-12-24 Thread Greg Kroah-Hartman
On Mon, Dec 24, 2012 at 11:55:46AM +, Arnd Bergmann wrote: > On Friday 21 December 2012, Amit Shah wrote: > > On (Thu) 20 Dec 2012 [14:11:21], Sasha Levin wrote: > > > debugfs_create_dir() returns NULL if it fails, there's little point in > > > calling PTR_ERR on it. > > > > debugfs_create_dir(

Re: [PATCH 3/4] signalfd: add ability to choose a private or shared queue

2012-12-24 Thread Oleg Nesterov
On 12/24, Andrey Vagin wrote: > > static ssize_t signalfd_dequeue(struct signalfd_ctx *ctx, siginfo_t *info, > - int nonblock) > + int nonblock, int queue) > { > ssize_t ret; > DECLARE_WAITQUEUE(wait, current); > > spin_loc

Re: [PATCH 1/4] signalfd: add ability to return siginfo in a raw format

2012-12-24 Thread Oleg Nesterov
On 12/24, Andrey Vagin wrote: > > signalfd should be called with the flag SFD_RAW for that. > > signalfd_siginfo is not full for siginfo with a negative si_code. > copy_siginfo_to_user() is copied a full siginfo to user-space, if > si_code is negative. signalfd_copyinfo() doesn't do that and can't

Re: [PATCH V3 00/11] ARM: delete struct sys_timer

2012-12-24 Thread Stephen Warren
On 11/19/2012 11:30 AM, Stephen Warren wrote: > Russell, I'm mainly reposting this to see if you have any comments or > opinion, in particular on the core ARM changes. > > The aim of this series is to remove ARM's struct sys_timer. As a pre- > cursor, this requires a little re-arrangement of arch_

Re: [PATCH] regulator: twl: Convert twl[6030|4030]fixed_ops to regulator_list_voltage_linear

2012-12-24 Thread Mark Brown
On Sat, Dec 22, 2012 at 01:31:19PM +0800, Axel Lin wrote: > Signed-off-by: Axel Lin Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH 1/4] regulator: core: Allow specify apply_[reg|bit] for regmap based voltage_sel operations

2012-12-24 Thread Mark Brown
On Tue, Dec 18, 2012 at 09:30:10AM +0800, Axel Lin wrote: > Some DVM regulators needs to update apply_bit after setting vsel_reg to > initiate voltage change on the output. This patch adds apply_reg and Applied all except patch 2 (where you've done a new version, I'll wait for Haojian to ack thou

Re: [PATCH v2] regulator: new driver for LP8755

2012-12-24 Thread Mark Brown
On Mon, Dec 17, 2012 at 10:24:06AM +0900, Daniel Jeong wrote: > This patch is for new lp8755 regulator dirver and > several unsed variables were deleted and then test was done. Applied, thanks. signature.asc Description: Digital signature

Porting problem: ndo_set_multicast_list removed

2012-12-24 Thread Woody Wu
Hi, list I am porting an ethernet driver from 2.6.x to 3.7.1. I found in the new kernel, the ndo_set_multicast_list method in the net_device_ops had been removed. What's the story behind? Can I simply ignore this method defined in an old driver? Please also help to check the below code, is the

Re: [PATCH RFC/RFT] regulator: core: Fix continuous_voltage_range case in regulator_can_change_voltage

2012-12-24 Thread Mark Brown
On Wed, Dec 12, 2012 at 09:22:46AM +0800, Axel Lin wrote: > Regulator drivers with continuous_voltage_range flag set allows not setting > n_voltages. Thus if continuous_voltage_range is set, check the constraint > range > instead. Applied, thanks. signature.asc Description: Digital signature

[PATCH 2/2] mfd: max77693: Init max77693->dev before using it

2012-12-24 Thread Axel Lin
Current code uses max77693->dev in the dev_err call before setting it to &i2c->dev. Fix it. This patch also includes below cleanups: - Move checking pdata earlier and show dev_err if no platform data found. - Remove unnecessary err_regmap goto label. - Unregister i2c devices if regmap init for

Re: [PATCH 2/2] regulator: lp3972: Convert to get_voltage_sel

2012-12-24 Thread Mark Brown
On Mon, Dec 10, 2012 at 07:50:12PM +0800, Axel Lin wrote: > regulator_list_voltage_table() returns -EINVAL if selector >= n_voltages. > Thus we don't need to check if reg is greater than LP3972_BUCK_VOL_MAX_IDX in > lp3972_dcdc_get_voltage_sel. Applied, thanks. signature.asc Description: Digital

Re: [PATCH 1/2] regulator: lp3971: Convert to get_voltage_sel

2012-12-24 Thread Mark Brown
On Mon, Dec 10, 2012 at 07:46:52PM +0800, Axel Lin wrote: > regulator_list_voltage_table() returns -EINVAL if selector >= n_voltages. > Thus we don't need to check if reg is greater than BUCK_TARGET_VOL_MAX_IDX in > lp3971_dcdc_get_voltage_sel. Applied, thanks. signature.asc Description: Digital

Re: [PATCH] regulator: s5m8767: Fix probe failure due to stack corruption

2012-12-24 Thread Mark Brown
On Wed, Dec 12, 2012 at 08:57:00AM +0530, Inderpal Singh wrote: > The function sec_reg_read invokes regmap_read which expects unsigned int * > as the destination address. The existing driver is passing address of local > variable "val" which is u8. This causes the stack corruption and following > d

Re: [PATCH] regmap: irq: enable wake support by default

2012-12-24 Thread Mark Brown
On Wed, Dec 19, 2012 at 07:42:28PM +0530, Laxman Dewangan wrote: > regmap-irq framework is used vastly by mfd drivers and some of > devices like TPS65910, TPS80036 do not support the wake base > register to enable wake. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH 17/25] wm97xx: don't use [delayed_]work_pending()

2012-12-24 Thread Mark Brown
On Sun, Dec 23, 2012 at 01:54:50AM -0800, Dmitry Torokhov wrote: > This is not 100% equivalent transformation as now we schedule first and > disable IRQ later... Anyway, I think the driver shoudl be converted to > threaded IRQ instead. Mark, does the patch below make any sense to you? I'm a bit n

[PATCH 1/2] mfd: max77686: Init max77686->dev before using it

2012-12-24 Thread Axel Lin
Current code uses max77686->dev in the dev_err call before setting it to &i2c->dev. Fix it. Signed-off-by: Axel Lin --- drivers/mfd/max77686.c | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/drivers/mfd/max77686.c b/drivers/mfd/max77686.c index f6878f8..4

Re: [PATCH 13/25] sound/wm8350: don't use [delayed_]work_pending()

2012-12-24 Thread Mark Brown
On Fri, Dec 21, 2012 at 05:57:03PM -0800, Tejun Heo wrote: > There's no need to test whether a (delayed) work item in pending > before queueing, flushing or cancelling it. Most uses are unnecessary > and quite a few of them are buggy. Applied, thanks. signature.asc Description: Digital signatur

Re: [PATCH 7/7] MFD:rtsx: Fix checkpatch warning

2012-12-24 Thread Borislav Petkov
On Mon, Dec 24, 2012 at 02:03:56PM +0800, wei_w...@realsil.com.cn wrote: > From: Wei WANG > > WARNING: Avoid CamelCase: > + u8 N, min_N, max_N, clk_divider; > > WARNING: Avoid CamelCase: > + u8 N, min_N, max_N, clk_divider; > > Signed-off-by: Wei WANG > --- > drivers/mfd/rtsx_pcr.c

Re: How kernel handle interrupts[AX88796B network controller]

2012-12-24 Thread Woody Wu
On Mon, Dec 24, 2012 at 10:10:17PM +0800, Woody Wu wrote: > On Fri, Dec 21, 2012 at 01:33:03PM -0800, anish kumar wrote: > > On Fri, 2012-12-21 at 23:34 +0800, Woody Wu wrote: > > > On Thu, Dec 20, 2012 at 10:05:05AM -0800, anish singh wrote: > > > > On Dec 20, 2012 6:30 AM, "Woody Wu" wrote: > >

Re: [RFC PATCH v4 1/9] CPU hotplug: Provide APIs to prevent CPU offline from atomic context

2012-12-24 Thread Srivatsa S. Bhat
On 12/23/2012 10:12 PM, Oleg Nesterov wrote: > On 12/23, Srivatsa S. Bhat wrote: >> >> On 12/20/2012 07:12 PM, Oleg Nesterov wrote: >>> >>> We need mb() + rmb(). Plust cli/sti unless this arch has optimized >>> this_cpu_add() like x86 (as you pointed out). >>> >> >> Hey, IIUC, we actually don't nee

Re: [B.A.T.M.A.N.] [PATCH] net, batman: don't crash on zero length strings in routing_algo

2012-12-24 Thread Pau Koning
On Mon, Dec 24, 2012 at 3:38 PM, Marek Lindner wrote: > On Monday, December 24, 2012 22:18:52 Pau Koning wrote: >> On Mon, Nov 19, 2012 at 9:08 PM, Sasha Levin wrote: >> > The code that works with routing_algo assumes that the string passed is >> > non >> >> > empty, this assumption is wrong: >>

Re: [PATCH] ASoC: core: giving WARN when device starting from non-off bias with idle_bias_off

2012-12-24 Thread Mark Brown
On Fri, Dec 21, 2012 at 06:17:12PM +0800, Chuansheng Liu wrote: > > Just found some cases that some codec drivers set the bias to _STANDBY and > set idle_bias_off to 1 during probing. > It will cause unpaired runtime_get_sync/put() issue. Also as Mark suggested, > there is no reason to start from

Re: [PATCH] kvm: Fix memory slot generation updates

2012-12-24 Thread Alex Williamson
On Sun, 2012-12-23 at 10:55 +0200, Gleb Natapov wrote: > On Fri, Dec 21, 2012 at 08:20:16AM -0700, Alex Williamson wrote: > > Previous patch "kvm: Minor memory slot optimization" overlooked the > > generation field of the memory slots. Re-using the original memory > > slots left us with with two s

Re: [PATCH 0/9] drivers: mailbox: framework creation

2012-12-24 Thread Ohad Ben-Cohen
Hi Omar, On Fri, Dec 21, 2012 at 9:33 PM, Omar Ramirez Luna wrote: > Yes, I made the changes, for tidspbridge and remoteproc, I will submit > both for review, based on this series. Great, thanks. Please note that when we do eventually merge this, we need your updates to be squashed into Loic's

Re: [PATCH] net, batman: don't crash on zero length strings in routing_algo

2012-12-24 Thread Pau Koning
On Mon, Nov 19, 2012 at 9:08 PM, Sasha Levin wrote: > The code that works with routing_algo assumes that the string passed is non > empty, this assumption is wrong: Why isn't this patch part of Linux 3.7? It seems to be a bugfix and it was sent early enough? -- To unsubscribe from this list: send

Re: [B.A.T.M.A.N.] [PATCH] net, batman: don't crash on zero length strings in routing_algo

2012-12-24 Thread Marek Lindner
On Monday, December 24, 2012 22:18:52 Pau Koning wrote: > On Mon, Nov 19, 2012 at 9:08 PM, Sasha Levin wrote: > > The code that works with routing_algo assumes that the string passed is > > non > > > empty, this assumption is wrong: > Why isn't this patch part of Linux 3.7? It seems to be a bugfi

Re: [Suggestion] drivers/staging/tidspbridge: strcpy and strncpy, src length checking issue.

2012-12-24 Thread Omar Ramirez Luna
Hi Gchen, On Mon, Dec 17, 2012 at 8:40 PM, Chen Gang wrote: > Hello Omar Ramirez Luna: > > excuse me to bother you (maybe you are busy in these days). > please help checking this suggestion when you have free time. Yes, I'm checking your suggestions, I was a little busy last week, for the co

Re: [Suggestion] drivers/staging/tidspbridge: pr_err and pr_debug for uninitialized buffer (name buf not initialized).

2012-12-24 Thread Omar Ramirez Luna
Hi, On Thu, Dec 13, 2012 at 7:30 PM, Chen Gang wrote: > also another suggestions: > I built ti otmap with ti dsp bridge by arm cross-compiler under i386 > platform. > the version tag is next-20121213 > I met 2 compiling issues. > > a: module dependency: > Multifunctio

Re: How kernel handle interrupts[AX88796B network controller]

2012-12-24 Thread Woody Wu
On Fri, Dec 21, 2012 at 01:33:03PM -0800, anish kumar wrote: > On Fri, 2012-12-21 at 23:34 +0800, Woody Wu wrote: > > On Thu, Dec 20, 2012 at 10:05:05AM -0800, anish singh wrote: > > > On Dec 20, 2012 6:30 AM, "Woody Wu" wrote: > > > > > > > > Hi, List > > > > > > > > Where is the Kernel code that

Re: [PATCH 1/3] Revert "perf: Require exclude_guest to use PEBS - kernel side enforcement"

2012-12-24 Thread Jiri Olsa
On Sat, Dec 22, 2012 at 12:21:23PM -0700, David Ahern wrote: > On 12/19/12 8:07 AM, Jiri Olsa wrote: > >This reverts commit 20b279ddb38ca42f8863cec07b4d45ec24589f13. > > > >Cc: Corey Ashford > >Cc: Frederic Weisbecker > >Cc: Ingo Molnar > >Cc: Namhyung Kim > >Cc: Paul Mackerras > >Cc: Peter Zi

  1   2   >