[PATCH] watchdog: hpwdt: add support for iLO5

2016-09-26 Thread Brian Boylston
iLO5 will offer the same watchdog timer as previous generations, but the PCI subsystem vendor ID will be PCI_VENDOR_ID_HP_3PAR (0x1590) instead of PCI_VENDOR_ID_HP (0x103c). Add 0x1590 to the whitelist and be more specific when ignoring the 103c,1979 device. Signed-off-by: Brian Boylston --- dr

Re: [PATCH] f2fs: introduce get_checkpoint_version for cleanup

2016-09-26 Thread Jaegeuk Kim
Hi Tiezhu, On Sun, Sep 25, 2016 at 05:50:44PM +0800, Tiezhu Yang wrote: > There exists almost same codes when get the value of pre_version > and cur_version in function validate_checkpoint, this patch adds > get_checkpoint_version to clean up redundant codes. > > Signed-off-by: Tiezhu Yang > ---

Re: modules still have .debug_* (was Re: [PATCH 0/3] ARC unwinder switch to .eh_frame)

2016-09-26 Thread Daniel Mentz
On Mon, Sep 26, 2016 at 5:38 AM, Alexey Brodkin wrote: > On Fri, 2016-09-23 at 18:04 -0700, Daniel Mentz wrote: >> On Fri, Sep 23, 2016 at 3:38 AM, Alexey Brodkin >> wrote: >> > >> > On Thu, 2016-09-22 at 15:37 -0700, Daniel Mentz wrote: >> > > >> > > Sorry, that was a misunderstanding. Buildroot

[PATCH 0/2] Moving runnable code from Documentation (last 2 patches)

2016-09-26 Thread Shuah Khan
This patch series contains the last 2 patches to complete moving runnable code from Documentation to selftests, samples, and tools. The first patch moves blackfin gptimers-example to samples and removes CONFIG_BUILD_DOCSRC. The second one updates 00-INDEX files under Documentation to reflect the

Re: [PATCH v2 5/5] ISDN-Gigaset: Enclose two expressions for the sizeof operator by parentheses

2016-09-26 Thread SF Markus Elfring
> Don't resubmit one single patch of a series (within hours!) > as you did here. I hope that I could reduce the confusion a bit which I introduced with a change in the last step of my questionable update suggestion. > I hope to have a look at your series within a few days. > Show some patience.

Re: [PATCH 0/2] Moving runnable code from Documentation (last 2 patches)

2016-09-26 Thread Kees Cook
On Mon, Sep 26, 2016 at 11:40 AM, Shuah Khan wrote: > This patch series contains the last 2 patches to complete moving runnable > code from Documentation to selftests, samples, and tools. > > The first patch moves blackfin gptimers-example to samples and removes > CONFIG_BUILD_DOCSRC. > > The seco

Re: hostap: Delete an unnecessary jump label in prism2_ioctl_priv_hostapd()

2016-09-26 Thread Joe Perches
On Mon, 2016-09-26 at 21:37 +0300, Kalle Valo wrote: > I talked with Jouni and we concluded marking this fully obsolete is the > best (so removing the "Maintained" part completelo the shmoo list > is not used anymore, that can be removed. Well, it would be best if Jouni submitted something.

Re: hostap: Delete an unnecessary jump label in prism2_ioctl_priv_hostapd()

2016-09-26 Thread Kalle Valo
Joe Perches writes: > On Mon, 2016-09-26 at 21:01 +0300, Kalle Valo wrote: >> hostap is an obsolete driver, it's waste of time doing style fixes to it >> as nobody maintains it anymore. > > Dunno know if Jouni is still maintaining this at all > but maybe a MAINTAINERS update to mark it obsolete s

Re: [PATCH 1/3] dt-bindings: Add a binding for the RPi firmware GPIO driver.

2016-09-26 Thread Stefan Wahren
> Stephen Warren hat am 26. September 2016 um 18:38 > geschrieben: > > > On 09/23/2016 12:39 PM, Stefan Wahren wrote: > > Hi Eric, > > > >> Eric Anholt hat am 19. September 2016 um 18:13 > >> geschrieben: > >> > >> > >> The RPi firmware exposes all of the board's GPIO lines through > >> proper

[PATCH 1/2] samples: move blackfin gptimers-example from Documentation

2016-09-26 Thread Shuah Khan
Move blackfin gptimers-example to samples and remove it from Documentation Makefile. Update samples Kconfig and Makefile to build gptimers-example. blackfin is the last CONFIG_BUILD_DOCSRC target in Documentation/Makefile, hence this patch also includes changes to remove CONFIG_BUILD_DOCSRC from M

[PATCH 2/2] Doc: update 00-INDEX files to reflect the runnable code move

2016-09-26 Thread Shuah Khan
Update 00-INDEX files with the current file list to reflect the runnable code move. Signed-off-by: Shuah Khan --- Documentation/00-INDEX | 2 -- Documentation/arm/00-INDEX | 2 -- Documentation/filesystems/00-INDEX | 2 -- Documentation/networking/00-INDEX | 2 -- Documentat

Re: ISDN-Gigaset: Enclose two expressions for the sizeof operator by parentheses

2016-09-26 Thread Paul Bolle
Where did the [PATCH 5/5] part of the subject go? You didn't drop it, did you? Because that's surprisingly annoying. Paul Bolle

Re: [PATCH 1/2] f2fs: fix to commit bio cache after flushing node pages

2016-09-26 Thread Jaegeuk Kim
Hi Chao, On Tue, Sep 27, 2016 at 12:09:52AM +0800, Chao Yu wrote: > From: Chao Yu > > In sync_node_pages, we won't check and commit last merged pages in private > bio cache of f2fs, as these pages were taged as writeback, someone who is > waiting for writebacking of the page will be blocked unti

Re: [PATHC v2 0/9] ima: carry the measurement list across kexec

2016-09-26 Thread Thiago Jung Bauermann
Hello Eric, Am Dienstag, 20 September 2016, 11:07:29 schrieb Eric W. Biederman: > Thiago Jung Bauermann writes: > > Am Samstag, 17 September 2016, 00:17:37 schrieb Eric W. Biederman: > >> Thiago Jung Bauermann writes: > > Is this what you had in mind? > > Sort of. > > I was just thinking that

Re: [PATCH v2 5/5] ISDN-Gigaset: Enclose two expressions for the sizeof operator by parentheses

2016-09-26 Thread Paul Bolle
On Mon, 2016-09-26 at 19:44 +0200, SF Markus Elfring wrote: > v2: Position the desired closing parenthesis behind the variable name for > a character buffer. Please wait a reasonable amount of time (say a week or two) to collect all feedback on all patches of a series. If there's feedback you

Re: [lustre-devel] [PATCH] staging/lustre/obdclass: add missing header dependencies

2016-09-26 Thread James Simmons
> We get 1 warning when building kernel with W=1: > drivers/staging/lustre/lustre/obdclass/linux/linux-sysctl.c:157:5: warning: > no previous prototype for 'obd_sysctl_init' [-Wmissing-prototypes] > > In fact, this function is declared in ../../include/obd_class.h, > so this patch add missing he

Re: [LKP] [lkp] [f2fs] ec795418c4: fsmark.files_per_sec -36.3% regression

2016-09-26 Thread Jaegeuk Kim
On Mon, Sep 26, 2016 at 02:26:06PM +0800, Huang, Ying wrote: > Hi, Jaegeuk, > > "Huang, Ying" writes: > > > Jaegeuk Kim writes: > > > >> Hello, > >> > >> On Sat, Aug 27, 2016 at 10:13:34AM +0800, Fengguang Wu wrote: > >>> Hi Jaegeuk, > >>> > >>> > > >> > - [lkp] [f2fs] b93f771286: aim7.jobs-pe

Re: [lustre-devel] [PATCH] staging/lustre/lov: add missing header dependencies

2016-09-26 Thread James Simmons
> We get 1 warning when building kernel with W=1: > drivers/staging/lustre/lustre/lov/lov_object.c:956:23: warning: no previous > prototype for 'lov_lsm_get' [-Wmissing-prototypes] > drivers/staging/lustre/lustre/lov/lov_object.c:972:6: warning: no previous > prototype for 'lov_lsm_put' [-Wmissi

Re: hostap: Delete an unnecessary jump label in prism2_ioctl_priv_hostapd()

2016-09-26 Thread Joe Perches
On Mon, 2016-09-26 at 21:01 +0300, Kalle Valo wrote: > hostap is an obsolete driver, it's waste of time doing style fixes to it > as nobody maintains it anymore. Dunno know if Jouni is still maintaining this at all but maybe a MAINTAINERS update to mark it obsolete so checkpatch warns on unnecessa

[PATCH trival 1/2] bpf: clean up put_cpu_var usage

2016-09-26 Thread Shaohua Li
put_cpu_var takes the percpu data, not the data returned from get_cpu_var. This doesn't change the behavior. Cc: Tejun Heo Cc: Alexei Starovoitov Signed-off-by: Shaohua Li --- kernel/bpf/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/bpf/core.c b/kernel/bpf/

[PATCH trival 2/2] lib: clean up put_cpu_var usage

2016-09-26 Thread Shaohua Li
put_cpu_var takes the percpu data, not the data returned from get_cpu_var. This doesn't change the behavior. Cc: Tejun Heo Signed-off-by: Shaohua Li --- lib/random32.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/random32.c b/lib/random32.c index 69ed593..915982b

Re: [PATCH V2] staging: rtl8172: usb_intf.c: Fixed block comment coding style

2016-09-26 Thread Larry Finger
On 09/26/2016 06:20 AM, Gonçalo Salazar wrote: 2016-09-26 1:53 GMT+01:00 Larry Finger : On 09/25/2016 06:00 PM, Gonçalo Salazar wrote: Fixed a block comment indentation in the rtl8712 usb_intf.c file. Made this as a first commit. Resubmitted with updated subject. Please let me know of any fee

[PATCH] x86/cmpxchg: Remove superfluous definitions

2016-09-26 Thread Nikolay Borisov
cmpxchg contained definitions for unused (x)add_* operations, dating back to the original ticket spinlock implementation. Nowadays these are unused so remove them. Signed-off-by: Nikolay Borisov --- arch/x86/include/asm/cmpxchg.h | 44 -- 1 file changed, 4

Re: [PATCH v2 net-next 3/3] net: bcmgenet: use new api ethtool_{get|set}_link_ksettings

2016-09-26 Thread Florian Fainelli
On 09/25/2016 09:29 AM, Philippe Reynes wrote: > The ethtool api {get|set}_settings is deprecated. > We move this driver to new api {get|set}_link_ksettings. > > Signed-off-by: Philippe Reynes > --- > drivers/net/ethernet/broadcom/genet/bcmgenet.c | 24 > ++-- > 1 files ch

Re: hostap: Delete an unnecessary jump label in prism2_ioctl_priv_hostapd()

2016-09-26 Thread SF Markus Elfring
> hostap is an obsolete driver, it's waste of time doing style fixes to it > as nobody maintains it anymore. Thanks for another bit of your software development attention and this information. Is it easier to understand than the previous response "Reason: The benefit is not clear."? Regards, M

Re: [PATCH] staging: vt6656: Add spaces around '+' operator

2016-09-26 Thread Joe Perches
On Mon, 2016-09-26 at 14:48 -0300, Martin Alonso wrote: > Fix coding style issue "spaces preferred around '+' > detected by checkpatch.pl in dpc.c file. [] > diff --git a/drivers/staging/vt6656/dpc.c b/drivers/staging/vt6656/dpc.c [] > @@ -75,8 +75,8 @@ int vnt_rx_data(struct vnt_private *priv, str

Re: hostap: Delete an unnecessary jump label in prism2_ioctl_priv_hostapd()

2016-09-26 Thread Kalle Valo
SF Markus Elfring writes: >> 9291771 [2/3] hostap: Delete an unnecessary jump label in >> prism2_ioctl_priv_hostapd() >> 9291775 [3/3] hostap: Delete unnecessary initialisations for the variable >> "ret" >> >> Reason: The benefit is not clear. > > How do you think about to reduce the source co

Re: ISDN-Gigaset: Enclose two expressions for the sizeof operator by parentheses

2016-09-26 Thread SF Markus Elfring
>> @@ -53,7 +53,7 @@ void gigaset_dbg_buffer(enum debuglevel level, const >> unsigned char *msg, >> { >> unsigned char outbuf[80]; >> unsigned char c; >> -size_t space = sizeof outbuf - 1; >> +size_t space = sizeof(outbuf - 1); > >What?! Does that compile? Yes, of course.

Re: [PATCH v3] bpf: Set register type according to is_valid_access()

2016-09-26 Thread Alexei Starovoitov
On Mon, Sep 26, 2016 at 04:49:17PM +0200, Daniel Borkmann wrote: > On 09/24/2016 08:01 PM, Mickaël Salaün wrote: > >This prevent future potential pointer leaks when an unprivileged eBPF > >program will read a pointer value from its context. Even if > >is_valid_access() returns a pointer type, the e

[PATCH v5 02/10] ima: on soft reboot, restore the measurement list

2016-09-26 Thread Mimi Zohar
The TPM PCRs are only reset on a hard reboot. In order to validate a TPM's quote after a soft reboot (eg. kexec -e), the IMA measurement list of the running kernel must be saved and restored on boot. This patch restores the measurement list. Changelog v5: - replace CONFIG_KEXEC_FILE with archite

[PATCH v5 03/10] ima: permit duplicate measurement list entries

2016-09-26 Thread Mimi Zohar
Measurements carried across kexec need to be added to the IMA measurement list, but should not prevent measurements of the newly booted kernel from being added to the measurement list. This patch adds support for allowing duplicate measurements. The "boot_aggregate" measurement entry is the delimi

Re: [x86-tip] strange nr_cpus= boot regression

2016-09-26 Thread Thomas Gleixner
Mike, On Mon, 26 Sep 2016, Thomas Gleixner wrote: > On Mon, 26 Sep 2016, Mike Galbraith wrote: > > > I've encountered a strange regression in tip, symptom is that if you > > boot with nr_cpus=nr_you_have, what actually boots is nr_you_have/2. > > Do not pass nr_cpus=, and all is well. > > Wha

Re: [bug] crypto/vmx/p8_ghash memory corruption in 4.8-rc7

2016-09-26 Thread Jan Stancek
- Original Message - > From: "Marcelo Cerri" > To: "Jan Stancek" > Cc: "rui y wang" , herb...@gondor.apana.org.au, > mhce...@linux.vnet.ibm.com, > leosi...@linux.vnet.ibm.com, pfsmor...@linux.vnet.ibm.com, > linux-cry...@vger.kernel.org, > linuxppc-...@lists.ozlabs.org, linux-kernel@

Re: brcmfmac: drop unused fields from struct brcmf_pub

2016-09-26 Thread Kalle Valo
Rafał Miłecki wrote: > From: Rafał Miłecki > > They seem to be there from the first day. We calculate these values but > never use them. > > Signed-off-by: Rafał Miłecki Patch applied to wireless-drivers-next.git, thanks. 2df86ad959c9 brcmfmac: drop unused fields from struct brcmf_pub -- ht

Re: brcmfmac: fix memory leak in brcmf_fill_bss_param

2016-09-26 Thread Kalle Valo
Rafał Miłecki wrote: > From: Rafał Miłecki > > This function is called from get_station callback which means that every > time user space was getting/dumping station(s) we were leaking 2 KiB. > > Signed-off-by: Rafał Miłecki > Fixes: 1f0dc59a6de ("brcmfmac: rework .get_station() callback") > Cc

[PATCH] staging: vt6656: Add spaces around '+' operator

2016-09-26 Thread Martin Alonso
Fix coding style issue "spaces preferred around '+' detected by checkpatch.pl in dpc.c file. Signed-off-by: Martin Alonso --- drivers/staging/vt6656/dpc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/vt6656/dpc.c b/drivers/staging/vt6656/dpc.c index 601

Re: [PATCH v2] raid6/test/test.c: bug fix: Specify aligned(alignment) attributes to the char arrays

2016-09-26 Thread Shaohua Li
On Thu, Sep 22, 2016 at 05:07:55PM -0700, Gayatri Kammela wrote: > Specifying the aligned attributes to the char recovi[PAGE_SIZE] > and char recovi[PAGE_SIZE] arrays, so that all malloc memory is page > boundary aligned. > > Without these alignment attributes, the test causes a segfault in > user

[PATCH v5 04/10] ima: maintain memory size needed for serializing the measurement list

2016-09-26 Thread Mimi Zohar
In preparation for serializing the binary_runtime_measurements, this patch maintains the amount of memory required. Changelog v5: - replace CONFIG_KEXEC_FILE with architecture CONFIG_HAVE_IMA_KEXEC (Thiago) Changelog v3: - include the ima_kexec_hdr size in the binary_runtime_measurement size. Si

[PATCH v2 5/5] ISDN-Gigaset: Enclose two expressions for the sizeof operator by parentheses

2016-09-26 Thread SF Markus Elfring
From: Markus Elfring Date: Mon, 26 Sep 2016 19:34:27 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script "checkpatch.pl" can point information out like the following. WARNING: sizeof … should be sizeof(…) Thus fix the affected source code p

Re: [bug] crypto/vmx/p8_ghash memory corruption in 4.8-rc7

2016-09-26 Thread Marcelo Cerri
Herbert, Wouldn't be enough to provide a pair of import/export functions as the padlock-sha driver does? -- Regards, Marcelo On Mon, Sep 26, 2016 at 10:59:34PM +0800, Herbert Xu wrote: > On Fri, Sep 23, 2016 at 08:22:27PM -0400, Jan Stancek wrote: > > > > This seems to directly correspond with:

Re: [PATCH v4 4/4] regulator: Prevent falling too fast

2016-09-26 Thread Doug Anderson
Hi, On Sat, Sep 24, 2016 at 11:41 AM, Mark Brown wrote: > On Mon, Sep 19, 2016 at 11:39:24AM -0700, Doug Anderson wrote: >> On Fri, Sep 16, 2016 at 9:32 AM, Mark Brown wrote: > >> > So the PWM is just configuring this external regulator chip (which >> > doesn't seem to be described in DT...) and

[PATCH] Staging: dgnc: Align the * on a block comment

2016-09-26 Thread Ramiro Oliveira
This patch corrects a misalligned * in a block comment warning found by checkpatch.pl Signed-off-by: Ramiro Oliveira --- drivers/staging/dgnc/dgnc_cls.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/dgnc/dgnc_cls.c b/drivers/staging/dgnc/dgnc_cls.c index 4e

Re: [x86-tip] strange nr_cpus= boot regression

2016-09-26 Thread Thomas Gleixner
CC'ed: Dou Liyang On Mon, 26 Sep 2016, Mike Galbraith wrote: > I've encountered a strange regression in tip, symptom is that if you > boot with nr_cpus=nr_you_have, what actually boots is nr_you_have/2. > Do not pass nr_cpus=, and all is well. What's the number of possible cpus in your system

Re: [PATCH 5/5] ISDN-Gigaset: Enclose two expressions for the sizeof operator by parentheses

2016-09-26 Thread Sergei Shtylyov
Hello. On 09/26/2016 06:44 PM, SF Markus Elfring wrote: From: Markus Elfring Date: Mon, 26 Sep 2016 17:03:56 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script "checkpatch.pl" can point information out like the following. WARNING: sizeo

Re: [PATCH v3] dmaengine: s3c24xx: Add dma_slave_map for s3c2440 devices

2016-09-26 Thread Vinod Koul
On Thu, Sep 22, 2016 at 08:51:15PM +0200, Sam Van Den Berge wrote: > This patch updates the s3c24xx dma driver to be able to pass a > dma_slave_map array via the platform data. This is needed to > be able to use the new, simpler dmaengine API [1]. > I used the virtual DMA channels as a parameter fo

[PATCH v5 07/10] ima: store the builtin/custom template definitions in a list

2016-09-26 Thread Mimi Zohar
The builtin and single custom templates are currently stored in an array. In preparation for being able to restore a measurement list containing multiple builtin/custom templates, this patch stores the builtin and custom templates as a linked list. This will permit defining more than one custom t

[PATCH v5 08/10] ima: support restoring multiple template formats

2016-09-26 Thread Mimi Zohar
The configured IMA measurement list template format can be replaced at runtime on the boot command line, including a custom template format. This patch adds support for restoring a measuremement list containing multiple builtin/custom template formats. Signed-off-by: Mimi Zohar --- security/inte

[PATCH v5 09/10] ima: define a canonical binary_runtime_measurements list format

2016-09-26 Thread Mimi Zohar
The IMA binary_runtime_measurements list is currently in platform native format. To allow restoring a measurement list carried across kexec with a different endianness than the targeted kernel, this patch defines little-endian as the canonical format. For big endian systems wanting to save/restor

[PATCH v5 06/10] ima: on soft reboot, save the measurement list

2016-09-26 Thread Mimi Zohar
The TPM PCRs are only reset on a hard reboot. In order to validate a TPM's quote after a soft reboot (eg. kexec -e), the IMA measurement list of the running kernel must be saved and restored on boot. This patch uses the kexec buffer passing mechanism to pass the serialized IMA binary_runtime_meas

Re: [PATCH] arch/arm: enable task isolation functionality

2016-09-26 Thread Chris Metcalf
On 9/22/2016 5:54 PM, Francis Giraldeau wrote: This patch is a port of the task isolation functionality to the arm 32-bit architecture. The task isolation needs an additional thread flag that requires to change the entry assembly code to accept a bitfield larger than one byte. The constants _TIF

[PATCH v5 05/10] powerpc: ima: Send the kexec buffer to the next kernel

2016-09-26 Thread Mimi Zohar
From: Thiago Jung Bauermann The IMA kexec buffer allows the currently running kernel to pass the measurement list via a kexec segment to the kernel that will be kexec'd. This is the architecture-specific part of setting up the IMA kexec buffer for the next kernel. It will be used in the next pat

[PATCH v5 01/10] powerpc: ima: Get the kexec buffer passed by the previous kernel

2016-09-26 Thread Mimi Zohar
From: Thiago Jung Bauermann The IMA kexec buffer allows the currently running kernel to pass the measurement list via a kexec segment to the kernel that will be kexec'd. The second kernel can check whether the previous kernel sent the buffer and retrieve it. This is the architecture-specific par

Re: [PATCH 00/24] ste_dma40: Fine-tuning for several function implementations

2016-09-26 Thread Vinod Koul
On Sat, Sep 17, 2016 at 05:05:16PM +0200, SF Markus Elfring wrote: > From: Markus Elfring > Date: Sat, 17 Sep 2016 16:56:56 +0200 > > Several update suggestions were taken into account > from static source code analysis. Applied all, thanks -- ~Vinod

[PATCH v5 00/10] ima: carry the measurement list across kexec

2016-09-26 Thread Mimi Zohar
The TPM PCRs are only reset on a hard reboot. In order to validate a TPM's quote after a soft reboot (eg. kexec -e), the IMA measurement list of the running kernel must be saved and then restored on the subsequent boot, possibly of a different architecture. The existing securityfs binary_runtime_

[PATCH v5 10/10] ima: platform-independent hash value

2016-09-26 Thread Mimi Zohar
From: Andreas Steffen For remote attestion it is important for the ima measurement values to be platform-independent. Therefore integer fields to be hashed must be converted to canonical format. Changelog: - Define canonical format as little endian (Mimi) Signed-off-by: Andreas Steffen Signed-

[PATCH v4 0/3] mm/hugetlb: memory offline issues with hugepages

2016-09-26 Thread Gerald Schaefer
This addresses several issues with hugepages and memory offline. While the first patch fixes a panic, and is therefore rather important, the last patch is just a performance optimization. The second patch fixes a theoretical issue with reserved hugepages, while still leaving some ugly usability is

[PATCH v4 3/3] mm/hugetlb: improve locking in dissolve_free_huge_pages()

2016-09-26 Thread Gerald Schaefer
For every pfn aligned to minimum_order, dissolve_free_huge_pages() will call dissolve_free_huge_page() which takes the hugetlb spinlock, even if the page is not huge at all or a hugepage that is in-use. Improve this by doing the PageHuge() and page_count() checks already in dissolve_free_huge_page

[PATCH v4 2/3] mm/hugetlb: check for reserved hugepages during memory offline

2016-09-26 Thread Gerald Schaefer
In dissolve_free_huge_pages(), free hugepages will be dissolved without making sure that there are enough of them left to satisfy hugepage reservations. Fix this by adding a return value to dissolve_free_huge_pages() and checking h->free_huge_pages vs. h->resv_huge_pages. Note that this may lead t

[PATCH v4 1/3] mm/hugetlb: fix memory offline with hugepage size > memory block size

2016-09-26 Thread Gerald Schaefer
dissolve_free_huge_pages() will either run into the VM_BUG_ON() or a list corruption and addressing exception when trying to set a memory block offline that is part (but not the first part) of a "gigantic" hugetlb page with a size > memory block size. When no other smaller hugetlb page sizes are p

cache flushing or cache line invalidation on page mapping changes

2016-09-26 Thread HONGIL YOON
Hello, I have read kernel codes to understand details of kernel operations when virtual to physical page mappings are changed or removed, especially for TLB and cache (line) invalidations (or flushes). Basically, my understanding is that dirty data from a page which is unmapped should be written

Re: Removal of regulator-boot-on/always-on when a consumer exists

2016-09-26 Thread Fabio Estevam
On Mon, Sep 26, 2016 at 1:57 PM, Mark Brown wrote: > Like I say I can't give you a definitive answer on this without looking > at the actual hardware and what it needs. Based on what you're saying > it sounds like it won't be an issue from the point of view of disrupting > other users but withou

Re: Warning from free_init_pages with large initrd

2016-09-26 Thread Sitsofe Wheeler
On 26 September 2016 at 07:47, H. Peter Anvin wrote: > On September 25, 2016 11:22:04 PM PDT, Sitsofe Wheeler > wrote: >>On 26 September 2016 at 03:14, H. Peter Anvin wrote: >>> On 09/24/16 08:32, Sitsofe Wheeler wrote: While trying to PXE boot a Fedora LiveISO on VMware ESXi the kern

Re: [PATCH] btrfs: fix btrfs_no_printk stub helper

2016-09-26 Thread David Sterba
On Fri, Sep 23, 2016 at 06:05:21PM +0200, Arnd Bergmann wrote: > The addition of btrfs_no_printk() caused a build failure when > CONFIG_PRINTK is disabled: > > fs/btrfs/send.c: In function 'send_rename': > fs/btrfs/ctree.h:3367:2: error: implicit declaration of function > 'btrfs_no_printk' [-Werr

[PATCH V2] staging: ks7010: Fix warnings on printk() usage

2016-09-26 Thread Sabitha George
This patch fixes the following warnings on ks7010_sdio.c 1. printk() should include KERN_ facility level 2. Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev, ... then pr_err(... to printk(KERN_ERR ... Signed-off-by: Sabitha George --- drivers/staging/ks7010/ks7010_sdio.c |

Re: [PATCH] ARM: dts: exynos: Add reboot reason support for Trats2

2016-09-26 Thread Krzysztof Kozlowski
On Sun, Sep 25, 2016 at 09:34:45AM +0200, Krzysztof Kozlowski wrote: > On Sat, Sep 24, 2016 at 11:04:49PM +0200, Wolfgang Wiedmeyer wrote: > > > > Krzysztof Kozlowski writes: > > > > > On Thu, Sep 22, 2016 at 08:59:03PM +0200, Wolfgang Wiedmeyer wrote: > > >> > > >> Krzysztof Kozlowski writes: >

Re: [PATCH] staging: skein: Remove extra blank lines

2016-09-26 Thread Martin Alonso
Ok Joe, Thanks and sorry for the annoyance. On Mon, Sep 26, 2016 at 09:26:12AM -0700, Joe Perches wrote: > On Mon, 2016-09-26 at 11:56 -0300, Martin wrote: > > Fix coding style issue "multiple blank lines" > > detected by checkpatch.pl in threefish_block.c file. > > This has been submitted an

Re: [PATCH 1/2] clk: rockchip: add I2S internal clock IDs for rk3288

2016-09-26 Thread John Keeping
On Sun, 25 Sep 2016 23:50:29 +0200, Heiko Stuebner wrote: > Am Mittwoch, 7. September 2016, 19:18:23 CEST schrieb John Keeping: > > On Wed, 07 Sep 2016 19:58:31 +0200, Heiko Stuebner wrote: > > > Am Mittwoch, 7. September 2016, 17:53:29 CEST schrieb John Keeping: > > > > To minimize jitter on

Re: [PATCH -v2 7/9] sched,tracing: Update trace_sched_pi_setprio()

2016-09-26 Thread Steven Rostedt
On Mon, 26 Sep 2016 14:32:20 +0200 Peter Zijlstra wrote: > Pass the PI donor task, instead of a numerical priority. > > Numerical priorities are not sufficient to describe state ever since > SCHED_DEADLINE. > > Annotate all sched tracepoints that are currently broken; fixing them > will bork us

Re: [PATCH -v2 6/9] sched/rtmutex: Refactor rt_mutex_setprio()

2016-09-26 Thread Steven Rostedt
On Mon, 26 Sep 2016 14:32:19 +0200 Peter Zijlstra wrote: > @@ -3586,9 +3625,7 @@ void rt_mutex_setprio(struct task_struct > goto out_unlock; > } > > - rt_mutex_update_top_task(p); > - > - trace_sched_pi_setprio(p, prio); > + trace_sched_pi_setprio(p, prio); /* br

[PATCH] staging: greybus/timesync: avoid divide by zero on X86 Qemu

2016-09-26 Thread Bryan O'Donoghue
A system configured without CONFIG_CPUFREQ will return 0 for cpufreq_get(). greybus-timesync can subsequently then do a divide-by-zero as result. This patch fixes by checking for a zero return value from cpufreq_get() and setting to a default value of 19.2MHz. Reported-by: Rui Miguel Silva Signed

Re: Removal of regulator-boot-on/always-on when a consumer exists

2016-09-26 Thread Mark Brown
On Mon, Sep 26, 2016 at 01:50:37PM -0300, Fabio Estevam wrote: > So regulator-boot-on and regulator-always-on can go away then? Like I say I can't give you a definitive answer on this without looking at the actual hardware and what it needs. Based on what you're saying it sounds like it won't b

Re: [PATCH v21 00/20] perf, tools: Add support for PMU events in JSON format

2016-09-26 Thread Andi Kleen
On Mon, Sep 26, 2016 at 12:03:43PM -0300, Arnaldo Carvalho de Melo wrote: > Em Mon, Sep 26, 2016 at 10:35:33AM +0200, Jiri Olsa escreveu: > > ping.. is that working for you? IMO we can include this > > as additional patch to the set.. > > No, it doesn't fails to build on the first cross env I trie

[PATCH V2 2/2] fs/super.c: don't fool lockdep in freeze_super() and thaw_super() paths

2016-09-26 Thread Oleg Nesterov
On 09/26, Jan Kara wrote: > > On Mon 26-09-16 18:08:06, Oleg Nesterov wrote: > > +/* > > + * Tell lockdep we are holding these locks before we call > > ->unfreeze_fs(sb). > > + */ > > +static void sb_freeze_acquire(struct super_block *sb) > > Can we call this lockdep_sb_freeze_acquire() or somethi

Re: [PATCH] net: tg3: use new api ethtool_{get|set}_link_ksettings

2016-09-26 Thread Michael Chan
On Sun, Sep 25, 2016 at 2:31 PM, Philippe Reynes wrote: > The ethtool api {get|set}_settings is deprecated. > We move this driver to new api {get|set}_link_ksettings. > > Signed-off-by: Philippe Reynes Looks good to me. Thanks. Acked-by: Michael Chan

Re: [Update][RFC/RFT][PATCH v3 2/5] driver core: Functional dependencies tracking support

2016-09-26 Thread Lukas Wunner
On Fri, Sep 23, 2016 at 03:42:31PM +0200, Rafael J. Wysocki wrote: > On Tuesday, September 20, 2016 12:46:30 AM Lukas Wunner wrote: > > On Fri, Sep 16, 2016 at 02:33:55PM +0200, Rafael J. Wysocki wrote: > > > +void device_links_no_driver(struct device *dev) > > > +{ > > > + struct device_link *link

Re: Removal of regulator-boot-on/always-on when a consumer exists

2016-09-26 Thread Michael Trimarchi
Hi On Mon, Sep 26, 2016 at 6:50 PM, Fabio Estevam wrote: > On Mon, Sep 26, 2016 at 1:49 PM, Mark Brown wrote: > >>> The can1 node is the only consumer of this regulator. >> >>> The reg_3p3v regulator models a discrete 3.3V power supply with no >>> software intervention. >> >> So if it's electric

Re: [PATCH 5/5] ISDN-Gigaset: Enclose two expressions for the sizeof operator by parentheses

2016-09-26 Thread Julia Lawall
On Mon, 26 Sep 2016, SF Markus Elfring wrote: > >>> @@ -53,7 +53,7 @@ void gigaset_dbg_buffer(enum debuglevel level, const > >>> unsigned char *msg, > >>> { > >>> unsigned char outbuf[80]; > >>> unsigned char c; > >>> - size_t space = sizeof outbuf - 1; > >>> + size_t space = sizeof(outbuf

Re: Removal of regulator-boot-on/always-on when a consumer exists

2016-09-26 Thread Mark Brown
On Mon, Sep 26, 2016 at 01:43:28PM -0300, Fabio Estevam wrote: > On Mon, Sep 26, 2016 at 1:34 PM, Mark Brown wrote: > > It really depends on what the actual board constraints are - are there > > other consumers that aren't visible to software, is it electrically safe > > to power down the rail?

Re: Removal of regulator-boot-on/always-on when a consumer exists

2016-09-26 Thread Fabio Estevam
On Mon, Sep 26, 2016 at 1:49 PM, Mark Brown wrote: >> The can1 node is the only consumer of this regulator. > >> The reg_3p3v regulator models a discrete 3.3V power supply with no >> software intervention. > > So if it's electrically safe it's electrically safe... So regulator-boot-on and regul

Re: new objtool warnings again...

2016-09-26 Thread Arnaldo Carvalho de Melo
Em Fri, Sep 23, 2016 at 02:19:04PM -0700, Linus Torvalds escreveu: > On Fri, Sep 23, 2016 at 2:16 PM, Josh Poimboeuf wrote: > > > > I just started seeing this problem today. I suspect it's a ccache > > issue, since it only showed up after ccache was updated. > > Ahh, I didn't even notice that cc

Re: [PATCH 2/3] gpio: Add a driver for the Raspberry Pi's firmware GPIO calls.

2016-09-26 Thread Stephen Warren
On 09/23/2016 07:15 AM, Eric Anholt wrote: Linus Walleij writes: On Mon, Sep 19, 2016 at 6:13 PM, Eric Anholt wrote: This driver will be used for accessing the FXL6408 GPIO exander on the Pi3. We can't drive it directly from Linux because the firmware is continuously polling one of the exp

Re: [PATCH 5/5] ISDN-Gigaset: Enclose two expressions for the sizeof operator by parentheses

2016-09-26 Thread SF Markus Elfring
>>> @@ -53,7 +53,7 @@ void gigaset_dbg_buffer(enum debuglevel level, const >>> unsigned char *msg, >>> { >>> unsigned char outbuf[80]; >>> unsigned char c; >>> - size_t space = sizeof outbuf - 1; >>> + size_t space = sizeof(outbuf - 1); > >> wrong ... >> think that is 7 instead of 79

Applied "ASoC: da7219: Disable AAD if codec is not a wake-up source" to the asoc tree

2016-09-26 Thread Mark Brown
The patch ASoC: da7219: Disable AAD if codec is not a wake-up source has been applied to the asoc tree at git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours)

Applied "ASoC: tlv320aic31xx: add explicit support for tlv320dac31xx" to the asoc tree

2016-09-26 Thread Mark Brown
The patch ASoC: tlv320aic31xx: add explicit support for tlv320dac31xx has been applied to the asoc tree at git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours)

Applied "regulator: dbx500: remove unused functions in dbx500-prcmu.c" to the regulator tree

2016-09-26 Thread Mark Brown
The patch regulator: dbx500: remove unused functions in dbx500-prcmu.c has been applied to the regulator tree at git://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git All being well this means that it will be integrated into the linux-next tree (usually sometime in the next

Applied "ASoC: da7219: Reset codec gracefully, if still active" to the asoc tree

2016-09-26 Thread Mark Brown
The patch ASoC: da7219: Reset codec gracefully, if still active has been applied to the asoc tree at git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and s

Applied "ASoC: sun4i-codec: Rename sun4i_codec_widgets for consistency" to the asoc tree

2016-09-26 Thread Mark Brown
The patch ASoC: sun4i-codec: Rename sun4i_codec_widgets for consistency has been applied to the asoc tree at git://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hour

Re: Removal of regulator-boot-on/always-on when a consumer exists

2016-09-26 Thread Fabio Estevam
Hi Mark, On Mon, Sep 26, 2016 at 1:34 PM, Mark Brown wrote: > It really depends on what the actual board constraints are - are there > other consumers that aren't visible to software, is it electrically safe > to power down the rail? The can1 node is the only consumer of this regulator. The re

Re: Removal of regulator-boot-on/always-on when a consumer exists

2016-09-26 Thread Mark Brown
On Mon, Sep 26, 2016 at 11:26:24AM -0300, Fabio Estevam wrote: > My suggestion is to remove regulator-boot-on/regulator-always-on: >reg_3p3v: regulator-3p3v { >compatible = "regulator-fixed"; >regulator-name = "3P3V"; >regulator-min-microvol

Re: [PATCH 1/3] dt-bindings: Add a binding for the RPi firmware GPIO driver.

2016-09-26 Thread Stephen Warren
On 09/23/2016 07:08 AM, Eric Anholt wrote: Linus Walleij writes: On Mon, Sep 19, 2016 at 6:13 PM, Eric Anholt wrote: The RPi firmware exposes all of the board's GPIO lines through property calls. Linux chooses to control most lines directly through the pinctrl driver, but for the FXL6408 G

Re: [PATCH 1/1 v8] ARM: imx: Added perf functionality to mmdc driver

2016-09-26 Thread Zhi Li
On Mon, Sep 19, 2016 at 12:57 PM, Frank Li wrote: > From: Zhengyu Shen > > MMDC is a multi-mode DDR controller that supports DDR3/DDR3L x16/x32/x64 > and LPDDR2 two channel x16/x32 memory types. MMDC is configurable, high > performance, and optimized. MMDC is present on i.MX6 Quad and i.MX6 > Qua

Re: [PATCH 1/3] dt-bindings: Add a binding for the RPi firmware GPIO driver.

2016-09-26 Thread Stephen Warren
On 09/23/2016 12:39 PM, Stefan Wahren wrote: Hi Eric, Eric Anholt hat am 19. September 2016 um 18:13 geschrieben: The RPi firmware exposes all of the board's GPIO lines through property calls. Linux chooses to control most lines directly through the pinctrl driver, but for the FXL6408 GPIO

Re: [linux-sunxi] [PATCH v6 0/3] ASoC: sun4i-codec: Distinguish sun4i from sun7i

2016-09-26 Thread Mark Brown
On Sun, Sep 25, 2016 at 10:08:27AM +0200, Danny Milosavljevic wrote: Please fix your mail client to word wrap within paragraphs at something substantially less than 80 columns. Doing this makes your messages much easier to read and reply to. > I see that the v5 patches 2 and 3 have already been

[tip:ras/core] x86/RAS/mce_amd_inj: Fix signed wrap around when decrementing index 'i'

2016-09-26 Thread tip-bot for Colin Ian King
Commit-ID: 8b44f00f8c952ab6eb658090383571b2ec7d253f Gitweb: http://git.kernel.org/tip/8b44f00f8c952ab6eb658090383571b2ec7d253f Author: Colin Ian King AuthorDate: Mon, 26 Sep 2016 10:31:51 +0200 Committer: Ingo Molnar CommitDate: Mon, 26 Sep 2016 11:13:17 +0200 x86/RAS/mce_amd_inj: Fix

[tip:ras/core] x86/RAS/mce_amd_inj: Remove debugfs dir recursively on exit

2016-09-26 Thread tip-bot for Borislav Petkov
Commit-ID: b199ac6c4943aa0db246163bf6b483e2bb53431b Gitweb: http://git.kernel.org/tip/b199ac6c4943aa0db246163bf6b483e2bb53431b Author: Borislav Petkov AuthorDate: Mon, 26 Sep 2016 10:31:52 +0200 Committer: Ingo Molnar CommitDate: Mon, 26 Sep 2016 11:13:17 +0200 x86/RAS/mce_amd_inj: Rem

Re: [PATCH 2/2] power: supply: max17042_battery: add technology property support

2016-09-26 Thread Krzysztof Kozlowski
On Mon, Sep 26, 2016 at 02:56:44PM +0200, Wolfgang Wiedmeyer wrote: > > Krzysztof Kozlowski writes: > > > On Sun, Sep 25, 2016 at 11:10:11PM +0200, Wolfgang Wiedmeyer wrote: > >> This patch reports the battery technology as Li-ion. > >> > >> Signed-off-by: Wolfgang Wiedmeyer > >> --- > >> driv

Re: [PATCH 5/5] ISDN-Gigaset: Enclose two expressions for the sizeof operator by parentheses

2016-09-26 Thread Joe Perches
On Mon, 2016-09-26 at 16:00 +, David Laight wrote: > From: SF Markus Elfring Sent: 26 September 2016 16:45 > > The script "checkpatch.pl" can point information out like the following. > > WARNING: sizeof … should be sizeof(…) > [] > > diff --git a/drivers/isdn/gigaset/common.c b/drivers/isdn/gi

Re: [PATCH v2] signals: Avoid unnecessary taking of sighand->siglock

2016-09-26 Thread Oleg Nesterov
On 09/23, Waiman Long wrote: > > > + /* > + * In case the signal mask hasn't changed, we won't need to take > + * the lock. The current blocked mask can be modified by other CPUs. > + * To be safe, we need to do an atomic read without lock. As a result, > + * this check wi

Re: [PATCH] staging: skein: Remove extra blank lines

2016-09-26 Thread Joe Perches
On Mon, 2016-09-26 at 11:56 -0300, Martin wrote: > Fix coding style issue "multiple blank lines" > detected by checkpatch.pl in threefish_block.c file. This has been submitted and rejected a couple times already. The extra blank lines are to separate blocks. Whatever the blocks are, perhaps _addi

Re: [RFC PATCH 6/6] perf: Disable IRQs in address filter sync path

2016-09-26 Thread Alexander Shishkin
Alexander Shishkin writes: > If PMU callbacks are executed in hardirq context, the address filter > sync code needs to disable interrupts when taking its spinlock to be > consistent with the rest of its users. This may happen if the PMU is > used in AUX sampling. Hi Mathieu, I've been meaning t

<    1   2   3   4   5   6   7   >