Re: [PATCH] PM-wakeup: Delete unnecessary checks before two function calls

2015-08-13 Thread Pavel Machek
On Sun 2015-06-28 12:21:53, SF Markus Elfring wrote: > From: Markus Elfring > Date: Sun, 28 Jun 2015 12:14:43 +0200 > > The functions dev_pm_disarm_wake_irq() and wakeup_source_unregister() test > whether their argument is NULL and then return immediately. > Thus the test around the calls is not

Re: [PATCH 1/6] net/bonding: enable LRO if one device supports it

2015-08-13 Thread Michal Kubecek
On Thu, Aug 13, 2015 at 02:02:55PM -0400, Jarod Wilson wrote: > Currently, all bonding devices come up, and claim to have LRO support, > which ethtool will let you toggle on and off, even if none of the > underlying hardware devices actually support it. While the bonding driver > takes precautions

Re: [PATCH v2] Input: drivers/joystick: use parallel port device model

2015-08-13 Thread Sudip Mukherjee
On Thu, Aug 13, 2015 at 09:26:02AM -0700, Dmitry Torokhov wrote: > On Thu, Aug 13, 2015 at 07:16:14PM +0530, Sudip Mukherjee wrote: > > On Sat, Aug 08, 2015 at 04:05:15PM +0530, Sudip Mukherjee wrote: > > > On Tue, Aug 04, 2015 at 07:55:51PM +0530, Sudip Mukherjee wrote: > > > > Modify db9 driver t

[PATCH v5 5/8] perf: Invert perf_read_group() loops

2015-08-13 Thread Sukadev Bhattiprolu
From: Peter Zijlstra In order to enable the use of perf_event_read(.group = true), we need to invert the sibling-child loop nesting of perf_read_group(). Currently we iterate the child list for each sibling, this precludes using group reads. Flip things around so we iterate each group for each c

[PATCH v5 0/8] perf: Implement group-read of events using txn interface

2015-08-13 Thread Sukadev Bhattiprolu
Unlike normal hardware PMCs, the 24x7 counters in Power8 are stored in memory and accessed via a hypervisor call (HCALL). A major aspect of the HCALL is that it allows retireving _several_ counters at once (unlike regular PMCs, which are read one at a time). By reading several counters at once, we

[PATCH v5 3/8] perf: Rename perf_event_read_{one,group}, perf_read_hw

2015-08-13 Thread Sukadev Bhattiprolu
From: "Peter Zijlstra (Intel)" In order to free up the perf_event_read_group() name: s/perf_event_read_\(one\|group\)/perf_read_\1/g s/perf_read_hw/__perf_read/g Signed-off-by: Peter Zijlstra (Intel) --- kernel/events/core.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletion

[PATCH v5 2/8] perf: Split perf_event_read() and perf_event_count()

2015-08-13 Thread Sukadev Bhattiprolu
perf_event_read() does two things: - call the PMU to read/update the counter value, and - compute the total count of the event and its children Not all callers need both. perf_event_reset() for instance needs the first piece but doesn't need the second. Similarly, when we impleme

[PATCH v5 7/8] Define PERF_PMU_TXN_READ interface

2015-08-13 Thread Sukadev Bhattiprolu
Define a new PERF_PMU_TXN_READ interface to read a group of counters at once. pmu->start_txn()// Initialize before first event for each event in group pmu->read(event); // Queue each event to be read rc = pmu->commit_txn() //

[PATCH v5 6/8] perf: Add return value for perf_event_read().

2015-08-13 Thread Sukadev Bhattiprolu
When we implement the ability to read several counters at once (using the PERF_PMU_TXN_READ transaction interface), perf_event_read() can fail when the 'group' parameter is true (eg: trying to read too many events at once). For now, have perf_event_read() return an integer. Ignore the return value

[PATCH v5 1/8] perf: Add a flags parameter to pmu txn interfaces

2015-08-13 Thread Sukadev Bhattiprolu
Currently, the PMU interface allows reading only one counter at a time. But some PMUs like the 24x7 counters in Power, support reading several counters at once. To leveage this functionality, extend the transaction interface to support a "transaction type". The first type, PERF_PMU_TXN_ADD, refers

[PATCH v5 4/8] perf: Add group reads to perf_event_read()

2015-08-13 Thread Sukadev Bhattiprolu
From: Peter Zijlstra Enable perf_event_read() to update entire groups at once, this will be useful for read transactions. Cc: Ingo Molnar Cc: Arnaldo Carvalho de Melo Cc: Michael Ellerman Cc: Sukadev Bhattiprolu Signed-off-by: Peter Zijlstra (Intel) Link: http://lkml.kernel.org/r/201507230

[PATCH v5 8/8] powerpc/perf/hv-24x7: Use PERF_PMU_TXN_READ interface

2015-08-13 Thread Sukadev Bhattiprolu
The 24x7 counters in Powerpc allow monitoring a large number of counters simultaneously. They also allow reading several counters in a single HCALL so we can get a more consistent snapshot of the system. Use the PMU's transaction interface to monitor and read several event counters at once. The id

Re: [PATCH RESEND V2 3/7] scsi: storvsc: Untangle the storage protocol negotiation from the vmbus protocol negotiation.

2015-08-13 Thread Johannes Thumshirn
KY Srinivasan writes: >> -Original Message- >> From: Johannes Thumshirn [mailto:jthumsh...@suse.de] >> Sent: Thursday, August 13, 2015 7:34 AM >> To: KY Srinivasan ; Keith Mange >> >> Cc: gre...@linuxfoundation.org; linux-kernel@vger.kernel.org; >> de...@linuxdriverproject.org; oher...@s

Re: [PATCH V4 3/3] mfd: arizona: Support Cirrus Logic CS47L24 and WM1831

2015-08-13 Thread Lee Jones
On Wed, 12 Aug 2015, Richard Fitzgerald wrote: > This patch adds the regmap configuration tables and > core MFD handling for the CS47L24 and WM1831 codecs. > > Note that compared to the other Arizona codecs, these devices > do not have an LDO1 or micsupp regulators, extcon driver, or > the DCVDD

Re: [PATCH v2 3/6] mailbox: Add support for ST's Mailbox IP

2015-08-13 Thread Lee Jones
On Thu, 13 Aug 2015, Jassi Brar wrote: > On Mon, Jul 27, 2015 at 3:14 PM, Lee Jones wrote: > > > + > > +static bool sti_mbox_tx_is_ready(struct mbox_chan *chan) > > +{ > > + struct sti_channel *chan_info = chan->con_priv; > > + struct sti_mbox_device *mdev = chan_info->mdev; > > +

Re: [PATCH v2 4/4] Added getsynctime64() callback

2015-08-13 Thread Richard Cochran
On Thu, Aug 13, 2015 at 09:10:36PM +, Hall, Christopher S wrote: > > > + if (!cpu_has_art) > > > + return -EOPNOTSUPP; > > > > Perform this check before registration, setting .getsynctime64 > > accordingly. > > The problem here is that ART initialization doesn't happen until we > inst

Re: [PATCH] scsi: Export SCSI Inquiry data to sysfs

2015-08-13 Thread Hannes Reinecke
On 08/13/2015 01:50 PM, Johannes Thumshirn wrote: > Export the RAW SCSI Inquiry to sysfs as binfile. This way the data can be > used by userlang without the need to have and ioctl or use the sg_inq tool. > userland! > Here is an example of the provided data > > linux:~ # hexdump /sys/class/scsi_

Re: [PATCH 2/2] crypto: qat - Don't move data inside output buffer

2015-08-13 Thread Herbert Xu
On Thu, Aug 13, 2015 at 11:14:11PM -0700, Tadeusz Struk wrote: > > Right, but we don't need that anymore. Why not? If you reduce the size without moving the buffer wouldn't it begin with a bunch of zeroes and wouldn't you lose the real bytes at the end? Cheers, -- Email: Herbert Xu Home Page:

RE: [PATCH v2] dmaengine: fsl-edma: add PM suspend/resume support

2015-08-13 Thread Yao Yuan
Hi Leo, Thanks for your review. About those two methods for DMA suspend that you have mentioned. We have a lot of the discussions in other DMA driver like DMA for Freescale PowerPC. Finally, we think the device which used the DMA transmission service should cancel the transmission service in it

Re: [PATCH v2] drm/panel: add lg4573 driver

2015-08-13 Thread Heiko Schocher
Hello Thierry, Am 13.08.2015 um 14:59 schrieb Thierry Reding: On Tue, Jun 09, 2015 at 07:51:22AM +0200, Heiko Schocher wrote: Add support for LG LG4573 480x800 4,3" panel. the LG4573 is used on the LG LCD LB043WV2-SD01, an industrial 4.3" TFT panel with SPI control interface. Signed-off-by: He

Re: [PATCH 2/2] crypto: qat - Don't move data inside output buffer

2015-08-13 Thread Tadeusz Struk
On 08/13/2015 10:14 PM, Herbert Xu wrote: >> diff --git a/drivers/crypto/qat/qat_common/qat_asym_algs.c >> b/drivers/crypto/qat/qat_common/qat_asym_algs.c >> > index fe352a6..6ddb13c 100644 >> > --- a/drivers/crypto/qat/qat_common/qat_asym_algs.c >> > +++ b/drivers/crypto/qat/qat_common/qat_asym_a

International Basic Sciences Programmer Letter Attached

2015-08-13 Thread International Basic Sciences Programmer
International Basic Sciences Programmer Award Letter Attached Mr. F.T Fagebuder International Basic Sciences Programmer.pdf Description: Adobe PDF document

[PATCH] Staging: sm750fb: Fix "foo* bar" should be "foo *bar" errors

2015-08-13 Thread Shah, Yash (Y.)
From: Yash Shah Fix "foo* bar" should be "foo *bar" errors as detected by checkpatch.pl Signed-off-by: Yash Shah --- drivers/staging/sm750fb/sm750.h| 14 +++--- drivers/staging/sm750fb/sm750_cursor.c | 4 ++-- drivers/staging/sm750fb/sm750_cursor.h | 4 ++-- drivers/staging/s

Re: [PATCH 1/2] staging: lustre: fix static declarations in libcfs

2015-08-13 Thread Sudip Mukherjee
On Wed, Aug 12, 2015 at 11:51:04AM +0200, Maxime Lorrillere wrote: > This patch fix the following sparse warnings in libcfs/linux/linux-debug.c: > >>> linux-debug.c:64:6: warning: symbol 'lnet_upcall' was not declared. > >>> Should it be static? > >>> linux-debug.c:65:6: warning: symbol 'lnet_debu

Re: [PATCH V2] thermal: cpu_cooling: fix lockdep problems in cpu_cooling

2015-08-13 Thread Viresh Kumar
On 12-08-15, 15:22, Viresh Kumar wrote: > From: Russell King > Fixes: 2dcd851fe4b4 ("thermal: cpu_cooling: Update always cpufreq policy with > Reviewed-by: Viresh Kumar > Signed-off-by: Russell King > Signed-off-by: Viresh Kumar > --- > > Hi Eduardo/Zhang, > > This is 4.2-rc material, please

Re: [PATCH] Documentation: minor typo fix in mailbox.txt

2015-08-13 Thread Leo Yan
On Thu, Aug 13, 2015 at 06:04:11PM -0600, Jonathan Corbet wrote: > On Mon, 3 Aug 2015 09:30:25 +0800 > Leo Yan wrote: > > > Fix minor typo so that can pass correct pointer variable for > > container_of(). > > Looks good to me, applied to the docs tree. I also took the opportunity > to eliminat

[PATCH v2] target: remove initiatorname field in se_acl_lun

2015-08-13 Thread Spencer Baugh
From: Chris Zankel The initiatorname field in se_acl_lun is only a copy of the same field in se_node_acl, so remove it and use the version in se_node_acl where needed (it's actually only used for pr_debug) Signed-off-by: Chris Zankel Signed-off-by: Spencer Baugh --- drivers/target/target_core

[PATCH v2] target: remove unused lun_flags field from se_lun

2015-08-13 Thread Spencer Baugh
From: Chris Zankel The lun_flags field is not used, so drop it. Signed-off-by: Chris Zankel Signed-off-by: Spencer Baugh --- include/target/target_core_base.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/target/target_core_base.h b/include/target/target_core_base.h index 17ae2d

linux-next: build warnings after merge of the drm-misc tree

2015-08-13 Thread Stephen Rothwell
Hi all, After merging the drm-misc tree, today's linux-next build (arm multi_v7_defconfig) produced these warnings: drivers/gpu/drm/exynos/exynos_drm_drv.c:290:2: warning: initialization from incompatible pointer type .get_vblank_counter = drm_vblank_count, ^ drivers/gpu/drm/exynos/exynos_dr

Re: [PATCHv4 man-pages 3/3] open.2: describe O_BENEATH flag

2015-08-13 Thread Michael Kerrisk (man-pages)
On 13 August 2015 at 19:38, Andy Lutomirski wrote: > On Thu, Aug 13, 2015 at 2:32 AM, David Drysdale wrote: >> Signed-off-by: David Drysdale > > What's the behavior wrt fcntl(F_GETFL, etc)? I would presume that O_BENEATH is one of the so-called "file creation flags". See this paragraph of the D

Re: [PATCH 2/2] crypto: qat - Don't move data inside output buffer

2015-08-13 Thread Herbert Xu
On Wed, Aug 12, 2015 at 08:54:45PM -0700, Tadeusz Struk wrote: > Don't need to move data inside of the output buffer > because SW doen't need to do this anymore sice the new MPI > mpi_read_buf() has been added. Just set the correct output len. > > Signed-off-by: Tadeusz Struk > --- > drivers/cry

Re: [PATCH] book3s_hv_rmhandlers:Pass the correct trap argument to kvmhv_commence_exit

2015-08-13 Thread Sam Bobroff
On Thu, May 21, 2015 at 01:57:04PM +0530, Gautham R. Shenoy wrote: > In guest_exit_cont we call kvmhv_commence_exit which expects the trap > number as the argument. However r3 doesn't contain the trap number at > this point and as a result we would be calling the function with a > spurious trap num

Re: [PATCH] mm/hwpoison: fix race between soft_offline_page and unpoison_memory

2015-08-13 Thread Wanpeng Li
On 8/14/15 12:19 PM, Naoya Horiguchi wrote: > On Thu, Aug 13, 2015 at 06:27:40PM +0800, Wanpeng Li wrote: >> On 8/13/15 6:04 PM, Naoya Horiguchi wrote: >>> On Thu, Aug 13, 2015 at 05:18:56PM +0800, Wanpeng Li wrote: On 8/13/15 4:53 PM, Naoya Horiguchi wrote: >>> ... > I think that unpoison

[PATCH v2] target: Fix handling of small allocation lengths in REPORT LUNS

2015-08-13 Thread Spencer Baugh
From: Roland Dreier REPORT LUNS should not fail just because the allocation length is less than 16. The relevant section of SPC-4 is: 4.2.5.6 Allocation length The ALLOCATION LENGTH field specifies the maximum number of bytes or blocks that an application client has allocated in the Data

Re: [PATCH] net: allow sleeping when modifying store_rps_map

2015-08-13 Thread David Miller
From: Sasha Levin Date: Thu, 13 Aug 2015 14:03:16 -0400 > Commit 10e4ea751 ("net: Fix race condition in store_rps_map") has moved the > manipulation of the rps_needed jump label under a spinlock. Since changing > the state of a jump label may sleep this is incorrect and causes warnings > during r

Re: [PATCH v3] powerpc: Add an inline function to update POWER8 HID0

2015-08-13 Thread Sam Bobroff
On Wed, Aug 05, 2015 at 12:38:31PM +0530, Gautham R. Shenoy wrote: > Section 3.7 of Version 1.2 of the Power8 Processor User's Manual > prescribes that updates to HID0 be preceded by a SYNC instruction and > followed by an ISYNC instruction (Page 91). > > Create an inline function name update_powe

[git pull] drm fixes

2015-08-13 Thread Dave Airlie
Hi Linus, back from holidays, found these in the cracks, one nouveau revert, one vmwgfx locking fix and a bunch of exynos fixes. Thanks, Dave. The following changes since commit 5b3e2e14eaa2a98232a4f292341fb88438685734: Merge tag 'dm-4.2-fixes-5' of git://git.kernel.org/pub/scm/linux/kernel

Re: [PATCH net-next 0/7] net: dsa: mv88e6xxx: add hardware VLAN support

2015-08-13 Thread David Miller
From: Vivien Didelot Date: Thu, 13 Aug 2015 12:52:16 -0400 > This patchset brings support to access hardware VLAN entries in DSA and > mv88e6xxx, through switchdev VLAN objects. Looks good, series applied, thanks Vivien. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel"

Re: [PATCH] iwlwifi: out-of-bounds access in iwl_init_sband_channels

2015-08-13 Thread Grumbach, Emmanuel
Hi, On 08/14/2015 03:36 AM, Adrien Schildknecht wrote: > Both loops of this function compare data from the 'chan' array and then > check if the index is valid. > > The 2 conditions should be inverted to avoid an out-of-bounds access. > Was that found by a static analyzer or any other automated

[PATCH v2] lib/decompressors: Use real out buf size for gunzip with kernel

2015-08-13 Thread Yinghai Lu
When loading x86 64bit kernel above 4GiB with patched grub2, got kernel gunzip error. | early console in decompress_kernel | decompress_kernel: | input: [0x807f2143b4-0x807ff61aee] | output: [0x807cc0-0x807f3ea29b] 0x027ea29c: output_len | boot via startup_64 | KASLR using RDTSC...

Re: [PATCH] mm/hwpoison: fix race between soft_offline_page and unpoison_memory

2015-08-13 Thread Naoya Horiguchi
On Thu, Aug 13, 2015 at 06:27:40PM +0800, Wanpeng Li wrote: > On 8/13/15 6:04 PM, Naoya Horiguchi wrote: > > On Thu, Aug 13, 2015 at 05:18:56PM +0800, Wanpeng Li wrote: > >> On 8/13/15 4:53 PM, Naoya Horiguchi wrote: > > ... > >>> I think that unpoison is used only in testing so this race never aff

Re: [PATCH 1/1 V2] ipv4: off-by-one in continuation handling in /proc/net/route

2015-08-13 Thread David Miller
From: Andy Whitcroft Date: Thu, 13 Aug 2015 20:49:01 +0100 > When generating /proc/net/route we emit a header followed by a line for > each route. When a short read is performed we will restart this process > based on the open file descriptor. When calculating the start point we > fail to take

Re: [PATCH] net/fddi: remove HWM_REVERSE() macro

2015-08-13 Thread David Miller
From: yalin wang Date: Thu, 13 Aug 2015 12:01:33 +0800 > HWM_REVERSE() macro is unused, remove it. > > Signed-off-by: yalin wang Applied. -- 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 htt

Re: [PATCH 29/31] parisc: handle page-less SG entries

2015-08-13 Thread David Miller
From: James Bottomley Date: Thu, 13 Aug 2015 20:59:20 -0700 > On Thu, 2015-08-13 at 20:30 -0700, Dan Williams wrote: >> On Thu, Aug 13, 2015 at 7:31 AM, Christoph Hellwig wrote: >> > On Wed, Aug 12, 2015 at 09:01:02AM -0700, Linus Torvalds wrote: >> >> I'm assuming that anybody who wants to use

Re: [PATCH] security/smack/smack_lsm.c: Fix build error when CONFIG_SECURITY_SMACK_BRINGUP is not defined

2015-08-13 Thread Casey Schaufler
On 8/13/2015 5:52 PM, Murilo Opsfelder Araujo wrote: > This change fixes the following build error when > CONFIG_SECURITY_SMACK_BRINGUP is not defined: > > security/smack/smack_lsm.c: In function ‘smack_parse_opts_str’: > security/smack/smack_lsm.c:618:26: error: ‘tokens’ undeclared (first use in

Re: [PATCH v7 0/9] Fixes and new clocks support for Mediatek MT8173

2015-08-13 Thread Daniel Kurtz
On Tue, Aug 11, 2015 at 12:43 PM, Daniel Kurtz wrote: > Hi James, > > On Mon, Aug 10, 2015 at 5:50 PM, James Liao wrote: >> This patchset is based on 4.2-rc2 and [1], and contains minor fixes and >> subsystem clocks support for Mediatek MT8173. >> >> The previous reviews can be found in [2]. >> >

Re: [PATCH 29/31] parisc: handle page-less SG entries

2015-08-13 Thread James Bottomley
On Thu, 2015-08-13 at 20:30 -0700, Dan Williams wrote: > On Thu, Aug 13, 2015 at 7:31 AM, Christoph Hellwig wrote: > > On Wed, Aug 12, 2015 at 09:01:02AM -0700, Linus Torvalds wrote: > >> I'm assuming that anybody who wants to use the page-less > >> scatter-gather lists always does so on memory th

linux-next: manual merge of the xen-tip tree with Linus' tree

2015-08-13 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the xen-tip tree got a conflict in: arch/x86/xen/Makefile between commit: fc5fee86bdd3 ("x86/xen: build "Xen PV" APIC driver for domU as well") from Linus' tree and commit: 6e3012fba294 ("xen/PMU: Initialization code for Xen PMU") from the xen-tip tr

Re: [PATCH] Revert "drm/nouveau/fifo/gk104: kick channels when deactivating them"

2015-08-13 Thread Alexandre Courbot
On Wed, Aug 12, 2015 at 6:59 PM, Afzal Mohammed wrote: > Hi, > > On Wed, Aug 12, 2015 at 04:40:57PM +0900, Alexandre Courbot wrote: > >> Great, thanks. Are you also on an optimus configuration with the >> NVIDIA card being the secondary GPU? > > Spec says graphic processor is NVIDIA GeForce NV14P-

RE: [PATCH v3 1/2] Documentation: DT: FTM: add FTM0 be used as alarm timer

2015-08-13 Thread Wang Dongsheng
Thank Yoder, Fix them in next version. Regards, -Dongsheng > -Original Message- > From: Stuart Yoder [mailto:b08...@gmail.com] > Sent: Friday, August 14, 2015 12:23 AM > To: Wang Dongsheng-B40534 > Cc: shawn@linaro.org; devicet...@vger.kernel.org; linux- > ker...@vger.kernel.org; lin

Re: [PATCH 29/31] parisc: handle page-less SG entries

2015-08-13 Thread Dan Williams
On Thu, Aug 13, 2015 at 7:31 AM, Christoph Hellwig wrote: > On Wed, Aug 12, 2015 at 09:01:02AM -0700, Linus Torvalds wrote: >> I'm assuming that anybody who wants to use the page-less >> scatter-gather lists always does so on memory that isn't actually >> virtually mapped at all, or only does so o

Re: [PATCH v1 1/3] usb: dwc2: reset AHB hclk domain before init

2015-08-13 Thread Yunzhi Li
在 2015/8/14 8:09, John Youn 写道: On 8/11/2015 12:57 AM, Yunzhi Li wrote: We initiate dwc2 usb controller in BIOS, when kernel driver start-up we should reset AHB hclk domain to reset all AHB interface registers to default. Without this the FIFO value setting might be incorrect because calculati

[PATCH v10 02/20] x86/asm: Add C versions of frame pointer macros

2015-08-13 Thread Josh Poimboeuf
Add C versions of the frame pointer macros which can be used to create a stack frame in inline assembly. Signed-off-by: Josh Poimboeuf --- arch/x86/include/asm/frame.h | 20 ++-- 1 file changed, 18 insertions(+), 2 deletions(-) diff --git a/arch/x86/include/asm/frame.h b/arch/x8

[PATCH v10 04/20] x86/stackvalidate: Add file and directory ignores

2015-08-13 Thread Josh Poimboeuf
Tell stackvalidate to skip validation of the following code: - boot image - vdso image - kexec purgatory - realmode - efi libstub - scripts/mod They all run outside the kernel's normal mode of operation and they don't affect runtime kernel stack traces, so they're free to skirt the stackvalidate

[PATCH v10 00/20] Compile-time stack validation

2015-08-13 Thread Josh Poimboeuf
This is v10 of the compile-time stack validation patch set, along with proposed fixes for many of the warnings it found. It's based on the tip/master branch. The frame pointer macros have been renamed from FRAME/ENDFRAME to FRAME_BEGIN/FRAME_END. v9 can be found here: https://lkml.kernel.org/

[PATCH v10 03/20] x86/stackvalidate: Compile-time stack validation

2015-08-13 Thread Josh Poimboeuf
This adds a CONFIG_STACK_VALIDATION option which enables a host tool named stackvalidate which runs at compile time. It analyzes every .o file and ensures the validity of its stack metadata. It enforces a set of rules on asm code and C inline assembly code so that stack traces can be reliable. C

[PATCH v10 05/20] x86/stackvalidate: Add ignore macros

2015-08-13 Thread Josh Poimboeuf
Add new stackvalidate ignore macros: STACKVALIDATE_IGNORE_INSN and STACKVALIDATE_IGNORE_FUNC. These can be used to tell stackvalidate to skip validation of an instruction or a function, respectively. Signed-off-by: Josh Poimboeuf --- arch/x86/include/asm/stackvalidate.h | 45 +++

[PATCH v10 07/20] x86/paravirt: Add stack frame dependency to PVOP inline asm calls

2015-08-13 Thread Josh Poimboeuf
If a PVOP call macro is inlined at the beginning of a function, gcc can insert the call instruction before setting up a stack frame, which breaks frame pointer convention if CONFIG_FRAME_POINTER is enabled and can result in a bad stack trace. Force a stack frame to be created if CONFIG_FRAME_POINT

[PATCH v10 08/20] x86/paravirt: Create a stack frame in PV_CALLEE_SAVE_REGS_THUNK

2015-08-13 Thread Josh Poimboeuf
A function created with the PV_CALLEE_SAVE_REGS_THUNK macro doesn't set up a new stack frame before the call instruction, which breaks frame pointer convention if CONFIG_FRAME_POINTER is enabled and can result in a bad stack trace. Also, the thunk functions aren't annotated as ELF callable functio

[PATCH v10 12/20] x86/asm/crypto: Create stack frames in aesni-intel_asm.S

2015-08-13 Thread Josh Poimboeuf
aesni-intel_asm.S has several callable non-leaf functions which don't honor CONFIG_FRAME_POINTER, which can result in bad stack traces. Create stack frames for them when CONFIG_FRAME_POINTER is enabled. Signed-off-by: Josh Poimboeuf Cc: Herbert Xu Cc: "David S. Miller" --- arch/x86/crypto/aes

[PATCH v10 09/20] x86/amd: Set ELF function type for vide()

2015-08-13 Thread Josh Poimboeuf
vide() is a callable function, but is missing the ELF function type, which confuses tools like stackvalidate. Properly annotate it to be a callable function. The generated code is unchanged. Signed-off-by: Josh Poimboeuf --- arch/x86/kernel/cpu/amd.c | 5 - 1 file changed, 4 insertions(+),

[PATCH v10 10/20] x86/reboot: Add ljmp instructions to stackvalidate whitelist

2015-08-13 Thread Josh Poimboeuf
stackvalidate reports a false positive warning for the ljmp instruction in machine_real_restart(). Normally, ljmp isn't allowed in a function, but this is a special case where it's jumping into real mode. Add the jumps to a whitelist which tells stackvalidate to ignore them. Signed-off-by: Josh

[PATCH v10 13/20] x86/asm/crypto: Move .Lbswap_mask data to .rodata section

2015-08-13 Thread Josh Poimboeuf
stackvalidate reports the following warning: stackvalidate: arch/x86/crypto/aesni-intel_asm.o: _aesni_inc_init(): can't find starting instruction stackvalidate gets confused when it tries to disassemble the following data in the .text section: .Lbswap_mask: .byte 15, 14, 13, 12, 1

[PATCH v10 14/20] x86/asm/crypto: Move jump_table to .rodata section

2015-08-13 Thread Josh Poimboeuf
stackvalidate reports the following warning: stackvalidate: arch/x86/crypto/crc32c-pcl-intel-asm_64.o: crc_pcl()+0x11dd: can't decode instruction It gets confused when trying to decode jump_table data. Move jump_table to the .rodata section which is a more appropriate home for read-only data.

[PATCH v10 11/20] x86/xen: Add xen_cpuid() and xen_setup_gdt() to stackvalidate whitelists

2015-08-13 Thread Josh Poimboeuf
stackvalidate reports the following false positive warnings: stackvalidate: arch/x86/xen/enlighten.o: xen_cpuid()+0x41: can't find jump dest instruction at .text+0x108 stackvalidate: arch/x86/xen/enlighten.o: xen_setup_gdt.constprop.23()+0x2e: kernel entry/exit from callable instruction The

[PATCH v10 18/20] x86/asm: Create stack frames in rwsem functions

2015-08-13 Thread Josh Poimboeuf
rwsem.S has several callable non-leaf functions which don't honor CONFIG_FRAME_POINTER, which can result in bad stack traces. Create stack frames for them when CONFIG_FRAME_POINTER is enabled. Signed-off-by: Josh Poimboeuf --- arch/x86/lib/rwsem.S | 11 ++- 1 file changed, 10 insertions

[PATCH v10 16/20] x86/asm/entry: Create stack frames in thunk functions

2015-08-13 Thread Josh Poimboeuf
Thunk functions are callable non-leaf functions that don't honor CONFIG_FRAME_POINTER, which can result in bad stack traces. Also they aren't annotated as ELF callable functions which can confuse tooling. Create stack frames for them when CONFIG_FRAME_POINTER is enabled and add the ELF function t

[PATCH v10 17/20] x86/asm/acpi: Create a stack frame in do_suspend_lowlevel()

2015-08-13 Thread Josh Poimboeuf
do_suspend_lowlevel() is a callable non-leaf function which doesn't honor CONFIG_FRAME_POINTER, which can result in bad stack traces. Create a stack frame for it when CONFIG_FRAME_POINTER is enabled. Signed-off-by: Josh Poimboeuf Acked-by: Pavel Machek Cc: "Rafael J. Wysocki" Cc: Len Brown --

[PATCH v10 20/20] x86/asm/power: Create stack frames in hibernate_asm_64.S

2015-08-13 Thread Josh Poimboeuf
swsusp_arch_suspend() and restore_registers() are callable non-leaf functions which don't honor CONFIG_FRAME_POINTER, which can result in bad stack traces. Also they aren't annotated as ELF callable functions which can confuse tooling. Create a stack frame for them when CONFIG_FRAME_POINTER is en

[PATCH v10 19/20] x86/asm/efi: Create a stack frame in efi_call()

2015-08-13 Thread Josh Poimboeuf
efi_call() is a callable non-leaf function which doesn't honor CONFIG_FRAME_POINTER, which can result in bad stack traces. Create a stack frame for it when CONFIG_FRAME_POINTER is enabled. Signed-off-by: Josh Poimboeuf Cc: Matt Fleming --- arch/x86/platform/efi/efi_stub_64.S | 3 +++ 1 file ch

[PATCH v10 15/20] x86/asm/crypto: Create stack frames in clmul_ghash_mul/update()

2015-08-13 Thread Josh Poimboeuf
clmul_ghash_mul() and clmul_ghash_update() are callable non-leaf functions which don't honor CONFIG_FRAME_POINTER, which can result in bad stack traces. Create stack frames for them when CONFIG_FRAME_POINTER is enabled. Signed-off-by: Josh Poimboeuf Cc: Herbert Xu Cc: "David S. Miller" --- ar

RE: [PATCH 2/2] soc/fsl: add ftm alarm driver for ls1021a platform

2015-08-13 Thread Wang Dongsheng
> -Original Message- > From: Linus Walleij [mailto:linus.wall...@linaro.org] > Sent: Thursday, August 13, 2015 9:54 PM > To: Wang Dongsheng-B40534; John Stultz; Alessandro Zummo; Alexandre Belloni > Cc: Shawn Guo; Nair, Sandeep; Hans de Goede; Wang Huan-B18965; linux-arm- > ker...@lists.i

[PATCH v10 01/20] x86/asm: Frame pointer macro cleanup

2015-08-13 Thread Josh Poimboeuf
The asm macros for setting up and restoring the frame pointer aren't currently being used. However, they will be needed soon to help asm functions to comply with stackvalidate. Rename FRAME/ENDFRAME to FRAME_BEGIN/FRAME_END for more symmetry. Also make the code more readable and improve the comm

[PATCH v10 06/20] x86/xen: Add stack frame dependency to hypercall inline asm calls

2015-08-13 Thread Josh Poimboeuf
If a hypercall is inlined at the beginning of a function, gcc can insert the call instruction before setting up a stack frame, which breaks frame pointer convention if CONFIG_FRAME_POINTER is enabled and can result in a bad stack trace. Force a stack frame to be created if CONFIG_FRAME_POINTER is

[PATCH] lib/decompress_unlzma: Do a NULL check for pointer

2015-08-13 Thread Fabio Estevam
From: Fabio Estevam Compare pointer-typed values to NULL rather than 0. The semantic patch that makes this change is available in scripts/coccinelle/null/badzero.cocci. Signed-off-by: Fabio Estevam --- lib/decompress_unlzma.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a

Re: [PATCH] kvm:powerpc:Fix return statements for wrapper functions in the file book3s_64_mmu_hv.c

2015-08-13 Thread Michael Ellerman
On Wed, 2015-08-12 at 21:06 +0200, Alexander Graf wrote: > > On 10.08.15 17:27, Nicholas Krause wrote: > > This fixes the wrapper functions kvm_umap_hva_hv and the function > > kvm_unmap_hav_range_hv to return the return value of the function > > kvm_handle_hva or kvm_handle_hva_range that they ar

Re: i915/kasan: out of bounds access in i915_cmd_parser_init_ring

2015-08-13 Thread Dave Airlie
On 14 August 2015 at 11:09, Dave Jones wrote: > I finally got around to playing with kasan. It didn't end well. > > I added some debugging to validate_cmds_sorted to print out the table > sizes right before the stack traces. oops typo, patch sent. Dave. -- To unsubscribe from this list: send th

Re: [PATCH v5 1/6] locking/pvqspinlock: Unconditional PV kick with _Q_SLOW_VAL

2015-08-13 Thread Waiman Long
On 08/08/2015 02:02 AM, Peter Zijlstra wrote: On Fri, Aug 07, 2015 at 11:17:56PM -0400, Waiman Long wrote: If _Q_SLOW_VAL has been set, the vCPU state must have been vcpu_hashed. The extra check at the end of __pv_queued_spin_unlock() is unnecessary and so is removed. This is half the patch it

linux-next: manual merge of the drm-misc tree with Linus' tree

2015-08-13 Thread Stephen Rothwell
Hi all, Today's linux-next merge of the drm-misc tree got a conflict in: drivers/gpu/drm/drm_irq.c between commit: 209e4dbc8dcd ("drm/vblank: Use u32 consistently for vblank counters") from Linus' tree and commit: b90180b057f7 ("drm/irq: More pipe/crtc consistency cleanups") from the d

Re: [PATCH] regulator: core: Spell fix

2015-08-13 Thread Viresh Kumar
On 13-08-15, 14:15, Mark Brown wrote: > On Thu, Aug 13, 2015 at 06:09:49PM +0530, Viresh Kumar wrote: > > Trivial spell fix, s/succesfully/successfully. > > That's "spelling fix" not "spell fix" :) :( Thanks for fixing/applying it :) -- viresh -- To unsubscribe from this list: send the line "u

Re: [regression] x86/signal/64: Fix SS handling for signals delivered to 64-bit programs breaks dosemu

2015-08-13 Thread Stas Sergeev
14.08.2015 04:37, Andy Lutomirski пишет: On Thu, Aug 13, 2015 at 6:32 PM, Stas Sergeev wrote: 14.08.2015 04:21, Andy Lutomirski пишет: On Thu, Aug 13, 2015 at 5:50 PM, Stas Sergeev wrote: 14.08.2015 03:27, Linus Torvalds пишет: On Thu, Aug 13, 2015 at 5:17 PM, Stas Sergeev wrote: For exa

linux-next: manual merge of the drm-tegra tree with the drm-misc tree

2015-08-13 Thread Stephen Rothwell
Hi Thierry, Today's linux-next merge of the drm-tegra tree got conflicts in: drivers/gpu/drm/tegra/dsi.c drivers/gpu/drm/tegra/hdmi.c drivers/gpu/drm/tegra/rgb.c drivers/gpu/drm/tegra/sor.c between commit: 9a69a9ac20f7 ("drm: Make the connector dpms callback return a value, v2") from

Re: [GIT PULL] MODSIGN: Use PKCS#7 for module signatures [ver #8a]

2015-08-13 Thread James Morris
On Thu, 13 Aug 2015, David Howells wrote: > Hi James, > > Can you pull this into security/next please? Its aim is twofold: firstly, > make the module signatures of PKCS#7/CMS format rather than a home-brewed > format and secondly to pave the way for use of the signing code for > firmware signatu

Re: [PATCH] pci: acpi: Generic function for setting up PCI device DMA coherency

2015-08-13 Thread Bjorn Helgaas
Hi Suravee, On Thu, Aug 13, 2015 at 04:58:45PM +0700, Suravee Suthikulpanit wrote: > This patch refactors of_pci_dma_configure() into a more generic > pci_dma_configure(), which can be reused by non-OF code. > Then, it adds support for setting up PCI device DMA coherency from > ACPI _CCA object th

[PATCH v4 1/2] mpt2sas: Refcount sas_device objects and fix unsafe list usage

2015-08-13 Thread Calvin Owens
These objects can be referenced concurrently throughout the driver, we need a way to make sure threads can't delete them out from under each other. This patch adds the refcount, and refactors the code to use it. Additionally, we cannot iterate over the sas_device_list without holding the lock, or

Re: [PATCH] usb: phy: phy-generic: Fix reset behaviour on legacy boot

2015-08-13 Thread Fabio Estevam
On Thu, Aug 13, 2015 at 7:28 AM, Roger Quadros wrote: > The gpio-desc migration done in v4.0 caused a regression > with legacy boots due to reversed reset logic. > e.g. omap3-beagle USB host breaks on legacy boot. > > Request the reset GPIO with GPIOF_ACTIVE_LOW flag so that > it matches the drive

[PATCH v4 2/2] mpt2sas: Refcount fw_events and fix unsafe list usage

2015-08-13 Thread Calvin Owens
The fw_event_work struct is concurrently referenced at shutdown, so add a refcount to protect it, and refactor the code to use it. Additionally, refactor _scsih_fw_event_cleanup_queue() such that it no longer iterates over the list without holding the lock, since _firmware_event_work() concurrentl

[PATCH v4 0/2] Fixes for memory corruption in mpt2sas

2015-08-13 Thread Calvin Owens
Hello all, This patchset attempts to address problems we've been having with panics due to memory corruption from the mpt2sas driver. Thanks, Calvin [PATCH v4 1/2] mpt2sas: Refcount sas_device objects and fix unsafe list [PATCH v4 2/2] mpt2sas: Refcount fw_events and fix unsafe list usage Tota

[PATCH RESEND] sched/nohz: Affine unpinned timers to housekeepers

2015-08-13 Thread Frederic Weisbecker
From: Vatika Harlalka The problem addressed in this patch is about affining unpinned timers. Adaptive or Full Dynticks CPUs are currently disturbed by unnecessary jitter due to firing of such timers on them. This patch will affine timers to online CPUs which are not full dynticks in NOHZ_FULL co

Re: [PATCH v3 1/2] mpt2sas: Refcount sas_device objects and fix unsafe list usage

2015-08-13 Thread Calvin Owens
On Monday 08/10 at 18:45 +0530, Sreekanth Reddy wrote: > On Sat, Aug 1, 2015 at 10:32 AM, Calvin Owens wrote: Sreekanth, Thanks for the review, responses below. I'll have a v4 out shortly. Calvin > > These objects can be referenced concurrently throughout the driver, we > > need a way to make

Re: [regression] x86/signal/64: Fix SS handling for signals delivered to 64-bit programs breaks dosemu

2015-08-13 Thread Andy Lutomirski
On Thu, Aug 13, 2015 at 6:32 PM, Stas Sergeev wrote: > 14.08.2015 04:21, Andy Lutomirski пишет: > >> On Thu, Aug 13, 2015 at 5:50 PM, Stas Sergeev wrote: >>> >>> 14.08.2015 03:27, Linus Torvalds пишет: On Thu, Aug 13, 2015 at 5:17 PM, Stas Sergeev wrote: > > For example because

RE: [PATCH 1/1] Drivers: hv: vmbus: fix init_vp_index() for reloading hv_netvsc

2015-08-13 Thread KY Srinivasan
> -Original Message- > From: James Bottomley [mailto:james.bottom...@hansenpartnership.com] > Sent: Thursday, August 13, 2015 4:09 PM > To: KY Srinivasan > Cc: gre...@linuxfoundation.org; linux-kernel@vger.kernel.org; > de...@linuxdriverproject.org; oher...@suse.com; > jbottom...@paralle

Re: [regression] x86/signal/64: Fix SS handling for signals delivered to 64-bit programs breaks dosemu

2015-08-13 Thread Stas Sergeev
14.08.2015 04:21, Andy Lutomirski пишет: On Thu, Aug 13, 2015 at 5:50 PM, Stas Sergeev wrote: 14.08.2015 03:27, Linus Torvalds пишет: On Thu, Aug 13, 2015 at 5:17 PM, Stas Sergeev wrote: For example because you can as well do: prctl(ARCH_SET_SIGNAL_SS, 0) which will mean "restore ss in sigha

Re: [regression] x86/signal/64: Fix SS handling for signals delivered to 64-bit programs breaks dosemu

2015-08-13 Thread Andy Lutomirski
On Thu, Aug 13, 2015 at 5:50 PM, Stas Sergeev wrote: > 14.08.2015 03:27, Linus Torvalds пишет: >> >> On Thu, Aug 13, 2015 at 5:17 PM, Stas Sergeev wrote: >>> >>> For example because you can as well do: >>> prctl(ARCH_SET_SIGNAL_SS, 0) >>> which will mean "restore ss in sighandler to its current v

Re: [PATCH 1/2] crypto: KEYS: convert public key to the akcipher API

2015-08-13 Thread Herbert Xu
On Thu, Aug 13, 2015 at 03:23:16PM +0100, David Howells wrote: > > > - /* Decode the public key */ > > - ret = asn1_ber_decoder(&x509_rsakey_decoder, ctx, > > - ctx->key, ctx->key_size); > > - if (ret < 0) > > + cert->pub->key = kmemdup(ctx->key, ctx->key_size, GFP

i915/kasan: out of bounds access in i915_cmd_parser_init_ring

2015-08-13 Thread Dave Jones
I finally got around to playing with kasan. It didn't end well. I added some debugging to validate_cmds_sorted to print out the table sizes right before the stack traces. Dave validate_cmds_sorted: table:a1fb4220 cmd_table_count:3 validate_cmds_sorted: table:a1fb4220 tabl

[PATCH v2] scripts/setlocalversion: fix false positive -dirty tag from ctime

2015-08-13 Thread Gregory Fong
git-diff-index cannot detect that a file hasn't actually changed when the file's ctime has changed (e.g. if chmod was used but didn't actually result in any changes), which was causing setlocalversion to incorrectly add a -dirty tag. Run git-update-index beforehand to make sure that the index is c

Re: [PATCH net-next 1/7] net: dsa: add support for switchdev VLAN objects

2015-08-13 Thread Andrew Lunn
On Thu, Aug 13, 2015 at 05:01:57PM -0400, Vivien Didelot wrote: > Hi Andrew, > > On 15-08-13 22:16:08, Andrew Lunn wrote: > > On Thu, Aug 13, 2015 at 12:52:17PM -0400, Vivien Didelot wrote: > > > Add new functions in DSA drivers to access hardware VLAN entries through > > > SWITCHDEV_OBJ_PORT_VLAN

Re: [PATCH V3 4/5] clk: core: add CLK_OPS_PARENT_ON flags to support clocks require parent on

2015-08-13 Thread Stephen Boyd
On 07/28, Dong Aisheng wrote: > On Freescale i.MX7D platform, all clocks operations, including > enable/disable, rate change and re-parent, requires its parent > clock on. Current clock core can not support it well. > This patch introduce a new flag CLK_OPS_PARENT_ON to handle this > special case i

Re: [PATCH v10 3/8] add fpga manager core

2015-08-13 Thread Moritz Fischer
Hi Alan, I've updated my Zynq driver (it can be found in an older version against your v8 in the Xilinx tree, too) https://github.com/mfischer/linux/tree/alan-fpga-mgr-v10 to use your v10 version of the patch. Either I'm using the API wrong , or it never gets to the 'operating state'. Comments i

  1   2   3   4   5   6   7   8   9   >