[rcu:rcu/next 42/42] kernel/rcu/tree_plugin.h:820:2: note: in expansion of macro 'pr_info'

2018-02-16 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git rcu/next head: 07180bfca95f9819250887160a5ccc53119967a7 commit: 07180bfca95f9819250887160a5ccc53119967a7 [42/42] EXP rcu: Add ->qsmask to assertion config: x86_64-randconfig-x013-201806 (attached as .config) compiler:

Re: [PATCH RFC v2 4/6] x86: Disable PTI on compatibility mode

2018-02-16 Thread Dmitry Safonov
2018-02-16 7:11 GMT+00:00 Cyrill Gorcunov : > On Thu, Feb 15, 2018 at 11:29:42PM +, Andy Lutomirski wrote: > ... >> >>> +bool pti_handle_segment_not_present(long error_code) >> >>> +{ >> >>> + if (!static_cpu_has(X86_FEATURE_PTI)) >> >>> + return false; >> >>> + >> >>> +

Re: [PATCH 0/2] efivars: reading variables can generate SMIs

2018-02-16 Thread Peter Jones
On Fri, Feb 16, 2018 at 09:09:30PM +, Luck, Tony wrote: > > That said, I'm not sure how many non-root users run the toolkit to > > extract their EFI certificates or check on the secure boot status of > > the system, but I suspect it might be non-zero: I can see the tinfoil > > hat people wantin

Re: [PATCH 0/2] efivars: reading variables can generate SMIs

2018-02-16 Thread Matthew Garrett
On Fri, Feb 16, 2018 at 2:02 PM Luck, Tony wrote: > > If the default is 600 then it makes sense to allow a privileged service to > > selectively make certain variables world readable at runtime. > As soon as you make one variable world readable you are vulnerable to > a local user launching a Do

Re: [PATCH 1/3] Kconfig: disable PROFILE_ALL_BRANCHES for compile testing

2018-02-16 Thread Steven Rostedt
On Fri, 16 Feb 2018 22:41:11 +0100 Arnd Bergmann wrote: > This can easily double the time for compiling a driver but does not > provide any benefit for the compile tester, so it's better left disabled. > > In addition, any 'inline' function that is not also 'static' and that > contains an 'if' c

Re: [PATCH v3 01/15] Documentation: add newcx initramfs format description

2018-02-16 Thread H. Peter Anvin
On 02/16/18 12:33, Taras Kondratiuk wrote: > Many of the Linux security/integrity features are dependent on file > metadata, stored as extended attributes (xattrs), for making decisions. > These features need to be initialized during initcall and enabled as > early as possible for complete security

RE: [PATCH 0/2] efivars: reading variables can generate SMIs

2018-02-16 Thread Luck, Tony
> If the default is 600 then it makes sense to allow a privileged service to > selectively make certain variables world readable at runtime. As soon as you make one variable world readable you are vulnerable to a local user launching a DoS attack by reading that variable over and over generating a

Re: [PATCH 03/11] fs: Allow superblock owner to change ownership of inodes

2018-02-16 Thread Eric W. Biederman
Miklos Szeredi writes: > On Fri, Dec 22, 2017 at 3:32 PM, Dongsu Park wrote: >> From: Eric W. Biederman >> >> Allow users with CAP_SYS_CHOWN over the superblock of a filesystem to >> chown files. Ordinarily the capable_wrt_inode_uidgid check is >> sufficient to allow access to files but when t

Re: [PATCH v21 1/5] xbitmap: Introduce xbitmap

2018-02-16 Thread Matthew Wilcox
On Fri, Feb 16, 2018 at 11:45:51PM +0200, Andy Shevchenko wrote: > Now, the question about test case. Why do you heavily use BUG_ON? > Isn't resulting statistics enough? No. If any of those tests fail, we want to stop dead. They'll lead to horrendous bugs throughout the kernel if they're wrong.

Re: [PATCH 0/2] efivars: reading variables can generate SMIs

2018-02-16 Thread Matthew Garrett
On Fri, Feb 16, 2018 at 1:45 PM Andy Lutomirski wrote: > I'm going to go out on a limb and suggest that the fact that > unprivileged users can read efi variables at all is a mistake > regardless of SMI issues. Why? They should never contain sensitive material. > Also, chmod() just shouldn't work

Re: [PATCH v3 01/15] Documentation: add newcx initramfs format description

2018-02-16 Thread Victor Kamensky
On Fri, 16 Feb 2018, Rob Landley wrote: On 02/16/2018 02:59 PM, H. Peter Anvin wrote: On 02/16/18 12:33, Taras Kondratiuk wrote: Many of the Linux security/integrity features are dependent on file metadata, stored as extended attributes (xattrs), for making decisions. These features need to

Re: [PATCH v5 00/11] FUSE mounts from non-init user namespaces

2018-02-16 Thread Eric W. Biederman
Miklos Szeredi writes: > On Fri, Dec 22, 2017 at 3:32 PM, Dongsu Park wrote: > >> Patches 1-2 deal with an additional flag of lookup_bdev() to check for >> additional inode permission. > > fuse_blk is less suitable for unprivileged mounting than plain fuse. > fusermount doesn't allow mounting fu

Re: [PATCH 08/11] fuse: Support fuse filesystems outside of init_user_ns

2018-02-16 Thread Eric W. Biederman
Miklos Szeredi writes: > On Mon, Feb 12, 2018 at 5:35 PM, Eric W. Biederman > wrote: >> Miklos Szeredi writes: >> >>> On Fri, Dec 22, 2017 at 3:32 PM, Dongsu Park wrote: From: Seth Forshee In order to support mounts from namespaces other than init_user_ns, fuse must transl

Re: [RFC PATCH] usb: hcd: complete URBs in threaded-IRQ context instead of tasklet

2018-02-16 Thread Alan Stern
On Fri, 16 Feb 2018, Sebastian Andrzej Siewior wrote: > On 2018-02-16 13:29:01 [-0500], Alan Stern wrote: > > We originally used tasklets because we didn't want to incur the delays > > associated with running in a process context. It seems odd to be > > reversing that decision now. > > The the

[PATCH 2/2] Staging: gdm724x: LTE: Fix argument list not aligned with parenthesis.

2018-02-16 Thread Quytelda Kahja
Fix coding style warning from checkpatch.pl. Signed-off-by: Quytelda Kahja --- drivers/staging/gdm724x/gdm_lte.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/gdm724x/gdm_lte.c b/drivers/staging/gdm724x/gdm_lte.c index ff3fed9c4a81..26a81fdd0044 100644 --

Re: [PATCH 0/2] efivars: reading variables can generate SMIs

2018-02-16 Thread Andy Lutomirski
On Fri, Feb 16, 2018 at 9:09 PM, Luck, Tony wrote: >> That said, I'm not sure how many non-root users run the toolkit to >> extract their EFI certificates or check on the secure boot status of >> the system, but I suspect it might be non-zero: I can see the tinfoil >> hat people wanting at least t

Re: [PATCH v21 1/5] xbitmap: Introduce xbitmap

2018-02-16 Thread Andy Shevchenko
On Fri, Feb 16, 2018 at 8:30 PM, Matthew Wilcox wrote: > On Fri, Feb 16, 2018 at 07:44:50PM +0200, Andy Shevchenko wrote: >> On Tue, Jan 9, 2018 at 1:10 PM, Wei Wang wrote: >> > From: Matthew Wilcox >> > >> > The eXtensible Bitmap is a sparse bitmap representation which is >> > efficient for set

[PATCH 3/3] Kconfig: add a default allrandom.config

2018-02-16 Thread Arnd Bergmann
Building randconfig kernels frequently leads to build errors from drivers that have additional build dependencies, or that we don't want to build for compile-testing for some other reason. We already have a couple of compile-time options that can deal with this problem, but the complete set is not

[PATCH 2/3] Kconfig: improve handling for all{rand,yes,no,}.config fragments

2018-02-16 Thread Arnd Bergmann
The kernel currently supports two methods of dealing with config fragments in the tree: a) Running "make foo.config" looks for arch/$(ARCH)/configs/foo.config and kernel/configs/foo.config, and applies the defaults from those files on top of the current configuration. b) Running "KCONFIG_AL

[PATCH 4/4] soc: ti: Add pm33xx driver for basic suspend support

2018-02-16 Thread Dave Gerlach
AM335x and AM437x support various low power modes as documented in section 8.1.4.3 of the AM335x Technical Reference Manual and section 6.4.3 of the AM437x Technical Reference Manual. DeepSleep0 mode offers the lowest power mode with limited wakeup sources without a system reboot and is mapped as

[PATCH 1/3] Kconfig: disable PROFILE_ALL_BRANCHES for compile testing

2018-02-16 Thread Arnd Bergmann
This can easily double the time for compiling a driver but does not provide any benefit for the compile tester, so it's better left disabled. In addition, any 'inline' function that is not also 'static' and that contains an 'if' causes a warning like include/linux/string.h:212:2: note: in expansi

[PATCH 1/4] ARM: OMAP2+: Introduce low-level suspend code for AM33XX

2018-02-16 Thread Dave Gerlach
In preparation for suspend-resume support for AM33XX, add the assembly file with the code which is copied to internal memory (OCMC RAM) during bootup and runs from there. As part of the low power entry (DeepSleep0 mode in AM33XX TRM), the code running from OCMC RAM does the following 1. Calls rout

[PATCH 0/4] ARM: OMAP2+: AM33XX/AM43XX: Add suspend-resume support

2018-02-16 Thread Dave Gerlach
Hi, This series contains the remaining code to enable suspend to mem and standby on am335x and am437x platforms. All dependencies have been merged and this is just a rebase on v4.16-rc1. For this to work on am437x-gp-evm this bug fix patch [1] is required but suspend works fine on am335x platforms

[PATCH 3/4] ARM: OMAP2+: pm33xx-core: Add platform code needed for PM

2018-02-16 Thread Dave Gerlach
Most of the PM code needed for am335x and am437x can be moved into a module under drivers but some core code must remain in mach-omap2 at the moment. This includes some internal clockdomain APIs and low-level ARM APIs which are also not exported for use by modules. Implement a few functions that h

[PATCH 0/3] fixing the last failures in randconfig builds

2018-02-16 Thread Arnd Bergmann
I've recently done some build testing on 4.14-stable and to my surprise found only a handful of bugfixes were required to build all x86 and arm64 randconfig output without any warnings using gcc-7.2.1. I have submitted the remaining trivial patches now, and this is the non-trivial remaining part, w

[PATCH 2/4] ARM: OMAP2+: Introduce low-level suspend code for AM43XX

2018-02-16 Thread Dave Gerlach
Although similar to AM33XX, introduce a new low-level asm file for suspend containing new context save and restore paths for EMIF and l2 cache disabling and enabling. Signed-off-by: Dave Gerlach --- arch/arm/mach-omap2/sleep43xx.S | 393 1 file changed, 3

[PATCH] Staging: gdm724x: LTE: Fix trailing open parenthesis code style issue.

2018-02-16 Thread Quytelda Kahja
Fix a coding style problem causing warnings from checkpatch.pl. Signed-off-by: Quytelda Kahja --- drivers/staging/gdm724x/gdm_lte.c | 50 ++- 1 file changed, 28 insertions(+), 22 deletions(-) diff --git a/drivers/staging/gdm724x/gdm_lte.c b/drivers/staging/g

Re: [PATCH][V2] net: dsa: mv88e6xxx: avoid unintended sign extension on a 16 bit shift

2018-02-16 Thread David Miller
From: Colin King Date: Fri, 16 Feb 2018 16:55:05 + > From: Colin Ian King > > The shifting of timehi by 16 bits to the left will be promoted to > a 32 bit signed int and then sign-extended to an u64. If the top bit > of timehi is set then all then all the upper bits of ns end up as also > b

Re: [RFC PATCH] ptr_ring: linked list fallback

2018-02-16 Thread David Miller
From: "Michael S. Tsirkin" Date: Fri, 16 Feb 2018 09:40:54 +0200 > So pointer rings work fine, but they have a problem: > make them too small and not enough entries fit. > Make them too large and you start flushing your cache > and running out of memory. > > This is a new idea of mine: a ring ba

[PATCH 2/3] nospec: Allow index argument to have const-qualified type

2018-02-16 Thread Dan Williams
From: Rasmus Villemoes The last expression in a statement expression need not be a bare variable, quoting gcc docs The last thing in the compound statement should be an expression followed by a semicolon; the value of this subexpression serves as the value of the entire construct. and we

[PATCH 3/3] nospec: Include asm/barrier.h dependency

2018-02-16 Thread Dan Williams
The nospec.h header expects the per-architecture header file asm/barrier.h to optionally define array_index_mask_nospec(). Include that dependency to prevent inadvertent fallback to the default array_index_mask_nospec() implementation. The default implementation may not provide a full mitigation on

Re: [PATCH v2 01/10] drivers: qcom: rpmh-rsc: add RPMH controller for QCOM SoCs

2018-02-16 Thread Evan Green
Hi Lina, On Thu, Feb 15, 2018 at 9:34 AM, Lina Iyer wrote: > Add controller driver for QCOM SoCs that have hardware based shared > resource management. The hardware IP known as RSC (Resource State > Coordinator) houses multiple Direct Resource Voter (DRV) for different > execution levels. A DRV i

[PATCH 0/3] nospec: Various fix-ups for include/linux/nospec.h

2018-02-16 Thread Dan Williams
Hi Ingo, Here is a small pile of cleanups and fixes for nospec.h after inspection from Linus, Rasmus, and Christian. Full changelogs below: These have received a build success notification from 0day across 126 configs. --- Dan Williams (2): nospec: Kill array_index_nospec_mask_check()

[PATCH 1/3] nospec: Kill array_index_nospec_mask_check()

2018-02-16 Thread Dan Williams
There are multiple problems with the dynamic sanity checking in array_index_nospec_mask_check(): * It causes unnecessary overhead in the 32-bit case since integer sized @index values will no longer cause the check to be compiled away like in the 64-bit case. * In the 32-bit case it may trigge

Re: [PATCH v3 01/15] Documentation: add newcx initramfs format description

2018-02-16 Thread Rob Landley
On 02/16/2018 02:59 PM, H. Peter Anvin wrote: > On 02/16/18 12:33, Taras Kondratiuk wrote: >> Many of the Linux security/integrity features are dependent on file >> metadata, stored as extended attributes (xattrs), for making decisions. >> These features need to be initialized during initcall and

Re: [PATCH v4] rtc: isl12026: Add driver.

2018-02-16 Thread Andy Shevchenko
On Fri, Feb 16, 2018 at 11:19 PM, David Daney wrote: > On 02/16/2018 12:13 PM, Andy Shevchenko wrote: >> On Fri, Feb 16, 2018 at 9:44 PM, David Daney >> wrote: >>> +config RTC_DRV_ISL12026 >>> + tristate "Intersil ISL12026" >> depends on OF > It doesn't depend on CONFIG_OF, it builds jus

Re: [PATCH net] rxrpc: Work around usercopy check

2018-02-16 Thread David Miller
From: David Howells Date: Thu, 15 Feb 2018 22:59:00 + > Due to a check recently added to copy_to_user(), it's now not permitted to > copy from slab-held data to userspace unless the slab is whitelisted. This > affects rxrpc_recvmsg() when it attempts to place an RXRPC_USER_CALL_ID > control

Re: [PATCH v4] rtc: isl12026: Add driver.

2018-02-16 Thread David Daney
On 02/16/2018 12:13 PM, Andy Shevchenko wrote: On Fri, Feb 16, 2018 at 9:44 PM, David Daney wrote: The ISL12026 is a combination RTC and EEPROM device with I2C interface. The standard RTC driver interface is provided. The EEPROM is accessed via the NVMEM interface via the "eeprom0" directory

Re: [PATCH v3 2/6] vfio/type1: Check reserve region conflict and update iova list

2018-02-16 Thread Alex Williamson
On Thu, 15 Feb 2018 09:45:00 + Shameer Kolothum wrote: > This retrieves the reserved regions associated with dev group and > checks for conflicts with any existing dma mappings. Also update > the iova list excluding the reserved regions. > > Signed-off-by: Shameer Kolothum > --- > drivers/

RE: [char-misc 4.16] mei: set device client to the disconnected state upon suspend.

2018-02-16 Thread Winkler, Tomas
> > On Wed, Feb 14, 2018 at 02:03:29PM +0200, Tomas Winkler wrote: > > This fixes regression introduced by > > commit 8d52af6795c0 ("mei: speed up the power down flow") > > In the future, put: > Fixes: 8d52af6795c0 ("mei: speed up the power down flow") > > in your signed-off-by area so tha

Re: [PATCH 0/3] Improve and extend checkpatch.pl Kconfig help text checks

2018-02-16 Thread Joe Perches
On Fri, 2018-02-16 at 21:22 +0100, Ulf Magnusson wrote: > Hello, > > This patchset contains some improvements for the Kconfig help text check in > scripts/checkconfig.pl: Seems sensible enough to me. Signed-off-by: Joe Perches > - Commits 1 and 2 make the check more robust by checking menuconf

RE: [PATCH 0/2] efivars: reading variables can generate SMIs

2018-02-16 Thread Luck, Tony
> That said, I'm not sure how many non-root users run the toolkit to > extract their EFI certificates or check on the secure boot status of > the system, but I suspect it might be non-zero: I can see the tinfoil > hat people wanting at least to check the secure boot status when they > log in. Anot

[PATCH v2 5/9] lib/vsprintf: Move pointer_string() upper

2018-02-16 Thread Andy Shevchenko
As preparatory patch to further clean up. No functional change. Signed-off-by: Andy Shevchenko --- lib/vsprintf.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/lib/vsprintf.c b/lib/vsprintf.c index c789d265311b..87dbced51b1a 100644 --- a/lib/

[PATCH v2 9/9] lib/vsprintf: Mark expected switch fall-through

2018-02-16 Thread Andy Shevchenko
In preparation to enabling -Wimplicit-fallthrough, mark switch cases where we are expecting to fall through. Signed-off-by: Andy Shevchenko --- lib/vsprintf.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lib/vsprintf.c b/lib/vsprintf.c index a49da00b79e7..28d7aca6a805 100644 --- a/lib/

[PATCH v2 3/9] lib/vsprintf: Make strspec global

2018-02-16 Thread Andy Shevchenko
There are places where default specification to print strings is in use. Make it global and convert existing users. Signed-off-by: Andy Shevchenko --- lib/vsprintf.c | 21 + 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/lib/vsprintf.c b/lib/vsprintf.c index

[PATCH v2 6/9] lib/vsprintf: Deduplicate pointer_string()

2018-02-16 Thread Andy Shevchenko
There is an exact code at the end of ptr_to_id(). Replace it by calling pointer_string() directly. This is followup to the commit ad67b74d2469 ("printk: hash addresses printed with %p"). Cc: Tobin C. Harding Signed-off-by: Andy Shevchenko --- lib/vsprintf.c | 27 +++

[PATCH v2 2/9] lib/vsprintf: Make dec_spec global

2018-02-16 Thread Andy Shevchenko
There are places where default specification to print decimal numbers is in use. Make it global and convert existing users. Signed-off-by: Andy Shevchenko --- lib/vsprintf.c | 21 + 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/lib/vsprintf.c b/lib/vsprintf.

[PATCH v2 1/9] lib/test_printf: Mark big constant with ULL

2018-02-16 Thread Andy Shevchenko
Sparse complains that constant is so bit for unsigned long on 64-bit architecture. lib/test_printf.c:217:54: warning: constant 0x0123456789ab is so big it is unsigned long lib/test_printf.c:246:54: warning: constant 0x0123456789ab is so big it is unsigned long To satisfy everyone, mark

[PATCH v2 7/9] lib/vsprintf: Replace space with '_' before crng is ready

2018-02-16 Thread Andy Shevchenko
From: Shunyong Yang Before crng is ready, output of "%p" composes of "(ptrval)" and left padding spaces for alignment as no random address can be generated. This seems a little strange when default string width is larger than strlen("(ptrval)"). For example, when irq domain names are built with

[PATCH v2 8/9] lib/vsprintf: Remove useless NULL checks

2018-02-16 Thread Andy Shevchenko
The pointer can't be NULL since it's first what has been done in the pointer(). Remove useless checks. Note we leave check for !CONFIG_HAVE_CLK to make compiler to optimize code away when possible. Cc: Petr Mladek Signed-off-by: Andy Shevchenko --- lib/vsprintf.c | 13 + 1 file ch

[PATCH v2 4/9] lib/vsprintf: Make flag_spec global

2018-02-16 Thread Andy Shevchenko
There are places where default specification to print flags as number is in use. Make it global and convert existing users. Signed-off-by: Andy Shevchenko --- lib/vsprintf.c | 21 - 1 file changed, 8 insertions(+), 13 deletions(-) diff --git a/lib/vsprintf.c b/lib/vsprintf.

Re: [RFC patch] checkpatch: test identifier lengths

2018-02-16 Thread Joe Perches
On Fri, 2018-02-16 at 12:31 -0800, Andrew Morton wrote: > On Fri, 16 Feb 2018 09:13:27 -0800 Joe Perches wrote: > > > On Fri, 2018-02-16 at 15:55 +0300, Dan Carpenter wrote: > > > On Fri, Feb 16, 2018 at 05:06:34PM +0530, Yash Omer wrote: > > > > This patch fix line should not end with open paren

Re: [PATCH 3/3] taint: Add taint for randstruct

2018-02-16 Thread Andrew Morton
On Thu, 15 Feb 2018 19:37:44 -0800 Kees Cook wrote: > --- a/Documentation/sysctl/kernel.txt > +++ b/Documentation/sysctl/kernel.txt > @@ -991,6 +991,7 @@ ORed together. The letters are seen in "Tainted" line of > Oops reports. > 16384 (L): A soft lockup has previously occurred on the system. >

Re: [PATCH 2/3] HID: steam: add serial number information.

2018-02-16 Thread Rodrigo Rivas Costa
On Fri, Feb 16, 2018 at 11:38:11AM +0100, Benjamin Tissoires wrote: > On Fri, Feb 16, 2018 at 10:57 AM, Rodrigo Rivas Costa > wrote: > > On Fri, Feb 16, 2018 at 10:31:35AM +0100, Benjamin Tissoires wrote: > >> > Ok, I'll do that. The weird thing, however, is that: > >> > > >> > hid_hw_raw_

Re: [PATCH 4.4 33/53] x86/apic/vector: Fix off by one in error path

2018-02-16 Thread Ben Hutchings
On Mon, 2018-01-22 at 09:40 +0100, Greg Kroah-Hartman wrote: > 4.4-stable review patch.  If anyone has any objections, please let me know. > > -- > > From: Thomas Gleixner > > commit 45d55e7bac4028af93f5fa324e69958a0b868e96 upstream. > > Keith reported the following warning: >

Re: [PATCH v3 33/43] drm/panel: simple: Change mode for Sharp lq123p1jx31

2018-02-16 Thread Doug Anderson
Hi, On Fri, Feb 16, 2018 at 4:34 AM, Enric Balletbo Serra wrote: > Hi, > > 2018-01-31 17:52 GMT+01:00 Doug Anderson : >> Hi, >> >> >> On Wed, Jan 31, 2018 at 7:16 AM, Sean Paul wrote: >>> On Wed, Jan 31, 2018 at 7:54 AM, Lucas Stach wrote: Am Dienstag, den 30.01.2018, 21:29 +0100 schrieb T

[PATCH] ARM: dts: sun7i: Enable HDMI support on the Orange Pi mini

2018-02-16 Thread Stefan Monnier
From: Stefan Monnier Enable the display pipeline and HDMI output for the Orange Pi mini Signed-off-by: Stefan Monnier --- arch/arm/boot/dts/sun7i-a20-orangepi-mini.dts | 25 + 1 file changed, 25 insertions(+) diff --git a/arch/arm/boot/dts/sun7i-a20-orangepi-mini.dts

Re: [PATCH 0/2] efivars: reading variables can generate SMIs

2018-02-16 Thread James Bottomley
On Fri, 2018-02-16 at 10:41 +, Ard Biesheuvel wrote: > On 15 February 2018 at 18:22, Joe Konno > wrote: > > > > From: Joe Konno > > > > It was pointed out that normal, unprivileged users reading certain > > EFI > > variables (through efivarfs) can generate SMIs. Given these nodes > > are cr

Re: [PATCH v3 1/6] vfio/type1: Introduce iova list and add iommu aperture validity check

2018-02-16 Thread Alex Williamson
On Thu, 15 Feb 2018 09:44:59 + Shameer Kolothum wrote: > This introduces an iova list that is valid for dma mappings. Make > sure the new iommu aperture window doesn't conflict with the current > one or with any existing dma mappings during attach. > > Signed-off-by: Shameer Kolothum > ---

Re: [PATCH 4.9 00/88] 4.9.82-stable review

2018-02-16 Thread Greg Kroah-Hartman
On Fri, Feb 16, 2018 at 12:39:23PM -0800, Guenter Roeck wrote: > On Fri, Feb 16, 2018 at 09:25:21PM +0100, Greg Kroah-Hartman wrote: > > On Fri, Feb 16, 2018 at 08:54:56PM +0100, Greg Kroah-Hartman wrote: > > > On Fri, Feb 16, 2018 at 08:21:17PM +0100, Greg Kroah-Hartman wrote: > > > > On Fri, Feb

Re: [RFC patch] checkpatch: test identifier lengths

2018-02-16 Thread Greg KH
On Fri, Feb 16, 2018 at 12:31:39PM -0800, Andrew Morton wrote: > On Fri, 16 Feb 2018 09:13:27 -0800 Joe Perches wrote: > > > On Fri, 2018-02-16 at 15:55 +0300, Dan Carpenter wrote: > > > On Fri, Feb 16, 2018 at 05:06:34PM +0530, Yash Omer wrote: > > > > This patch fix line should not end with ope

Re: [RESEND v2] mm: don't defer struct page initialization for Xen pv guests

2018-02-16 Thread Andrew Morton
On Fri, 16 Feb 2018 16:41:01 +0100 Juergen Gross wrote: > --- a/mm/page_alloc.c > +++ b/mm/page_alloc.c > @@ -347,6 +347,9 @@ static inline bool update_defer_init(pg_data_t *pgdat, > /* Always populate low zones for address-constrained allocations */ > if (zone_end < pgdat_end_pfn(pgd

Re: [PATCH v1 1/7] lib/test_printf: Mark big constant with ULL

2018-02-16 Thread Andy Shevchenko
On Fri, 2018-02-16 at 21:28 +0200, Andy Shevchenko wrote: > Sparse complains that constant is so bit for unsigned long on 64-bit > architecture. > > lib/test_printf.c:217:54: warning: constant 0x0123456789ab is so > big it is unsigned long > lib/test_printf.c:246:54: warning: constant 0x01

Re: [RESEND v2] mm: don't defer struct page initialization for Xen pv guests

2018-02-16 Thread Andrew Morton
On Fri, 16 Feb 2018 16:41:01 +0100 Juergen Gross wrote: > Commit f7f99100d8d95dbcf09e0216a143211e79418b9f ("mm: stop zeroing > memory during allocation in vmemmap") broke Xen pv domains in some > configurations, as the "Pinned" information in struct page of early > page tables could get lost. Thi

Re: [PATCH 1/1] perf: Add CPU hotplug support for events

2018-02-16 Thread Peter Zijlstra
On Fri, Feb 16, 2018 at 10:06:29AM -0800, Raghavendra Rao Ananta wrote: > > No this is absolutely disguisting. You can simply keep the events in the > > dead CPU's context. It's really not that hard. > Keeping the events in the dead CPU's context was also an idea that we had. > However, detaching t

Re: [PATCH 4.9 00/88] 4.9.82-stable review

2018-02-16 Thread Guenter Roeck
On Fri, Feb 16, 2018 at 09:25:21PM +0100, Greg Kroah-Hartman wrote: > On Fri, Feb 16, 2018 at 08:54:56PM +0100, Greg Kroah-Hartman wrote: > > On Fri, Feb 16, 2018 at 08:21:17PM +0100, Greg Kroah-Hartman wrote: > > > On Fri, Feb 16, 2018 at 06:19:46AM -0800, Guenter Roeck wrote: > > > > On 02/15/201

[PATCH v3 02/15] initramfs: replace states with function pointers

2018-02-16 Thread Taras Kondratiuk
Currently the FSM states are mapped directly to function pointers. Extra level of intirection is not needed and makes navigation over the code harder. One can't jump between states directly when browsing code (e.g. with cscope). Need to go through actions[] array each time. Replace states with the

[PATCH v3 04/15] initramfs: remove unnecessary symlinks processing shortcut

2018-02-16 Thread Taras Kondratiuk
Special handling of symlinks in do_header() assumes that name and body entries are sequential and reads them together. This shortcut has no real performance benefits, but it complicates changes to the state machine. Make handling of symlinks more similar to a regular files. Store name in name_buf

[PATCH v3 00/15] extend initramfs archive format to support xattrs

2018-02-16 Thread Taras Kondratiuk
Many of the Linux security/integrity features are dependent on file metadata, stored as extended attributes (xattrs), for making decisions. These features need to be initialized during initcall and enabled as early as possible for complete security coverage. Initramfs (tmpfs) supports xattrs, but

[PATCH v3 05/15] initramfs: move files creation into separate state

2018-02-16 Thread Taras Kondratiuk
Move most of the file creation logic into a separate state. This splits collection of data stage from data processing and makes it easier to add additional states for a new archive format. Signed-off-by: Taras Kondratiuk --- init/initramfs.c | 52 ++---

Re: [RFC PATCH] usb: hcd: complete URBs in threaded-IRQ context instead of tasklet

2018-02-16 Thread Sebastian Andrzej Siewior
On 2018-02-16 13:29:01 [-0500], Alan Stern wrote: > We originally used tasklets because we didn't want to incur the delays > associated with running in a process context. It seems odd to be > reversing that decision now. The theaded interrupt runs SCHED_FIFO priority 50 by default. The only thi

[PATCH v3 06/15] initramfs: separate reading cpio method from header

2018-02-16 Thread Taras Kondratiuk
From: Mimi Zohar In preparation for adding xattr support, read the CPIO method separately from the rest of the header. Signed-off-by: Mimi Zohar Signed-off-by: Taras Kondratiuk --- init/initramfs.c | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/init/initr

[PATCH v3 07/15] initramfs: split header layout information from parsing function

2018-02-16 Thread Taras Kondratiuk
Header parsing has hardcoded assumption about header field size and layout. It is hard to modify the function to parse a new format. Move information about size and layout into a data structure to make parsing code more generic and simplify adding a new format. This also removes some magic numbers

[PATCH v3 01/15] Documentation: add newcx initramfs format description

2018-02-16 Thread Taras Kondratiuk
Many of the Linux security/integrity features are dependent on file metadata, stored as extended attributes (xattrs), for making decisions. These features need to be initialized during initcall and enabled as early as possible for complete security coverage. Initramfs (tmpfs) supports xattrs, but

[PATCH v3 10/15] gen_init_cpio: move header formatting into function

2018-02-16 Thread Taras Kondratiuk
CPIO header is generated in multiple places with the same sprintf() format string. Move formatting into a single function in preparation to adding a new cpio format. Signed-off-by: Taras Kondratiuk --- usr/gen_init_cpio.c | 186 ++-- 1 file changed

[PATCH v3 03/15] initramfs: store file name in name_buf

2018-02-16 Thread Taras Kondratiuk
There is already name_buf buffer pre-allocated for a file name. No need to allocate vcollected for every file. More over a name can be already stored in name_buf by read_info() function. Add memcpy_optional() function to handle such case. Signed-off-by: Taras Kondratiuk --- init/initramfs.c | 1

[PATCH v3 09/15] initramfs: set extended attributes

2018-02-16 Thread Taras Kondratiuk
From: Mimi Zohar This patch writes out the extended attributes included in the cpio file. As the "security.ima" xattr needs to be written after the file data. this patch separates extracting and setting the xattrs by defining new do_setxattrs state. [kamensky: fixed restoring of xattrs for symbo

[PATCH v3 08/15] initramfs: add newcx format

2018-02-16 Thread Taras Kondratiuk
Add 'newcx' format that adds extended attributes and increased size of c_mtime and c_filesize fields. Refer to Documentation/early-userspace/buffer-format.txt for detailed format description. Signed-off-by: Taras Kondratiuk --- init/initramfs.c | 121 +++-

[PATCH v3 12/15] gen_init_cpio: set extended attributes for newcx format

2018-02-16 Thread Taras Kondratiuk
gen_init_cpio creates CPIO archive according to cpio_list manifest file that contains list of archive entries (one per line). To be able to store extended attributes in newcx CPIO format we need to pass them via cpio_list file. One way of doing it would be to append xattrs to each entry line, but

[PATCH v3 14/15] selinux: allow setxattr on rootfs so initramfs code can set them

2018-02-16 Thread Taras Kondratiuk
From: Victor Kamensky initramfs code supporting extended cpio format have ability to fill extended attributes from cpio archive, but if SELinux enabled and security server is not initialized yet, selinux callback would refuse setxattr made by initramfs code. Solution enable SBLABEL_MNT on rootfs

[PATCH v3 11/15] gen_init_cpio: add newcx format

2018-02-16 Thread Taras Kondratiuk
Add "newcx" format that supports extended attributes and has increased size of c_mtime and c_filesize fields. Added -x option to select "newcx" format. Default is "newc". Refer to Documentation/early-userspace/buffer-format.txt for detailed format description. Signed-off-by: Taras Kondratiuk --

[PATCH v3 14/14] selinux: delay sid population for rootfs till init is complete

2018-02-16 Thread Taras Kondratiuk
From: Victor Kamensky With initramfs cpio format that supports extended attributes we need to skip sid population on sys_lsetxattr call from initramfs for rootfs if security server is not initialized yet. Otherwise callback in selinux_inode_post_setxattr will try to translate give security.selin

[PATCH v3 15/15] selinux: delay sid population for rootfs till init is complete

2018-02-16 Thread Taras Kondratiuk
From: Victor Kamensky With initramfs cpio format that supports extended attributes we need to skip sid population on sys_lsetxattr call from initramfs for rootfs if security server is not initialized yet. Otherwise callback in selinux_inode_post_setxattr will try to translate give security.selin

[PATCH v3 13/15] gen_initramfs_list.sh: add -x option to enable newcx format

2018-02-16 Thread Taras Kondratiuk
From: Mimi Zohar -x option populates extended attributes in cpio_list file passed to get_init_cpio and selects newcx CPIO format. Signed-off-by: Mimi Zohar Signed-off-by: Taras Kondratiuk --- scripts/gen_initramfs_list.sh | 13 - usr/Kconfig | 11 +++ usr

[PATCH v3 12/14] gen_initramfs_list.sh: add -x option to enable newcx format

2018-02-16 Thread Taras Kondratiuk
From: Mimi Zohar -x option populates extended attributes in cpio_list file passed to get_init_cpio and selects newcx CPIO format. Signed-off-by: Mimi Zohar Signed-off-by: Taras Kondratiuk --- scripts/gen_initramfs_list.sh | 13 - usr/Kconfig | 11 +++ usr

Re: [PATCH v3 08/11] watchdog/hpwdt: Programable Pretimeout NMI

2018-02-16 Thread Guenter Roeck
On Thu, Feb 15, 2018 at 04:43:57PM -0700, Jerry Hoemann wrote: > Make whether or not the hpwdt watchdog delivers a pretimeout NMI > programable by the user. > > The underlying iLO hardware is programmable as to whether or not > a pre-timeout NMI is delivered to the system before the iLO resets > t

Re: [PATCH] PCI: Add quirk for Cavium Thunder-X2 PCIe erratum #173

2018-02-16 Thread Bjorn Helgaas
On Fri, Feb 16, 2018 at 01:40:37PM +0100, Rafael J. Wysocki wrote: > On Friday, February 16, 2018 12:39:00 AM CET Bjorn Helgaas wrote: > > On Thu, Feb 15, 2018 at 10:57:25PM +0100, Rafael J. Wysocki wrote: > > > On Wednesday, February 14, 2018 9:16:53 PM CET Bjorn Helgaas wrote: > > > > On Wed, Feb

[PATCH v3 13/14] selinux: allow setxattr on rootfs so initramfs code can set them

2018-02-16 Thread Taras Kondratiuk
From: Victor Kamensky initramfs code supporting extended cpio format have ability to fill extended attributes from cpio archive, but if SELinux enabled and security server is not initialized yet, selinux callback would refuse setxattr made by initramfs code. Solution enable SBLABEL_MNT on rootfs

Re: [RFC patch] checkpatch: test identifier lengths

2018-02-16 Thread Andrew Morton
On Fri, 16 Feb 2018 09:13:27 -0800 Joe Perches wrote: > On Fri, 2018-02-16 at 15:55 +0300, Dan Carpenter wrote: > > On Fri, Feb 16, 2018 at 05:06:34PM +0530, Yash Omer wrote: > > > This patch fix line should not end with open parenthesis found by > > > checkpatch.plscript. > > > > > > Signed-of

Re: [PATCH 4.9 00/88] 4.9.82-stable review

2018-02-16 Thread Greg Kroah-Hartman
On Fri, Feb 16, 2018 at 08:54:56PM +0100, Greg Kroah-Hartman wrote: > On Fri, Feb 16, 2018 at 08:21:17PM +0100, Greg Kroah-Hartman wrote: > > On Fri, Feb 16, 2018 at 06:19:46AM -0800, Guenter Roeck wrote: > > > On 02/15/2018 07:16 AM, Greg Kroah-Hartman wrote: > > > > This is the start of the stabl

[PATCH 2/3] checkpatch: kconfig: check help texts for menuconfig and choice

2018-02-16 Thread Ulf Magnusson
Currently, only Kconfig symbols are checked for a missing or short help text, and are only checked if they are defined with the 'config' keyword. To make the check more general, extend it to also check help texts for choices and for symbols defined with the 'menuconfig' keyword. This increases th

[PATCH 3/3] checkpatch: kconfig: prefer 'help' over '---help---'

2018-02-16 Thread Ulf Magnusson
IMO, we should discourage '---help---' for new help texts, even in cases where it would be consistent with other help texts in the file. This will help if we ever want to get rid of '---help---' in the future. Also simplify the code to only check for exactly '---help---'. Since commit c2264564df3d

[PATCH 1/3] checkpatch: kconfig: recognize more prompts when checking help texts

2018-02-16 Thread Ulf Magnusson
The check for a missing or short help text only considers symbols with a prompt, but doesn't recognize any of the following as a prompt: bool 'foo' tristate 'foo' prompt "foo" prompt 'foo' Make the check recognize those too. Signed-off-by: Ulf Magnusson --- scri

[PATCH 0/3] Improve and extend checkpatch.pl Kconfig help text checks

2018-02-16 Thread Ulf Magnusson
Hello, This patchset contains some improvements for the Kconfig help text check in scripts/checkconfig.pl: - Commits 1 and 2 make the check more robust by checking menuconfig symbols and choices as well as symbols defined with 'config', and by making the detection of definition boundaries

[PATCH v2] staging: android: ion: Initialize dma_address of new sg list

2018-02-16 Thread Liam Mark
Fix the dup_sg_table function to initialize the dma_address of the new sg list entries instead of the source dma_address entries. Since ION duplicates the sg_list this issue does not appear to result in an actual bug. Signed-off-by: Liam Mark Acked-by: Laura Abbott --- Changes in v2: - Add to

Re: [PATCH] drm/sun4i: fix HSYNC and VSYNC polarity

2018-02-16 Thread Maxime Ripard
On Thu, Feb 15, 2018 at 06:54:48PM +0100, Giulio Benetti wrote: > Differently from other Lcd signals, HSYNC and VSYNC signals > result inverted if their bits are cleared to 0. > > Invert their settings of IO_POL register. > > Signed-off-by: Giulio Benetti Applied, thanks! Maxime -- Maxime Rip

Re: [PATCH v4] rtc: isl12026: Add driver.

2018-02-16 Thread Andy Shevchenko
On Fri, Feb 16, 2018 at 9:44 PM, David Daney wrote: > The ISL12026 is a combination RTC and EEPROM device with I2C > interface. The standard RTC driver interface is provided. The EEPROM > is accessed via the NVMEM interface via the "eeprom0" directory in the > sysfs entry for the device. > +con

Re: [PATCH 3.18 00/45] 3.18.95-stable review

2018-02-16 Thread Greg Kroah-Hartman
On Fri, Feb 16, 2018 at 12:07:32PM -0800, Kevin Hilman wrote: > kernelci.org bot writes: > > > stable-rc/linux-3.18.y boot: 63 boots: 2 failed, 51 passed with 5 offline, > > 5 conflicts (v3.18.94-46-g72255f521206) > > > > Full Boot Summary: > > https://kernelci.org/boot/all/job/stable-rc/branch

Re: [PATCH 2/3] x86/mm: introduce __PAGE_KERNEL_GLOBAL

2018-02-16 Thread Dave Hansen
On 02/16/2018 11:54 AM, Nadav Amit wrote: >> But I don't really want to hide that gunk in a macro like that. It >> might make more sense as a static inline. I'll give that a shot and resent. > Since determining whether PTI is on is done in several places in the kernel, > maybe there should a sing

Re: [PATCH 4.9 00/88] 4.9.82-stable review

2018-02-16 Thread Greg Kroah-Hartman
On Fri, Feb 16, 2018 at 08:21:17PM +0100, Greg Kroah-Hartman wrote: > On Fri, Feb 16, 2018 at 06:19:46AM -0800, Guenter Roeck wrote: > > On 02/15/2018 07:16 AM, Greg Kroah-Hartman wrote: > > > This is the start of the stable review cycle for the 4.9.82 release. > > > There are 88 patches in this se

<    1   2   3   4   5   6   7   8   9   >