Re: Is iommu_num_pages() broken ?

2012-08-04 Thread Benjamin Herrenschmidt
On Sun, 2012-08-05 at 08:39 +0200, Andreas Schwab wrote: > addr & (io_page_size - 1) computes the offset into the page pointed to > by addr. Looks right to me. > > addr & ~(io_page_size - 1) would round addr down to the start of the > page. Ah that's right, for some reason I'm so used to seeing

Re: Is iommu_num_pages() broken ?

2012-08-04 Thread Andreas Schwab
Benjamin Herrenschmidt writes: > static inline unsigned long iommu_num_pages(unsigned long addr, > unsigned long len, > unsigned long io_page_size) > { > unsigned long size = (addr & (io_page_size - 1)) + le

[PATCH] x86: Changing date reference in CONFIG_SWIOTLB help from a relative to fixed point

2012-08-04 Thread Joe Millenbach
CONFIG_SWIOTLB's existing help information refers to "the current generation of Intel's x86-64 CPUs", but the statement is no longer true. After researching on Intel's ark site, I changed the statement to the also true "all Intel CPUs released before 2008". Signed-off-by: Joe Millenbach Reviewed

Re: Linux kernel boot process

2012-08-04 Thread Kumar Sukhani
On Sun, Aug 5, 2012 at 7:40 AM, Guenter Roeck wrote: > > What would be the point ? Linux already boots much faster than Windows, even > with Windows running in "hybrid" mode. And, from what I heard, things will get > even better with systemd. > Whether It will not add any enhancement in boot time?

Re: [Nouveau] [PATCH] nouveau: Do not use nva3 engine for 0xaf chipset

2012-08-04 Thread Ben Skeggs
On Sat, Aug 04, 2012 at 08:00:45AM +0200, Henrik Rydberg wrote: > The nva3 copy engine exhibits random memory corruption in at least one > case, the GeForce 320M (nv50, 0xaf) in the MacBookAir3,1. This patch > omits creating the engine for the specific chipset, falling back to > M2MF, which kills

Re: Fwd: Linux kernel boot process

2012-08-04 Thread Guenter Roeck
On Sat, Aug 04, 2012 at 12:57:55PM +0530, Kumar Sukhani wrote: > I heard that windows 8 have change its boot process to a hybrid mood. > In which it shutdowns the user applications and hibernates the kernel > and device drivers[1]. > Can the same be introduce in Linux kernel? > > [1] http://www.z

[PATCH 02/13] SCTP: register protocol for namespace handling.

2012-08-04 Thread Jan Ariyasu
Add namespace-helper functions to access per-namespace parameters and variables. Create per-namespace control socket for out-of-the- blue packets, initialize per-namespace protocol parameters, and register SCTP protocol for namespace handling. Signed-off-by: Jan Ariyasu --- include/net/sctp/sct

Re: [RFC v2 2/7] user_ns: use new hashtable implementation

2012-08-04 Thread Eric W. Biederman
Sasha Levin writes: > Switch user_ns to use the new hashtable implementation. This reduces the > amount of > generic unrelated code in user_ns. Just looking at this ick. - Your comparison function is broken. - The naming is awkward. hash_get without a reference count being incremented? -

Re: [PATCH] time: Fix adjustment cleanup bug in timekeeping_adjust()

2012-08-04 Thread Tetsuo Handa
Ingo Molnar wrote: > > Bisected to 2a8c0883 "time: Move xtime_nsec adjustment > > underflow handling timekeeping_adjust". > > > > Would you check? > > probably caused by this stray return in timekeeping_adjust(): > > } else > return; > > which should really be a proper

Re: [RFC v2 6/7] tracepoint: use new hashtable implementation

2012-08-04 Thread Steven Rostedt
FYI, Mathieu is the author of this file. -- Steve On Fri, 2012-08-03 at 16:23 +0200, Sasha Levin wrote: > Switch tracepoints to use the new hashtable implementation. This reduces the > amount of > generic unrelated code in the tracepoints. > > Signed-off-by: Sasha Levin > --- > kernel/tracep

Re: [RFC v2] ARM: sched_clock: update epoch_cyc on resume

2012-08-04 Thread Linus Walleij
On Fri, Aug 3, 2012 at 3:28 AM, Colin Cross wrote: > On Fri, Jul 27, 2012 at 8:30 PM, Colin Cross wrote: >> On Fri, Jul 27, 2012 at 6:15 PM, Colin Cross wrote: >>> That patch was merged in 3.4, and my patch is on top of it. Your >>> patch updates epoch_cyc and epoch_ns in suspend, but if the fi

Is iommu_num_pages() broken ?

2012-08-04 Thread Benjamin Herrenschmidt
Hi folks ! I stumbled upon this today: static inline unsigned long iommu_num_pages(unsigned long addr, unsigned long len, unsigned long io_page_size) { unsigned long size = (addr & (io_page_size - 1))

Re: [RFC] page-table walkers vs memory order

2012-08-04 Thread Dr. David Alan Gilbert
* Paul E. McKenney (paul...@linux.vnet.ibm.com) wrote: > On Sat, Aug 04, 2012 at 11:59:10PM +0100, Dr. David Alan Gilbert wrote: > > A compiler could decide to dereference it using a non-faulting load, > > do the calculations or whatever on the returned value of the non-faulting > > load, and th

Re: [PATCH 00/13] SCTP: Enable netns

2012-08-04 Thread David Miller
From: Jan Ariyasu Date: Sat, 4 Aug 2012 15:30:37 -0600 > The following set of patches enable network-namespaces for the SCTP protocol. > > The multitude of global parameters are stored in a net_generic > structure, and the bulk of the patches enable the protocol to access > the parameters on a

Re: [PATCH] drivers/dma/sirf-dma.c: fix usage of devm functions

2012-08-04 Thread Barry Song
2012/8/4 Julia Lawall > > From: Julia Lawall > > Fix some problems with the use of devm_ functions. > > devm_kzalloc: devm_kfree is not needed > > devm_ioremap: iounmap should not be used, no free is needed > > devm_request_irq: the devm_free_irq is followed by irq_dispose_mapping. I > don't kno

Re: [PATCH V1 0/3] Extend TCA6424 support

2012-08-04 Thread Linus Walleij
On Sat, Jul 28, 2012 at 8:05 AM, Chandrabhanu Mahapatra wrote: > These patches were based on mainline kernel v3.5rc7. Aha. Rebase to v3.6-rc1 because much of this is now upstream. Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a me

Re: [PATCH V1 1/3] GPIO: PCA953X: Increase size of invert variable to support 24 bit

2012-08-04 Thread Linus Walleij
On Sat, Jul 28, 2012 at 8:07 AM, Chandrabhanu Mahapatra wrote: > TCA6424 is a low voltage 24 bit I2C and SMBus I/O expander of pca953x family > similar to its 16 bit predecessor TCA6416. It comes with three 8-bit active > Input, Output, Polarity Inversion and Configuration registers each. The >

Re: [PATCH] Forbid invocation of kexec_load() outside initial PID namespace

2012-08-04 Thread Serge Hallyn
Eric, during the container reboot discussion, the agreement was reached that rebooting for real fron non-init pid ns is not safe. Restarting userspace (in pidns caller owns) is. I argue the same reasoning supports this. I haven't had a chance to review the patch, but the idea gets my ack. I'

Re: [RFC] page-table walkers vs memory order

2012-08-04 Thread Paul E. McKenney
On Sat, Aug 04, 2012 at 11:59:10PM +0100, Dr. David Alan Gilbert wrote: > * Andrea Arcangeli (aarca...@redhat.com) wrote: > > On Sat, Aug 04, 2012 at 03:02:45PM -0700, Paul E. McKenney wrote: > > > OK, I'll bite. ;-) > > > > :)) > > > > > The most sane way for this to happen is with feedback-dri

Re: [PATCH] gpio: msic: Fix calculating register address in msic_gpio_to_oreg()

2012-08-04 Thread Linus Walleij
On Tue, Jul 24, 2012 at 12:35 PM, Axel Lin wrote: > In the case offset is 20 ... 23, the equation to get the register should be: > INTEL_MSIC_GPIO1HV0CTLO - offset + 20 > > With above equation, we can get below mapping between offset and the register: > offset is 20: INTEL_MSIC_GP

Re: [PATCH] gpiolib: Use seq_file's iterator interface

2012-08-04 Thread Linus Walleij
On Mon, Jul 23, 2012 at 2:12 PM, Thierry Reding wrote: > On Sat, May 12, 2012 at 12:56:14PM +, Arnd Bergmann wrote: >> > This patch was requested by Arnd Bergmann here: >> > >> > http://article.gmane.org/gmane.linux.ports.tegra/3535 >> > >> > Signed-off-by: Thierry Reding >> > Cc: Li

Re: [RFC] page-table walkers vs memory order

2012-08-04 Thread Paul E. McKenney
On Sun, Aug 05, 2012 at 12:47:05AM +0200, Andrea Arcangeli wrote: > On Sat, Aug 04, 2012 at 03:02:45PM -0700, Paul E. McKenney wrote: > > OK, I'll bite. ;-) > > :)) > > > The most sane way for this to happen is with feedback-driven techniques > > involving profiling, similar to what is done for

Re: [RFC] page-table walkers vs memory order

2012-08-04 Thread Dr. David Alan Gilbert
* Andrea Arcangeli (aarca...@redhat.com) wrote: > On Sat, Aug 04, 2012 at 03:02:45PM -0700, Paul E. McKenney wrote: > > OK, I'll bite. ;-) > > :)) > > > The most sane way for this to happen is with feedback-driven techniques > > involving profiling, similar to what is done for basic-block reorde

Re: [RFC] page-table walkers vs memory order

2012-08-04 Thread Andrea Arcangeli
On Sat, Aug 04, 2012 at 03:02:45PM -0700, Paul E. McKenney wrote: > OK, I'll bite. ;-) :)) > The most sane way for this to happen is with feedback-driven techniques > involving profiling, similar to what is done for basic-block reordering > or branch prediction. The idea is that you compile the

Re: [PATCH] fix NULL-pointer dereference on scsi_run_queue

2012-08-04 Thread Mike Christie
On 08/04/2012 03:18 PM, Bart Van Assche wrote: > On 08/04/12 16:46, Mike Christie wrote: >> I think we have to have scsi-ml do a get_device when a sdev is added to >> the starved entry and then do a put_device when it is removed (must do >> these under the host lock for the starved entry case too).

Re: [RFC] page-table walkers vs memory order

2012-08-04 Thread Paul E. McKenney
On Sat, Aug 04, 2012 at 04:37:19PM +0200, Andrea Arcangeli wrote: > On Tue, Jul 24, 2012 at 02:51:05PM -0700, Hugh Dickins wrote: > > Since then, I think THP has made the rules more complicated; but I > > believe Andrea paid a great deal of attention to that kind of issue. > > There were many issu

Re: [GIT PULL] GPIO changes for v3.6

2012-08-04 Thread Linus Walleij
On Mon, Jul 30, 2012 at 8:57 AM, NeilBrown wrote: > it means that if !gpio_is_valid(gpio), the error returned is EPROBE_DEFER > which isn't right (an invalid gpio number will never become valid). > If a driver happened to use gpio_request to check the validity of the gpio > rather than doing it i

Re: [Regression, post-3.5] System suspend broken on the Mackerel board

2012-08-04 Thread Rafael J. Wysocki
On Wednesday, August 01, 2012, Paul Mundt wrote: > On Sat, Jul 28, 2012 at 12:53:11AM +0200, Rafael J. Wysocki wrote: > > Unfortunately, your commit > > > > commit ca5481c68e9fbcea62bb3c78ae6cccf99ca8fb73 > > Author: Paul Mundt > > Date: Tue Jul 10 12:08:14 2012 +0900 > > > > sh: pfc: Rudi

Re: [PATCH] intel_idle: Check cpu_idle_get_driver() for NULL before dereferencing it.

2012-08-04 Thread Rafael J. Wysocki
On Thursday, August 02, 2012, Konrad Rzeszutek Wilk wrote: > If the machine is booted without any cpu_idle driver set > (b/c disable_cpuidle() has been called) we should follow > other users of cpu_idle API and check the return value > for NULL before using it. > > Reported-by: Mark van Dijk > Te

Re: [BUGFIX 2/4] PCI/PM: Keep parent bridge active when probing device

2012-08-04 Thread Rafael J. Wysocki
On Friday, August 03, 2012, Huang Ying wrote: > This patch fixes the following bug: > > http://marc.info/?l=linux-pci&m=134329923124234&w=2 > > The root cause of the bug is as follow. > > If a device is not bound with the corresponding driver, the device > runtime PM will be disabled and the dev

Re: [BUGFIX 1/4] PCI/PM: enable D3/D3cold by default for most devices

2012-08-04 Thread Rafael J. Wysocki
On Friday, August 03, 2012, Huang Ying wrote: > This patch fixes the following bug: > > http://marc.info/?l=linux-usb&m=134318961120825&w=2 > > Originally, device lower power states include D1, D2, D3. After that, > D3 is further divided into D3hot and D3cold. To support both scenario > safely,

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

2012-08-04 Thread Rafael J. Wysocki
On Saturday, August 04, 2012, Rafael J. Wysocki wrote: > On Friday, August 03, 2012, Alan Stern wrote: > > On Fri, 3 Aug 2012, Huang Ying wrote: > > > > > This patch fixes the following bug: > > > > > > http://marc.info/?l=linux-pci&m=134338059022620&w=2 > > > > > > Where lspci does not work pro

[PATCH 13/13] SCTP: make socket-initialization use per-namespace protocol parameters.

2012-08-04 Thread Jan Ariyasu
Made the last few protocol parameters per-namespace; sndbuf_policy, rcvbuf_policy, addip_enable, addip_noauth_enable and prsctp_enable. Signed-off-by: Jan Ariyasu --- net/sctp/endpointola.c |6 ++--- net/sctp/protocol.c |6 ++--- net/sctp/sm_statefuns.c | 12 ++--- net/sctp

[PATCH 12/13] SCTP: Enable chunk-auth verification (RFC4895).

2012-08-04 Thread Jan Ariyasu
This patch replaces the global parameter sctp_auth_enable with the per-namespace parameter stored in struct net. Signed-off-by: Jan Ariyasu --- net/sctp/endpointola.c |4 +++- net/sctp/sm_statetable.c | 11 +++ net/sctp/socket.c| 32 3 f

[PATCH 10/13] SCTP: enable per-namespace authentication configuration.

2012-08-04 Thread Jan Ariyasu
This is the first of three patches to set authentication by namespace. In this patch, the relevant routines in auth.c now check to see if auth_enable is set as a namespace parameter instead of checking the global parameter sctp_auth_enable. Signed-off-by: Jan Ariyasu --- include/net/sctp/auth.h

[PATCH 09/13] SCTP: Make transmitted packets use per-namespace parameters.

2012-08-04 Thread Jan Ariyasu
This patch set makes transmitted packets use per-namespace protocol parameters such as rto_initial, max_retrans_path, etc. Signed-off-by: Jan Ariyasu --- include/net/sctp/structs.h |5 +++-- net/sctp/associola.c |2 +- net/sctp/output.c |2 +- net/sctp/outqueue.c

[PATCH 08/13] SCTP: Make associations use per-namespace configuration.

2012-08-04 Thread Jan Ariyasu
These changes make associations namespace aware, including association ID lists and their associated locks. Signed-off-by: Jan Ariyasu --- include/net/sctp/sctp.h|3 --- include/net/sctp/structs.h |3 ++- net/sctp/associola.c | 37 - net/sc

[PATCH 07/13] SCTP: Make primitive functions pass per-namespace parameters.

2012-08-04 Thread Jan Ariyasu
This patch sets up the primitive functions to pass on struct net in preparation for SCTP to use per-net protocol parameters. Signed-off-by: Jan Ariyasu --- include/net/sctp/sctp.h | 15 -- include/net/sctp/sm.h|6 -- net/sctp/associola.c |3 ++- net/sctp/endpoi

[PATCH 05/13] SCTP: register per-namespace sysctl.

2012-08-04 Thread Jan Ariyasu
Updated sysctl registration to be per-netns and to use per-net protocol parameters. Signed-off-by: Jan Ariyasu --- include/net/sctp/sctp.h |8 +- net/sctp/protocol.c | 24 +- net/sctp/sysctl.c | 218 +-- 3 files changed, 221 insert

[PATCH 04/13] SCTP: enable per-namespace address handling.

2012-08-04 Thread Jan Ariyasu
Make a separate local_addr_list for each netns, and use that list to when creating associations. Signed-off-by: Jan Ariyasu --- include/net/sctp/sctp.h|8 ++- include/net/sctp/structs.h | 28 net/sctp/associola.c |6 +- net/sctp/bind_addr.c | 25 --- net/

[PATCH 03/13] SCTP: These changes register procfs for netns.

2012-08-04 Thread Jan Ariyasu
Modified proc to initialization to be namespace compatible. Signed-off-by: Jan Ariyasu --- include/net/sctp/sctp.h | 17 +- net/sctp/objcnt.c |8 ++--- net/sctp/proc.c | 77 +--- net/sctp/protocol.c | 81

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

2012-08-04 Thread Rafael J. Wysocki
On Friday, August 03, 2012, Alan Stern wrote: > On Fri, 3 Aug 2012, Huang Ying wrote: > > > This patch fixes the following bug: > > > > http://marc.info/?l=linux-pci&m=134338059022620&w=2 > > > > Where lspci does not work properly if a device and the corresponding > > parent bridge (such as PCIe

[PATCH 01/13] SCTP: Preparation for namespace enablement

2012-08-04 Thread Jan Ariyasu
Introduce data structures and break up module initialization and exit into subroutines with common functionality for namespace registration. Signed-off-by: Jan Ariyasu --- include/net/sctp/structs.h | 150 net/sctp/protocol.c| 332 +++---

[PATCH 00/13] SCTP: Enable netns

2012-08-04 Thread Jan Ariyasu
The following set of patches enable network-namespaces for the SCTP protocol. The multitude of global parameters are stored in a net_generic structure, and the bulk of the patches enable the protocol to access the parameters on a per-namespace basis. The first five patches enable netns handling o

[Resend][PATCH] PCI / PM: Fix messages printed by acpi_pci_set_power_state()

2012-08-04 Thread Rafael J. Wysocki
If a PCI device is put into D3_cold by acpi_bus_set_power(), the message printed by acpi_pci_set_power_state() says that its power state has been changed to D4, which doesn't make sense. In turn, if the device is put into D3_hot, the message simply says "D3" without specifying the variant of the D

[PATCH] drivers/mtd/devices/spear_smi.c: use devm_ functions consistently

2012-08-04 Thread Julia Lawall
From: Julia Lawall Use devm_kzalloc for all calls to kzalloc and not just the first. Use devm functions for other allocations as well. Move the call to platform_get_resource(pdev, IORESOURCE_MEM, 0) closer to where its result is passed to devm_request_and_ioremap to make the lack of need for a

Re: [PATCH] fix NULL-pointer dereference on scsi_run_queue

2012-08-04 Thread Bart Van Assche
On 08/04/12 16:46, Mike Christie wrote: > I think we have to have scsi-ml do a get_device when a sdev is added to > the starved entry and then do a put_device when it is removed (must do > these under the host lock for the starved entry case too). I am not sure > if that is just a hack/papering-ove

[Regression, post-3.5][PATCH] Revert "NMI watchdog: fix for lockup detector breakage on resume"

2012-08-04 Thread Rafael J. Wysocki
Revert commit 45226e9 (NMI watchdog: fix for lockup detector breakage on resume) which breaks resume from system suspend on my SH7372 Mackerel board (by causing a NULL pointer dereference to happen) and is generally wrong, because it abuses CPU hotplug notifications in a shamelessly blatant way.

Re: [RFC PATCH 2/3] ACPIHP: ACPI system device hotplug slot enumerator

2012-08-04 Thread Yinghai Lu
On Sat, Jul 28, 2012 at 4:42 AM, Jiang Liu wrote: > The first is an ACPI hotplug slot enumerator, which enumerates ACPI hotplug > slots on load and provides callbacks to manage those hotplug slots. > An ACPI hotplug slot is an abstraction of receptacles, where a group of > system devices could be

Re: [PATCH] drivers/i2c/i2c-smbus.c: convert kzalloc to devm_kzalloc

2012-08-04 Thread Jean Delvare
Hi Julia, On Sat, 4 Aug 2012 19:10:48 +0200, Julia Lawall wrote: > From: Julia Lawall > > Converting kzalloc to devm_kzalloc simplifies the code and ensures that the > result, alert, is freed after the irq allocated by the subsequent > devm_request_irq. This in turn ensures that when an interr

[PATCH] tcp_output: fix sparse warning for tcp_wfree

2012-08-04 Thread Silviu-Mihai Popescu
Fix sparse warning: * symbol 'tcp_wfree' was not declared. Should it be static? Signed-off-by: Silviu-Mihai Popescu --- net/ipv4/tcp_output.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/ipv4/tcp_output.c b/net/ipv4/tcp_output.c index 3f1bcff..18475e0 100644

[PATCH] time: Fix adjustment cleanup bug in timekeeping_adjust()

2012-08-04 Thread Ingo Molnar
* Tetsuo Handa wrote: > Hello. > > Bisected to 2a8c0883 "time: Move xtime_nsec adjustment > underflow handling timekeeping_adjust". > > Would you check? probably caused by this stray return in timekeeping_adjust(): } else return; which should really be a proper goto

Re: TCP Delayed ACK in FIN/ACK

2012-08-04 Thread Sławek Janecki
I have no such info in dmsg. I think I will file a issue to Node.js Dev Team. Thanks for reply. On Sat, Aug 4, 2012 at 6:34 PM, Eric Dumazet wrote: > On Sat, 2012-08-04 at 16:51 +0200, richard -rw- weinberger wrote: >> On Sat, Aug 4, 2012 at 4:45 PM, Sławek Janecki wrote: >> > I have a node.js c

Re: linux-next: manual merge of the signal tree with the arm-current tree

2012-08-04 Thread Geert Uytterhoeven
Hi Al, On Sun, Jul 15, 2012 at 2:04 PM, Al Viro wrote: > On Sun, Jul 15, 2012 at 01:47:59PM +0200, Geert Uytterhoeven wrote: >> On Sun, Jul 15, 2012 at 1:27 PM, Al Viro wrote: >> > On Thu, Jul 05, 2012 at 08:52:04AM +0100, Russell King wrote: >> >> Will has concerns with Al's proposed fixes for

Re: [PATCH] CRIS: Fix I/O macros

2012-08-04 Thread Geert Uytterhoeven
Hi Corey, On Sat, Aug 4, 2012 at 7:25 PM, Corey Minyard wrote: > Can we get something like this in? It would fix a lot of build regressions. Yes, would be nice. I also noticed these issues, causing obscure errors like: drivers/block/paride/bpck.c: In function 'bpck_read_regr': drivers/block/p

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

2012-08-04 Thread Julia Lawall
From: Julia Lawall Using devm_kzalloc and devm_clk_get 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. The problem of a free after a devm_request_irq wa

[PATCH] pci: Account for virtual buses in pci_acs_path_enabled

2012-08-04 Thread Alex Williamson
It's possible to have buses without an associated bridge (bus->self == NULL). SR-IOV can generate such buses. When we find these, skip to the parent bus to look for the next ACS test. Signed-off-by: Alex Williamson --- David Ahern reported an oops from iommu drivers passing NULL into this func

[PATCH 2/2] staging: media: cxd2099: use kzalloc to allocate ci pointer of type struct cxd in cxd2099_attach

2012-08-04 Thread Devendra Naga
this does kmalloc and followed by memset, calling kzalloc will actually sets the allocated memory to zero, use kzalloc instead Signed-off-by: Devendra Naga --- drivers/staging/media/cxd2099/cxd2099.c |3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/media/c

[PATCH 1/2] staging: media: cxd2099: fix sparse warnings in cxd2099_attach

2012-08-04 Thread Devendra Naga
The following sparse warnings were fixed drivers/staging/media/cxd2099/cxd2099.c:686:26: warning: Using plain integer as NULL pointer drivers/staging/media/cxd2099/cxd2099.c:691:24: warning: Using plain integer as NULL pointer drivers/staging/media/cxd2099/cxd2099.c:696:24: warning: Using plain

[PATCH 1/2] intel-iommu: Fix ACS path checking

2012-08-04 Thread Alex Williamson
SR-IOV can create buses without a bridge. There may be other cases where this happens as well. In these cases skip to the parent bus and continue testing devices there. Signed-off-by: Alex Williamson --- drivers/iommu/intel-iommu.c | 25 ++--- 1 file changed, 22 insertio

[PATCH 2/2] amd-iommu: Fix ACS path checking

2012-08-04 Thread Alex Williamson
SR-IOV can create buses without a bridge. There may be other cases where this happens as well. In these cases skip to the parent bus and continue testing devices there. Signed-off-by: Alex Williamson --- drivers/iommu/amd_iommu.c | 25 ++--- 1 file changed, 22 insert

[PATCH 0/2] Fix ACS path checking

2012-08-04 Thread Alex Williamson
David Ahern reported an oops caused by passing a NULL start device to pci_acs_path_enabled(). This can happen when a bus is created for sr-iov devices without an associated bridge. To handle this, skip over to the parent bus and look for a bridge there. Continue until we find something or reach

Re: [PATCH] CRIS: Fix I/O macros

2012-08-04 Thread Corey Minyard
Can we get something like this in? It would fix a lot of build regressions. -corey On 07/09/2012 03:22 PM, miny...@acm.org wrote: From: Corey Minyard The inb/outb macros for CRIS are broken from a number of points of view, missing () around parameters and they have an unprotected if statemen

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

2012-08-04 Thread Devendra Naga
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. fix it by copying the name before calling alloc_fbinfo. Signed-off-b

Re: [ 33/73] x86, microcode: Sanitize per-cpu microcode reloading interface

2012-08-04 Thread Ben Hutchings
On Sat, 2012-08-04 at 13:07 -0300, Henrique de Moraes Holschuh wrote: > On Sat, 04 Aug 2012, Ben Hutchings wrote: > > On Fri, 2012-08-03 at 14:27 +0200, Borislav Petkov wrote: > > > On Fri, Aug 03, 2012 at 11:43:14AM +0200, Borislav Petkov wrote: > > > > On Fri, Aug 03, 2012 at 11:04:06AM +0200, Sv

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

2012-08-04 Thread Devendra Naga
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. fix it by copying the name before calling alloc_fbinfo Signed-off-by

[PATCH] drivers/i2c/i2c-smbus.c: convert kzalloc to devm_kzalloc

2012-08-04 Thread Julia Lawall
From: Julia Lawall Converting kzalloc to devm_kzalloc simplifies the code and ensures that the result, alert, is freed after the irq allocated by the subsequent devm_request_irq. This in turn ensures that when an interrupt can be triggered, the alert structure is still available. The problem of

i915 regression on 3.6-rc1: lid blanks screen

2012-08-04 Thread Hugh Dickins
Sorry to report that with 3.6-rc1, closing and opening the lid on this ThinkPad T420s leaves the screen blank, and I have to reboot. I understand there's also an nVidia graphics device in here, but I have that configured out, preferring to use the i915: 00:02.0 VGA compatible controller: Intel Co

Re: [RFC PATCH 3/5] drm/i915: register LVDS connector even if we can't get a panel mode

2012-08-04 Thread Seth Forshee
On Fri, Aug 03, 2012 at 05:27:02PM +0100, Matthew Garrett wrote: > On Fri, Aug 03, 2012 at 11:24:51AM -0500, Seth Forshee wrote: > > > This is one of the things I wasn't so sure about. There are various > > checks in intel_lvds_init() that can cause it to bail out before we try > > to get the EDID

[PATCH 2/6] drivers/mfd: drop frees of devm allocated data

2012-08-04 Thread Julia Lawall
From: Julia Lawall devm_kfree should not have to be explicitly used. devm_regmap_init_i2c also does not require an explicit free. The semantic patch that fixes the first problem is as follows: (http://coccinelle.lip6.fr/) // @@ expression x,d; @@ x = devm_kzalloc(...) ... ?-devm_kfree(d,x); /

[PATCH 1/6] drivers/thermal/rcar_thermal.c: drop frees of devm allocated data

2012-08-04 Thread Julia Lawall
From: Julia Lawall devm_kfree and devm_iounmap should not have to be explicitly used. The semantic patch that fixes one of these problems is as follows: (http://coccinelle.lip6.fr/) // @@ expression x,d; @@ x = devm_kzalloc(...) ... ?-devm_kfree(d,x); // Signed-off-by: Julia Lawall --- d

[PATCH 4/6] drivers/pinctrl/pinctrl-nomadik.c: drop devm_kfree of devm_kzalloc'd data

2012-08-04 Thread Julia Lawall
From: Julia Lawall devm_kfree should not have to be explicitly used. The semantic patch that fixes this problem is as follows: (http://coccinelle.lip6.fr/) // @@ expression x,d; @@ x = devm_kzalloc(...) ... ?-devm_kfree(d,x); // Signed-off-by: Julia Lawall --- drivers/pinctrl/pinctrl-nom

Re: Why is there no nouveau driver maintainer listed in MAINTAINERS?

2012-08-04 Thread Maarten Lankhorst
Hey, Op 04-08-12 18:33, Calvin Walton schreef: > On Sat, 2012-08-04 at 17:49 +0200, richard -rw- weinberger wrote: >> On Sat, Aug 4, 2012 at 5:36 PM, Miles Lane wrote: >>> I would like to report an issue in the nouveau driver, but don't know >>> who to report it to. >> We have a script for this.

[PATCH 5/6] drivers/video/backlight/da9052_bl.c: drop devm_kfree of devm_kzalloc'd data

2012-08-04 Thread Julia Lawall
From: Julia Lawall devm_kfree should not have to be explicitly used. The semantic patch that fixes this problem is as follows: (http://coccinelle.lip6.fr/) // @@ expression x,d; @@ x = devm_kzalloc(...) ... ?-devm_kfree(d,x); // Signed-off-by: Julia Lawall --- drivers/video/backlight/da9

[PATCH 3/6] drivers/char/hw_random/octeon-rng.c: drop frees of devm allocated data

2012-08-04 Thread Julia Lawall
From: Julia Lawall devm_kfree and devm_iounmap should not have to be explicitly used. The semantic patch that fixes this problem is as follows: (http://coccinelle.lip6.fr/) // @@ expression x,d; @@ x = devm_kzalloc(...) ... ?-devm_kfree(d,x); // Signed-off-by: Julia Lawall --- drivers/ch

[PATCH 6/6] drivers/net/phy/mdio-mux-gpio.c: drop devm_kfree of devm_kzalloc'd data

2012-08-04 Thread Julia Lawall
From: Julia Lawall devm_kfree should not have to be explicitly used. The semantic patch that fixes this problem is as follows: (http://coccinelle.lip6.fr/) // @@ expression x,d; @@ x = devm_kzalloc(...) ... ?-devm_kfree(d,x); // Signed-off-by: Julia Lawall --- drivers/net/phy/mdio-mux-gp

Re: [PATCH] fix NULL-pointer dereference on scsi_run_queue

2012-08-04 Thread Mike Christie
On 08/04/2012 04:01 AM, Bart Van Assche wrote: > On 08/02/12 08:41, Chanho Min wrote: >> This patch is to fix a oops from a torn down device. When >> scsi_run_queue process starved queues, scsi_request_fn can race with >> scsi_remove_device. In this case, rarely, scsi_request_fn release the >> last

Re: TCP Delayed ACK in FIN/ACK

2012-08-04 Thread Eric Dumazet
On Sat, 2012-08-04 at 16:51 +0200, richard -rw- weinberger wrote: > On Sat, Aug 4, 2012 at 4:45 PM, Sławek Janecki wrote: > > I have a node.js client (10.177.62.7) requesting some data from http > > rest service from server (10.177.0.1). > > Client is simply using nodejs http.request() method (age

Re: Why is there no nouveau driver maintainer listed in MAINTAINERS?

2012-08-04 Thread Calvin Walton
On Sat, 2012-08-04 at 17:49 +0200, richard -rw- weinberger wrote: > On Sat, Aug 4, 2012 at 5:36 PM, Miles Lane wrote: > > I would like to report an issue in the nouveau driver, but don't know > > who to report it to. > > We have a script for this. > > rw@raccoon:/media/data1/linux-2.6 (master)>

[3.5 regression] DRM: Massive (EDID-probing?) X startup delay on ATI Radeon RV770 (HD4870)

2012-08-04 Thread Nix
Possibly-relevant info: - Two DVI monitors, identical specs, one dual-head graphics card (so no VGA switcheroo or awesome-yet-terrifying PRIME madness needed) - KMS, Xserver 1.12.3, driver 6.14.6-28 (trunk current as of today), Mesa 8.0.4, libdrm 2.4.37 As of kernel 3.5 EDID probing of t

Re: [ 33/73] x86, microcode: Sanitize per-cpu microcode reloading interface

2012-08-04 Thread Henrique de Moraes Holschuh
On Sat, 04 Aug 2012, Ben Hutchings wrote: > On Fri, 2012-08-03 at 14:27 +0200, Borislav Petkov wrote: > > On Fri, Aug 03, 2012 at 11:43:14AM +0200, Borislav Petkov wrote: > > > On Fri, Aug 03, 2012 at 11:04:06AM +0200, Sven Joachim wrote: > > > > On 2012-07-31 06:43 +0200, Ben Hutchings wrote: > >

Re: Why is there no nouveau driver maintainer listed in MAINTAINERS?

2012-08-04 Thread richard -rw- weinberger
On Sat, Aug 4, 2012 at 5:36 PM, Miles Lane wrote: > I would like to report an issue in the nouveau driver, but don't know > who to report it to. We have a script for this. rw@raccoon:/media/data1/linux-2.6 (master)> ./scripts/get_maintainer.pl -f ./drivers/gpu/drm/nouveau Ben Skeggs (commit_sig

Re: [ 33/73] x86, microcode: Sanitize per-cpu microcode reloading interface

2012-08-04 Thread Ben Hutchings
On Fri, 2012-08-03 at 14:27 +0200, Borislav Petkov wrote: > On Fri, Aug 03, 2012 at 11:43:14AM +0200, Borislav Petkov wrote: > > On Fri, Aug 03, 2012 at 11:04:06AM +0200, Sven Joachim wrote: > > > On 2012-07-31 06:43 +0200, Ben Hutchings wrote: > > > > > > > 3.2-stable review patch. If anyone has

Why is there no nouveau driver maintainer listed in MAINTAINERS?

2012-08-04 Thread Miles Lane
I would like to report an issue in the nouveau driver, but don't know who to report it to. Thanks, Miles -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-

[PATCH 3/5] cputime: Allow dynamic switch between tick/virtual based cputime accounting

2012-08-04 Thread Frederic Weisbecker
Allow to dynamically switch between tick and virtual based cputime accounting. This way we can provide a kind of "on-demand" virtual based cputime accounting. In this mode, the kernel will rely on the user hooks subsystem to dynamically hook on kernel boundaries. This is in preparation for beeing

[PATCH 1/5] context_tracking: New context tracking susbsystem

2012-08-04 Thread Frederic Weisbecker
Create a new subsystem that probes on kernel boundaries to keep track of the transitions between level contexts with two basic initial contexts: user or kernel. This is an abstraction of some RCU code that use such tracking to implement its userspace extended quiescent state. We need to pull this

[PATCH 2/5] cputime: Don't allow virtual and irq finegrained cputime accounting simultaneously

2012-08-04 Thread Frederic Weisbecker
We may soon be able to run both at the same time. But we need to rework a bit the irq finegrained accounting before that. Just do a mutual exclusion for now. Signed-off-by: Frederic Weisbecker Cc: Alessio Igor Bogani Cc: Andrew Morton Cc: Avi Kivity Cc: Chris Metcalf Cc: Christoph Lameter Cc

[PATCH 5/5] cputime: Generic on-demand virtual cputime accounting

2012-08-04 Thread Frederic Weisbecker
If we want to stop the tick further idle, we need to be able to account the cputime without using the tick. Virtual based cputime accounting solves that problem by hooking into kernel/user boundaries. However implementing CONFIG_VIRT_CPU_ACCOUNTING requires to set low level hooks and involves mor

[PATCH 4/5] cputime: Rename account_system_vtime to account_vtime

2012-08-04 Thread Frederic Weisbecker
account_system_vtime() can be called from random places: hard/softirq entry/exit, kvm guest entry/exit, and even context switches on powerpc. Rename it to the even more generic account_vtime() name, this reflect well that we are in a random place in the kernel where we have either system, idle or

[PATCH 0/5] cputime: Generic virtual based cputime accounting v3

2012-08-04 Thread Frederic Weisbecker
Hi, Changes since v2 (https://lwn.net/Articles/509575/) are only about converting the "code domain tracking" naming to "context tracking" as per Ingo's suggestion. And the result indeed looks much better. Ring 0 dark knights can pull from: git://github.com/fweisbec/linux-dynticks.git vti

Re: TCP Delayed ACK in FIN/ACK

2012-08-04 Thread richard -rw- weinberger
On Sat, Aug 4, 2012 at 4:45 PM, Sławek Janecki wrote: > I have a node.js client (10.177.62.7) requesting some data from http > rest service from server (10.177.0.1). > Client is simply using nodejs http.request() method (agent=false). > Client is on Ubuntu 11.10 box. > Why client sends FIN ACK aft

Re: [PATCH V3 1/4] [SCSI] drivers/scsi/ufs: Seggregate PCI Specific Code

2012-08-04 Thread vinayak holikatti
On Wed, Aug 1, 2012 at 8:08 PM, Arnd Bergmann wrote: > On Thursday 26 July 2012, Vinayak Holikatti wrote: > >> -static void ufshcd_remove(struct pci_dev *pdev) >> +void ufshcd_remove(struct ufs_hba *hba) >> { >> - struct ufs_hba *hba = pci_get_drvdata(pdev); >> - >> /* disable interrupt

Re: [bisected] nouveau: "Failed to idle channel x" after resume

2012-08-04 Thread Maxim Levitsky
On Mon, 2012-07-23 at 18:25 +0300, Aioanei Rares wrote: > On Thu, Jul 5, 2012 at 11:24 PM, Martin Nyhus wrote: > > > > On Mon, 11 Jun 2012 23:18:42 +0200 Martin Nyhus wrote: > > > after resuming from suspend nouveau starts writing Failed to idle > > > channel x (where x is 2 or 3) to the log and

Re: [RFC] page-table walkers vs memory order

2012-08-04 Thread Andrea Arcangeli
On Tue, Jul 24, 2012 at 02:51:05PM -0700, Hugh Dickins wrote: > Since then, I think THP has made the rules more complicated; but I > believe Andrea paid a great deal of attention to that kind of issue. There were many issues, one unexpected was 1a5a9906d4e8d1976b701f889d8f35d54b928f25. Keep in mi

[PATCH/v2] [trivial] dma: Fix typo various drivers in dma

2012-08-04 Thread Masanari Iida
Correct spelling typo in drivers/dma. Signed-off-by: Masanari Iida --- drivers/dma/at_hdmac.c | 8 drivers/dma/ep93xx_dma.c | 2 +- drivers/dma/fsldma.c | 2 +- drivers/dma/imx-dma.c| 4 ++-- drivers/dma/intel_mid_dma_regs.h | 6 +++--- drivers/

Re: [RFC][PATCH v3 1/3] runtime interpreted power sequences

2012-08-04 Thread Mark Brown
On Fri, Aug 03, 2012 at 10:15:46AM +0900, Alex Courbot wrote: > On Fri 03 Aug 2012 03:11:12 AM JST, Mark Brown wrote: > >I missed some of the earlier bits of the thread here but why can't we do > >device based lookups? > That is because the phandles would not be properties of the device > node bu

Re: [RFC PATCH 08/23 V2] hugetlb: use N_MEMORY instead N_HIGH_MEMORY

2012-08-04 Thread Hillf Danton
On Thu, Aug 2, 2012 at 2:01 PM, Lai Jiangshan wrote: > N_HIGH_MEMORY stands for the nodes that has normal or high memory. > N_MEMORY stands for the nodes that has any memory. > > The code here need to handle with the nodes which have memory, we should > use N_MEMORY instead. > > Signed-off-by: Lai

Re: [3.6-rc1] sched: clock counts too quickly

2012-08-04 Thread Tetsuo Handa
Hello. Bisected to 2a8c0883 "time: Move xtime_nsec adjustment underflow handling timekeeping_adjust". Would you check? Regards. Tetsuo Handa wrote: > Tetsuo Handa wrote: > > > Under unknown condition, clock starts counting up too quickly which is > > > enough > > > to confuse hangcheck timer t

Re: [Xen-devel] Regression in xen-netfront on v3.6 (git commit c48a11c7ad2623b99bbd6859b0b4234e7f11176f, netvm: propagate page->pfmemalloc to skb)

2012-08-04 Thread Mel Gorman
On Sat, Aug 04, 2012 at 07:03:55AM -0400, Konrad Rzeszutek Wilk wrote: > On Fri, Aug 03, 2012 at 08:04:14AM -0400, Konrad Rzeszutek Wilk wrote: > > On Wed, Aug 01, 2012 at 03:02:27PM -0400, Konrad Rzeszutek Wilk wrote: > > > So I hadn't done a git bisection yet. But if I choose git commit: > > > 4b

Re: [PATCH] spi: omap2-mcspi: Fix the error handling in probe

2012-08-04 Thread Mark Brown
On Thu, Aug 02, 2012 at 04:41:25PM +0530, Shubhrajyoti D wrote: > The kfree() is taken care of by the spi core (spi_master_release() function) > that is called once the last reference to the underlying struct device has > been released. So the driver need not call kfree. Applied, thanks. -- To uns

  1   2   >