Re: [PATCH 23/26] Generic dynamic per cpu refcounting

2012-12-27 Thread Fubo Chen
On Mon, Dec 3, 2012 at 9:58 PM, Kent Overstreet wrote: > +#define pr_fmt(fmt) "%s: " fmt "\n", __func__ Dynamic debug already allows to insert the function name. Please consider leaving this line out entirely and move the "\n" to the end of individual printed lines. > +#define PCPU_REF_PTR

RE: [PATCH 0/6 v8] cpufreq: add support for Calxeda ECX-1000 (highbank)

2012-12-27 Thread Mark Langsdorf
I thought I had addressed all the issues with the v9 version. The only comments I got on it was Mike's reviewed-by and I didn't think that warranted a resend. If you do want me to send it again, do you want it updated to current top-of-tree? That may take a bit, since there's an unrelated bug

Re: [RFC PATCH v1 11/31] ARC: Low level IRQ/Trap/Exception(non-MMU) Handling

2012-12-27 Thread Vineet Gupta
On Thursday 27 December 2012 02:30 PM, Vineet Gupta wrote: > On Friday 16 November 2012 10:28 AM, Al Viro wrote: >>> + ; - check for signals/restore-sigmask >>> + bbit0 r9, TIF_SIGPENDING, chk_next_work >>> + >>> + ; save CALLEE Regs. >>> + ; (i) If this signal

Re: [Alternative][PATCH] ACPI / PCI: Set root bridge ACPI handle in advance

2012-12-27 Thread Rafael J. Wysocki
On Thursday, December 27, 2012 01:47:22 PM Rafael J. Wysocki wrote: > On Wednesday, December 26, 2012 04:10:32 PM Yinghai Lu wrote: > > On Wed, Dec 26, 2012 at 2:36 PM, Rafael J. Wysocki wrote: > > > On Wednesday, December 26, 2012 12:41:05 PM Yinghai Lu wrote: > > >> On Wed, Dec 26, 2012 at

Re: [PATCH 1/2] vhost_net: correct error hanlding in vhost_net_set_backend()

2012-12-27 Thread Michael S. Tsirkin
On Thu, Dec 27, 2012 at 02:39:19PM +0800, Jason Wang wrote: > Fix the leaking of oldubufs and fd refcnt when fail to initialized used ring. > > Signed-off-by: Jason Wang > --- > drivers/vhost/net.c | 14 +++--- > 1 files changed, 11 insertions(+), 3 deletions(-) > > diff --git

Re: [PATCH 0/6 v8] cpufreq: add support for Calxeda ECX-1000 (highbank)

2012-12-27 Thread Rafael J. Wysocki
On Wednesday, December 05, 2012 10:48:35 AM Mark Langsdorf wrote: > This patch series adds cpufreq support for the Calxeda > ECX-1000 (highbank) SoCs. The EnergyCore Management Engine (ECME) on > the ECX-1000 manages the voltage for the part and communications with > Linux through a pl320 mailbox.

Re: [REGRESSION][3.8.-rc1][ INFO: possible circular locking dependency detected ]

2012-12-27 Thread Peter Hurley
On Thu, 2012-12-27 at 16:36 +0800, Shawn Guo wrote: > On Wed, Dec 26, 2012 at 10:34:39AM +0800, Shawn Guo wrote: > > It seems that I'm running into the same locking issue. My setup is: > > > > - i.MX28 (ARM) > > - v3.8-rc1 > > - mxs_defconfig > - The warning is seen when LCD is blanking > > >

Re: [PATCH 1/2] vhost_net: correct error hanlding in vhost_net_set_backend()

2012-12-27 Thread Michael S. Tsirkin
On Thu, Dec 27, 2012 at 02:39:20PM +0800, Jason Wang wrote: > Currently, polling error were ignored in vhost. This may lead some issues (e.g > kenrel crash when passing a tap fd to vhost before calling TUNSETIFF). Fix > this > by: > > - extend the idea of vhost_net_poll_state to all vhost_polls

Re: [rtc-linux] [PATCH] rtc: recycle id when unloading a rtc driver

2012-12-27 Thread Alexander Holler
Am 19.12.2012 09:55, schrieb Alexander Holler: Unlikely, as I've worked hard to get one of the first drivers for pluggable RTCs into the kernel. ;) BTW. maybe you could have a look at the patch for that: https://lkml.org/lkml/2012/12/15/64 It should already be in your mailbox. I had to

Re: [Alternative][PATCH] ACPI / PCI: Set root bridge ACPI handle in advance

2012-12-27 Thread Rafael J. Wysocki
On Wednesday, December 26, 2012 04:10:32 PM Yinghai Lu wrote: > On Wed, Dec 26, 2012 at 2:36 PM, Rafael J. Wysocki wrote: > > On Wednesday, December 26, 2012 12:41:05 PM Yinghai Lu wrote: > >> On Wed, Dec 26, 2012 at 12:16 PM, Yinghai Lu wrote: > >> > On Wed, Dec 26, 2012 at 12:10 PM, Bjorn

Re: 3.8-rc1 build failure with MIPS/SPARSEMEM

2012-12-27 Thread Aaro Koskinen
Hi, On Wed, Dec 26, 2012 at 02:34:35AM +0200, Kirill A. Shutemov wrote: > On MIPS if SPARSEMEM is enabled we've got this: > > In file included from > /home/kas/git/public/linux/arch/mips/include/asm/pgtable.h:552, > from include/linux/mm.h:44, > from

[PATCH 2/6] of: Return -ENXIO from of_parse_phandle_with_args for too large index and return errors from of_gpio_named_count

2012-12-27 Thread Andreas Larsson
This lets of_gpio_named_count return an errno on errors by being able to distinguish between reaching the end of the phandle list and getting some other error from of_parse_phandle_with_args. Return error from of_spi_register_master when there is an "cs-gpios" list for which gp_gpio_named_count

[PATCH 6/6] of_spi: Initialize cs_gpios and cs_gpio with -EEXIST

2012-12-27 Thread Andreas Larsson
Holes in the cs-gpios DT phandle list is supposed to mark that native chipselects is to be used. The value returned from of_get_named_gpio_flags in this case is -EEXIST. By initializing cs_gpios and cs_gpio with -EEXIST, this and only this errno will indicate to a spi controller driver that a

[PATCH 1/6] of: Return -EEXIST from of_parse_phandle_with_args for holes in phandle list

2012-12-27 Thread Andreas Larsson
Return value for an empty phandle was -EEXIST before commit 15c9a0ac, that changed the return value in this case to -ENOENT. However, of_gpio_named_count relies upon the return value to be -EEXIST and relies upon being able to distinguish this case from the case of no list at all which also

Re: kernel BUG at mm/huge_memory.c:1798!

2012-12-27 Thread Hillf Danton
On Thu, Dec 27, 2012 at 8:31 AM, Alexander Beregalov wrote: > On 25 December 2012 16:05, Hillf Danton wrote: >> On Tue, Dec 25, 2012 at 12:38 PM, Zhouping Liu wrote: >>> Hello all, >>> >>> I found the below kernel bug using latest mainline(637704cbc95), >>> my hardware has 2 numa nodes, and

[PATCH 5/6] of_spi: Add fetching of of_gpio flags to of_spi_register_master

2012-12-27 Thread Andreas Larsson
When using a gpio chip select with a OF_GPIO_ACTIVE_LOW flag, this needs to be known to the controller driver. Signed-off-by: Andreas Larsson --- Documentation/devicetree/bindings/spi/spi-bus.txt |3 +- drivers/spi/spi.c | 24 ++--

[PATCH 4/6] of_spi: Document cs_gpios and cs_gpio in kernel-doc

2012-12-27 Thread Andreas Larsson
This adds missing kernel-doc entries for cs_gpios in struct spi_master and cs_gpio in struct spi_device. Signed-off-by: Andreas Larsson --- include/linux/spi/spi.h |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/include/linux/spi/spi.h b/include/linux/spi/spi.h index

[PATCH 3/6] of_spi: Initialize cs_gpios properly

2012-12-27 Thread Andreas Larsson
Using memset does not set an array of integers properly Signed-off-by: Andreas Larsson --- drivers/spi/spi.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drivers/spi/spi.c b/drivers/spi/spi.c index 9c2acf1..a4baa0a 100644 --- a/drivers/spi/spi.c +++

[PATCH 0/6] of, of_gpio, of_spi: Fix and improve of_parse_phandle_with_args, of_gpio_named_count and of_spi_register_master

2012-12-27 Thread Andreas Larsson
This patch series fixes a bug where of_gpio_named count relied upon a return value that was no longer returned from of_parse_phandle_with_args and adds the possibility for of_gpio_named_count to return error values. In addition, for of_spi_register_master it fixes a bug, adds documentation, adds

Re: [PATCH v5 14/14] memory-hotplug: free node_data when a node is offlined

2012-12-27 Thread Wen Congyang
At 12/26/2012 11:55 AM, Kamezawa Hiroyuki Wrote: > (2012/12/24 21:09), Tang Chen wrote: >> From: Wen Congyang >> >> We call hotadd_new_pgdat() to allocate memory to store node_data. So we >> should free it when removing a node. >> >> Signed-off-by: Wen Congyang > > I'm sorry but is it safe to

Re: [PATCH 2/2] USB: ehci-s5p: Add to get interrupt from DT

2012-12-27 Thread Dongjin Kim
Hi Sergei, Yes, you are right. I made this patch to read its interrupt number from dtb directly. But now platform_get_irq() returns correct irq since "OF_DEV_AUXDATA(...)" is added as my first patch. This patch is useless. Thanks and best regards, Dongjin. On Thu, Dec 27, 2012 at 5:18 AM,

Re: [PATCH v4] usb: phy: samsung: Add support to set pmu isolation

2012-12-27 Thread Vivek Gautam
Hi Sylwester, On Thu, Dec 27, 2012 at 4:00 AM, Sylwester Nawrocki wrote: > On 12/26/2012 01:28 PM, Vivek Gautam wrote: >> >> Adding support to parse device node data in order to get >> required properties to set pmu isolation for usb-phy. >> >> Signed-off-by: Vivek Gautam >> --- >>

Re: [PATCH] [media] vivi: Constify structures

2012-12-27 Thread Hans Verkuil
On Wed December 26 2012 16:29:43 Kirill Smelkov wrote: > Most of *_ops and other structures in vivi.c were already declared const > but some have not. Constify and code/data will take less space: > > $ size drivers/media/platform/vivi.o > textdata bss dec hex

Re: [RFC PATCH] virtio-net: reset virtqueue affinity when doing cpu hotplug

2012-12-27 Thread Michael S. Tsirkin
On Thu, Dec 27, 2012 at 11:34:16AM +0800, Jason Wang wrote: > On 12/26/2012 06:46 PM, Michael S. Tsirkin wrote: > > On Wed, Dec 26, 2012 at 03:06:54PM +0800, Wanlong Gao wrote: > >> Add a cpu notifier to virtio-net, so that we can reset the > >> virtqueue affinity if the cpu hotplug happens. It

echo 1 > /proc/sys/vm/compact_memory -> sh: write error: Bad address

2012-12-27 Thread Liu Hui-R64343
I met one error when does the following command: sh/$ echo 1 > /proc/sys/vm/compact_memory sh/$ sh: write error: Bad address After using strace, I find the following: ... write(1, "1\n", 2) = 3 write(1, "", 4294967295)= -1 EFAULT (Bad address)

Re: [PATCH] iio: (max1363) Fix probe error path

2012-12-27 Thread Jonathan Cameron
On 12/24/2012 06:24 AM, Guenter Roeck wrote: > Instantiating the driver with no available regulator results in: > > [39711.686393] i2c i2c-7: new_device: Instantiated device max1139 at 0x35 > [39711.688687] BUG: unable to handle kernel paging request at fe13 > [39711.688734] IP: []

Re: [PATCH] ehci-mxc: remove Efika MX-specific CHRGVBUS hack

2012-12-27 Thread Sascha Hauer
On Sun, Dec 23, 2012 at 05:16:02AM -0600, Matt Sealey wrote: > Since Efika MX platform support (pre-devicetree) was removed from the tree > this code no longer has any possibility of running and clutters up the > driver which is being replaced by the chipidea host in the future anyway. > >

[RESEND PATCH] modpost: For ARC Port submission

2012-12-27 Thread Vineet Gupta
Hi, This is following up on ARC Linux Port submission bits: Basic Port: https://lkml.org/lkml/2012/11/7/114 Addons: https://lkml.org/lkml/2012/11/12/121 I'm resending this single patch to hopefully get ACK on simple modpost change. Thx, -Vineet Vineet Gupta (1): modpost: Ignore ARC

[PATCH] modpost: Ignore ARC specific non-alloc sections

2012-12-27 Thread Vineet Gupta
ARC relocatable object files contain one/more .gnu.linkonce.arcextmap.* sections (collated by kernel/vmlinux.lds into .arcextmap in final link). This section is used by debuggers to display the extension instructions and need-not be loaded by target (hence !SHF_ALLOC) The final kernel binary only

Re: [PATCH] drivers/platform/x86/thinkpad_acpi.c: Handle HKEY event 0x6040

2012-12-27 Thread Henrique de Moraes Holschuh
On Wed, 26 Dec 2012, Richard Hartmann wrote: > + case TP_HKEY_EV_AC_CHANGED: > + pr_info("AC status has changed\n"); > + /* X120e, x121e, X220, X220i, X220t, X230, T420, T420s, W520: > + * AC status changed; can be triggered by plugging or > +

Re: [PATCH 2/2] vhost: handle polling failure

2012-12-27 Thread Wanlong Gao
On 12/27/2012 02:39 PM, Jason Wang wrote: > Currently, polling error were ignored in vhost. This may lead some issues (e.g > kenrel crash when passing a tap fd to vhost before calling TUNSETIFF). Fix > this > by: Can this kernel crash be reproduced by hand? Thanks, Wanlong Gao > > - extend

Re: [PATCH 04/26] aio: remove retry-based AIO

2012-12-27 Thread Fubo Chen
On Mon, Dec 3, 2012 at 9:58 PM, Kent Overstreet wrote: > We want to remove retry-based AIO because it is fundemantally unsafe. Instead of "fundemantally" that should have been "fundamentally" I guess ? Thanks, Fubo. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

Re: [PATCH 1/3] staging: drm/imx: check return value of ipu_reset()

2012-12-27 Thread Sascha Hauer
On Tue, Dec 25, 2012 at 03:58:37PM +0100, Lothar Waßmann wrote: > ipu_reset() can fail with a timeout. Check the return value and act > appropriately. > > Signed-off-by: Lothar Waßmann All: Acked-by: Sascha Hauer > --- > drivers/staging/imx-drm/ipu-v3/ipu-common.c |5 - > 1 files

Re: [PATCH 2/2] tracing: Use sched_clock_cpu for trace_clock_global

2012-12-27 Thread Namhyung Kim
On Thu, 27 Dec 2012 11:49:45 +0900, Namhyung Kim wrote: > From: Namhyung Kim > > For systems have unstable sched_clock, all cpu_clock() does is enable/ > disable local irq during call to sched_clock(). And for stable systems Oops, I meant s/sched_clock/scheck_clock_cpu/. Thanks, Namhyung >

Re: What does ISA/PCI really mean to ARM architecture?

2012-12-27 Thread Geert Uytterhoeven
On Thu, Dec 27, 2012 at 9:27 AM, Woody Wu wrote: > Can a peripheral chip that claims to be ISA or PCI device be used in a > ARM based embedded system? For these kind of chips, I only concern > about the planar kind of devices, means they are not on a dedicated > expansion card. > > From hardware

Re: [PATCH 0/3] Thunderbolt workarounds

2012-12-27 Thread Mika Westerberg
On Wed, Dec 26, 2012 at 05:50:42PM -0700, Bjorn Helgaas wrote: > On Thu, Dec 13, 2012 at 12:25 PM, Kirill A. Shutemov > wrote: > > From: "Kirill A. Shutemov" > > > > I had chance to test two PC setups with Thunderbolt: Acer Aspire S5 and > > Intel DZ77RE-75K motherboard. Unfortunately, both of

Re: [PATCH v4] usb: phy: samsung: Add support to set pmu isolation

2012-12-27 Thread Vivek Gautam
Hi Russell, On Thu, Dec 27, 2012 at 5:56 AM, Russell King - ARM Linux wrote: > On Wed, Dec 26, 2012 at 05:58:32PM +0530, Vivek Gautam wrote: >> + if (!ret) >> + sphy->phyctrl_pmureg = ioremap(reg[0], reg[1]); >> + >> + of_node_put(usbphy_pmu); >> + >> + if

[PATCH 03/10] perf sort: Fix --sort pid output

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim The "pid" sort key prints "Command: Pid" output but it's misaligned. It's because of the offset of 6 was added to the column length during the calculation in order to reserve an space for Pid part but it isn't honored when printed. The output before this patch was like this:

[PATCH 05/10] perf sort: Calculate parent column width too

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim When hists__calc_col_len() called, most of column length are refreshed but it missed parent column. So if the parent sort key was used along with other keys rests will be misalinged since parent has no proper column width. Cc: Stephane Eranian Signed-off-by: Namhyung Kim

[PATCH 04/10] perf sort: Align cpu column to right

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim Since cpu number is a natural number, it'd be more appropriate aligning it to right. Before: # Overhead CPU Command: Pid Shared Object # ... . . # 8.91% 8gnome-shell: 1497 perf-1497.map

[PATCHSET 00/10] perf tools: Cleanups and bug fixes on sort keys

2012-12-27 Thread Namhyung Kim
Hi, There's a bug report from Stefan Beller that gets segfault on using some sort keys. In addition to it, I had some TODO items related to it, hence this patchset. :) This is mostly simple few liners and more complex ones might come later. Stephane, it'd be great if you check I'm not messed

[PATCH 07/10] perf sort: Check return value of strdup()

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim When setup_sorting() is called, 'str' is passed to strtok_r() but it's not checked to have a valid pointer. As strtok_r() accepts NULL pointer on a first argument and use the third argument in that case, it can cause a trouble since our third argument, tmp, is not

[PATCH 10/10] perf report: Update documentation for sort keys

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim Add description of sort keys to the perf-report document and also add missing cpu and srcline keys to the command line help string. Cc: Stephane Eranian Signed-off-by: Namhyung Kim --- tools/perf/Documentation/perf-report.txt | 38 +---

[PATCH 06/10] perf sort: Drop ip_[lr] arguments from _sort__sym_cmp()

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim Current _sort__sym_cmp() function is used for comparing symbols between two hist entries on symbol, symbol_from and symbol_to sort keys. Those functions pass addresses of symbols but it's meaningless since it gets over-written inside of the _sort__sym_cmp function to a start

[PATCH 02/10] perf sort: Get rid of unnecessary __maybe_unused

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim Some functions have set __maybe_unused on its arguments that are used actually. Remove them. Cc: Stephane Eranian Signed-off-by: Namhyung Kim --- tools/perf/util/sort.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/tools/perf/util/sort.c

[PATCH 09/10] perf sort: Separate out branch stack specific sort keys

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim Current perf report gets segmentation fault when a branch stack specific sort key is provided by --sort option to a perf.data file which contains no branch infomation. It's because those sort keys reference branch info of a hist entry unconditionally. Maybe we can change it

[PATCH 01/10] perf sort: Move misplaced sort entry functions

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim Some functions are misplaced along with other entries. Move them to a right place so that it can be found together with related functions. No functional change intended. Cc: Stephane Eranian Signed-off-by: Namhyung Kim --- tools/perf/util/sort.c | 119

[PATCH 08/10] perf sort: Clean up sort__first_dimension setting

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim It doesn't need to compare to every sort key names since the index already has the required information. Cc: Stephane Eranian Signed-off-by: Namhyung Kim --- tools/perf/util/sort.c | 26 ++ 1 file changed, 2 insertions(+), 24 deletions(-) diff

Re: [RFC PATCH v1 11/31] ARC: Low level IRQ/Trap/Exception(non-MMU) Handling

2012-12-27 Thread Vineet Gupta
On Friday 16 November 2012 10:28 AM, Al Viro wrote: >> +; - check for signals/restore-sigmask >> +bbit0 r9, TIF_SIGPENDING, chk_next_work >> + >> +; save CALLEE Regs. >> +; (i) If this signal causes coredump - full regfile needed >> +; (ii) If signal is

Re: [PATCH v2] Add the values related to buddy system for filtering free pages.

2012-12-27 Thread Atsushi Kumagai
Hello, On Thu, 20 Dec 2012 18:00:11 -0800 ebied...@xmission.com (Eric W. Biederman) wrote: > "Hatayama, Daisuke" writes: > > >> From: kexec-boun...@lists.infradead.org > >> [mailto:kexec-boun...@lists.infradead.org] On Behalf Of Atsushi Kumagai > >> Sent: Thursday, December 20, 2012 11:21 AM >

Re: [REGRESSION][3.8.-rc1][ INFO: possible circular locking dependency detected ]

2012-12-27 Thread Shawn Guo
On Wed, Dec 26, 2012 at 10:34:39AM +0800, Shawn Guo wrote: > It seems that I'm running into the same locking issue. My setup is: > > - i.MX28 (ARM) > - v3.8-rc1 > - mxs_defconfig - The warning is seen when LCD is blanking > The warning disappears after reverting patch daee779 (console:

What does ISA/PCI really mean to ARM architecture?

2012-12-27 Thread Woody Wu
Hi, list I know this might be a very basic question. But I really don't clear at it. Can a peripheral chip that claims to be ISA or PCI device be used in a ARM based embedded system? For these kind of chips, I only concern about the planar kind of devices, means they are not on a dedicated

Re: [tip:x86/build] x86: Default to ARCH= x86 to avoid overriding CONFIG_64BIT

2012-12-27 Thread David Woodhouse
On Wed, 2012-12-26 at 17:52 -0800, David Rientjes wrote: > On Wed, 26 Dec 2012, David Woodhouse wrote: > > > Thanks. I'll look into this. I presume it was *always* failing, but > > nobody happened to come across it because our test coverage of x86 > > configs without CONFIG_64BIT wasn't

Re: [tip:x86/build] x86: Default to ARCH= x86 to avoid overriding CONFIG_64BIT

2012-12-27 Thread David Woodhouse
On Wed, 2012-12-26 at 17:52 -0800, David Rientjes wrote: On Wed, 26 Dec 2012, David Woodhouse wrote: Thanks. I'll look into this. I presume it was *always* failing, but nobody happened to come across it because our test coverage of x86 configs without CONFIG_64BIT wasn't particularly

What does ISA/PCI really mean to ARM architecture?

2012-12-27 Thread Woody Wu
Hi, list I know this might be a very basic question. But I really don't clear at it. Can a peripheral chip that claims to be ISA or PCI device be used in a ARM based embedded system? For these kind of chips, I only concern about the planar kind of devices, means they are not on a dedicated

Re: [REGRESSION][3.8.-rc1][ INFO: possible circular locking dependency detected ]

2012-12-27 Thread Shawn Guo
On Wed, Dec 26, 2012 at 10:34:39AM +0800, Shawn Guo wrote: It seems that I'm running into the same locking issue. My setup is: - i.MX28 (ARM) - v3.8-rc1 - mxs_defconfig - The warning is seen when LCD is blanking The warning disappears after reverting patch daee779 (console: implement

Re: [PATCH v2] Add the values related to buddy system for filtering free pages.

2012-12-27 Thread Atsushi Kumagai
Hello, On Thu, 20 Dec 2012 18:00:11 -0800 ebied...@xmission.com (Eric W. Biederman) wrote: Hatayama, Daisuke d.hatay...@jp.fujitsu.com writes: From: kexec-boun...@lists.infradead.org [mailto:kexec-boun...@lists.infradead.org] On Behalf Of Atsushi Kumagai Sent: Thursday, December 20,

Re: [RFC PATCH v1 11/31] ARC: Low level IRQ/Trap/Exception(non-MMU) Handling

2012-12-27 Thread Vineet Gupta
On Friday 16 November 2012 10:28 AM, Al Viro wrote: +; - check for signals/restore-sigmask +bbit0 r9, TIF_SIGPENDING, chk_next_work + +; save CALLEE Regs. +; (i) If this signal causes coredump - full regfile needed +; (ii) If signal is

[PATCH 01/10] perf sort: Move misplaced sort entry functions

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim namhyung@lge.com Some functions are misplaced along with other entries. Move them to a right place so that it can be found together with related functions. No functional change intended. Cc: Stephane Eranian eran...@google.com Signed-off-by: Namhyung Kim

[PATCH 08/10] perf sort: Clean up sort__first_dimension setting

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim namhyung@lge.com It doesn't need to compare to every sort key names since the index already has the required information. Cc: Stephane Eranian eran...@google.com Signed-off-by: Namhyung Kim namhy...@kernel.org --- tools/perf/util/sort.c | 26 ++ 1

[PATCH 09/10] perf sort: Separate out branch stack specific sort keys

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim namhyung@lge.com Current perf report gets segmentation fault when a branch stack specific sort key is provided by --sort option to a perf.data file which contains no branch infomation. It's because those sort keys reference branch info of a hist entry unconditionally.

[PATCH 02/10] perf sort: Get rid of unnecessary __maybe_unused

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim namhyung@lge.com Some functions have set __maybe_unused on its arguments that are used actually. Remove them. Cc: Stephane Eranian eran...@google.com Signed-off-by: Namhyung Kim namhy...@kernel.org --- tools/perf/util/sort.c | 11 --- 1 file changed, 4

[PATCH 07/10] perf sort: Check return value of strdup()

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim namhyung@lge.com When setup_sorting() is called, 'str' is passed to strtok_r() but it's not checked to have a valid pointer. As strtok_r() accepts NULL pointer on a first argument and use the third argument in that case, it can cause a trouble since our third argument,

[PATCH 10/10] perf report: Update documentation for sort keys

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim namhyung@lge.com Add description of sort keys to the perf-report document and also add missing cpu and srcline keys to the command line help string. Cc: Stephane Eranian eran...@google.com Signed-off-by: Namhyung Kim namhy...@kernel.org ---

[PATCH 06/10] perf sort: Drop ip_[lr] arguments from _sort__sym_cmp()

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim namhyung@lge.com Current _sort__sym_cmp() function is used for comparing symbols between two hist entries on symbol, symbol_from and symbol_to sort keys. Those functions pass addresses of symbols but it's meaningless since it gets over-written inside of the _sort__sym_cmp

[PATCHSET 00/10] perf tools: Cleanups and bug fixes on sort keys

2012-12-27 Thread Namhyung Kim
Hi, There's a bug report from Stefan Beller that gets segfault on using some sort keys. In addition to it, I had some TODO items related to it, hence this patchset. :) This is mostly simple few liners and more complex ones might come later. Stephane, it'd be great if you check I'm not messed

[PATCH 05/10] perf sort: Calculate parent column width too

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim namhyung@lge.com When hists__calc_col_len() called, most of column length are refreshed but it missed parent column. So if the parent sort key was used along with other keys rests will be misalinged since parent has no proper column width. Cc: Stephane Eranian

[PATCH 04/10] perf sort: Align cpu column to right

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim namhyung@lge.com Since cpu number is a natural number, it'd be more appropriate aligning it to right. Before: # Overhead CPU Command: Pid Shared Object # ... . . # 8.91% 8gnome-shell: 1497

[PATCH 03/10] perf sort: Fix --sort pid output

2012-12-27 Thread Namhyung Kim
From: Namhyung Kim namhyung@lge.com The pid sort key prints Command: Pid output but it's misaligned. It's because of the offset of 6 was added to the column length during the calculation in order to reserve an space for Pid part but it isn't honored when printed. The output before this patch

Re: [PATCH v4] usb: phy: samsung: Add support to set pmu isolation

2012-12-27 Thread Vivek Gautam
Hi Russell, On Thu, Dec 27, 2012 at 5:56 AM, Russell King - ARM Linux li...@arm.linux.org.uk wrote: On Wed, Dec 26, 2012 at 05:58:32PM +0530, Vivek Gautam wrote: + if (!ret) + sphy-phyctrl_pmureg = ioremap(reg[0], reg[1]); + + of_node_put(usbphy_pmu); + + if

Re: [PATCH 0/3] Thunderbolt workarounds

2012-12-27 Thread Mika Westerberg
On Wed, Dec 26, 2012 at 05:50:42PM -0700, Bjorn Helgaas wrote: On Thu, Dec 13, 2012 at 12:25 PM, Kirill A. Shutemov kirill.shute...@linux.intel.com wrote: From: Kirill A. Shutemov kirill.shute...@linux.intel.com I had chance to test two PC setups with Thunderbolt: Acer Aspire S5 and

Re: What does ISA/PCI really mean to ARM architecture?

2012-12-27 Thread Geert Uytterhoeven
On Thu, Dec 27, 2012 at 9:27 AM, Woody Wu narkewo...@gmail.com wrote: Can a peripheral chip that claims to be ISA or PCI device be used in a ARM based embedded system? For these kind of chips, I only concern about the planar kind of devices, means they are not on a dedicated expansion card.

Re: [PATCH 2/2] tracing: Use sched_clock_cpu for trace_clock_global

2012-12-27 Thread Namhyung Kim
On Thu, 27 Dec 2012 11:49:45 +0900, Namhyung Kim wrote: From: Namhyung Kim namhyung@lge.com For systems have unstable sched_clock, all cpu_clock() does is enable/ disable local irq during call to sched_clock(). And for stable systems Oops, I meant s/sched_clock/scheck_clock_cpu/.

Re: [PATCH 1/3] staging: drm/imx: check return value of ipu_reset()

2012-12-27 Thread Sascha Hauer
On Tue, Dec 25, 2012 at 03:58:37PM +0100, Lothar Waßmann wrote: ipu_reset() can fail with a timeout. Check the return value and act appropriately. Signed-off-by: Lothar Waßmann l...@karo-electronics.de All: Acked-by: Sascha Hauer s.ha...@pengutronix.de ---

Re: [PATCH 04/26] aio: remove retry-based AIO

2012-12-27 Thread Fubo Chen
On Mon, Dec 3, 2012 at 9:58 PM, Kent Overstreet koverstr...@google.com wrote: We want to remove retry-based AIO because it is fundemantally unsafe. Instead of fundemantally that should have been fundamentally I guess ? Thanks, Fubo. -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH 2/2] vhost: handle polling failure

2012-12-27 Thread Wanlong Gao
On 12/27/2012 02:39 PM, Jason Wang wrote: Currently, polling error were ignored in vhost. This may lead some issues (e.g kenrel crash when passing a tap fd to vhost before calling TUNSETIFF). Fix this by: Can this kernel crash be reproduced by hand? Thanks, Wanlong Gao - extend the idea

Re: [PATCH] drivers/platform/x86/thinkpad_acpi.c: Handle HKEY event 0x6040

2012-12-27 Thread Henrique de Moraes Holschuh
On Wed, 26 Dec 2012, Richard Hartmann wrote: + case TP_HKEY_EV_AC_CHANGED: + pr_info(AC status has changed\n); + /* X120e, x121e, X220, X220i, X220t, X230, T420, T420s, W520: + * AC status changed; can be triggered by plugging or + *

[RESEND PATCH] modpost: For ARC Port submission

2012-12-27 Thread Vineet Gupta
Hi, This is following up on ARC Linux Port submission bits: Basic Port: https://lkml.org/lkml/2012/11/7/114 Addons: https://lkml.org/lkml/2012/11/12/121 I'm resending this single patch to hopefully get ACK on simple modpost change. Thx, -Vineet Vineet Gupta (1): modpost: Ignore ARC

[PATCH] modpost: Ignore ARC specific non-alloc sections

2012-12-27 Thread Vineet Gupta
ARC relocatable object files contain one/more .gnu.linkonce.arcextmap.* sections (collated by kernel/vmlinux.lds into .arcextmap in final link). This section is used by debuggers to display the extension instructions and need-not be loaded by target (hence !SHF_ALLOC) The final kernel binary only

Re: [PATCH] ehci-mxc: remove Efika MX-specific CHRGVBUS hack

2012-12-27 Thread Sascha Hauer
On Sun, Dec 23, 2012 at 05:16:02AM -0600, Matt Sealey wrote: Since Efika MX platform support (pre-devicetree) was removed from the tree this code no longer has any possibility of running and clutters up the driver which is being replaced by the chipidea host in the future anyway.

Re: [PATCH] iio: (max1363) Fix probe error path

2012-12-27 Thread Jonathan Cameron
On 12/24/2012 06:24 AM, Guenter Roeck wrote: Instantiating the driver with no available regulator results in: [39711.686393] i2c i2c-7: new_device: Instantiated device max1139 at 0x35 [39711.688687] BUG: unable to handle kernel paging request at fe13 [39711.688734] IP:

echo 1 /proc/sys/vm/compact_memory - sh: write error: Bad address

2012-12-27 Thread Liu Hui-R64343
I met one error when does the following command: sh/$ echo 1 /proc/sys/vm/compact_memory sh/$ sh: write error: Bad address After using strace, I find the following: ... write(1, 1\n, 2) = 3 write(1, , 4294967295)= -1 EFAULT (Bad address) write(2,

Re: [RFC PATCH] virtio-net: reset virtqueue affinity when doing cpu hotplug

2012-12-27 Thread Michael S. Tsirkin
On Thu, Dec 27, 2012 at 11:34:16AM +0800, Jason Wang wrote: On 12/26/2012 06:46 PM, Michael S. Tsirkin wrote: On Wed, Dec 26, 2012 at 03:06:54PM +0800, Wanlong Gao wrote: Add a cpu notifier to virtio-net, so that we can reset the virtqueue affinity if the cpu hotplug happens. It improve

Re: [PATCH] [media] vivi: Constify structures

2012-12-27 Thread Hans Verkuil
On Wed December 26 2012 16:29:43 Kirill Smelkov wrote: Most of *_ops and other structures in vivi.c were already declared const but some have not. Constify and code/data will take less space: $ size drivers/media/platform/vivi.o textdata bss dec hex filename

Re: [PATCH v4] usb: phy: samsung: Add support to set pmu isolation

2012-12-27 Thread Vivek Gautam
Hi Sylwester, On Thu, Dec 27, 2012 at 4:00 AM, Sylwester Nawrocki sylvester.nawro...@gmail.com wrote: On 12/26/2012 01:28 PM, Vivek Gautam wrote: Adding support to parse device node data in order to get required properties to set pmu isolation for usb-phy. Signed-off-by: Vivek

Re: [PATCH 2/2] USB: ehci-s5p: Add to get interrupt from DT

2012-12-27 Thread Dongjin Kim
Hi Sergei, Yes, you are right. I made this patch to read its interrupt number from dtb directly. But now platform_get_irq() returns correct irq since OF_DEV_AUXDATA(...) is added as my first patch. This patch is useless. Thanks and best regards, Dongjin. On Thu, Dec 27, 2012 at 5:18 AM, Sergei

Re: [PATCH v5 14/14] memory-hotplug: free node_data when a node is offlined

2012-12-27 Thread Wen Congyang
At 12/26/2012 11:55 AM, Kamezawa Hiroyuki Wrote: (2012/12/24 21:09), Tang Chen wrote: From: Wen Congyang we...@cn.fujitsu.com We call hotadd_new_pgdat() to allocate memory to store node_data. So we should free it when removing a node. Signed-off-by: Wen Congyang we...@cn.fujitsu.com I'm

[PATCH 0/6] of, of_gpio, of_spi: Fix and improve of_parse_phandle_with_args, of_gpio_named_count and of_spi_register_master

2012-12-27 Thread Andreas Larsson
This patch series fixes a bug where of_gpio_named count relied upon a return value that was no longer returned from of_parse_phandle_with_args and adds the possibility for of_gpio_named_count to return error values. In addition, for of_spi_register_master it fixes a bug, adds documentation, adds

[PATCH 3/6] of_spi: Initialize cs_gpios properly

2012-12-27 Thread Andreas Larsson
Using memset does not set an array of integers properly Signed-off-by: Andreas Larsson andr...@gaisler.com --- drivers/spi/spi.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drivers/spi/spi.c b/drivers/spi/spi.c index 9c2acf1..a4baa0a 100644 --- a/drivers/spi/spi.c

[PATCH 5/6] of_spi: Add fetching of of_gpio flags to of_spi_register_master

2012-12-27 Thread Andreas Larsson
When using a gpio chip select with a OF_GPIO_ACTIVE_LOW flag, this needs to be known to the controller driver. Signed-off-by: Andreas Larsson andr...@gaisler.com --- Documentation/devicetree/bindings/spi/spi-bus.txt |3 +- drivers/spi/spi.c | 24

[PATCH 4/6] of_spi: Document cs_gpios and cs_gpio in kernel-doc

2012-12-27 Thread Andreas Larsson
This adds missing kernel-doc entries for cs_gpios in struct spi_master and cs_gpio in struct spi_device. Signed-off-by: Andreas Larsson andr...@gaisler.com --- include/linux/spi/spi.h |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/include/linux/spi/spi.h

[PATCH 6/6] of_spi: Initialize cs_gpios and cs_gpio with -EEXIST

2012-12-27 Thread Andreas Larsson
Holes in the cs-gpios DT phandle list is supposed to mark that native chipselects is to be used. The value returned from of_get_named_gpio_flags in this case is -EEXIST. By initializing cs_gpios and cs_gpio with -EEXIST, this and only this errno will indicate to a spi controller driver that a

[PATCH 1/6] of: Return -EEXIST from of_parse_phandle_with_args for holes in phandle list

2012-12-27 Thread Andreas Larsson
Return value for an empty phandle was -EEXIST before commit 15c9a0ac, that changed the return value in this case to -ENOENT. However, of_gpio_named_count relies upon the return value to be -EEXIST and relies upon being able to distinguish this case from the case of no list at all which also

Re: kernel BUG at mm/huge_memory.c:1798!

2012-12-27 Thread Hillf Danton
On Thu, Dec 27, 2012 at 8:31 AM, Alexander Beregalov a.berega...@gmail.com wrote: On 25 December 2012 16:05, Hillf Danton dhi...@gmail.com wrote: On Tue, Dec 25, 2012 at 12:38 PM, Zhouping Liu z...@redhat.com wrote: Hello all, I found the below kernel bug using latest mainline(637704cbc95),

[PATCH 2/6] of: Return -ENXIO from of_parse_phandle_with_args for too large index and return errors from of_gpio_named_count

2012-12-27 Thread Andreas Larsson
This lets of_gpio_named_count return an errno on errors by being able to distinguish between reaching the end of the phandle list and getting some other error from of_parse_phandle_with_args. Return error from of_spi_register_master when there is an cs-gpios list for which gp_gpio_named_count

Re: 3.8-rc1 build failure with MIPS/SPARSEMEM

2012-12-27 Thread Aaro Koskinen
Hi, On Wed, Dec 26, 2012 at 02:34:35AM +0200, Kirill A. Shutemov wrote: On MIPS if SPARSEMEM is enabled we've got this: In file included from /home/kas/git/public/linux/arch/mips/include/asm/pgtable.h:552, from include/linux/mm.h:44, from

Re: [Alternative][PATCH] ACPI / PCI: Set root bridge ACPI handle in advance

2012-12-27 Thread Rafael J. Wysocki
On Wednesday, December 26, 2012 04:10:32 PM Yinghai Lu wrote: On Wed, Dec 26, 2012 at 2:36 PM, Rafael J. Wysocki r...@sisk.pl wrote: On Wednesday, December 26, 2012 12:41:05 PM Yinghai Lu wrote: On Wed, Dec 26, 2012 at 12:16 PM, Yinghai Lu ying...@kernel.org wrote: On Wed, Dec 26, 2012 at

Re: [rtc-linux] [PATCH] rtc: recycle id when unloading a rtc driver

2012-12-27 Thread Alexander Holler
Am 19.12.2012 09:55, schrieb Alexander Holler: Unlikely, as I've worked hard to get one of the first drivers for pluggable RTCs into the kernel. ;) BTW. maybe you could have a look at the patch for that: https://lkml.org/lkml/2012/12/15/64 It should already be in your mailbox. I had to

Re: [PATCH 1/2] vhost_net: correct error hanlding in vhost_net_set_backend()

2012-12-27 Thread Michael S. Tsirkin
On Thu, Dec 27, 2012 at 02:39:20PM +0800, Jason Wang wrote: Currently, polling error were ignored in vhost. This may lead some issues (e.g kenrel crash when passing a tap fd to vhost before calling TUNSETIFF). Fix this by: - extend the idea of vhost_net_poll_state to all vhost_polls -

Re: [REGRESSION][3.8.-rc1][ INFO: possible circular locking dependency detected ]

2012-12-27 Thread Peter Hurley
On Thu, 2012-12-27 at 16:36 +0800, Shawn Guo wrote: On Wed, Dec 26, 2012 at 10:34:39AM +0800, Shawn Guo wrote: It seems that I'm running into the same locking issue. My setup is: - i.MX28 (ARM) - v3.8-rc1 - mxs_defconfig - The warning is seen when LCD is blanking The warning

<    1   2   3   4   5   6   >