[PATCH v5 1/1] security: Add mechanism to safely (un)load LSMs after boot time

2018-04-07 Thread Sargun Dhillon
This patch introduces a mechanism to add mutable hooks and immutable hooks to the callback chain. It adds an intermediary item to the chain which separates mutable and immutable hooks. Immutable hooks are then marked as read-only, as well as the hook heads. This does not preclude some hooks being a

[PATCH v5 0/1] Safe LSM (un)loading, and immutable hooks

2018-04-07 Thread Sargun Dhillon
The primary security benefit of this patchset is the introduction of read-only hooks, even if some security modules have mutable hooks. Currently, if you have any LSMs with mutable hooks it will render all heads, and list nodes mutable. These are a prime place to attack because being able to manipu

Re: [PATCH v3 2/4] mm/sparsemem: Defer the ms->section_mem_map clearing

2018-04-07 Thread Baoquan He
Hi Dave, Thanks a lot for your careful reviewing! On 04/06/18 at 07:23am, Dave Hansen wrote: > On 02/27/2018 07:26 PM, Baoquan He wrote: > > In sparse_init(), if CONFIG_SPARSEMEM_ALLOC_MEM_MAP_TOGETHER=y, system > > will allocate one continuous memory chunk for mem maps on one node and > > popula

[PATCH] x86/boot/e820: add new chareater "-" to free BIOS memory in memmap bootargs

2018-04-07 Thread zoucao
From: zoucao Normally every BIOS reserved memory is used for some features, we can not use them, but in some conditions, users can ensure some BIOS memorys are not used and reserved memory is well to free, they have not a good way to free these memroys, here add a new chareater "-" in memmap to

[PATCH] free e820 reserved memory in bootargs memmap

2018-04-07 Thread zoucao
add a new chareater "-" in bootargs memmap, you can use this chareater to free e820 reserved memory, for example: reserved iomem: cat /proc/iomem: .. 0010-6aefefff : System RAM 0100-0165537a : Kernel code 0165537b-01a8873f : Kernel data

Re: [PATCH 2/4] arm64: usercopy: implement arch_within_stack_frames

2018-04-07 Thread Keun-O Park
Hi Kees, On Thu, Apr 5, 2018 at 2:55 AM, Kees Cook wrote: > On Thu, Mar 1, 2018 at 2:19 AM, wrote: >> From: James Morse >> >> This implements arch_within_stack_frames() for arm64 that should >> validate if a given object is contained by a kernel stack frame. >> >> Signed-off-by: James Morse >

Running syzkaller repros using kvm-xfstests

2018-04-07 Thread Theodore Y. Ts'o
On Thu, Apr 05, 2018 at 09:37:41PM -0400, Theodore Y. Ts'o wrote: > Note that I haven't just been complaining about it. I've been working > on ways so that the gce-xfstests and kvm-xfstests test appliances can > more easily be used to work on Syzbot reports. If I can make myself > more efficient,

Re: [GIT PULL] SELinux patches for v4.17

2018-04-07 Thread Xin Long
On Sat, Apr 7, 2018 at 7:07 AM, Linus Torvalds wrote: > On Tue, Apr 3, 2018 at 6:37 PM, Paul Moore wrote: >> >> Everything passes the selinux-testsuite, but there are a few known >> merge conflicts. The first is with the netdev tree and is in >> net/sctp/socket.c. Unfortunately it is a bit ugly

Re: WARNING in kmem_cache_free

2018-04-07 Thread Matthew Wilcox
On Fri, Apr 06, 2018 at 03:33:36PM +0200, Dmitry Vyukov wrote: > On Fri, Apr 6, 2018 at 3:24 PM, syzbot > wrote: > > Unfortunately, I don't have any reproducer for this crash yet. > > Interesting type of bug, I think we see this for the first time. Can you focus syzbot to try to find a reproduce

Re: WARNING in kmem_cache_free

2018-04-07 Thread Al Viro
On Fri, Apr 06, 2018 at 03:33:36PM +0200, Dmitry Vyukov wrote: > Interesting type of bug, I think we see this for the first time. > > Al, do you see how this can happen? putname() on something that hasn't come from getname(). Short of reproducer, I don't see what can be done - it can be any kin

Re: [PATCH v2 1/3] resource: Use list_head to link sibling resource

2018-04-07 Thread kbuild test robot
Hi Baoquan, I love your patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v4.16 next-20180406] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Ba

Re: [PATCH v1] ringbuffer: Don't choose the process with adj equal OOM_SCORE_ADJ_MIN

2018-04-07 Thread Zhaoyang Huang
On Sun, Apr 8, 2018 at 11:48 AM, Steven Rostedt wrote: > On Sun, 8 Apr 2018 10:16:23 +0800 > Zhaoyang Huang wrote: > >> Don't choose the process with adj == OOM_SCORE_ADJ_MIN which >> over-allocating pages for ring buffers. > > Why? > > -- Steve because in oom_evaluate_task, the process with adj

[no subject]

2018-04-07 Thread Sorina Secosan
Benötigen Sie ein persönliches oder geschäftliches Darlehen mit 2% Zinssatz? Wir bieten Projektfinanzierung Business Loans Personal Loans. für weitere Informationen kontaktieren Sie uns bitte

Re: Use struct page for filename

2018-04-07 Thread Al Viro
On Sat, Apr 07, 2018 at 08:16:46PM -0700, Matthew Wilcox wrote: > On Fri, Apr 06, 2018 at 03:33:36PM +0200, Dmitry Vyukov wrote: > > On Fri, Apr 6, 2018 at 3:24 PM, syzbot > > wrote: > > > cache_from_obj: Wrong slab cache. names_cache but object is from > > > kmalloc-96 > > > > Al, do you see ho

[PATCH] ASoC: fsl_ssi: Fix mode setting when changing channel number

2018-04-07 Thread Nicolin Chen
This is a partial revert (in a cleaner way) of commit ebf08ae3bc90 ("ASoC: fsl_ssi: Keep ssi->i2s_net updated") to fix a regression at test cases when switching between mono and stereo audio. The problem is that ssi->i2s_net is initialized in set_dai_fmt() only, while this set_dai_fmt() is only ca

Re: __GFP_LOW

2018-04-07 Thread Matthew Wilcox
On Fri, Apr 06, 2018 at 08:09:53AM +0200, Michal Hocko wrote: > OK, we already split the documentation into these categories. So we got > at least the structure right ;) Yes, this part of the documentation makes sense to me :-) > > - What kind of memory to allocate (DMA, NORMAL, HIGHMEM) > > -

Re: [PATCH v2 1/3] resource: Use list_head to link sibling resource

2018-04-07 Thread kbuild test robot
Hi Baoquan, I love your patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v4.16 next-20180406] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Ba

RE: [PATCH 1/1] target:separate tx/rx cmd_puds

2018-04-07 Thread Zhang Zhuoyu
> -Original Message- > From: David Disseldorp [mailto:dd...@suse.de] > Sent: Friday, April 6, 2018 8:10 PM > To: Zhang Zhuoyu > Cc: n...@linux-iscsi.org; linux-s...@vger.kernel.org; target- > de...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: Re: [PATCH 1/1] target:separate t

Re: [PATCH v1] ringbuffer: Don't choose the process with adj equal OOM_SCORE_ADJ_MIN

2018-04-07 Thread Steven Rostedt
On Sun, 8 Apr 2018 10:16:23 +0800 Zhaoyang Huang wrote: > Don't choose the process with adj == OOM_SCORE_ADJ_MIN which > over-allocating pages for ring buffers. Why? -- Steve

[PATCH -mm] mm, pagemap: Fix swap offset value for PMD migration entry

2018-04-07 Thread Huang, Ying
From: Huang Ying The swap offset reported by /proc//pagemap may be not correct for PMD migration entry. If addr passed into pagemap_range() isn't aligned with PMD start address, the swap offset reported doesn't reflect this. And in the loop to report information of each sub-page, the swap offse

[PATCH v2] ARM64: dts: meson-axg: enable the eMMC controller

2018-04-07 Thread Yixun Lan
From: Nan Li The IP of eMMC controller in AXG is similiar to Meson-GX series. Here we add the initial support of the HS200 mode with clock running at 166MHz (to be safe), since we found some eMMC chip fail to run at 200MHz due to tunning phase error. Signed-off-by: Nan Li Signed-off-by: Yixun L

[PATCH] f2fs: remove redundant block plug

2018-04-07 Thread Chao Yu
For buffered IO, we don't need to use block plug to cache bio, for direct IO, generic f2fs_direct_IO has already added block plug, so let's remove redundant one in .write_iter. As Yunlei described in his patch: -f2fs_file_write_iter -blk_start_plug -__generic_file_write_iter ...

[PATCH] f2fs: fix to show missing bits in FS_IOC_GETFLAGS

2018-04-07 Thread Chao Yu
This patch fixes to show missing encrypt/inline_data flag in FS_IOC_GETFLAGS like ext4 does. Signed-off-by: Chao Yu --- fs/f2fs/file.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/fs/f2fs/file.c b/fs/f2fs/file.c index 3fd485395f9d..79cfaad21705 100644 --- a/fs/

[PATCH] f2fs: remove unneeded F2FS_PROJINHERIT_FL

2018-04-07 Thread Chao Yu
Now F2FS_FL_USER_VISIBLE and F2FS_FL_USER_MODIFIABLE has included F2FS_PROJINHERIT_FL, so remove unneeded F2FS_PROJINHERIT_FL when using visible/modifiable flag macro. Signed-off-by: Chao Yu --- fs/f2fs/file.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/fs/f2fs/fi

[PATCH v4 5/7] clk: meson-axg: Add AO Clock and Reset controller driver

2018-04-07 Thread Yixun Lan
From: Qiufang Dai Adds a Clock and Reset controller driver for the Always-On part of the Amlogic Meson-AXG SoC. Signed-off-by: Qiufang Dai Signed-off-by: Yixun Lan --- drivers/clk/meson/Makefile| 2 +- drivers/clk/meson/axg-aoclk.c | 164 ++ drive

[PATCH v4 6/7] clk: meson: drop CLK_SET_RATE_PARENT flag

2018-04-07 Thread Yixun Lan
The clk81 is not expected to be changed, so drop this flag. Signed-off-by: Yixun Lan --- drivers/clk/meson/gxbb-aoclk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/meson/gxbb-aoclk.c b/drivers/clk/meson/gxbb-aoclk.c index 59db8e92f8cf..0a2641d6fd33 100644 ---

Re: [PATCH 01/15] mm/hmm: documentation editorial update to HMM documentation

2018-04-07 Thread Randy Dunlap
On 03/22/2018 05:55 PM, jgli...@redhat.com wrote: > From: Ralph Campbell > > This patch updates the documentation for HMM to fix minor typos and > phrasing to be a bit more readable. > > Signed-off-by: Ralph Campbell > Signed-off-by: Jérôme Glisse > Cc: Stephen Bates > Cc: Jason Gunthorpe >

[PATCH v4 7/7] clk: meson: drop CLK_IGNORE_UNUSED flag

2018-04-07 Thread Yixun Lan
Rely on drivers to request the clock explicitly. Previous the kernel will leave the clock on while bootloader adready initilized the clock, this wasn't optimal way, so fix it here. Signed-off-by: Yixun Lan --- drivers/clk/meson/axg-aoclk.c | 1 - drivers/clk/meson/gxbb-aoclk.c | 1 - 2 files c

[PATCH v4 0/7] clk: meson-axg: Add AO Cloclk and Reset driver

2018-04-07 Thread Yixun Lan
This patch try to add AO clock and Reset driver for Amlogic's Meson-AXG SoC. Please note that patch 7 need to wait for the DTS changes[3] merged into mainline first, otherwise it will break the serial console. patch 1: factor the common code into a dedicated file patch 3-5: add the aoclk dri

[PATCH v4 2/7] clk: meson: migrate to devm_of_clk_add_hw_provider API

2018-04-07 Thread Yixun Lan
There is a protential memory leak, as of_clk_del_provider is never called if of_clk_add_hw_provider has been executed. Fix this by using devm variant API. Suggested-by: Stephen Boyd Signed-off-by: Yixun Lan --- drivers/clk/meson/meson-aoclk.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-

[PATCH v4 3/7] dt-bindings: clock: axg-aoclkc: New binding for Meson-AXG SoC

2018-04-07 Thread Yixun Lan
Update the dt-binding documentation to support new compatible string for the Amlogic's Meson-AXG SoC. Reviewed-by: Rob Herring Signed-off-by: Yixun Lan --- Documentation/devicetree/bindings/clock/amlogic,gxbb-aoclkc.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetre

Re: [PATCH 2/6] crypto: ctr - avoid VLA use

2018-04-07 Thread Herbert Xu
On Sat, Apr 07, 2018 at 08:38:19PM +0200, Salvatore Mesoraca wrote: > > @@ -206,6 +207,14 @@ static struct crypto_instance *crypto_ctr_alloc(struct > rtattr **tb) > if (alg->cra_blocksize < 4) > goto out_put_alg; > > + /* Block size must be <= MAX_BLOCKSIZE. */ > + if

[PATCH v4 1/7] clk: meson: aoclk: refactor common code into dedicated file

2018-04-07 Thread Yixun Lan
We try to refactor the common code into one dedicated file, while preparing to add new Meson-AXG aoclk driver, this would help us to better share the code by all aoclk drivers. Suggested-by: Jerome Brunet Signed-off-by: Yixun Lan --- drivers/clk/meson/Makefile | 2 +- drivers/clk/meson/gx

[PATCH v4 4/7] dt-bindings: clock: reset: Add AXG AO Clock and Reset Bindings

2018-04-07 Thread Yixun Lan
Add dt-bindings headers for the Meson-AXG's AO clock and reset controller. Reviewed-by: Rob Herring Signed-off-by: Yixun Lan --- include/dt-bindings/clock/axg-aoclkc.h | 26 ++ include/dt-bindings/reset/axg-aoclkc.h | 20 2 files changed, 46 insertio

Re: [PATCH V3 4/4] genirq/affinity: irq vector spread among online CPUs as far as possible

2018-04-07 Thread Ming Lei
On Fri, Apr 06, 2018 at 11:49:47PM +0200, Thomas Gleixner wrote: > On Fri, 6 Apr 2018, Thomas Gleixner wrote: > > > On Fri, 6 Apr 2018, Ming Lei wrote: > > > > > > I will post V4 soon by using cpu_present_mask in the 1st stage irq spread. > > > And it should work fine for Kashyap's case in normal

Re: [PATCH 3/6] crypto: api - avoid VLA use

2018-04-07 Thread Herbert Xu
On Sat, Apr 07, 2018 at 08:38:20PM +0200, Salvatore Mesoraca wrote: > > int crypto_init_cipher_ops(struct crypto_tfm *tfm) > { > + const unsigned long alignmask = crypto_tfm_alg_alignmask(tfm); > + const unsigned int size = crypto_tfm_alg_blocksize(tfm); > struct cipher_tfm *ops = &

Use struct page for filename

2018-04-07 Thread Matthew Wilcox
On Fri, Apr 06, 2018 at 03:33:36PM +0200, Dmitry Vyukov wrote: > On Fri, Apr 6, 2018 at 3:24 PM, syzbot > wrote: > > cache_from_obj: Wrong slab cache. names_cache but object is from kmalloc-96 > > Al, do you see how this can happen? I don't see how it happened, but when looking at this bug, I th

Re: [PATCH 1/6] crypto: api - laying macros for statically allocated buffers

2018-04-07 Thread Herbert Xu
On Sat, Apr 07, 2018 at 08:38:18PM +0200, Salvatore Mesoraca wrote: > Creating 2 new compile-time constants for internal use, > in preparation for the removal of VLAs[1] from crypto code. > All ciphers implemented in Linux have a block size less than or > equal to 16 bytes and the most demanding hw

Re: [PATCH v2 1/3] mm/gup_benchmark: handle gup failures

2018-04-07 Thread Michael S. Tsirkin
On Sat, Apr 07, 2018 at 01:08:43PM -0700, Linus Torvalds wrote: > On Thu, Apr 5, 2018 at 2:03 PM, Michael S. Tsirkin wrote: > > > > nr = get_user_pages_fast(addr, nr, gup->flags & 1, pages + > > i); > > - i += nr; > > + if (nr > 0) > > +

Re: [PATCH v8 15/18] mm, fs, dax: handle layout changes to pinned dax mappings

2018-04-07 Thread Paul E. McKenney
On Sat, Apr 07, 2018 at 12:38:24PM -0700, Dan Williams wrote: > [ adding Paul and Josh ] > > On Wed, Apr 4, 2018 at 2:46 AM, Jan Kara wrote: > > On Fri 30-03-18 21:03:30, Dan Williams wrote: > >> Background: > >> > >> get_user_pages() in the filesystem pins file backed memory pages for > >> acces

Re: [PATCH v2 0/3] mm/get_user_pages_fast fixes, cleanups

2018-04-07 Thread Michael S. Tsirkin
On Fri, Apr 06, 2018 at 01:08:02PM -0700, Andrew Morton wrote: > On Fri, 6 Apr 2018 00:03:48 +0300 "Michael S. Tsirkin" > wrote: > > > Turns out get_user_pages_fast and __get_user_pages_fast return different > > values on error when given a single page: __get_user_pages_fast returns > > 0. get_

[PATCH 2/2] hwspinlock: Introduce one new mode for hwspinlock

2018-04-07 Thread Baolin Wang
In some scenarios, user need do some time-consuming or sleepable operations under the hardware spinlock protection for synchronization between the multiple subsystems. For example, there is one PMIC efuse on Spreadtrum platform, which need to be accessed under one hardware lock. But during the har

[PATCH 1/2] hwspinlock: Convert to use 'switch' statement

2018-04-07 Thread Baolin Wang
We have different hwspinlock modes to select, thus it will be more readable to handle different modes with using 'switch' statement instead of 'if' statement. Signed-off-by: Baolin Wang --- drivers/hwspinlock/hwspinlock_core.c | 33 - 1 file changed, 24 insertio

Re: [PATCH v3 1/6] clk: meson: aoclk: refactor common code into dedicated file

2018-04-07 Thread Yixun Lan
HI Stephen thanks for the review On 04/07/18 02:39, Stephen Boyd wrote: > Quoting Yixun Lan (2018-03-27 19:50:45) >> diff --git a/drivers/clk/meson/gxbb-aoclk.c b/drivers/clk/meson/gxbb-aoclk.c >> index 9ec23ae9a219..5a922639a264 100644 >> --- a/drivers/clk/meson/gxbb-aoclk.c >> +++ b/drivers/clk

[PATCH] Documentation: fix reST markup error in driver-api/usb/typec.rst

2018-04-07 Thread changbin . du
From: Changbin Du There is an format error in driver-api/usb/typec.rst that breaks sphinx docs building. reST markup error: /home/changbin/work/linux/Documentation/driver-api/usb/typec.rst:215: (SEVERE/4) Unexpected section title or transition. Documentation/Makefile:6

Re: linux-next: build failure after merge of the vfs tree

2018-04-07 Thread Stephen Rothwell
Hi Al, On Sun, 8 Apr 2018 03:19:56 +0100 Al Viro wrote: > > > > Caused by commit > > > > > > 92afc556e622 ("buffer.c: call thaw_super during emergency thaw") > > > > > > I have reverted that commit for today. > > > > I am still doing that revert ... > > That's interesting, seeing that t

[PATCH v2 2/3] resource: add walk_system_ram_res_rev()

2018-04-07 Thread Baoquan He
This function, being a variant of walk_system_ram_res() introduced in commit 8c86e70acead ("resource: provide new functions to walk through resources"), walks through a list of all the resources of System RAM in reversed order, i.e., from higher to lower. It will be used in kexec_file code. Signe

[PATCH v2 3/3] kexec_file: Load kernel at top of system RAM if required

2018-04-07 Thread Baoquan He
For kexec_file loading, if kexec_buf.top_down is 'true', the memory which is used to load kernel/initrd/purgatory is supposed to be allocated from top to down. This is what we have been doing all along in the old kexec loading interface and the kexec loading is still default setting in some distrib

[PATCH v2 1/3] resource: Use list_head to link sibling resource

2018-04-07 Thread Baoquan He
The struct resource uses singly linked list to link siblings. It's not easy to do reverse iteration on sibling list. So replace it with list_head. And this makes codes in kernel/resource.c more readable after refactoring than pointer operation. Suggested-by: Andrew Morton Signed-off-by: Baoquan

[PATCH v2 0/3] resource: Use list_head to link sibling resource

2018-04-07 Thread Baoquan He
This post mainly converts strut resource's sibling list from singly linked list to doubly linked list, list_head. This is suggested by Andrew. Since I need a reversed searching on iomem_resource's IORESOURCE_SYSTEM_RAM children, the old singly linked list way makes the code in v1 post really ugly.

Re: simultaneous voice/data works (was Re: call/normal switch was Re: omap4-droid4: voice call support was)

2018-04-07 Thread Dan Williams
On Sat, 2018-04-07 at 14:22 +0200, Pavel Machek wrote: > On Sat 2018-04-07 10:10:00, Pavel Machek wrote: > > Hi! > > > > It seems qmicli can be used while unicsy_demo/ofone talks to the > > modem > > using the AT commands. > > > > So I could do: > > > > qmicli -d /dev/cdc-wdm0 --wds-follow-netwo

Re: WARNING in ext4_superblock_csum_set

2018-04-07 Thread Theodore Y. Ts'o
#syz fix: ext4: always initialize the crc32c checksum driver

Re: [PATCH] drm: Add crtc_queue_syncobj and crtc_get_syncobj ioctls

2018-04-07 Thread Keith Packard
Jason Ekstrand writes: > Yeah, I suppose an application could ask for 10k frames in the future or > something ridiculous like that. For sync_file, people strongly want a > finite time guarantee for security/deadlock reasons. I don't know how > happy they would be with a finite time of 10 minute

Re: linux-next: build failure after merge of the vfs tree

2018-04-07 Thread Al Viro
On Tue, Apr 03, 2018 at 12:26:29PM +1000, Stephen Rothwell wrote: > Hi Al, > > On Mon, 19 Mar 2018 17:06:27 +1100 Stephen Rothwell > wrote: > > > > After merging the vfs tree, today's linux-next build (x86_64 allnoconfig) > > failed like this: > > > > fs/super.c: In function 'do_thaw_all_callba

答复: 答复: [PATCH 1/2] x86/mce: new centaur CPUs support MCE broadcasting

2018-04-07 Thread David Wang
> -邮件原件- > 发件人: Borislav Petkov [mailto:b...@alien8.de] > 发送时间: 2018年4月4日 19:18 > 收件人: David Wang > 抄送: tony.l...@intel.com; t...@linutronix.de; mi...@redhat.com; > h...@zytor.com; x...@kernel.org; linux-e...@vger.kernel.org; > linux-kernel@vger.kernel.org; Bruce Chang (VAS) > ; Cooper Yan

[PATCH v1] ringbuffer: Don't choose the process with adj equal OOM_SCORE_ADJ_MIN

2018-04-07 Thread Zhaoyang Huang
Don't choose the process with adj == OOM_SCORE_ADJ_MIN which over-allocating pages for ring buffers. Signed-off-by: Zhaoyang Huang --- kernel/trace/ring_buffer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/trace/ring_buffer.c b/kernel/trace/ring_buffer.c index 5f38

Re: [PATCH] checkpatch: relax check for revert commit

2018-04-07 Thread Jia He
Hi Andy & Joe Although it is minor, it is a real bug, I thought. Is there any comment? Thank you --- Cheers, Jia On 4/2/2018 10:28 PM, Jia He Wrote: For revert commit, it might has two double quotation marks in its commit log. Relax the check condition for revert commit to avoid checkpatc

Re: [PATCH v7 2/5] arm: arm64: page_alloc: reduce unnecessary binary search in memblock_next_valid_pfn()

2018-04-07 Thread Jia He
Thanks for your comments, Russell On 4/6/2018 5:09 PM, Russell King - ARM Linux Wrote: On Thu, Apr 05, 2018 at 05:50:54AM -0700, Matthew Wilcox wrote: On Thu, Apr 05, 2018 at 08:44:12PM +0800, Jia He wrote: On 4/5/2018 7:34 PM, Matthew Wilcox Wrote: On Thu, Apr 05, 2018 at 01:04:35AM -0700,

Re: [PATCH] mmc: dw_mmc-k3: Fix DDR52 mode by setting required clock divisor

2018-04-07 Thread Shawn Lin
On 2018/4/6 21:41, Ryan Grachek wrote: On Wed, Apr 4, 2018 at 7:51 PM, Shawn Lin wrote: [+ Zhangfei Gao who added support for hi6220] On 2018/4/4 23:31, Ryan Grachek wrote: On Tue, Apr 3, 2018 at 6:31 AM, Shawn Lin mailto:shawn@rock-chips.com>> wrote: On 2018/3/30 2:24, oscardagrac

Re: [PATCH 1/3] hexagon: fix ffz/fls/ffs return type

2018-04-07 Thread Richard Kuo
On Fri, Apr 06, 2018 at 04:28:21PM +0200, Arnd Bergmann wrote: > Let's use the same return type as the major architectures to > avoid warnings like > > drivers/ata/libahci_platform.c: In function 'ahci_platform_init_host': > drivers/ata/libahci_platform.c:561:12: warning: comparison of distinct >

Re: [PATCH 3/3] hexagon: export csum_partial_copy_nocheck

2018-04-07 Thread Richard Kuo
On Fri, Apr 06, 2018 at 04:28:23PM +0200, Arnd Bergmann wrote: > This is needed to link ipv6 as a loadable module, which in turn happens > in allmodconfig. > > Signed-off-by: Arnd Bergmann > --- > arch/hexagon/lib/checksum.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/hexagon/

Re: [PATCH 2/3] hexagon: add memset_io() helper

2018-04-07 Thread Richard Kuo
On Fri, Apr 06, 2018 at 04:28:22PM +0200, Arnd Bergmann wrote: > We already have memcpy_toio(), but not memset_io(), so let's > add the obvious version to allow building an allmodconfig kernel > without errors like > > drivers/gpu/drm/ttm/ttm_bo_util.c: In function 'ttm_bo_move_memcpy': > drivers/

Re: [PATCH] f2fs: enlarge block plug coverage

2018-04-07 Thread Chao Yu
On 2018/4/5 11:51, Jaegeuk Kim wrote: > On 04/04, Chao Yu wrote: >> This patch enlarges block plug coverage in __issue_discard_cmd, in >> order to collect more pending bios before issuing them, to avoid >> being disturbed by previous discard I/O in IO aware discard mode. > > Hmm, then we need to w

Re: [PATCH] f2fs: fix to show encrypt flag in FS_IOC_GETFLAGS

2018-04-07 Thread Chao Yu
On 2018/4/5 11:46, Jaegeuk Kim wrote: > On 04/03, Chao Yu wrote: >> On 2018/4/3 4:21, Jaegeuk Kim wrote: >>> On 04/02, Chao Yu wrote: This patch fixes to show encrypt flag in FS_IOC_GETFLAGS like ext4 does. >>> >>> Actually, we have to show internal flags owned by f2fs, not generic ones. >>> W

Re: KMSAN: uninit-value in tipc_node_get_mtu

2018-04-07 Thread syzbot
syzbot has found reproducer for the following crash on https://github.com/google/kmsan.git/master commit e2ab7e8abba47a2f2698216258e5d8727ae58717 (Fri Apr 6 16:24:31 2018 +) kmsan: temporarily disable visitAsmInstruction() to help syzbot syzbot dashboard link: https://syzkaller.appspot.com

[PATCH] tools/memory-model: Update required version of herdtools7

2018-04-07 Thread Akira Yokosawa
>From 64542a47acf159d26b8290480c0af9c2a05d20d6 Mon Sep 17 00:00:00 2001 From: Akira Yokosawa Date: Sun, 8 Apr 2018 07:44:55 +0900 Subject: [PATCH] tools/memory-model: Update required version of herdtools7 Code generated by klitmus7 version 7.48 doesn't compile with kernel header of 4.15 and later

Re: [PATCH] openrisc: define mb() as its mandatory

2018-04-07 Thread Stafford Horne
On Sat, Apr 07, 2018 at 11:09:05AM +0200, Peter Zijlstra wrote: > On Sat, Apr 07, 2018 at 05:58:49AM +0900, Stafford Horne wrote: > > Following Peter Z's patch ("asm-generic: Disallow no-op mb() for SMP > > systems") which makes mb() mandatory for SMP architectures we define it > > as l.msync. On

[GIT PULL][SECURITY] Integrity updates for v4.17

2018-04-07 Thread James Morris
Please pull these updates for the Integrity subsystem, via Mimi: a mixture of bug fixes, code cleanup, and continues to close IMA-measurement, IMA-appraisal, and IMA-audit gaps. Also note the addition of a new cred_getsecid LSM hook by Matthew Garrett: For IMA purposes, we want to b

RE: [PATCH 1/2] tools/power turbostat: Correct SNB_C1/C3_AUTO_UNDEMOTE defines

2018-04-07 Thread Brown, Len
You just did it:-) -Original Message- From: Matt Turner [mailto:matts...@gmail.com] Sent: Saturday, April 07, 2018 6:25 PM To: Brown, Len Cc: Thomas Gleixner ; Ingo Molnar ; H. Peter Anvin ; Borislav Petkov ; LKML ; x...@kernel.org; Matt Turner Subject: Re: [PATCH 1/2] tools/power tu

[GIT PULL][SECURITY] TPM updates for v4.17

2018-04-07 Thread James Morris
Please pull these changes for TPM, via Jarkko: "This release contains only bug fixes. There are no new major features added." The following changes since commit 5893ed18a26d1f56b97c0290b0cbbc2d49d6de28: Merge tag 'v4.16-rc6' into next-general (2018-03-23 08:26:16 +1100) are available in the

Re: [PATCH 02/10] locking/qspinlock: Remove unbounded cmpxchg loop from locking slowpath

2018-04-07 Thread Paul E. McKenney
On Sat, Apr 07, 2018 at 10:47:32AM +0200, Peter Zijlstra wrote: > On Fri, Apr 06, 2018 at 02:09:53PM -0700, Paul E. McKenney wrote: > > It would indeed be good to not be in the position of having to trade off > > forward-progress guarantees against performance, but that does appear to > > be where

Re: [git pull] sparc syscall stuff

2018-04-07 Thread Al Viro
On Sat, Apr 07, 2018 at 02:37:24PM -0700, Linus Torvalds wrote: > On Sat, Apr 7, 2018 at 1:44 PM, Al Viro wrote: > > > > Conflicts in arch/sparc/kernel/sys_sparc32.c are not hard to > > resolve, > > but keep in mind that casts to u64 are needed there. I'd pushed a variant > > of > > res

Re: [PATCH v3 1/4] Ktest: add email support

2018-04-07 Thread Steven Rostedt
On Fri, 6 Apr 2018 16:09:28 -0700 Tim Tianyang Chen wrote: > > sub _sendmail_send { > > my ($subject, $message) = @_; > > -system("echo -e \"Subject: $subject\n\n$message\" | sendmail -t > > $mailto"); > > + > > +if (!defined($mail_exec)) { > > + $mail_exec = "/usr/sbin/sendma

Re: [PATCH] scripts : sync the var type between memory and print

2018-04-07 Thread Steven Rostedt
On Sat, 7 Apr 2018 18:01:35 +0800 Wenle Chen wrote: > From: Wenle Chen > > The %d in printf need get a int > But the real type of these vars is unsigned. > It may let some compilers show a warning. Thanks but the merge window is currently open. It's best to not send patches during it. Ple

Re: [PATCH 1/2] tools/power turbostat: Correct SNB_C1/C3_AUTO_UNDEMOTE defines

2018-04-07 Thread Matt Turner
On Tue, Feb 13, 2018 at 11:12 AM, Matt Turner wrote: > According to the Intel Software Developers' Manual, Vol. 4, Order No. > 335592, these macros have been reversed since they were added. > > Fixes: 889facbee3e6 ("tools/power turbostat: v3.0: monitor Watts and > Temperature") > Signed-off-by: M

Re: [PATCH v2] add -Wpointer-arith sparse flag to toggle sizeof(void) warnings

2018-04-07 Thread Linus Torvalds
Ack, looks sane. Linus

[PATCH v2] add -Wpointer-arith sparse flag to toggle sizeof(void) warnings

2018-04-07 Thread Joey Pabalinas
Recent changes to the min()/max() macros in include/linux/kernel.h have added a lot of noise when compiling the kernel with Sparse checking enabled. This mostly is due to the *huge* increase in the number of sizeof(void) warnings, a larger number of which can safely be ignored. Add the -Wpointer-a

Re: [PATCH v3 2/2] MIPS: io: add a barrier after register read in readX()

2018-04-07 Thread Sinan Kaya
On 4/6/2018 5:26 PM, James Hogan wrote: > On Fri, Apr 06, 2018 at 02:15:57PM -0400, Sinan Kaya wrote: >> On 4/5/2018 9:34 PM, Sinan Kaya wrote: >>> Can we get these merged to 4.17? >>> >>> There was a consensus to fix the architectures having API violation issues. >>> https://www.mail-archive.com/

Re: [git pull] sparc syscall stuff

2018-04-07 Thread Linus Torvalds
On Sat, Apr 7, 2018 at 1:44 PM, Al Viro wrote: > > Conflicts in arch/sparc/kernel/sys_sparc32.c are not hard to resolve, > but keep in mind that casts to u64 are needed there. I'd pushed a variant of > resolution into #test-merge. You say "keep in mind that casts to u64 are needed there"

[PATCH] drm/stm: ltdc: add user update info in plane print state

2018-04-07 Thread Philippe Cornu
This patch adds the user update information in frames-per-second into the drm debugfs plane state. Signed-off-by: Philippe Cornu --- drivers/gpu/drm/stm/ltdc.c | 22 ++ drivers/gpu/drm/stm/ltdc.h | 8 2 files changed, 30 insertions(+) diff --git a/drivers/gpu/drm/s

[PATCH] drm/stm: move enable/disable_vblank to crtc

2018-04-07 Thread Philippe Cornu
enable/disable_vblank() functions at drm_driver level are deprecated. Move them to the ltdc drm_crtc_funcs structure. Signed-off-by: Philippe Cornu --- drivers/gpu/drm/stm/drv.c | 2 -- drivers/gpu/drm/stm/ltdc.c | 10 ++ drivers/gpu/drm/stm/ltdc.h | 2 -- 3 files changed, 6 insertion

Re: [PATCH 4.16 00/31] 4.16.1-stable review

2018-04-07 Thread Dan Rue
On Sat, Apr 07, 2018 at 07:54:27PM +0200, Greg Kroah-Hartman wrote: > On Sat, Apr 07, 2018 at 10:37:33PM +0530, Naresh Kamboju wrote: > > >> Results from Linaro’s test farm. > > >> No regressions on arm64, arm and x86_64. > > >> > > >> Summary > > >>

[git pull] alpha syscall stuff

2018-04-07 Thread Al Viro
a couple of SYSCALL_DEFINE conversions and removal of pointless (and bitrotten) piece stuck in ret_from_kernel_thread since the kernel_exceve/kernel_thread conversions six years ago. The following changes since commit 0c8efd610b58cb23cefdfa12015799079aef94ae: Linux 4.16-rc5 (2018-03-11

[git pull] sparc syscall stuff

2018-04-07 Thread Al Viro
sparc syscall stuff - killing pointless wrappers, conversions to {COMPAT_,}SYSCALL_DEFINE. Conflicts in arch/sparc/kernel/sys_sparc32.c are not hard to resolve, but keep in mind that casts to u64 are needed there. I'd pushed a variant of resolution into #test-merge. The following

Re: git source files with rw permissions

2018-04-07 Thread Linus Torvalds
On Thu, Apr 5, 2018 at 5:53 PM, Joe Perches wrote: > > A trivial script on a clean git tree like: > > $ git ls-files -- "*.[chsS]" | \ > xargs ls -la | \ > grep -P "^[^l](?:.{2,2}x|.{5,5}x|.{8,8}x)" | \ > awk '{print $9}' A simpler version of this is: git ls-files -s '*.[chsS]' | grep '^

[PATCH] can: ti_hecc: fix close when napi poll is active

2018-04-07 Thread Jeroen Hofstee
When closing this CAN interface while napi poll is active, for example with: `ip link set can0 down` several interfaces freeze. This seemed to be caused by napi_disable called from ti_hecc_close expecting the scheduled probe to either return quota or call napi_complete. Since the poll functions has

Re: [PATCH 1/3] mm: replace S_IRUGO with 0444

2018-04-07 Thread Joe Perches
On Sat, 2018-04-07 at 19:47 +0100, Paul McQuade wrote: > Fix checkpatch warnings about S_IRUGO being less readable than > providing the permissions octal as '0444'. Hey Paul. I sent a cleanup a couple weeks ago to Andrew Morton for the same thing. https://lkml.org/lkml/2018/3/26/638 Andrew said

Re: [PATCH v2 1/3] mm/gup_benchmark: handle gup failures

2018-04-07 Thread Linus Torvalds
On Thu, Apr 5, 2018 at 2:03 PM, Michael S. Tsirkin wrote: > > nr = get_user_pages_fast(addr, nr, gup->flags & 1, pages + i); > - i += nr; > + if (nr > 0) > + i += nr; Can we just make this robust while at it, and just make it

Re: [PATCH 0/6] Remove several VLAs in the crypto subsystem

2018-04-07 Thread Kees Cook
On Sat, Apr 7, 2018 at 11:38 AM, Salvatore Mesoraca wrote: > As suggested by Laura Abbott[2], I'm resending my patch with > MAX_BLOCKSIZE and MAX_ALIGNMASK defined in an header, so they > can be used in other places. > I take this opportuinuty to deal with some other VLAs not > handled in the old

Re: [PATCH v8 15/18] mm, fs, dax: handle layout changes to pinned dax mappings

2018-04-07 Thread Dan Williams
[ adding Paul and Josh ] On Wed, Apr 4, 2018 at 2:46 AM, Jan Kara wrote: > On Fri 30-03-18 21:03:30, Dan Williams wrote: >> Background: >> >> get_user_pages() in the filesystem pins file backed memory pages for >> access by devices performing dma. However, it only pins the memory pages >> not the

Re: make xmldocs failed with error after 4.17 merge period

2018-04-07 Thread Johannes Berg
On Fri, 2018-04-06 at 13:38 +0200, Markus Heiser wrote: > > Sorry, not yet. Johannes started a discussion about this. Its a long > time ago and I do not have any new ideas yet :/ see: > > https://www.mail-archive.com/linux-doc@vger.kernel.org/msg07035.html I still think the tools themselves do

Re: [PATCH v2] writeback: safer lock nesting

2018-04-07 Thread kbuild test robot
://github.com/0day-ci/linux/commits/Greg-Thelen/writeback-safer-lock-nesting/20180407-122200 config: mips-db1xxx_defconfig (attached as .config) compiler: mipsel-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin

[PATCH 3/3] mm: replace S_IRUSR | S_IWUSR with 0600

2018-04-07 Thread Paul McQuade
Fix checkpatch warnings about S_IRUSR | S_IWUSR being less readable than providing the permissions octal as '0600'. Signed-off-by: Paul McQuade --- mm/failslab.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/failslab.c b/mm/failslab.c index 1f2f248e3601..b135ebb88b6f 100

[PATCH 1/3] mm: replace S_IRUGO with 0444

2018-04-07 Thread Paul McQuade
Fix checkpatch warnings about S_IRUGO being less readable than providing the permissions octal as '0444'. Signed-off-by: Paul McQuade --- mm/cleancache.c | 8 mm/cma_debug.c | 12 ++-- mm/dmapool.c| 2 +- mm/frontswap.c | 8 4 files changed, 15 insertions(+), 1

[PATCH 2/3] mm: Replace S_IWUSR with 0200

2018-04-07 Thread Paul McQuade
Fix checkpatch warnings about S_IWUSR being less readable than providing the permissions octal as '0200'. Signed-off-by: Paul McQuade --- mm/cma_debug.c | 4 ++-- mm/compaction.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/mm/cma_debug.c b/mm/cma_debug.c index 6494c7a

[PATCH 4/6] crypto: pcbc - avoid VLA use

2018-04-07 Thread Salvatore Mesoraca
We avoid 2 VLAs[1] by always allocating MAX_BLOCKSIZE bytes. We also check the selected cipher at instance creation time, if it doesn't comply with these limits, the creation will fail. [1] http://lkml.kernel.org/r/CA+55aFzCG-zNmZwX4A2FQpadafLfEzK6CC=qpxydaacu1rq...@mail.gmail.com Signed-off-by:

[PATCH 1/6] crypto: api - laying macros for statically allocated buffers

2018-04-07 Thread Salvatore Mesoraca
Creating 2 new compile-time constants for internal use, in preparation for the removal of VLAs[1] from crypto code. All ciphers implemented in Linux have a block size less than or equal to 16 bytes and the most demanding hw require 16 bytes alignment for the block buffer. [1] http://lkml.kernel.o

[PATCH 6/6] crypto: cfb - avoid VLA use

2018-04-07 Thread Salvatore Mesoraca
We avoid 3 VLAs[1] by always allocating MAX_BLOCKSIZE bytes or, when needed for alignement, MAX_BLOCKSIZE + MAX_ALIGNMASK bytes. We also check the selected cipher at instance creation time, if it doesn't comply with these limits, the creation will fail. [1] http://lkml.kernel.org/r/CA+55aFzCG-zNm

[PATCH 3/6] crypto: api - avoid VLA use

2018-04-07 Thread Salvatore Mesoraca
We avoid a VLA[1] by always allocating MAX_BLOCKSIZE + MAX_ALIGNMASK bytes. We also check the selected cipher at initialization time, if it doesn't comply with these limits, the initialization will fail. [1] http://lkml.kernel.org/r/CA+55aFzCG-zNmZwX4A2FQpadafLfEzK6CC=qpxydaacu1rq...@mail.gmail.c

[PATCH 5/6] crypto: cts - avoid VLA use

2018-04-07 Thread Salvatore Mesoraca
We avoid 2 VLAs[1] by always allocating MAX_BLOCKSIZE*2 bytes. We also check the selected cipher at instance creation time, if it doesn't comply with these limits, the creation will fail. [1] http://lkml.kernel.org/r/CA+55aFzCG-zNmZwX4A2FQpadafLfEzK6CC=qpxydaacu1rq...@mail.gmail.com Signed-off-b

  1   2   3   >