[PATCH] x86/fixup_irq: using the cpu_online_mask instead of cpu_all_mask

2012-08-13 Thread Liu, Chuansheng
From: liu chuansheng Subject: [PATCH] x86/fixup_irq: using the cpu_online_mask instead of cpu_all_mask When one CPU is going down, and this CPU is the last one in irq affinity, current code is setting cpu_all_mask as the new affinity for that irq. But for some system the firmware maybe send the

[PATCH 0/5] fix error return code

2012-08-13 Thread Julia Lawall
These patches fix cases where the return code appears to be unintentially 0. The complete semantic match that finds the problem is as follows: (http://coccinelle.lip6.fr/) // @r@ identifier f; expression ret,e; constant C; @@ f(...) { <+... ( return -C; | ret = -C ... when != ret = e return ret

[PATCH 3/5] drivers/usb/wusbcore/wa-hc.c: fix error return code

2012-08-13 Thread Julia Lawall
From: Julia Lawall Convert a 0 error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ identifier ret; expression e,e1,e2,e3,e4,x; @@ ( if (\(ret != 0\|ret

[PATCH 2/5] drivers/tty/moxa.c: fix error return code

2012-08-13 Thread Julia Lawall
From: Julia Lawall Convert a 0 error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ identifier ret; expression e,e1,e2,e3,e4,x; @@ ( if (\(ret != 0\|ret

[PATCH 1/5] drivers/usb/gadget/s3c-hsotg.c: fix error return code

2012-08-13 Thread Julia Lawall
From: Julia Lawall Convert a 0 error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ identifier ret; expression e,e1,e2,e3,e4,x; @@ ( if (\(ret != 0\|ret

[PATCH 4/5] drivers/usb/host/ohci-platform.c: fix error return code

2012-08-13 Thread Julia Lawall
From: Julia Lawall Convert a possibly 0 error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ identifier ret; expression e,e1,e2,e3,e4,x; @@ ( if (\(ret !

[PATCH 5/5] drivers/usb/host/ehci-platform.c: fix error return code

2012-08-13 Thread Julia Lawall
From: Julia Lawall Convert a possibly 0 error return code to a negative one, as returned elsewhere in the function. A simplified version of the semantic match that finds this problem is as follows: (http://coccinelle.lip6.fr/) // @@ identifier ret; expression e,e1,e2,e3,e4,x; @@ ( if (\(ret !

Re: [PATCH 0/7] HID: picoLCD updates

2012-08-13 Thread Bruno Prémont
Hello Tejun, [Tejun: sorry for duplicate, did hit "reply" instead of "reply to all"] On Mon, 13 Aug 2012 16:27:08 Tejun Heo wrote: > On Thu, Aug 09, 2012 at 08:09:47PM +0200, Bruno Prémont wrote: > > As you are working on workqueues and related code, could you have a look > > at my usage of them

Re: [PATCH] drivers/media/video/mx2_emmaprp.c: use devm_kzalloc and devm_clk_get

2012-08-13 Thread Julia Lawall
Well, I've massively applied hundreds of patches today, but not much on this driver. Maybe it is better for you to wait for a couple of days for these to be at -next, or use, instead, our tree as the basis for it: git://linuxtv.org/media_tree.git staging/for_v3.7 I cloned this, but it do

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

2012-08-13 Thread Minchan Kim
Hi Greg, On Mon, Aug 13, 2012 at 07:35:30PM -0700, Greg Kroah-Hartman wrote: > On Wed, Aug 08, 2012 at 03:12:13PM +0900, 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] a

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

2012-08-13 Thread Minchan Kim
Hi Nitin, On Mon, Aug 13, 2012 at 10:36:47PM -0700, Nitin Gupta wrote: > On 08/13/2012 07:35 PM, Greg Kroah-Hartman wrote: > > On Wed, Aug 08, 2012 at 03:12:13PM +0900, Minchan Kim wrote: > >> This patchset promotes zram/zsmalloc from staging. > >> Both are very clean and zram is used by many embe

Re: thermal patches in linux-next

2012-08-13 Thread Amit Kachhap
On 10 August 2012 11:21, Zhang Rui wrote: > 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: >> >> > >> >>

Re: [PATCH 0/1] hopefully fix null pointer dereference on i915 load

2012-08-13 Thread Jani Nikula
On Mon, 13 Aug 2012, Mihai Moldovan wrote: > Had another look at the code and would like to apologize for the confusion... No worries Mihai, thanks for testing! BR, Jani. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.

Re: [PATCH V5 09/18] MIPS: Loongson: Add swiotlb to support big memory (>4GB).

2012-08-13 Thread Huacai Chen
Hi, David, Seems like you are the original author of code in arch/mips/cavium-octeon/dma-octeon.c. Could you please tell me why we need mb() in alloc_coherent(), map_page(), map_sg()? It seems like because of cache coherency (CPU write some data, then map the page for a device, if without mb(), th

Re: psmouse synaptics: "hardware appears to be different" bail out

2012-08-13 Thread Andreas Mohr
Hi, On Fri, Jul 20, 2012 at 08:21:45PM +0200, Andreas Mohr wrote: > Hi, > > On Mon, Jul 16, 2012 at 03:07:19PM -0700, Dmitry Torokhov wrote: > > On Mon, Jul 16, 2012 at 11:44:53PM +0200, Andreas Mohr wrote: > > > Hi, > > > > > > just found the following in dmesg after resume > > > (currently run

Re: [ 10/44] ALSA: hda - remove quirk for Dell Vostro 1015

2012-08-13 Thread Takashi Iwai
At Tue, 14 Aug 2012 07:17:43 +0200, David Henningsson wrote: > > On 08/14/2012 12:02 AM, Greg Kroah-Hartman wrote: > > From: Greg KH > > > > 3.0-stable review patch. If anyone has any objections, please let me know. > > As my previous email states, I still object to this patch being applied >

Re: [PATCH v5 02/12] dm: Use bioset's front_pad for dm_rq_clone_bio_info

2012-08-13 Thread Jun'ichi Nomura
On 08/07/12 07:08, Kent Overstreet wrote: > struct dm_rq_clone_bio_info { > struct bio *orig; > struct dm_rq_target_io *tio; > + struct bio clone; > }; ... > - pools->bs = bioset_create(pool_size, 0); > + pools->bs = bioset_create(pool_size, > +

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

2012-08-13 Thread Nitin Gupta
On 08/13/2012 07:35 PM, Greg Kroah-Hartman wrote: > On Wed, Aug 08, 2012 at 03:12:13PM +0900, 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 linux-n

[PATCH 2/2] drivers/mmc: Add realtek pcie sdmmc host driver

2012-08-13 Thread wei_wang
From: Wei WANG Realtek PCI-E SD/MMC card host driver is used to access SD/MMC card, with the help of Realtek PCI-E card reader MFD driver. Signed-off-by: Wei WANG --- drivers/mmc/host/Kconfig |7 + drivers/mmc/host/Makefile |2 + drivers/mmc/host/rtsx_pci_sdmmc.c | 134

[PATCH v2] media: davinci: fix section mismatch warnings

2012-08-13 Thread Prabhakar Lad
From: Lad, Prabhakar This patch fixes section mismatch warnings for davinci video drivers. Signed-off-by: Lad, Prabhakar Signed-off-by: Manjunath Hadli --- Changes for v2: 1: Annotate probe with __devinit. 2: Fixed the commit message. drivers/media/video/davinci/dm355_ccdc.c |2 +-

Re: [ 10/44] ALSA: hda - remove quirk for Dell Vostro 1015

2012-08-13 Thread David Henningsson
On 08/14/2012 12:02 AM, Greg Kroah-Hartman wrote: From: Greg KH 3.0-stable review patch. If anyone has any objections, please let me know. As my previous email states, I still object to this patch being applied to 3.0 (it is for 3.2+). Repeating that text now: This patch will not work on

Re: [PATCH] m68k: select CONFIG_GENERIC_ATOMIC64 for all m68k CPU types

2012-08-13 Thread Fengguang Wu
On Tue, Aug 14, 2012 at 02:45:33PM +1000, g...@snapgear.com wrote: > From: Greg Ungerer > > There is no specific atomic64 support code for any m68k CPUs, so we should > select CONFIG_GENERIC_ATOMC64 for all. Remove the existing per CPU selection > of this and select it for all m68k. > > Signed-of

Re: [PATCH] davinci: fix build warning when CONFIG_DEBUG_SECTION_MISMATCH is enabled

2012-08-13 Thread Prabhakar Lad
Hi Mauro, Please drop this patch which is queued for 3.7, the actual fix should be to annotate probe with __devinit. I'll be sending V2 with the correct fix. Thx, --Prabhakar On Monday 13 August 2012 03:02 PM, Prabhakar Lad wrote: > From: Lad, Prabhakar > > Signed-off-by: Lad, Prabhakar > Sig

[PATCH] m68k: select CONFIG_GENERIC_ATOMIC64 for all m68k CPU types

2012-08-13 Thread gerg
From: Greg Ungerer There is no specific atomic64 support code for any m68k CPUs, so we should select CONFIG_GENERIC_ATOMC64 for all. Remove the existing per CPU selection of this and select it for all m68k. Signed-off-by: Greg Ungerer --- arch/m68k/Kconfig |1 + arch/m68k/Kconfig.cpu |

Re: [PATCH v3 00/32] provide interfaces to access PCIe capabilities registers

2012-08-13 Thread Bjorn Helgaas
On Wed, Aug 1, 2012 at 8:54 AM, Jiang Liu wrote: > From: Jiang Liu > > As suggested by Bjorn Helgaas and Don Dutile in threads > http://www.spinics.net/lists/linux-pci/msg15663.html, we could improve access > to PCIe capabilities register in to way: > 1) cache content of PCIe Capabilities Registe

Re: Backports mailing list

2012-08-13 Thread David Miller
From: "Luis R. Rodriguez" Date: Mon, 13 Aug 2012 13:42:53 -0700 > I was hoping we can get a mailing list set up on vger for the > project, perhaps backpo...@vger.kernel.org, so we don't flood the > other mailing lists with random kernel backport junk. Please let me > know if this is reasonable.

Re: [PATCH v2] mmc: card: Skip secure option for MoviNAND.

2012-08-13 Thread Kyungmin Park
and also you should update "MMC_QUIRK_MOVINAND_SECURE" also. On 8/14/12, Kyungmin Park wrote: > Hi Ian, > > "MMC_QUIRK_SKIP_SECURE"? It seems it support secure discard but want > to skip. but it's wrong. it's broken feature at listed MoviNAND. > As linus suggested, SECURE_BROKEN is better. > > Th

Re: [PATCH v2] mmc: card: Skip secure option for MoviNAND.

2012-08-13 Thread Kyungmin Park
Hi Ian, "MMC_QUIRK_SKIP_SECURE"? It seems it support secure discard but want to skip. but it's wrong. it's broken feature at listed MoviNAND. As linus suggested, SECURE_BROKEN is better. Thank you, Kyungmin Park On 8/14/12, IAN CHEN wrote: > For several MoviNAND, there are some known issue with

Re: [PATCH 5/5] Fixes a potential bug in android/logger.c

2012-08-13 Thread Cruz Julian Bishop
On Mon, 2012-08-13 at 19:01 -0700, Greg KH wrote: > On Thu, Aug 02, 2012 at 09:50:44AM +1000, Ryan Mallon wrote: > > On 01/08/12 14:54, Cruz Julian Bishop wrote: > > > Previously, when calling is_between(a, b, c), the calculation was wrong. > > > It counted C as between A and B if C was equal to B,

[PATCH v2] mmc: card: Skip secure option for MoviNAND.

2012-08-13 Thread IAN CHEN
For several MoviNAND, there are some known issue with secure option. For these specific MoviNAND device, we skip secure option. Signed-off-by: Ian Chen --- drivers/mmc/card/block.c | 24 +++- include/linux/mmc/card.h |1 + 2 files changed, 24 insertions(+), 1 deletions(

Re: [PATCH repost] kvm: drop parameter validation

2012-08-13 Thread Gleb Natapov
On Mon, Aug 13, 2012 at 05:59:32PM -0300, Marcelo Tosatti wrote: > On Mon, Aug 13, 2012 at 05:30:53PM +0300, Gleb Natapov wrote: > > On Mon, Aug 13, 2012 at 01:43:58PM +0300, Michael S. Tsirkin wrote: > > > We validate irq pin number when routing is setup, so > > > code handling illegal irq # in pi

linux-next: Tree for Aug 14

2012-08-13 Thread Stephen Rothwell
Hi all, Changes since 20120813: I have still reverted 3 commits from the signal tree at the request of the arm maintainer. I have created today's linux-next tree at git://git.kernel.org/pub/scm/linux/kernel/git

Re: [PATCH] mmc: card: Skip secure option for MoviNAND.

2012-08-13 Thread Namjae Jeon
Hi. Ian. Okay, I see. plz add this address in changelog. I also agree about Linus's suggestion. So Would you change flag name to be neutral ? We can distinguish samsung movinand by CID_MANFID_SAMSUNG. And will reuse quirk flag when same issue occurred from other vendor's mmc deivce. Thanks. 201

Re: [PATCH v3 5/6] floppy: use disk_registered for checking if a drive is present

2012-08-13 Thread Ben Hutchings
On Mon, 2012-08-13 at 15:16 -0300, Herton Ronaldo Krzesinski wrote: > Simplify/cleanup code, replacing remaining checks for drives present > using disk_registered array. > > Acked-by: Vivek Goyal > Signed-off-by: Herton Ronaldo Krzesinski > --- > drivers/block/floppy.c | 10 +++--- > 1 fi

Re: [PATCH v3 4/6] floppy: properly handle failure on add_disk loop

2012-08-13 Thread Ben Hutchings
On Mon, 2012-08-13 at 15:16 -0300, Herton Ronaldo Krzesinski wrote: > On do_floppy_init, if something failed inside the loop we call add_disk, > there was no cleanup of previous iterations in the error handling. > > Cc: sta...@vger.kernel.org > Acked-by: Vivek Goyal > Signed-off-by: Herton Ronald

Re: [PATCH v3 3/6] floppy: avoid leaking extra reference to queue on do_floppy_init error handling

2012-08-13 Thread Ben Hutchings
On Mon, 2012-08-13 at 15:16 -0300, Herton Ronaldo Krzesinski wrote: > After commit 3f9a5aa ("floppy: Cleanup disk->queue before caling > put_disk() if add_disk() was never called"), if something fails in the > add_disk loop, we unconditionally set disks[dr]->queue to NULL. But > that's wrong, since

RE: [PATCH V2 18/18] Tools: hv: Properly manage open streams

2012-08-13 Thread KY Srinivasan
> -Original Message- > From: Ben Hutchings [mailto:b...@decadent.org.uk] > Sent: Monday, August 13, 2012 9:58 PM > To: KY Srinivasan > Cc: gre...@linuxfoundation.org; linux-kernel@vger.kernel.org; > de...@linuxdriverproject.org; virtualizat...@lists.osdl.org; o...@aepfle.de; > a...@canoni

Re: [GIT PULL] Update LZO compression

2012-08-13 Thread Andi Kleen
On Tue, Aug 14, 2012 at 01:44:02AM +0200, Markus F.X.J. Oberhumer wrote: > Hi all, > > as suggested on the mailing list I have converted the updated LZO > code into git, so please pull my "lzo-update" branch from > > git://github.com/markus-oberhumer/linux.git lzo-update > > You can browse the

RE: [PATCH V2 02/18] Drivers: hv: Add KVP definitions for IP address injection

2012-08-13 Thread KY Srinivasan
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Monday, August 13, 2012 9:38 PM > To: KY Srinivasan > Cc: linux-kernel@vger.kernel.org; de...@linuxdriverproject.org; > virtualizat...@lists.osdl.org; o...@aepfle.de; a...@canonical.com; > b...@decadent.org.u

Re: [PATCH v7 2/2] kvm: KVM_EOIFD, an eventfd for EOIs

2012-08-13 Thread Alex Williamson
On Tue, 2012-08-14 at 02:00 +0300, Michael S. Tsirkin wrote: > On Mon, Aug 13, 2012 at 04:41:05PM -0600, Alex Williamson wrote: > > On Tue, 2012-08-14 at 01:06 +0300, Michael S. Tsirkin wrote: > > > On Mon, Aug 13, 2012 at 03:34:01PM -0600, Alex Williamson wrote: > > > > On Sun, 2012-08-12 at 11:36

RE: [PATCH V2 06/18] Tools: hv: Further refactor kvp_get_ip_address()

2012-08-13 Thread KY Srinivasan
> -Original Message- > From: Ben Hutchings [mailto:b...@decadent.org.uk] > Sent: Monday, August 13, 2012 9:46 PM > To: KY Srinivasan > Cc: gre...@linuxfoundation.org; linux-kernel@vger.kernel.org; > de...@linuxdriverproject.org; virtualizat...@lists.osdl.org; o...@aepfle.de; > a...@canoni

Re: [net PATCH v1 2/2] net: netprio: fd passed in SCM_RIGHTS datagram not set correctly

2012-08-13 Thread John Fastabend
On 8/13/2012 7:31 PM, Al Viro wrote: On Mon, Aug 13, 2012 at 06:39:45PM -0700, John Fastabend wrote: /* Bump the usage count and install the file. */ get_file(fp[i]); fd_install(new_fd, fp[i]); + sock = sock_from_file(fp[i], &err); +

Re: [net PATCH v1 1/2] net: netprio: fix files lock and remove useless d_path bits

2012-08-13 Thread John Fastabend
On 8/13/2012 7:29 PM, Al Viro wrote: On Mon, Aug 13, 2012 at 06:39:39PM -0700, John Fastabend wrote: - path = d_path(&file->f_path, tmp, PAGE_SIZE); What's using tmp after that patch and do you still need to bother allocating it? nothing removed it in v2. -- To unsubsc

[net PATCH v2 2/2] net: netprio: fd passed in SCM_RIGHTS datagram not set correctly

2012-08-13 Thread John Fastabend
A socket fd passed in a SCM_RIGHTS datagram was not getting updated with the new tasks cgrp prioidx. This leaves IO on the socket tagged with the old tasks priority. To fix this add a check in the scm recvmsg path to update the sock cgrp prioidx with the new tasks value. Thanks to Al Viro for cat

[net PATCH v2 1/2] net: netprio: fix files lock and remove useless d_path bits

2012-08-13 Thread John Fastabend
Add lock to prevent a race with a file closing and also remove useless and ugly sscanf code. The extra code was never needed and the case it supposedly protected against is in fact handled correctly by sock_from_file as pointed out by Al Viro. CC: Neil Horman Reported-by: Al Viro Signed-off-by:

Re: [PATCH v5 00/12] KVM: introduce readonly memslot

2012-08-13 Thread Xiao Guangrong
On 08/14/2012 01:39 AM, Marcelo Tosatti wrote: > On Sat, Aug 11, 2012 at 11:36:20AM +0800, Xiao Guangrong wrote: >> On 08/11/2012 02:14 AM, Marcelo Tosatti wrote: >>> On Tue, Aug 07, 2012 at 05:47:15PM +0800, Xiao Guangrong wrote: Changelog: - introduce KVM_PFN_ERR_RO_FAULT instead of dum

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

2012-08-13 Thread Greg Kroah-Hartman
On Wed, Aug 08, 2012 at 03:12:13PM +0900, 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 linux-next yet but needed > it as base for [4-5] which promotes

[PATCH v2 3/3] rtc: add new lp8788 rtc driver

2012-08-13 Thread Kim, Milo
Patch v2. (a) use irq domain for handling alarm IRQ (b) replace module_init() and module_exit() with module_platform_driver() (c) clean up code for _probe() and _remove() Signed-off-by: Milo(Woogyom) Kim --- drivers/rtc/Kconfig |6 + drivers/rtc/Makefile |1 + drivers/rtc/rtc-l

[PATCH v3 2/3] power_supply: add new lp8788 charger driver

2012-08-13 Thread Kim, Milo
Patch v3. (a) use irq domain for handling charger interrupts (b) use scaled adc value rather than raw value : replace iio_read_channel_raw() with iio_read_channel_scale() (c) clean up charger-platform-data code (d) remove goto statement in _probe() (e) name change : from lp8788_unregister_psy(

[PATCH v3 1/3] mfd: add lp8788 mfd driver

2012-08-13 Thread Kim, Milo
Patch v3. replace generic irq code with irq domain (a) use linear irq domain (b) remove irq_base from lp8788 platform data (c) changes on lp8788-charger and rtc-lp8788 drivers : mapping hwirq to linux IRQ number (seperate patches will be sent) Signed-off-by: Milo(Woogyom) Kim --- driver

Re: [net PATCH v1 2/2] net: netprio: fd passed in SCM_RIGHTS datagram not set correctly

2012-08-13 Thread Al Viro
On Mon, Aug 13, 2012 at 06:39:45PM -0700, John Fastabend wrote: > /* Bump the usage count and install the file. */ > get_file(fp[i]); > fd_install(new_fd, fp[i]); > + sock = sock_from_file(fp[i], &err); > + if (sock) > +

Re: [net PATCH v1 1/2] net: netprio: fix files lock and remove useless d_path bits

2012-08-13 Thread Al Viro
On Mon, Aug 13, 2012 at 06:39:39PM -0700, John Fastabend wrote: > - path = d_path(&file->f_path, tmp, PAGE_SIZE); What's using tmp after that patch and do you still need to bother allocating it? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the b

Re: [PATCH V2] staging: sm7xxfb: copy name of the device before calling smtc_alloc_fb_info

2012-08-13 Thread Greg Kroah-Hartman
On Sat, Aug 04, 2012 at 11:10:20PM +0545, Devendra Naga wrote: > as we do a strcpy(smdrv_ptr->fb_struct->fix->id, name), and the name here in > sm7xxx_probe is not having any assignment, and which leads to copying of the > garbage value > into the id field of the fix struct of fb interface struct.

Re: [PATCH V5 09/18] MIPS: Loongson: Add swiotlb to support big memory (>4GB).

2012-08-13 Thread Huacai Chen
Most of the code are copied from arch/mips/cavium-octeon/dma-octeon.c and they work well. Anyway, I'll try your suggestions, thank you. On Tue, Aug 14, 2012 at 1:54 AM, Konrad Rzeszutek Wilk wrote: >> +static void *loongson_dma_alloc_coherent(struct device *dev, size_t size, >> +

Re: [PATCH] staging/olpc_dcon: fix checkpatch warnings

2012-08-13 Thread Greg Kroah-Hartman
On Fri, Jul 27, 2012 at 08:45:52PM +0900, Toshiaki Yamane wrote: > fixed some checkpatch warnings. > (Excluding -WARNING: msleep < 20ms can sleep for up to 20ms-) Then fix your changelog entry: > > $ find drivers/staging/olpc_dcon/ -name "*.[ch]"|xargs > ./scripts/checkpatch.pl \ > -f --terse -

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

2012-08-13 Thread Hugh Dickins
On Fri, 10 Aug 2012, Hillf Danton wrote: > On Fri, Aug 10, 2012 at 9:26 AM, Hugh Dickins wrote: > > 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.

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

2012-08-13 Thread Xiao Guangrong
On 08/11/2012 04:37 AM, David Ahern wrote: > Thanks for resubmitting this; it was on my to-do list as well. Thanks for you liking it and your precious comments, David! :) > > On 8/9/12 9:19 PM, Dong Hao wrote: >> +static bool kvm_events_exist(const char *event) >> +{ >> +char evt_path[MAXPAT

RE: [PATCH] mmc: card: Skip secure option for MoviNAND.

2012-08-13 Thread IAN CHEN
Dear Mr. Jeon, This is workaround for some Samsung eMMC, MoviNAND. There are known issue about secure erase & secure trim with these eMMC device which I listed by product_name. Therefore, for these eMMC devices, we just skip mmc_blk_issue_secdiscard_rq() and use mmc_blk_issue_discard_rq() instea

RE: [PATCH] mmc: card: Skip secure option for MoviNAND.

2012-08-13 Thread IAN CHEN
Dear Mr. Jeon, This is workaround for some Samsung eMMC, MoviNAND. There are known issue about secure erase & secure trim with these eMMC device which I listed by product_name. Therefore, for these eMMC devices, we just skip mmc_blk_issue_secdiscard_rq() and use mmc_blk_issue_discard_rq() instea

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

2012-08-13 Thread Xiao Guangrong
On 08/10/2012 10:14 PM, Arnaldo Carvalho de Melo wrote: > Em Fri, Aug 10, 2012 at 11:19:10AM +0800, Dong Hao escreveu: >> From: Xiao Guangrong >> >> Add 'perf kvm stat' support to analyze kvm vmexit/mmio/ioport smartly. > > Some comments below about recent changes in my perf/core branch. Arnaldo

Re: [PATCH v3 2/6] floppy: do put_disk on current dr if blk_init_queue fails

2012-08-13 Thread Ben Hutchings
On Mon, 2012-08-13 at 15:16 -0300, Herton Ronaldo Krzesinski wrote: > If blk_init_queue fails, we do not call put_disk on the current dr > (dr is decremented first in the error handling loop). > > Cc: sta...@vger.kernel.org > Signed-off-by: Herton Ronaldo Krzesinski Reviewed-by: Ben Hutchings

[PATCH] perf script: Fix a NULL pointer dereference

2012-08-13 Thread Namhyung Kim
From: Namhyung Kim If 'perf script --gen-script' was called with a perf.data which contains no tracepoint event, it'd segfault due to NULL pevent pointer. Fix it. Cc: Feng Tang Signed-off-by: Namhyung Kim --- tools/perf/util/trace-event-parse.c | 2 +- 1 file changed, 1 insertion(+), 1 deleti

Re: [PATCH 0/5] Android: Small documentation changes and a bug fix

2012-08-13 Thread Greg KH
On Wed, Aug 01, 2012 at 02:54:15PM +1000, Cruz Julian Bishop wrote: > Hi, > > This set of patches completes more documentation in android/logger.c, as well > as fixing a bug there and a comment formatting issue in android/ashmem.c. > > Sorry if kernel-doc was not supposed to be applied to driver

Re: [PATCH v3 1/6] floppy: don't call alloc_ordered_workqueue inside the alloc_disk loop

2012-08-13 Thread Ben Hutchings
On Mon, 2012-08-13 at 15:16 -0300, Herton Ronaldo Krzesinski wrote: > Since commit 070ad7e ("floppy: convert to delayed work and single-thread > wq"), we end up calling alloc_ordered_workqueue multiple times inside > the loop, which shouldn't be intended. Besides the leak, other side > effect in th

Re: [PATCH 5/5] Fixes a potential bug in android/logger.c

2012-08-13 Thread Greg KH
On Thu, Aug 02, 2012 at 09:50:44AM +1000, Ryan Mallon wrote: > On 01/08/12 14:54, Cruz Julian Bishop wrote: > > Previously, when calling is_between(a, b, c), the calculation was wrong. > > It counted C as between A and B if C was equal to B, but not A. > > > > Example of this are: > > > > is_betw

Re: [PATCH 4/5] Redocument some functions in android/logger.c

2012-08-13 Thread Greg KH
On Wed, Aug 01, 2012 at 02:54:19PM +1000, Cruz Julian Bishop wrote: > I will document the rest later if they remain unchanged > Normally, I would do them all at once, but I don't have the chance to do them > all at the moment > > Signed-off-by: Cruz Julian Bishop > --- > drivers/staging/android

Re: [PATCH 0/5] staging/csr: cleanup patches

2012-08-13 Thread Greg Kroah-Hartman
On Sat, Aug 04, 2012 at 01:00:08PM +0545, Devendra Naga wrote: > Actually i have a question about the dbg macros, unifi_warning, > unifi_error, and unifi_trace, > > I think these can be implemented using dev_warn, dev_dbg, functions, > i mean we can remove these unifi_warning and unifi_error and u

Re: [PATCH V2 18/18] Tools: hv: Properly manage open streams

2012-08-13 Thread Ben Hutchings
On Mon, 2012-08-13 at 10:07 -0700, K. Y. Srinivasan wrote: > Close the open streams properly. > > Signed-off-by: K. Y. Srinivasan > Reviewed-by: Haiyang Zhang > Reviewed-by: Olaf Hering > Reviewed-by: Ben Hutchings This isn't reviewed by me, I'm the author! Ben. > --- > tools/hv/hv_kvp_dae

Re: [PATCH V2 02/18] Drivers: hv: Add KVP definitions for IP address injection

2012-08-13 Thread Ben Hutchings
On Mon, 2012-08-13 at 18:38 -0700, Greg KH wrote: > On Mon, Aug 13, 2012 at 10:06:51AM -0700, K. Y. Srinivasan wrote: > > Add the necessary definitions for supporting the IP injection functionality. [...] > > --- a/include/linux/hyperv.h > > +++ b/include/linux/hyperv.h > > @@ -122,12 +122,53 @@ >

[net PATCH v1 2/2] net: netprio: fd passed in SCM_RIGHTS datagram not set correctly

2012-08-13 Thread John Fastabend
A socket fd passed in a SCM_RIGHTS datagram was not getting updated with the new tasks cgrp prioidx. This leaves IO on the socket tagged with the old tasks priority. To fix this add a check in the scm recvmsg path to update the sock cgrp prioidx with the new tasks value. Thanks to Al Viro for cat

[net PATCH v1 1/2] net: netprio: fix files lock and remove useless d_path bits

2012-08-13 Thread John Fastabend
Add lock to prevent a race with a file closing and also remove useless and ugly sscanf code. The extra code was never needed and the case it supposedly protected against is in fact handled correctly by sock_from_file as pointed out by Al Viro. CC: Neil Horman Reported-by: Al Viro Signed-off-by:

[PATCH 1/6] workqueue: make all workqueues non-reentrant

2012-08-13 Thread Tejun Heo
By default, each per-cpu part of a bound workqueue operates separately and a work item may be executing concurrently on different CPUs. The behavior avoids some cross-cpu traffic but leads to subtle weirdities and not-so-subtle contortions in the API. * There's no sane usefulness in allowing a si

[PATCH 3/6] workqueue: gut system_nrt[_freezable]_wq()

2012-08-13 Thread Tejun Heo
Now that all workqueues are non-reentrant, system[_freezable]_wq() are equivalent to system_nrt[_freezable]_wq(). Replace the latter with wrappers around system[_freezable]_wq(). The wrapping goes through inline functions so that __deprecated can be added easily. Signed-off-by: Tejun Heo --- i

[PATCH 6/6] workqueue: deprecate WQ_NON_REENTRANT

2012-08-13 Thread Tejun Heo
WQ_NON_REENTRANT is now spurious. Make alloc_workqueue() trigger WARN_ON() if the flag is specified and drop it from all in-kernel users. If you're cc'd and wondering what's going on: Now all workqueues are non-reentrant, so there's no reason to use WQ_NON_REENTRANT. Some of the conversions in t

[PATCH 5/6] workqueue: deprecate system_nrt[_freezable]_wq

2012-08-13 Thread Tejun Heo
system_nrt[_freezable]_wq are now spurious. Mark them deprecated and convert all users to system[_freezable]_wq. If you're cc'd and wondering what's going on: Now all workqueues are non-reentrant, so there's no reason to use system_nrt[_freezable]_wq. Please use system[_freezable]_wq instead. Th

[PATCH 2/6] workqueue: gut flush[_delayed]_work_sync()

2012-08-13 Thread Tejun Heo
Now that all workqueues are non-reentrant, flush[_delayed]_work_sync() are equivalent to flush[_delayed]_work(). Drop the separate implementation and make them thin wrappers around flush[_delayed]_work(). * start_flush_work() no longer takes @wait_executing as the only left user - flush_work()

[PATCHSET] workqueue: make all workqueues non-reentrant

2012-08-13 Thread Tejun Heo
Hello, Unless explicitly specified, a workqueued is reentrant, which might not even be the correct term. If a work item is executing on a CPU but not pending anywhere, the work item can be queued on a different CPU, and that CPU might start executing the work item whether the previous execution o

Re: [PATCH V2 06/18] Tools: hv: Further refactor kvp_get_ip_address()

2012-08-13 Thread Ben Hutchings
On Mon, 2012-08-13 at 10:06 -0700, K. Y. Srinivasan wrote: > In preparation for making kvp_get_ip_address() more generic, factor out > the code for handling IP addresses. > > Signed-off-by: K. Y. Srinivasan > Reviewed-by: Haiyang Zhang > Reviewed-by: Olaf Hering > Reviewed-by: Ben Hutchings I

[PATCH v3 1/1] HID:hid-multitouch: Add ELAN production request when resume.

2012-08-13 Thread Scott Liu
Add ELAN production request when resume Signed-off-by: Scott Liu Suggested-off-by: Benjamin Tissoires --- drivers/hid/hid-multitouch.c | 27 +++ 1 file changed, 27 insertions(+) diff --git a/drivers/hid/hid-multitouch.c b/drivers/hid/hid-multitouch.c index 59c8b5c..e8

Re: [PATCH] drivers/media/video/mx2_emmaprp.c: use devm_kzalloc and devm_clk_get

2012-08-13 Thread Mauro Carvalho Chehab
Em 13-08-2012 17:20, Julia Lawall escreveu: > From: Julia Lawall > > Using devm_kzalloc simplifies the code and ensures that the use of > devm_request_irq is safe. When kzalloc and kfree were used, the interrupt > could be triggered after the handler's data argument had been freed. > > This als

Re: [PATCH V2 02/18] Drivers: hv: Add KVP definitions for IP address injection

2012-08-13 Thread Greg KH
On Mon, Aug 13, 2012 at 10:06:51AM -0700, K. Y. Srinivasan wrote: > Add the necessary definitions for supporting the IP injection functionality. > > Signed-off-by: K. Y. Srinivasan > Reviewed-by: Haiyang Zhang > Reviewed-by: Olaf Hering > Reviewed-by: Ben Hutchings > --- > drivers/hv/hv_util.

RE: [PATCH 1/2 v2] staging: comedi: new adl_pci7x3x driver

2012-08-13 Thread H Hartley Sweeten
On Monday, August 13, 2012 6:30 PM, Greg KH wrote: > On Fri, Aug 03, 2012 at 10:28:18AM -0700, H Hartley Sweeten wrote: >> Currently the ADLink PCI-7230 and PCI-7432 Isolated Digital >> I/O Boards are supported using two drivers (adl_pci7230 and >> adl_pci7432). These drivers are very similar and o

Re: [PATCH RFT 1/2] gpio: gpio-ml-ioh: Use spinlock for register access protection

2012-08-13 Thread Feng Tang
On Mon, 13 Aug 2012 14:01:31 +0200 Linus Walleij wrote: > On Sun, Jul 29, 2012 at 4:54 AM, Axel Lin wrote: > > > gpio_chip.can_sleep is 0, but current code uses mutex in ioh_gpio_set, > > ioh_gpio_get and ioh_gpio_direction_input functions. > > Thus those functions are not callable from interru

Re: [PATCH 1/2 v2] staging: comedi: new adl_pci7x3x driver

2012-08-13 Thread Greg KH
On Fri, Aug 03, 2012 at 10:28:18AM -0700, H Hartley Sweeten wrote: > Currently the ADLink PCI-7230 and PCI-7432 Isolated Digital > I/O Boards are supported using two drivers (adl_pci7230 and > adl_pci7432). These drivers are very similar and only differ > in the total number of di/do channels provi

Re: [PATCH 1/1] boot: Put initcall_debug into its own Kconfig option DEBUG_INITCALL

2012-08-13 Thread Josh Triplett
On Mon, Aug 13, 2012 at 03:39:54PM -0700, Randy Dunlap wrote: > On 08/13/2012 03:08 PM, Thai Bui wrote: > >Hi all, > > > >I am as part of a capstone group at Portland State University is working > >to tinify the kernel as small as possible. The ultimate goal is to make > >the kernel small enough to

Re: [PATCH V2 2/2] apple_gmux: Add support for newer hardware

2012-08-13 Thread Seth Forshee
On Mon, Aug 13, 2012 at 06:52:49PM -0400, Matthew Garrett wrote: > New gmux devices have a different method for accessing the registers. > Update the driver to cope. Incorporates feedback from Bernhard Froemel. > > Signed-off-by: Matthew Garrett > Cc: Bernhard Froemel > Cc: Seth Forshee Acked-

Re: [PATCH V2 1/2] gmux: Add generic write32 function

2012-08-13 Thread Seth Forshee
On Mon, Aug 13, 2012 at 06:52:48PM -0400, Matthew Garrett wrote: > Move the special-cased backlight update function to a generic gmux_write32 > function. > > Signed-off-by: Matthew Garrett > Cc: Seth Forshee Acked-by: Seth Forshee -- To unsubscribe from this list: send the line "unsubscribe l

Re: linux-next: Tree for July 26 (uml)

2012-08-13 Thread Rusty Russell
y_relocate_add' was here > > make[2]: *** [arch/x86/um/../kernel/module.o] Error 1 > > > > > > > > > > Adding Rusty. > > > This build error is still happening for uml on x86_64 > in linux-next 20120813. No huge surprise, I'll

Re: [PATCH v7 2/4] virtio_balloon: introduce migration primitives to balloon pages

2012-08-13 Thread Rusty Russell
On Mon, 13 Aug 2012 11:41:23 +0300, "Michael S. Tsirkin" wrote: > On Fri, Aug 10, 2012 at 02:55:15PM -0300, Rafael Aquini wrote: > > +/* > > + * Populate balloon_mapping->a_ops->freepage method to help compaction on > > + * re-inserting an isolated page into the balloon page list. > > + */ > > +v

Re: [PATCH] MIPS: fix module.c build for 32 bit

2012-08-13 Thread Rusty Russell
On Mon, 13 Aug 2012 16:39:42 +0100, David Howells wrote: > > Hi Rusty, > > I've fixed up my patch for ARM and pulled Jonas's patch on top of that. Do > you want me to merge them together? Yep, thanks. And might as well sent them straight to Linus; since linux-next didn't catch this, there's l

[PATCH wq/for-3.7] workqueue: add missing wmb() in clear_work_data()

2012-08-13 Thread Tejun Heo
>From 23657bb192f14b789e4c478def8f11ecc95b4f6c Mon Sep 17 00:00:00 2001 From: Tejun Heo Date: Mon, 13 Aug 2012 17:08:19 -0700 Any operation which clears PENDING should be preceded by a wmb to guarantee that the next PENDING owner sees all the changes made before PENDING release. There are only t

mmotm 2012-08-13-16-55 uploaded

2012-08-13 Thread akpm
The mm-of-the-moment snapshot 2012-08-13-16-55 has been uploaded to http://www.ozlabs.org/~akpm/mmotm/ mmotm-readme.txt says README for mm-of-the-moment: http://www.ozlabs.org/~akpm/mmotm/ This is a snapshot of my -mm patch queue. Uploaded at random hopefully more than once a week. You wi

Re: [PATCH] md/linear: rcu_dereference outside read-lock section

2012-08-13 Thread NeilBrown
On Tue, 14 Aug 2012 01:08:12 +0400 Denis Efremov wrote: > According to the comment in linear_stop function > rcu_dereference in linear_start and linear_stop functions > occurs under reconfig_mutex. The patch represents this > agreement in code and prevents lockdep complaint. > > Found by Linux D

[GIT PULL] Update LZO compression

2012-08-13 Thread Markus F.X.J. Oberhumer
Hi all, as suggested on the mailing list I have converted the updated LZO code into git, so please pull my "lzo-update" branch from git://github.com/markus-oberhumer/linux.git lzo-update You can browse the branch at https://github.com/markus-oberhumer/linux/compare/lzo-update I'd ask some

Re: [PATCHSET] workqueue: use irqsafe timer in delayed_work

2012-08-13 Thread Tejun Heo
Hello, On Wed, Aug 08, 2012 at 02:37:55PM -0700, Tejun Heo wrote: > This patchset makes delayed_work use the irqsafe timer added by the > pending "timer: clean up initializers and implement irqsafe timers" > patchset[1]. This enables try_to_grab_pending() to be used from any > context which in tu

Re: [PATCH] [NETFILTER] bridge: rcu_deref outside read-lock section

2012-08-13 Thread Stephen Hemminger
On Tue, 14 Aug 2012 02:47:42 +0400 Denis Efremov wrote: > As it noted in the comment before the br_handle_frame_finish > function, this function should be called under rcu_read_lock. > > The problem callgraph: > br_dev_xmit -> br_nf_pre_routing_finish_bridge_slow -> > -> br_handle_frame_finish -

Re: [PATCHSET] timer: clean up initializers and implement irqsafe timers

2012-08-13 Thread Tejun Heo
Hello, On Wed, Aug 08, 2012 at 11:10:24AM -0700, Tejun Heo wrote: > Timer internals are protected by irqsafe lock but the lock is > naturally dropped and irq enabled while a timer is executed. This > makes dequeueing timer for execution and the actual execution > non-atomic against IRQs. No matt

[git pull] drm fixes

2012-08-13 Thread Dave Airlie
Hi Linus, radeon and intel fixes mostly, one fix to the mgag200 driver to not hang on certain server variants. Dave. The following changes since commit 0d7614f09c1ebdbaa1599a5aba7593f147bf96ee: Linux 3.6-rc1 (2012-08-02 16:38:10 -0700) are available in the git repository at: git://peopl

Re: [PATCH 0/7] HID: picoLCD updates

2012-08-13 Thread Tejun Heo
Hello, On Thu, Aug 09, 2012 at 08:09:47PM +0200, Bruno Prémont wrote: > As you are working on workqueues and related code, could you have a look > at my usage of them in combination with db_defio? > > The delayed memory corruptions or system reboots after unbinding/unplugging > the PicoLCD seem v

Re: [PATCH 5/5] x86: Move enabling of PSE and PGE out of init_memory_mapping

2012-08-13 Thread Tejun Heo
Hello, On Mon, Aug 13, 2012 at 09:34:40AM +0200, Borislav Petkov wrote: > On Sat, Aug 11, 2012 at 01:01:33PM -0700, Tejun Heo wrote: > > Shouldn't this happen before init_memory_mapping() is called multiple > > times? > > It does. > > Those CR4 flags are set before the loop which calls init_memo

  1   2   3   4   5   6   7   8   >