Re: [PATCHv2 4/4] gpio: smscece: Add support for gpio IO expander feature

2012-09-05 Thread Poddar, Sourav
Hi Vaibhav, On Thu, Sep 6, 2012 at 12:29 AM, Vaibhav Hiremath wrote: > > > On 9/5/2012 5:07 PM, Sourav Poddar wrote: >> smsc can be used as an gpio io expander device also. So adding >> support for configuring smsc pins as a gpio. >> >> Cc: Benoit Cousson >> Cc: Felipe Balbi >> Cc: Santosh

Re: [PATCHv2 1/4] mfd: smsc: Add support for smsc gpio io/keypad driver

2012-09-05 Thread Poddar, Sourav
Hi, On Wed, Sep 5, 2012 at 11:23 PM, Vaibhav Hiremath wrote: > > > On 9/5/2012 5:06 PM, Sourav Poddar wrote: >> smsc ece1099 is a keyboard scan or gpio expansion device. >> The patch create keypad and gpio expander child for this >> multi function smsc driver. >> >> Cc: Samuel Ortiz >> Cc:

Re: [PATCH] OMAP GPIO - don't wake from suspend unless requested.

2012-09-05 Thread Shilimkar, Santosh
On Thu, Sep 6, 2012 at 8:35 AM, NeilBrown wrote: > On Mon, 3 Sep 2012 22:59:06 -0700 "Shilimkar, Santosh" > wrote: > >> On Sun, Aug 26, 2012 at 6:29 PM, Shilimkar, Santosh >> wrote: >> > On Sun, Aug 26, 2012 at 3:53 PM, NeilBrown wrote: >> >> >> >> On Sun, 26 Aug 2012 09:47:50 +0530

linux-next: manual merge of the arm-soc tree with the usb tree

2012-09-05 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the arm-soc tree got a conflict in drivers/usb/host/Kconfig between commit 952230d774bb ("usb: ohci: Fix Kconfig dependency on USB_ISP1301") from the usb tree and commit d684f05f2d55 ("ARM: mach-pnx4008: Remove architecture") from the arm-soc tree. I fixed it

Re: [PATCHv2 2/4] Input: keypad: Add smsc ece1099 keypad driver

2012-09-05 Thread Poddar, Sourav
+ other external mailing list (did a reply to by mistake) On Thu, Sep 6, 2012 at 11:04 AM, Poddar, Sourav wrote: > Hi Vaibhav, > > On Thu, Sep 6, 2012 at 12:01 AM, Vaibhav Hiremath wrote: >> >> >> On 9/5/2012 5:06 PM, Sourav Poddar wrote: >>> From: G, Manjunath Kondaiah >>> >>> SMSC ECE1099 is

[PATCH v2] pwm: pwm-tiehrpwm: Fix conflicting channel period setting

2012-09-05 Thread Philip, Avinash
EHRPWM hardware supports 2 independent PWM channels. However the device uses only one register to handle period setting for both channels. So both channels should be configured for same period (in nsec). Fix the same by returning error for conflicting period values. However, allow 1.

Re: [PATCH 2/2] mm: support MIGRATE_DISCARD

2012-09-05 Thread Minchan Kim
Hi Mel, On Wed, Sep 05, 2012 at 11:56:11AM +0100, Mel Gorman wrote: > On Wed, Sep 05, 2012 at 05:11:13PM +0900, Minchan Kim wrote: > > This patch introudes MIGRATE_DISCARD mode in migration. > > It drops *clean cache pages* instead of migration so that > > migration latency could be reduced by

[PATCH V2 2/2] pwm: pwm-tiehrpwm: Add support for configuring polarity of PWM

2012-09-05 Thread Philip, Avinash
EHRPWM hardware supports polarity configuration of PWM output. However configuration of polarity done in hardware only in .enable() to ensure PWM output present only after enabling PWM. This commit adds support for polarity configuration for EHRPWM. When being here, remove configuring of polarity

[PATCH V2 1/2] pwm: pwm-tiecap: Add support for configuring polarity of PWM

2012-09-05 Thread Philip, Avinash
ECAP APWM hardware supports polarity configuration of PWM output. This commit adds support for polarity configuration of ECAP APWM. Signed-off-by: Philip, Avinash --- :100644 100644 4b66889... a3d21e7... M drivers/pwm/pwm-tiecap.c drivers/pwm/pwm-tiecap.c | 22 ++ 1 files

[PATCH V2 0/2] Support for configuring polarity of PWM

2012-09-05 Thread Philip, Avinash
These patch set adds support for configuring polarity of eCAP & eHRPWM. Philip, Avinash (2): pwm: pwm-tiecap: Add support for configuring polarity of PWM pwm: pwm-tiehrpwm: Add support for configuring polarity of PWM drivers/pwm/pwm-tiecap.c | 22 +

[PATCH] usb: phy: fix build break

2012-09-05 Thread Venu Byravarasu
During phy interface separation from otg.h, as the enum "usb_otg_state" was having multiple otg states info and removal of member 'state' of this enum type from usb_phy struct did not generate any compilation issues, I removed member state from struct usb_phy. As this is causing build break in

Re: [RFC 1/2] virtio_console: Add support for DMA memory allocation

2012-09-05 Thread Michael S. Tsirkin
On Thu, Sep 06, 2012 at 11:34:25AM +0930, Rusty Russell wrote: > "Michael S. Tsirkin" writes: > > On Tue, Sep 04, 2012 at 06:58:47PM +0200, Sjur Brændeland wrote: > >> Hi Michael, > >> > >> > Exactly. Though if we just fail load it will be much less code. > >> > > >> > Generally, using a feature

[PATCH] audit: use list_move instead of list_del/list_add in audit-tree

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun Using list_move() instead of list_del() + list_add(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun --- kernel/audit_tree.c | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git

Re: [GIT] kbuild rc fixes for v3.6

2012-09-05 Thread Stephen Rothwell
Hi Linus, On Wed, 5 Sep 2012 20:32:00 -0700 Linus Torvalds wrote: > > On Mon, Sep 3, 2012 at 12:18 PM, Michal Marek wrote: > > > > The following changes since commit 0d7614f09c1ebdbaa1599a5aba7593f147bf96ee: > > > > Linux 3.6-rc1 (2012-08-02 16:38:10 -0700) > > > > are available in the git

[PATCH] clockevents: use list_move instead of list_del/list_add

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun Using list_move() instead of list_del() + list_add(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun --- kernel/time/clockevents.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git

Re: [PATCH v2 1/1] mmc: block: Add write packing control

2012-09-05 Thread merez
Hi Venkat, Sorry for the late response. I came back from a long vacation and had many issues to take care of. If you still need a rebased version of the packed commands patches, I can send a rebased version of the write packing control patch once Seungwon Jeon will send the rebased version of the

[PATCH v2 3/3] memory-hotplug: bug fix race between isolation and allocation

2012-09-05 Thread Minchan Kim
Like below, memory-hotplug makes race between page-isolation and page-allocation so it can hit BUG_ON in __offline_isolated_pages. CPU A CPU B start_isolate_page_range set_migratetype_isolate spin_lock_irqsave(zone->lock)

[PATCH v2 2/3] mm: remain migratetype in freed page

2012-09-05 Thread Minchan Kim
The page allocator caches the pageblock information in page->private while it is in the PCP freelists but this is overwritten with the order of the page when freed to the buddy allocator. This patch stores the migratetype of the page in the page->index field so that it is available at all times

[PATCH v2 1/3] use get_page_migratetype instead of page_private

2012-09-05 Thread Minchan Kim
page allocator uses set_page_private and page_private for handling migratetype when it frees page. Let's replace them with [set|get] _freepage_migratetype to make it more clear. * from v1 * Change set_page_migratetype with set_freepage_migratetype * Add comment on set_freepage_migratetype

[PATCH v2 0/3] memory-hotplug: handle page race between allocation and isolation

2012-09-05 Thread Minchan Kim
Memory hotplug has a subtle race problem so this patchset fixes the problem (Look at [3/3] for detail and please confirm the problem before review other patches in this series.) [1/3] is just clean up and help for [2/3]. [2/3] keeps the migratetype information to freed page's index field

linux-next: build failure after merge of the tty tree

2012-09-05 Thread Stephen Rothwell
Hi Greg, After merging the tty tree, today's linux-next build (x86_64 allmodconfig) failed like this: In file included from drivers/tty/serial/sccnxp.c:20:0: include/linux/serial_core.h: In function 'uart_handle_break': include/linux/serial_core.h:543:30: error: dereferencing pointer to

Re: A workaround for request_firmware() stuck in module_init

2012-09-05 Thread Benjamin Herrenschmidt
On Thu, 2012-09-06 at 00:18 +0100, Alan Cox wrote: > > But then you have cases where probe() -> register_with_my_subsystem() -> > > open(). Network devices come to mind. IE. udev must be able to deal with > > a synchronous firmware load from probe I'm afraid. > > I don't believe so. You have > >

Re: [PATCH v2 2/2] virtio-ring: Allocate indirect buffers from cache when possible

2012-09-05 Thread Michael S. Tsirkin
On Thu, Sep 06, 2012 at 10:32:48AM +0930, Rusty Russell wrote: > Sasha Levin writes: > >> On Wed, Aug 29, 2012 at 05:03:03PM +0200, Sasha Levin wrote: > >>> I've also re-ran it on a IBM server type host instead of my laptop. Here > >>> are the > >>> results: > >>> > >>> Vanilla kernel: > >>> >

[PATCH] [v2] sata_fsl: add workaround for data length mismatch on freescale V2 controller

2012-09-05 Thread Shaohui Xie
The freescale V2 SATA controller checks if the received data length matches the programmed length 'ttl', if not, it assumes that this is an error. In ATAPI, the 'ttl' is based on max allocation length and not the actual data transfer length, controller will raise 'DLM' (Data length Mismatch) error

linux-next: build failure after merge of the driver-core tree (workqueues tree related)

2012-09-05 Thread Stephen Rothwell
Hi Greg, Tejun, After merging the driver-core tree, today's linux-next build (x86_64 allmodconfig) failed like this: drivers/extcon/extcon-adc-jack.c: In function 'adc_jack_probe': drivers/extcon/extcon-adc-jack.c:146:2: error: implicit declaration of function 'INIT_DELAYED_WORK_DEFERRABLE'

Re: [PATCH 3/3] memory-hotplug: bug fix race between isolation and allocation

2012-09-05 Thread Minchan Kim
On Wed, Sep 05, 2012 at 10:40:41AM +0100, Mel Gorman wrote: > On Wed, Sep 05, 2012 at 04:26:02PM +0900, Minchan Kim wrote: > > Like below, memory-hotplug makes race between page-isolation > > and page-allocation so it can hit BUG_ON in __offline_isolated_pages. > > > > CPU A

[PATCH] staging: wlags49_h2: use list_move instead of list_del/list_add

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun Using list_move() instead of list_del() + list_add(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun --- drivers/staging/wlags49_h2/wl_netdev.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff

[PATCH] staging: gdm72xx: use list_move instead of list_del/list_add

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun Using list_move() instead of list_del() + list_add(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun --- drivers/staging/gdm72xx/gdm_usb.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff

[PATCH] iommu/exynos: use list_del_init instead of list_del/INIT_LIST_HEAD

2012-09-05 Thread Wei Yongjun
From: Wei Yongjun Using list_del_init() instead of list_del() + INIT_LIST_HEAD(). spatch with a semantic match is used to found this problem. (http://coccinelle.lip6.fr/) Signed-off-by: Wei Yongjun --- drivers/iommu/exynos-iommu.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff

[PATCH v3] ARM: S3C24XX: Add WIZnet W5300E01-ARM board support

2012-09-05 Thread Taehun Kim
- The gpio routines are changed to use the gpio_request() functions from comments of Vasily and Sylwester. - The mistake of adding a character by my email client automatically is fixed. Please review this patch and apply it if do not have any problems. Signed-off-by: Taehun Kim ---

Re: A workaround for request_firmware() stuck in module_init

2012-09-05 Thread Ming Lei
On Thu, Sep 6, 2012 at 12:30 AM, Alan Cox wrote: >> Yes, deferring the load may fix the built in case, but which also >> introduces much work on changes of current drivers. In fact, >> there are few guys who complained the built in case. > > It fixes the modular case too. Sorry, I don't see

[PATCH] extcon: use IRQF_ONESHOT

2012-09-05 Thread Fengguang Wu
Generated by: scripts/coccinelle/misc/irqf_oneshot.cocci Make sure threaded IRQs without a primary handler are always requested with IRQF_ONESHOT. Signed-off-by: Fengguang Wu --- Please take the patch only if it's a positive warning. Thanks! cocci-output-4211-882baf-extcon-max77693.c |2

linux-next: build failure after merge of the spi-mb tree

2012-09-05 Thread Stephen Rothwell
38ab18caa0ad ("spi: spi-gpio: Add DT bindings"). Please build test these sort of patches with and without CONFIG_OF. I have used the spi-mb tree from next-20120905 for today. -- Cheers, Stephen Rothwells...@canb.auug.org.au pgppgL0onmWo8.pgp Description: PGP signature

Re: Re: [PATCH -next] extcon: extcon-arizona depends on INPUT

2012-09-05 Thread Greg KH
On Thu, Sep 06, 2012 at 08:54:16AM +0530, anish singh wrote: > On Thu, Sep 6, 2012 at 5:36 AM, Greg KH wrote: > > On Wed, Sep 05, 2012 at 11:47:31PM +, MyungJoo Ham wrote: > >> > On Wed, Sep 05, 2012 at 11:28:03AM +0800, Mark Brown wrote: > >> > > On Tue, Sep 04, 2012 at 04:51:00PM -0700,

Re: [GIT] kbuild rc fixes for v3.6

2012-09-05 Thread Linus Torvalds
On Mon, Sep 3, 2012 at 12:18 PM, Michal Marek wrote: > > 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://git.kernel.org/pub/scm/linux/kernel/git/mmarek/kbuild-2.6.git

Re: [tip:sched/core] sched: Fix load avg vs cpu-hotplug

2012-09-05 Thread Rakib Mullick
On 9/6/12, Peter Zijlstra wrote: > On Wed, 2012-09-05 at 19:01 +0200, Peter Zijlstra wrote: >> > Please do a delta. > > OK, so I suppose something like the below ought to do. Paul its slightly > different than the one in your tree, given the changelog below, do you > see anything wrong with it? >

Re: Re: [PATCH -next] extcon: extcon-arizona depends on INPUT

2012-09-05 Thread anish singh
On Thu, Sep 6, 2012 at 5:36 AM, Greg KH wrote: > On Wed, Sep 05, 2012 at 11:47:31PM +, MyungJoo Ham wrote: >> > On Wed, Sep 05, 2012 at 11:28:03AM +0800, Mark Brown wrote: >> > > On Tue, Sep 04, 2012 at 04:51:00PM -0700, Randy Dunlap wrote: >> > > >> > > > This build error still happens in

linux-next: manual merge of the trivial tree with the powerpc tree

2012-09-05 Thread Stephen Rothwell
Hi Jiri, Today's linux-next merge of the trivial tree got a conflict in drivers/scsi/ipr.c between commit d3dbeef657fd ("powerpc: Rename 64-bit PVR constants to PVR_foo") from the powerpc tree and commit 203fa3fe9c9d ("ipr: fix small coding style issues") from the trivial tree. Just context

Re: [dm-devel] [PATCH v8 3/8] dm: Use bioset's front_pad for dm_rq_clone_bio_info

2012-09-05 Thread Jun'ichi Nomura
On 09/06/12 05:27, Kent Overstreet wrote: > @@ -2718,7 +2705,8 @@ struct dm_md_mempools *dm_alloc_md_mempools(unsigned > type, unsigned integrity) > if (!pools->tio_pool) > goto free_io_pool_and_out; > > - pools->bs = bioset_create(pool_size, 0); > + pools->bs =

Re: [PATCH] OMAP GPIO - don't wake from suspend unless requested.

2012-09-05 Thread NeilBrown
On Mon, 3 Sep 2012 22:59:06 -0700 "Shilimkar, Santosh" wrote: > On Sun, Aug 26, 2012 at 6:29 PM, Shilimkar, Santosh > wrote: > > On Sun, Aug 26, 2012 at 3:53 PM, NeilBrown wrote: > >> > >> On Sun, 26 Aug 2012 09:47:50 +0530 "Shilimkar, Santosh" > >> wrote: > >> > >> > + Jon, > >> > > >> > On

Re: [PATCH] slab: fix the DEADLOCK issue on l3 alien lock

2012-09-05 Thread Michael Wang
On 09/05/2012 09:55 PM, Christoph Lameter wrote: > On Wed, 5 Sep 2012, Michael Wang wrote: > >> Since the cachep and cachep->slabp_cache's l3 alien are in the same lock >> class, >> fake report generated. > > Ahh... That is a key insight into why this occurs. > >> This should not happen since

Re: [PATCH] Add a page cache-backed balloon device driver.

2012-09-05 Thread Rusty Russell
Paolo Bonzini writes: > Il 02/07/2012 02:29, Rusty Russell ha scritto: >> VIRTIO_BALLOON_F_MUST_TELL_HOST >> implies you should tell the host (eventually). I don't know if any >> implementations actually care though. > > This is indeed broken, because it is a "negative" feature: it tells you >

Re: [PATCH 1/2] module: allow loading module from fd

2012-09-05 Thread Rusty Russell
Kees Cook writes: > This changes the init_module syscall so that when the first argument > (blob address) is NULL, the second argument is used as a file descriptor > to the module (instead of length). The third argument (module arguments) > remains unchanged. Do you know why Linus hates ioctls?

Re: [PATCH] virtio-blk: Fix kconfig option

2012-09-05 Thread Rusty Russell
"Michael S. Tsirkin" writes: > On Tue, Sep 04, 2012 at 03:53:53PM +0930, Rusty Russell wrote: >> Kent Overstreet writes: >> >> > CONFIG_VIRTIO isn't exposed, everything else is supposed to select it >> > instead. >> >> This is a slight mis-understanding. It's supposed to be selected by >> the

Re: [PATCH] virtio: support reserved vqs

2012-09-05 Thread Rusty Russell
"Michael S. Tsirkin" writes: > virtio network device multiqueue support reserves > vq 3 for future use (useful both for future extensions and to make it > pretty - this way receive vqs have even and transmit - odd numbers). > Make it possible to skip initialization for > specific vq numbers by

Re: [PATCH v2 2/2] virtio-ring: Allocate indirect buffers from cache when possible

2012-09-05 Thread Rusty Russell
Sasha Levin writes: >> On Wed, Aug 29, 2012 at 05:03:03PM +0200, Sasha Levin wrote: >>> I've also re-ran it on a IBM server type host instead of my laptop. Here >>> are the >>> results: >>> >>> Vanilla kernel: >>> >>> MIGRATED TCP STREAM TEST from 0.0.0.0 (0.0.0.0) port 0 AF_INET to >>>

Re: [PATCH 1/5] virtio-ring: move queue_index to vring_virtqueue

2012-09-05 Thread Rusty Russell
Paolo Bonzini writes: > From: Jason Wang > > Instead of storing the queue index in transport-specific virtio structs, > this patch moves them to vring_virtqueue and introduces an helper to get > the value. This lets drivers simplify their management and tracing of > virtqueues. > >

Re: [PATCH v2 2/2] virtio-ring: Allocate indirect buffers from cache when possible

2012-09-05 Thread Rusty Russell
Sasha Levin writes: > On 08/28/2012 03:20 PM, Michael S. Tsirkin wrote: >> On Tue, Aug 28, 2012 at 03:04:03PM +0200, Sasha Levin wrote: >>> Currently if VIRTIO_RING_F_INDIRECT_DESC is enabled we will >>> use indirect descriptors and allocate them using a simple >>> kmalloc(). >>> >>> This patch

Re: [PATCH 2/5] virtio: introduce an API to set affinity for a virtqueue

2012-09-05 Thread Rusty Russell
Paolo Bonzini writes: > From: Jason Wang > > Sometimes, virtio device need to configure irq affinity hint to maximize the > performance. Instead of just exposing the irq of a virtqueue, this patch > introduce an API to set the affinity for a virtqueue. > > The api is best-effort, the affinity

Re: [RFC] module: signature infrastructure

2012-09-05 Thread Rusty Russell
Mimi Zohar writes: > On Wed, 2012-09-05 at 09:59 +0930, Rusty Russell wrote: >> "Kasatkin, Dmitry" writes: >> > Hi, >> > >> > Please read bellow... >> > >> > On Tue, Sep 4, 2012 at 8:55 AM, Rusty Russell >> > wrote: >> >> OK, I took a look at the module.c parts of David and Dmitry's

Re: [RFC 1/2] virtio_console: Add support for DMA memory allocation

2012-09-05 Thread Rusty Russell
"Michael S. Tsirkin" writes: > On Tue, Sep 04, 2012 at 06:58:47PM +0200, Sjur Brændeland wrote: >> Hi Michael, >> >> > Exactly. Though if we just fail load it will be much less code. >> > >> > Generally, using a feature bit for this is a bit of a problem though: >> > normally driver is expected

Re: [PATCH 3/3] memory-hotplug: bug fix race between isolation and allocation

2012-09-05 Thread Minchan Kim
Hello Xishi, On Thu, Sep 06, 2012 at 10:35:39AM +0800, qiuxishi wrote: > On 2012/9/5 17:40, Mel Gorman wrote: > > > On Wed, Sep 05, 2012 at 04:26:02PM +0900, Minchan Kim wrote: > >> Like below, memory-hotplug makes race between page-isolation > >> and page-allocation so it can hit BUG_ON in

[RFC v2] memory-hotplug: remove MIGRATE_ISOLATE from free_area->free_list

2012-09-05 Thread Minchan Kim
Normally, MIGRATE_ISOLATE type is used for memory-hotplug. But it's irony type because the pages isolated would exist as free page in free_area->free_list[MIGRATE_ISOLATE] so people can think of it as allocatable pages but it is *never* allocatable. It ends up confusing NR_FREE_PAGES vmstat so it

Re: A workaround for request_firmware() stuck in module_init

2012-09-05 Thread Linus Torvalds
On Wed, Sep 5, 2012 at 9:30 AM, Alan Cox wrote: > > It's not about a workaround but about doing it properly for the long term > and doing it in one place. It's also not a "great change", its a small > change. udev needs to get fixed regardless. Stop this "we can break stuff" crap. Who maintains

Re: [RFC PATCH v2] trace: Move trace event enable from fs_initcall to early_initcall

2012-09-05 Thread Steven Rostedt
On Wed, 2012-09-05 at 13:53 -0400, Steven Rostedt wrote: > On Tue, 2012-09-04 at 22:18 -0300, Ezequiel Garcia wrote: > > Hi Steven, > > > > On 8/17/12, Steven Rostedt wrote: > > > On Fri, 2012-08-17 at 09:03 -0300, Ezequiel Garcia wrote: > > >> This patch splits trace event initialization in two

Re: [PATCH 3/3] memory-hotplug: bug fix race between isolation and allocation

2012-09-05 Thread qiuxishi
On 2012/9/5 17:40, Mel Gorman wrote: > On Wed, Sep 05, 2012 at 04:26:02PM +0900, Minchan Kim wrote: >> Like below, memory-hotplug makes race between page-isolation >> and page-allocation so it can hit BUG_ON in __offline_isolated_pages. >> >> CPU A CPU B >>

Re: [PATCH] drm/exynos: fix double call of drm_prime_(init/destroy)_file_private

2012-09-05 Thread InKi Dae
Hi, 2012/9/6 Mandeep Singh Baines : > The double invocations are incorrect but seem to be safe so I don't > think this will fix any bugs. > > Before: > > [7.639366] drm_prime_init_file ee3675d0 > [7.639377] drm_prime_init_file ee3675d0 > [7.639507] drm_prime_destroy_file ee3675d0 > [

Re: [RFC PATCH 2/3] tcm_iscsi: support multiple sizes in the scatterlist

2012-09-05 Thread Nicholas A. Bellinger
On Wed, 2012-09-05 at 17:13 +0200, Paolo Bonzini wrote: > The next patch will use multiple orders to satisfy the allocation > of the data buffers. > > We need to support this in iscsi_map_iovec and iscsi_unmap_iovec. > The idea here is to walk each relevant page in the scatterlist > (which may

[PATCH] serial_core: fix sizeof(pointer)

2012-09-05 Thread Fengguang Wu
sizeof when applied to a pointer typed expression gives the size of the pointer. Generated by: scripts/coccinelle/misc/noderef.cocci Signed-off-by: Fengguang Wu --- cocci-output-38612-39d907-serial_core.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---

Re: [PATCH 2/3] mm: remain migratetype in freed page

2012-09-05 Thread Minchan Kim
On Wed, Sep 05, 2012 at 10:25:34AM +0100, Mel Gorman wrote: > On Wed, Sep 05, 2012 at 04:26:01PM +0900, Minchan Kim wrote: > > Page allocator doesn't keep migratetype information to page > > when the page is freed. This patch remains the information > > to freed page's index field which isn't used

Re: [RFC PATCH 3/3] target: try satisfying memory requests with contiguous blocks

2012-09-05 Thread Nicholas A. Bellinger
On Wed, 2012-09-05 at 17:13 +0200, Paolo Bonzini wrote: > transport_generic_get_mem's strategy of allocating pages one-by-one > for the data scatterlist may fail for even not-so-big data transfers > if the underlying device poses a small limit on the number of segments. > > This patch fixes this

[PATCH 2/2] perf header: Prepare tracepoint events regardless of name

2012-09-05 Thread Namhyung Kim
From: Namhyung Kim Current perf_evlist__set_tracepoint_names is a misnomer because it finds and sets correspoding event_format in addition to the name. So skipping it when a event has set name already caused a trouble. Rename it and set name only a event doesn't have one. Reported-by: David

[PATCH 1/2] perf header: Fix a typo on evsel

2012-09-05 Thread Namhyung Kim
From: Namhyung Kim For checking return value of the strdup, 'event' should be 'evsel'. Signed-off-by: Namhyung Kim --- tools/perf/util/header.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/util/header.c b/tools/perf/util/header.c index

Re: [PATCH 1/3] mm: use get_page_migratetype instead of page_private

2012-09-05 Thread Minchan Kim
Hi Kame, On Thu, Sep 06, 2012 at 11:02:47AM +0900, Kamezawa Hiroyuki wrote: > (2012/09/05 16:26), Minchan Kim wrote: > > page allocator uses set_page_private and page_private for handling > > migratetype when it frees page. Let's replace them with [set|get] > > _page_migratetype to make it more

Re: [PATCH 1/3] mm: use get_page_migratetype instead of page_private

2012-09-05 Thread Minchan Kim
Hi Mel, On Wed, Sep 05, 2012 at 10:09:55AM +0100, Mel Gorman wrote: > On Wed, Sep 05, 2012 at 04:26:00PM +0900, Minchan Kim wrote: > > page allocator uses set_page_private and page_private for handling > > migratetype when it frees page. Let's replace them with [set|get] > > _page_migratetype to

Re: [PATCH] drm/exynos: fix double call of drm_prime_(init/destroy)_file_private

2012-09-05 Thread 김승우
Hi Mandeep, On 2012년 09월 06일 06:47, Mandeep Singh Baines wrote: > The double invocations are incorrect but seem to be safe so I don't > think this will fix any bugs. > > Before: > > [7.639366] drm_prime_init_file ee3675d0 > [7.639377] drm_prime_init_file ee3675d0 > [7.639507]

Re: [RFC PATCH] perf: add SLANG_INC for slang.h

2012-09-05 Thread Liang Li
On 2012-09-06 06:41, Arnaldo Carvalho de Melo wrote: > Em Wed, Sep 05, 2012 at 10:23:03PM +0100, Richard Purdie escreveu: > > On Wed, 2012-09-05 at 13:54 -0700, Arnaldo Carvalho de Melo wrote: > > > Em Fri, Aug 24, 2012 at 11:10:39AM +0800, Liang Li escreveu: > > > > CFLAGS was previously hard

Re: [PATCH 03/11 V5] workqueue: new day don't need WORKER_REBIND for busy rebinding

2012-09-05 Thread Lai Jiangshan
On 09/06/2012 02:31 AM, Tejun Heo wrote: > On Wed, Sep 05, 2012 at 06:37:40PM +0800, Lai Jiangshan wrote: >> because old busy_worker_rebind_fn() have to wait until all idle worker >> finish. >> so we have to use two flags WORKER_UNBOUND and WORKER_REBIND to avoid >> prematurely clear all

Re: [RFC] memory-hotplug: remove MIGRATE_ISOLATE from free_area->free_list

2012-09-05 Thread Minchan Kim
Hi Michal, On Wed, Sep 05, 2012 at 07:28:23PM +0200, Michal Nazarewicz wrote: > On Wed, Sep 05 2012, Minchan Kim wrote: > > Normally, MIGRATE_ISOLATE type is used for memory-hotplug. > > But it's irony type because the pages isolated would exist > > as free page in

Re: [PATCH 1/3] mm: use get_page_migratetype instead of page_private

2012-09-05 Thread Kamezawa Hiroyuki
(2012/09/05 16:26), Minchan Kim wrote: > page allocator uses set_page_private and page_private for handling > migratetype when it frees page. Let's replace them with [set|get] > _page_migratetype to make it more clear. > > Signed-off-by: Minchan Kim Hmm. one request from me. > --- >

[PATCH RESEND] USB/host: Cleanup unneccessary irq disable code

2012-09-05 Thread Liu, Chuansheng
Because the IRQF_DISABLED as the flag is now a NOOP and has been deprecated and in hardirq context the interrupt is disabled. so in usb/host code: Removing the usage of flag IRQF_DISABLED; Removing the calling local_irq save/restore actions in irq handler usb_hcd_irq(); Signed-off-by: liu

Re: [RFC PATCH 0/3] target: try satisfying memory requests with higher-order allocations

2012-09-05 Thread Nicholas A. Bellinger
On Wed, 2012-09-05 at 17:13 +0200, Paolo Bonzini wrote: > Hi all, > > while testing PSCSI I noticed that even requests for a smallish amount > of data (approximately 700 KB) failed due to an excessive number of > segments in the request. In fact, using alloc_page resulted in a > completely

[GIT PULL] MMC fixes for 3.6-rc5

2012-09-05 Thread Chris Ball
Hi Linus, Please pull from: git://git.kernel.org/pub/scm/linux/kernel/git/cjb/mmc.git tags/mmc-fixes-for-3.6-rc5 to receive MMC fixes for 3.6-rc5. There are no merge conflicts, and the patches have been tested in linux-next. Thanks. The following changes since commit

Re: [PATCH 04/10 V4] workqueue: add manage_workers_slowpath()

2012-09-05 Thread Lai Jiangshan
On 09/05/2012 09:12 AM, Tejun Heo wrote: > Hello, Lai. > > On Sun, Sep 02, 2012 at 12:28:22AM +0800, Lai Jiangshan wrote: >> If hotplug code grabbed the manager_mutex and worker_thread try to create >> a worker, the manage_worker() will return false and worker_thread go to >> process work items.

Re: [PATCH 2/2] workqueue: fix possible deadlock in idle worker rebinding

2012-09-05 Thread Lai Jiangshan
On 09/06/2012 07:11 AM, Tejun Heo wrote: > On Tue, Sep 04, 2012 at 11:16:32PM -0700, Tejun Heo wrote: >> Currently, rebind_workers() and idle_worker_rebind() are two-way >> interlocked. rebind_workers() waits for idle workers to finish >> rebinding and rebound idle workers wait for

Re: [PATCH 14/17] perf header: Set tracepoint event name only if not set

2012-09-05 Thread Namhyung Kim
On Wed, 05 Sep 2012 18:01:16 -0600, David Ahern wrote: > On 9/5/12 5:08 PM, Arnaldo Carvalho de Melo wrote: >> From: Namhyung Kim >> >> The event name can be set already by processing a event_desc data. >> >> So check it before setting to prevent possible leak. >> >> Signed-off-by: Namhyung Kim

Re: 3.5-rc7: nouveau doesn't X on NVC0

2012-09-05 Thread Dave Airlie
On Thu, Sep 6, 2012 at 11:15 AM, Linus Torvalds wrote: > [ This got dropped somehow - it's in my draft folder. The bisection > may be irrelevant now: does it work with current git, since we've had > some nouveau changes? ] > > On Tue, Aug 28, 2012 at 8:26 AM, Alexey Dobriyan wrote: >> Ping! >>

Re: [PATCH 1/1] x86/oprofile: Fix the calltrace upon profiling some specified events with oprofile

2012-09-05 Thread wyang1
On 09/04/2012 06:24 PM, Robert Richter wrote: Wei, see my comments below. On 27.08.12 09:32:13, wei.y...@windriver.com wrote: From: Wei Yang Upon enabling the call-graph functionality of oprofile, A few minutes later the following calltrace will always occur. BUG: unable to handle kernel

[PATCH v6] uartclk value from serial_core exposed to sysfs

2012-09-05 Thread Tomas Hlavacek
Added file /sys/devices/.../tty/ttySX/uartclk to allow reading uartclk value in struct uart_port in serial_core via sysfs. tty_register_device() has been generalized and refactored in order to add support for setting drvdata and attribute_group to the device. Signed-off-by: Tomas Hlavacek ---

Re: [PATCH 02/11 V5] workqueue: async idle rebinding

2012-09-05 Thread Lai Jiangshan
On 09/06/2012 02:06 AM, Tejun Heo wrote: > Hello, Lai. > > Ooh, I like the approach. That said, I think it's a bit too invasive > for 3.6-fixes. I'll merge the two patches I posted yesterday in > 3.6-fixes. Let's do this restructuring in for-3.7. OK for me. it is too complicated for 3.6. >

Re: [PATCH] perf: clean target should do clean for lib/traceevent too

2012-09-05 Thread Namhyung Kim
On Wed, 5 Sep 2012 18:53:36 -0600, David Ahern wrote: > It's built as part of perf, so it should be cleaned too. > > Signed-off-by: David Ahern > Cc: Steven Rostedt Tested-by: Namhyung Kim Thanks, Namhyung > --- > tools/perf/Makefile |5 - > 1 file changed, 4 insertions(+), 1

RE: [PATCH] USB/host: Cleanup unneccessary irq disable code

2012-09-05 Thread Liu, Chuansheng
> > From: liu chuansheng > > Subject: [PATCH] USB/host: Cleanup unneccessary irq disable code > > Why is this in the patch? > > Please resend it in a format that I do not have to manually edit the patch. > > greg k-h Thanks your teaching, resend again. Because the IRQF_DISABLED as the flag

Re: 3.5-rc7: nouveau doesn't X on NVC0

2012-09-05 Thread Linus Torvalds
[ This got dropped somehow - it's in my draft folder. The bisection may be irrelevant now: does it work with current git, since we've had some nouveau changes? ] On Tue, Aug 28, 2012 at 8:26 AM, Alexey Dobriyan wrote: > Ping! > > No X for me with 3.6-rc2. Can you possibly bisect it, at least

Re: [PATCH] USB/host: Cleanup unneccessary irq disable code

2012-09-05 Thread gre...@linuxfoundation.org
On Thu, Sep 06, 2012 at 12:39:09AM +, Liu, Chuansheng wrote: > > This patch is corrupted and can not be applied at all. Please fix your > > email > > client and try again. > > > > greg k-h > > I am very sorry to waste your time, resend it again. > > From: liu chuansheng > Subject:

Re: [PATCH 05/11 V5] workqueue: Add @bind arguement back without change any thing

2012-09-05 Thread Lai Jiangshan
On 09/06/2012 03:49 AM, Tejun Heo wrote: > Hello, > > On Wed, Sep 05, 2012 at 06:37:42PM +0800, Lai Jiangshan wrote: >> Ensure the gcwq->flags is only accessed with gcwq->lock held. >> And make the code more easier to understand. >> >> In all current callsite of create_worker(), DISASSOCIATED

Re: [PATCHv5 1/1] uartclk value from serial_core exposed to sysfs

2012-09-05 Thread Tomas Hlavacek
Hello Greg, On Thu, Sep 6, 2012 at 1:42 AM, Greg KH wrote: > On Thu, Sep 06, 2012 at 01:16:56AM +0200, Tomas Hlavacek wrote: >> Added file /sys/devices/.../tty/ttySX/uartclk to allow reading >> uartclk value in struct uart_port in serial_core via sysfs. >> >> tty_register_device() has been

RE: [Patch 0/1]drm_irq: Introducing the irq_thread support

2012-09-05 Thread Liu, Chuansheng
> Well, you cant use the pre_install/post_install hooks the drm_irq code > provides, > but yes, just do the request_irq in your driver code at the right time, with > the > right parameters. Much easier than adding code to a part of the drm core > fraught with backwards-compat stuff no one really

[PATCH] perf: clean target should do clean for lib/traceevent too

2012-09-05 Thread David Ahern
It's built as part of perf, so it should be cleaned too. Signed-off-by: David Ahern Cc: Steven Rostedt Cc: Namhyung Kim --- tools/perf/Makefile |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/tools/perf/Makefile b/tools/perf/Makefile index 90d8e09..e7e62b2 100644

Re: [PATCH] perf: clean should remove libtraceevent.a too

2012-09-05 Thread David Ahern
On 9/5/12 6:42 PM, Steven Rostedt wrote: On Wed, 2012-09-05 at 18:10 -0600, David Ahern wrote: It's built as part of perf, so it should be cleaned too. Signed-off-by: David Ahern Cc: Steven Rostedt Cc: Namhyung Kim --- tools/perf/Makefile |1 + 1 file changed, 1 insertion(+) diff

RE: [Patch 0/1]drm_irq: Introducing the irq_thread support

2012-09-05 Thread Liu, Chuansheng
> For a kms drm driver (and tbh, doing a non-kms driver today is not a great > idea), > there's no reason to use the drm_irq_install/_unistall helpers. > Can not understand well, I found many GPU drivers are using drm_irq helpers' function, including ours:) -- To unsubscribe from this list:

Re: [PATCH -next] soc: fix wm0010.c printk format warning

2012-09-05 Thread Mark Brown
On Wed, Sep 05, 2012 at 05:11:08PM -0700, Randy Dunlap wrote: > On 09/05/2012 05:06 PM, Mark Brown wrote: > > Somone already sent this. As ever please try to use subject lines > > appropriate for the subsystem. Please also consider who you're CCing - > Unfortunately the MAINTAINERS file does

Re: [PATCH] perf: clean should remove libtraceevent.a too

2012-09-05 Thread Steven Rostedt
On Wed, 2012-09-05 at 18:10 -0600, David Ahern wrote: > It's built as part of perf, so it should be cleaned too. > > Signed-off-by: David Ahern > Cc: Steven Rostedt > Cc: Namhyung Kim > --- > tools/perf/Makefile |1 + > 1 file changed, 1 insertion(+) > > diff --git a/tools/perf/Makefile

RE: [Patch 0/1]drm_irq: Introducing the irq_thread support

2012-09-05 Thread Liu, Chuansheng
> This possibly ought to be submitted in parallel with the code that uses it so > that > the whole proposal can be evaluated as one thing ? > > Alan Patch is here, thanks. From: liu chuansheng Subject: [PATCH] drm_irq: Introducing the irq_thread support For some GPUs, the irq handler need

Re: [PATCH 3.6 0/3] Fix use-after-free in PSCSI

2012-09-05 Thread Nicholas A. Bellinger
On Wed, 2012-09-05 at 17:09 +0200, Paolo Bonzini wrote: > Hi, > > this series fixes the bug I reported with wrong sense data. The > memory corruption is caused by using the sense data after freeing > it. The series corrects it by moving the copy of the sense data > earlier, to the

Re: PCI/e1000 BUG: unable to handle kernel paging request at 0ffff163

2012-09-05 Thread Fengguang Wu
On Wed, Sep 05, 2012 at 11:41:04AM -0700, Yinghai Lu wrote: > On Tue, Sep 4, 2012 at 11:51 PM, Fengguang Wu wrote: > > Yinghai, > > > > There are many kernel paging errors showing up in tree: > > > > git://git.kernel.org/pub/scm/linux/kernel/git/yinghai/linux-yinghai.git > >

RE: [PATCH] USB/host: Cleanup unneccessary irq disable code

2012-09-05 Thread Liu, Chuansheng
> This patch is corrupted and can not be applied at all. Please fix your email > client and try again. > > greg k-h I am very sorry to waste your time, resend it again. From: liu chuansheng Subject: [PATCH] USB/host: Cleanup unneccessary irq disable code Because the IRQF_DISABLED as the flag

Re: [PATCH] pch_uart: Add eg20t_port lock field, avoid recursive spinlocks

2012-09-05 Thread Darren Hart
On 09/05/2012 05:25 PM, Darren Hart wrote: > > > On 09/05/2012 05:18 PM, Greg Kroah-Hartman wrote: >> On Wed, Sep 05, 2012 at 05:14:48PM -0700, Greg Kroah-Hartman wrote: >>> On Wed, Sep 05, 2012 at 05:04:07PM -0700, Darren Hart wrote: The following patch has been included in linux-next

Re: [PATCH v2] checkpatch: Suggest using min_t or max_t

2012-09-05 Thread Joe Perches
On Thu, 2012-09-06 at 02:16 +0200, Philippe De Muyter wrote: > On Wed, Sep 5, 2012 at 7:07 PM, Joe Perches wrote: > > On Wed, 2012-09-05 at 13:21 +0200, Philippe De Muyter wrote: > >> > v2: Make $match_balanced_parentheses work in perl 5.8 > >> > >> Has this been applied ? > >> > >> v3.3 version

Re: [PATCH] pch_uart: Add eg20t_port lock field, avoid recursive spinlocks

2012-09-05 Thread Darren Hart
On 09/05/2012 05:18 PM, Greg Kroah-Hartman wrote: > On Wed, Sep 05, 2012 at 05:14:48PM -0700, Greg Kroah-Hartman wrote: >> On Wed, Sep 05, 2012 at 05:04:07PM -0700, Darren Hart wrote: >>> The following patch has been included in linux-next >>> (fe89def79c48e2149abdd1e816523e69a9067191) but has

Re: [PATCH -next] soc: fix wm0010.c printk format warning

2012-09-05 Thread Sedat Dilek
On Thu, Sep 6, 2012 at 2:11 AM, Randy Dunlap wrote: > On 09/05/2012 05:06 PM, Mark Brown wrote: > >> On Wed, Sep 05, 2012 at 04:24:57PM -0700, Randy Dunlap wrote: >>> From: Randy Dunlap >>> >>> Fix printk format warning: >> >> Somone already sent this. As ever please try to use subject lines >>

  1   2   3   4   5   6   7   8   9   10   >