Re: [PATCH linux v2 5/6] hwmon: occ: Add hwmon implementation for the P8 OCC

2017-01-15 Thread Guenter Roeck
On 01/11/2017 10:10 AM, eajames@gmail.com wrote: From: "Edward A. James" Add code to tie the hwmon sysfs code and the POWER8 OCC code together, as well as probe the entire driver from the I2C bus. I2C is the communication method between the BMC and the P8 OCC.

Re: [PATCH linux v2 5/6] hwmon: occ: Add hwmon implementation for the P8 OCC

2017-01-15 Thread Guenter Roeck
On 01/11/2017 10:10 AM, eajames@gmail.com wrote: From: "Edward A. James" Add code to tie the hwmon sysfs code and the POWER8 OCC code together, as well as probe the entire driver from the I2C bus. I2C is the communication method between the BMC and the P8 OCC. Signed-off-by: Edward A.

Re: [PATCH v3 0/5] MIPS: Add per-cpu IRQ stack

2017-01-15 Thread Jason A. Donenfeld
FYI, LEDE/OpenWRT is shipping this patchset now: https://github.com/lede-project/source/commit/1708644f1915eb7587a904d81da0ef0b559d1567

Re: [PATCH v3 0/5] MIPS: Add per-cpu IRQ stack

2017-01-15 Thread Jason A. Donenfeld
FYI, LEDE/OpenWRT is shipping this patchset now: https://github.com/lede-project/source/commit/1708644f1915eb7587a904d81da0ef0b559d1567

Re: [PATCH net-next v3 06/10] net: dsa: Migrate to device_find_class()

2017-01-15 Thread Florian Fainelli
On 01/15/2017 09:39 AM, Greg KH wrote: > On Sun, Jan 15, 2017 at 09:27:22AM -0800, Florian Fainelli wrote: >> >> >> On 01/15/2017 03:06 AM, Greg KH wrote: >>> On Sat, Jan 14, 2017 at 01:47:09PM -0800, Florian Fainelli wrote: Now that the base device driver code provides an identical

Re: [PATCH net-next v3 06/10] net: dsa: Migrate to device_find_class()

2017-01-15 Thread Florian Fainelli
On 01/15/2017 09:39 AM, Greg KH wrote: > On Sun, Jan 15, 2017 at 09:27:22AM -0800, Florian Fainelli wrote: >> >> >> On 01/15/2017 03:06 AM, Greg KH wrote: >>> On Sat, Jan 14, 2017 at 01:47:09PM -0800, Florian Fainelli wrote: Now that the base device driver code provides an identical

Re: [PATCH net-next v3 00/10] net: dsa: Support for pdata in dsa2

2017-01-15 Thread Greg KH
On Sun, Jan 15, 2017 at 09:40:24AM -0800, Florian Fainelli wrote: > On 01/15/2017 03:08 AM, Greg KH wrote: > > On Sat, Jan 14, 2017 at 01:47:03PM -0800, Florian Fainelli wrote: > >> Hi all, > >> > >> This is not exactly new, and was sent before, although back then, I did not > >> have an user of

Re: [PATCH net-next v3 00/10] net: dsa: Support for pdata in dsa2

2017-01-15 Thread Greg KH
On Sun, Jan 15, 2017 at 09:40:24AM -0800, Florian Fainelli wrote: > On 01/15/2017 03:08 AM, Greg KH wrote: > > On Sat, Jan 14, 2017 at 01:47:03PM -0800, Florian Fainelli wrote: > >> Hi all, > >> > >> This is not exactly new, and was sent before, although back then, I did not > >> have an user of

Re: [PATCH resend] swiotlb: ensure that page-sized mappings are page-aligned

2017-01-15 Thread Konrad Rzeszutek Wilk
On Wed, Jan 11, 2017 at 09:56:31PM +0300, Nikita Yushchenko wrote: > Some drivers do depend on page mappings to be page aligned. > > Swiotlb already enforces such alignment for mappings greater than page, > extend that to page-sized mappings as well. > > Without this fix, nvme hits BUG() in

Re: [PATCH resend] swiotlb: ensure that page-sized mappings are page-aligned

2017-01-15 Thread Konrad Rzeszutek Wilk
On Wed, Jan 11, 2017 at 09:56:31PM +0300, Nikita Yushchenko wrote: > Some drivers do depend on page mappings to be page aligned. > > Swiotlb already enforces such alignment for mappings greater than page, > extend that to page-sized mappings as well. > > Without this fix, nvme hits BUG() in

Re: [PATCH net-next v3 00/10] net: dsa: Support for pdata in dsa2

2017-01-15 Thread Florian Fainelli
On 01/15/2017 03:08 AM, Greg KH wrote: > On Sat, Jan 14, 2017 at 01:47:03PM -0800, Florian Fainelli wrote: >> Hi all, >> >> This is not exactly new, and was sent before, although back then, I did not >> have an user of the pre-declared MDIO board information, but now we do. Note >> that I have

Re: [PATCH net-next v3 07/10] net: Relocate dev_to_net_device() into core

2017-01-15 Thread Greg KH
On Sun, Jan 15, 2017 at 09:20:06AM -0800, Florian Fainelli wrote: > > > On 01/15/2017 03:07 AM, Greg KH wrote: > > On Sat, Jan 14, 2017 at 01:47:10PM -0800, Florian Fainelli wrote: > >> dev_to_net_device() is moved from net/dsa/dsa.c to net/core/dev.c since > >> it going to be used by

Re: [PATCH net-next v3 00/10] net: dsa: Support for pdata in dsa2

2017-01-15 Thread Florian Fainelli
On 01/15/2017 03:08 AM, Greg KH wrote: > On Sat, Jan 14, 2017 at 01:47:03PM -0800, Florian Fainelli wrote: >> Hi all, >> >> This is not exactly new, and was sent before, although back then, I did not >> have an user of the pre-declared MDIO board information, but now we do. Note >> that I have

Re: [PATCH net-next v3 07/10] net: Relocate dev_to_net_device() into core

2017-01-15 Thread Greg KH
On Sun, Jan 15, 2017 at 09:20:06AM -0800, Florian Fainelli wrote: > > > On 01/15/2017 03:07 AM, Greg KH wrote: > > On Sat, Jan 14, 2017 at 01:47:10PM -0800, Florian Fainelli wrote: > >> dev_to_net_device() is moved from net/dsa/dsa.c to net/core/dev.c since > >> it going to be used by

Re: [PATCH net-next v3 06/10] net: dsa: Migrate to device_find_class()

2017-01-15 Thread Greg KH
On Sun, Jan 15, 2017 at 09:27:22AM -0800, Florian Fainelli wrote: > > > On 01/15/2017 03:06 AM, Greg KH wrote: > > On Sat, Jan 14, 2017 at 01:47:09PM -0800, Florian Fainelli wrote: > >> Now that the base device driver code provides an identical > >> implementation of dev_find_class() utilize

Re: [PATCH net-next v3 06/10] net: dsa: Migrate to device_find_class()

2017-01-15 Thread Greg KH
On Sun, Jan 15, 2017 at 09:27:22AM -0800, Florian Fainelli wrote: > > > On 01/15/2017 03:06 AM, Greg KH wrote: > > On Sat, Jan 14, 2017 at 01:47:09PM -0800, Florian Fainelli wrote: > >> Now that the base device driver code provides an identical > >> implementation of dev_find_class() utilize

Re: [PATCH net-next v3 05/10] drivers: base: Add device_find_class()

2017-01-15 Thread Florian Fainelli
On 01/15/2017 03:04 AM, Greg KH wrote: > On Sat, Jan 14, 2017 at 01:47:08PM -0800, Florian Fainelli wrote: >> Add a helper function to lookup a device reference given a class name. >> This is a preliminary patch to remove adhoc code from net/dsa/dsa.c and >> make it more generic. >> >>

[PATCH] ASoC: tegra: constify tegra30_ahub_soc_data structures

2017-01-15 Thread Bhumika Goyal
The tegra30_ahub_soc_data structures are only stored in the data field of of_device_id structures. This field is of type const, so tegra30_ahub_soc_data structures having the same property can be declared const too. File size before: sound/soc/tegra/tegra30_ahub.o textdata bss dec

Re: [PATCH net-next v3 05/10] drivers: base: Add device_find_class()

2017-01-15 Thread Florian Fainelli
On 01/15/2017 03:04 AM, Greg KH wrote: > On Sat, Jan 14, 2017 at 01:47:08PM -0800, Florian Fainelli wrote: >> Add a helper function to lookup a device reference given a class name. >> This is a preliminary patch to remove adhoc code from net/dsa/dsa.c and >> make it more generic. >> >>

[PATCH] ASoC: tegra: constify tegra30_ahub_soc_data structures

2017-01-15 Thread Bhumika Goyal
The tegra30_ahub_soc_data structures are only stored in the data field of of_device_id structures. This field is of type const, so tegra30_ahub_soc_data structures having the same property can be declared const too. File size before: sound/soc/tegra/tegra30_ahub.o textdata bss dec

sctp: kernel memory overwrite attempt detected in sctp_getsockopt_assoc_stats

2017-01-15 Thread Dmitry Vyukov
Hello, I've enabled CONFIG_HARDENED_USERCOPY_PAGESPAN on syzkaller fuzzer and now I am seeing lots of: usercopy: kernel memory overwrite attempt detected to 8801a74f6f10 () (256 bytes) kernel BUG at mm/usercopy.c:75! invalid opcode: [#1] SMP KASAN Dumping ftrace buffer: (ftrace

sctp: kernel memory overwrite attempt detected in sctp_getsockopt_assoc_stats

2017-01-15 Thread Dmitry Vyukov
Hello, I've enabled CONFIG_HARDENED_USERCOPY_PAGESPAN on syzkaller fuzzer and now I am seeing lots of: usercopy: kernel memory overwrite attempt detected to 8801a74f6f10 () (256 bytes) kernel BUG at mm/usercopy.c:75! invalid opcode: [#1] SMP KASAN Dumping ftrace buffer: (ftrace

Re: [PATCH net-next v3 06/10] net: dsa: Migrate to device_find_class()

2017-01-15 Thread Florian Fainelli
On 01/15/2017 03:06 AM, Greg KH wrote: > On Sat, Jan 14, 2017 at 01:47:09PM -0800, Florian Fainelli wrote: >> Now that the base device driver code provides an identical >> implementation of dev_find_class() utilize device_find_class() instead >> of our own version of it. >> >> Signed-off-by:

Re: [PATCH net-next v3 06/10] net: dsa: Migrate to device_find_class()

2017-01-15 Thread Florian Fainelli
On 01/15/2017 03:06 AM, Greg KH wrote: > On Sat, Jan 14, 2017 at 01:47:09PM -0800, Florian Fainelli wrote: >> Now that the base device driver code provides an identical >> implementation of dev_find_class() utilize device_find_class() instead >> of our own version of it. >> >> Signed-off-by:

Re: [PATCH net-next v3 07/10] net: Relocate dev_to_net_device() into core

2017-01-15 Thread Florian Fainelli
On 01/15/2017 03:07 AM, Greg KH wrote: > On Sat, Jan 14, 2017 at 01:47:10PM -0800, Florian Fainelli wrote: >> dev_to_net_device() is moved from net/dsa/dsa.c to net/core/dev.c since >> it going to be used by net/dsa/dsa2.c and the namespace of the function >> justifies making it available to

Re: [PATCH net-next v3 07/10] net: Relocate dev_to_net_device() into core

2017-01-15 Thread Florian Fainelli
On 01/15/2017 03:07 AM, Greg KH wrote: > On Sat, Jan 14, 2017 at 01:47:10PM -0800, Florian Fainelli wrote: >> dev_to_net_device() is moved from net/dsa/dsa.c to net/core/dev.c since >> it going to be used by net/dsa/dsa2.c and the namespace of the function >> justifies making it available to

[tip:timers/core 1/1] arch/tile/include/asm/div64.h:5:1: error: unknown type name 'u64'

2017-01-15 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git timers/core head: 9e3d6223d2093a8903c8f570a06284453ee59944 commit: 9e3d6223d2093a8903c8f570a06284453ee59944 [1/1] math64, timers: Fix 32bit mul_u64_u32_shr() and friends config: tile-allmodconfig (attached as .config) compiler:

[tip:timers/core 1/1] arch/tile/include/asm/div64.h:5:1: error: unknown type name 'u64'

2017-01-15 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git timers/core head: 9e3d6223d2093a8903c8f570a06284453ee59944 commit: 9e3d6223d2093a8903c8f570a06284453ee59944 [1/1] math64, timers: Fix 32bit mul_u64_u32_shr() and friends config: tile-allmodconfig (attached as .config) compiler:

Re: kvm: use-after-free in process_srcu

2017-01-15 Thread Dmitry Vyukov
On Fri, Jan 13, 2017 at 10:19 AM, Dmitry Vyukov wrote: > On Fri, Jan 13, 2017 at 4:30 AM, Steve Rutherford > wrote: >> I'm not that familiar with the kernel's workqueues, but this seems >> like the classic "callback outlives the memory it references"

Re: kvm: use-after-free in process_srcu

2017-01-15 Thread Dmitry Vyukov
On Fri, Jan 13, 2017 at 10:19 AM, Dmitry Vyukov wrote: > On Fri, Jan 13, 2017 at 4:30 AM, Steve Rutherford > wrote: >> I'm not that familiar with the kernel's workqueues, but this seems >> like the classic "callback outlives the memory it references" >> use-after-free, where the process_srcu

Re: [PATCH] scsi: mpt3sas: fix hang on ata passthru commands

2017-01-15 Thread James Bottomley
On Tue, 2017-01-03 at 15:46 -0500, Jason Baron wrote: > On 01/01/2017 12:39 PM, James Bottomley wrote: > > + /* > > +* Bug work around for firmware SATL handling > > +*/ > > + if (sas_device_priv_data->ata_command_pending) { > > + scmd->result = SAM_STAT_BUSY; > > +

Re: [PATCH] scsi: mpt3sas: fix hang on ata passthru commands

2017-01-15 Thread James Bottomley
On Tue, 2017-01-03 at 15:46 -0500, Jason Baron wrote: > On 01/01/2017 12:39 PM, James Bottomley wrote: > > + /* > > +* Bug work around for firmware SATL handling > > +*/ > > + if (sas_device_priv_data->ata_command_pending) { > > + scmd->result = SAM_STAT_BUSY; > > +

Re: [PATCH] Revert "scsi: mpt3sas: Fix secure erase premature termination"

2017-01-15 Thread James Bottomley
On Sun, 2017-01-15 at 10:19 +0100, Ingo Molnar wrote: > So there's a new mpt3sas SCSI driver boot regression, introduced in > this merge window, which made one of my servers unbootable. We're not reverting a fix that would cause regressions for others. However, The fix was manifestly wrong, so

Re: [PATCH] Revert "scsi: mpt3sas: Fix secure erase premature termination"

2017-01-15 Thread James Bottomley
On Sun, 2017-01-15 at 10:19 +0100, Ingo Molnar wrote: > So there's a new mpt3sas SCSI driver boot regression, introduced in > this merge window, which made one of my servers unbootable. We're not reverting a fix that would cause regressions for others. However, The fix was manifestly wrong, so

Dropping device tree pinmux nodes for GPIO usage (Was: [PATCH 3/5] arm64: dts: sun50i: add MMC nodes)

2017-01-15 Thread Rask Ingemann Lambertsen
On Tue, Jan 03, 2017 at 10:52:12AM +0800, Chen-Yu Tsai wrote: > On Tue, Jan 3, 2017 at 7:03 AM, Andre Przywara wrote: > > + > > + mmc0_default_cd_pin: mmc0_cd_pin@0 { > > + pins = "PF6"; > > +

Dropping device tree pinmux nodes for GPIO usage (Was: [PATCH 3/5] arm64: dts: sun50i: add MMC nodes)

2017-01-15 Thread Rask Ingemann Lambertsen
On Tue, Jan 03, 2017 at 10:52:12AM +0800, Chen-Yu Tsai wrote: > On Tue, Jan 3, 2017 at 7:03 AM, Andre Przywara wrote: > > + > > + mmc0_default_cd_pin: mmc0_cd_pin@0 { > > + pins = "PF6"; > > + function = "gpio_in";

Re: [PATCHSET v6] blk-mq scheduling framework

2017-01-15 Thread Jens Axboe
On 01/15/2017 03:12 AM, Paolo Valente wrote: > >> Il giorno 11 gen 2017, alle ore 22:39, Jens Axboe ha scritto: >> >> Another year, another posting of this patchset. The previous posting >> was here: >> >> https://www.spinics.net/lists/kernel/msg2406106.html >> >> (yes, I've

Re: [PATCHSET v6] blk-mq scheduling framework

2017-01-15 Thread Jens Axboe
On 01/15/2017 03:12 AM, Paolo Valente wrote: > >> Il giorno 11 gen 2017, alle ore 22:39, Jens Axboe ha scritto: >> >> Another year, another posting of this patchset. The previous posting >> was here: >> >> https://www.spinics.net/lists/kernel/msg2406106.html >> >> (yes, I've skipped v5, it was

Re: [PATCH v9 3/3] serial: 8250_pci: remove exar code

2017-01-15 Thread Andy Shevchenko
On Sun, Jan 15, 2017 at 12:50 AM, Sudip Mukherjee wrote: > From: Sudip Mukherjee > > Remove the Exar specific codes from 8250_pci and blacklist those chips > so that the new Exar serial driver binds to the devices. > Reviewed-by: Andy

Re: [PATCH v9 3/3] serial: 8250_pci: remove exar code

2017-01-15 Thread Andy Shevchenko
On Sun, Jan 15, 2017 at 12:50 AM, Sudip Mukherjee wrote: > From: Sudip Mukherjee > > Remove the Exar specific codes from 8250_pci and blacklist those chips > so that the new Exar serial driver binds to the devices. > Reviewed-by: Andy Shevchenko > Signed-off-by: Sudip Mukherjee > --- >

[PATCH 46/46] selinuxfs: Use seq_puts() in sel_avc_stats_seq_show()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 14:04:53 +0100 A string which did not contain data format specifications should be put into a sequence. Thus use the corresponding function "seq_puts". This issue was detected by using the Coccinelle software.

[PATCH 46/46] selinuxfs: Use seq_puts() in sel_avc_stats_seq_show()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 14:04:53 +0100 A string which did not contain data format specifications should be put into a sequence. Thus use the corresponding function "seq_puts". This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring ---

Re: [PATCH v9 3/3] serial: 8250_pci: remove exar code

2017-01-15 Thread Andy Shevchenko
On Sun, Jan 15, 2017 at 12:50 AM, Sudip Mukherjee wrote: > From: Sudip Mukherjee > > Remove the Exar specific codes from 8250_pci and blacklist those chips > so that the new Exar serial driver binds to the devices. > #define

Re: [PATCH v9 3/3] serial: 8250_pci: remove exar code

2017-01-15 Thread Andy Shevchenko
On Sun, Jan 15, 2017 at 12:50 AM, Sudip Mukherjee wrote: > From: Sudip Mukherjee > > Remove the Exar specific codes from 8250_pci and blacklist those chips > so that the new Exar serial driver binds to the devices. > #define UART_EXAR_MPIOINT_7_0 0x8f/* MPIOINT[7:0] */ > #define

[PATCH 45/46] selinux: Use common error handling code in sidtab_insert()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 13:45:45 +0100 Add a jump target so that a bit of exception handling can be better reused at the end of this function. Signed-off-by: Markus Elfring --- security/selinux/ss/sidtab.c | 18

[PATCH 45/46] selinux: Use common error handling code in sidtab_insert()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 13:45:45 +0100 Add a jump target so that a bit of exception handling can be better reused at the end of this function. Signed-off-by: Markus Elfring --- security/selinux/ss/sidtab.c | 18 -- 1 file changed, 8 insertions(+), 10

Re: [RFT PATCH] ARM64: dts: meson-gxbb: Add reserved memory zone and usable memory range

2017-01-15 Thread Andreas Färber
Am 23.12.2016 um 10:42 schrieb Heinrich Schuchardt: > it really makes a difference if we write > > memory@0 { > device_type = "memory"; > linux,usable-memory = <0x0 0x100 0x0 0x7f00>; > }; > > or > > memory@0 { > device_type =

[PATCH 44/46] selinux: Adjust two checks for null pointers

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 13:30:20 +0100 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script "checkpatch.pl" pointed information out like the following. Comparison to NULL could be written !… Thus

Re: [RFT PATCH] ARM64: dts: meson-gxbb: Add reserved memory zone and usable memory range

2017-01-15 Thread Andreas Färber
Am 23.12.2016 um 10:42 schrieb Heinrich Schuchardt: > it really makes a difference if we write > > memory@0 { > device_type = "memory"; > linux,usable-memory = <0x0 0x100 0x0 0x7f00>; > }; > > or > > memory@0 { > device_type =

[PATCH 44/46] selinux: Adjust two checks for null pointers

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 13:30:20 +0100 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script "checkpatch.pl" pointed information out like the following. Comparison to NULL could be written !… Thus fix affected source code

[PATCH 43/46] selinux: Use kmalloc_array() in sidtab_init()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 13:13:19 +0100 A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". This issue was detected

[PATCH 43/46] selinux: Use kmalloc_array() in sidtab_init()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 13:13:19 +0100 A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". This issue was detected by using the Coccinelle

[PATCH 42/46] selinux: One function call less in roles_init() after error detection

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 12:56:54 +0100 The kfree() function was called in up to two cases by the roles_init() function during error handling even if the passed variable contained a null pointer. * Adjust a jump target according to the Linux

[PATCH 42/46] selinux: One function call less in roles_init() after error detection

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 12:56:54 +0100 The kfree() function was called in up to two cases by the roles_init() function during error handling even if the passed variable contained a null pointer. * Adjust a jump target according to the Linux coding style convention. * Delete

[PATCH 41/46] selinux: Move two assignments for the variable "rc" in roles_init()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 12:40:35 +0100 One local variable was set to an error code in two cases before a concrete error situation was detected. Thus move the corresponding assignments into if branches to indicate a software failure there.

[PATCH 41/46] selinux: Move two assignments for the variable "rc" in roles_init()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 12:40:35 +0100 One local variable was set to an error code in two cases before a concrete error situation was detected. Thus move the corresponding assignments into if branches to indicate a software failure there. Signed-off-by: Markus Elfring ---

Re: [PATCH v9 2/3] serial: exar: split out the exar code from 8250_pci

2017-01-15 Thread Andy Shevchenko
On Sun, Jan 15, 2017 at 12:50 AM, Sudip Mukherjee wrote: > From: Sudip Mukherjee > > Add the serial driver for the Exar chips. And also register the > platform device for the GPIO provided by the Exar chips. Thanks! Still v10 is

Re: [PATCH v9 2/3] serial: exar: split out the exar code from 8250_pci

2017-01-15 Thread Andy Shevchenko
On Sun, Jan 15, 2017 at 12:50 AM, Sudip Mukherjee wrote: > From: Sudip Mukherjee > > Add the serial driver for the Exar chips. And also register the > platform device for the GPIO provided by the Exar chips. Thanks! Still v10 is needed for my opinion. See my comments below. (Telling ahead, I

[PATCH 40/46] selinux: Return directly after a failed kzalloc() in roles_init()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 12:10:09 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1

[PATCH 40/46] selinux: Return directly after a failed kzalloc() in roles_init()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 12:10:09 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH 39/46] selinux: Move two assignments for the variable "rc" in ocontext_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 11:30:12 +0100 One local variable was set to an error code in two cases before a concrete error situation was detected. Thus move the corresponding assignments into if branches to indicate a software failure there.

[PATCH 39/46] selinux: Move two assignments for the variable "rc" in ocontext_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 11:30:12 +0100 One local variable was set to an error code in two cases before a concrete error situation was detected. Thus move the corresponding assignments into if branches to indicate a software failure there. Signed-off-by: Markus Elfring ---

[PATCH 38/46] selinux: One function call less in five functions after null pointer detection

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 11:28:02 +0100 Call the function "kfree" at the end only after it was determined that the passed parameter contained a non-null pointer. Signed-off-by: Markus Elfring ---

[PATCH 38/46] selinux: One function call less in five functions after null pointer detection

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 11:28:02 +0100 Call the function "kfree" at the end only after it was determined that the passed parameter contained a non-null pointer. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 10 +- 1 file changed, 5

[PATCH 37/46] selinux: Move an assignment for the variable "rc" in policydb_load_isids()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 11:24:51 +0100 A local variable was set to an error code in one case before a concrete error situation was detected. Thus move the corresponding assignment into an if branch to indicate a software failure there.

[PATCH 37/46] selinux: Move an assignment for the variable "rc" in policydb_load_isids()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 11:24:51 +0100 A local variable was set to an error code in one case before a concrete error situation was detected. Thus move the corresponding assignment into an if branch to indicate a software failure there. Signed-off-by: Markus Elfring ---

[PATCH 36/46] selinux: Move an assignment for the variable "rc" in mls_read_range_helper()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 11:22:23 +0100 A local variable was set to an error code in one case before a concrete error situation was detected. Thus move the corresponding assignment into an if branch to indicate a software failure there.

[PATCH 36/46] selinux: Move an assignment for the variable "rc" in mls_read_range_helper()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 11:22:23 +0100 A local variable was set to an error code in one case before a concrete error situation was detected. Thus move the corresponding assignment into an if branch to indicate a software failure there. Signed-off-by: Markus Elfring ---

[PATCH 35/46] selinux: Return directly after a failed kzalloc() in perm_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 11:20:13 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1

[PATCH 35/46] selinux: Return directly after a failed kzalloc() in perm_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 11:20:13 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH 34/46] selinux: Return directly after a failed kzalloc() in common_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 11:15:19 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1

[PATCH 34/46] selinux: Return directly after a failed kzalloc() in common_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 11:15:19 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH 33/46] selinux: Move an assignment for the variable "rc" in class_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 11:10:39 +0100 A local variable was set to an error code in one case before a concrete error situation was detected. Thus move the corresponding assignment into an if branch to indicate a software failure there.

[PATCH 33/46] selinux: Move an assignment for the variable "rc" in class_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 15 Jan 2017 11:10:39 +0100 A local variable was set to an error code in one case before a concrete error situation was detected. Thus move the corresponding assignment into an if branch to indicate a software failure there. Signed-off-by: Markus Elfring ---

[PATCH 32/46] selinux: Return directly after a failed kzalloc() in class_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 22:30:51 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1

[PATCH 32/46] selinux: Return directly after a failed kzalloc() in class_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 22:30:51 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH 31/46] selinux: Move an assignment for the variable "rc" in role_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 22:25:20 +0100 A local variable was set to an error code in one case before a concrete error situation was detected. Thus move the corresponding assignment into an if branch to indicate a software failure there.

[PATCH 31/46] selinux: Move an assignment for the variable "rc" in role_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 22:25:20 +0100 A local variable was set to an error code in one case before a concrete error situation was detected. Thus move the corresponding assignment into an if branch to indicate a software failure there. Signed-off-by: Markus Elfring ---

[PATCH 30/46] selinux: Return directly after a failed kzalloc() in role_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 22:20:25 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1

[PATCH 30/46] selinux: Return directly after a failed kzalloc() in role_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 22:20:25 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH 29/46] selinux: Return directly after a failed kzalloc() in type_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 22:15:54 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1

[PATCH 29/46] selinux: Return directly after a failed kzalloc() in type_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 22:15:54 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH 28/46] selinux: Return directly after a failed kzalloc() in user_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 22:08:22 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1

[PATCH 28/46] selinux: Return directly after a failed kzalloc() in user_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 22:08:22 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH 27/46] selinux: Move an assignment for the variable "rc" in sens_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 22:02:14 +0100 A local variable was set to an error code in one case before a concrete error situation was detected. Thus move the corresponding assignment into an if branch to indicate a software failure there.

[PATCH 27/46] selinux: Move an assignment for the variable "rc" in sens_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 22:02:14 +0100 A local variable was set to an error code in one case before a concrete error situation was detected. Thus move the corresponding assignment into an if branch to indicate a software failure there. Signed-off-by: Markus Elfring ---

[PATCH 26/46] selinux: Improve another size determination in sens_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 21:52:55 +0100 Replace the specification of a data type by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style

[PATCH 26/46] selinux: Improve another size determination in sens_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 21:52:55 +0100 Replace the specification of a data type by a pointer dereference as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. Signed-off-by: Markus

[PATCH 25/46] selinux: Return directly after a failed kzalloc() in sens_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 21:42:02 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1

[PATCH 25/46] selinux: Return directly after a failed kzalloc() in sens_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 21:42:02 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

[PATCH 24/46] selinux: Return directly after a failed kzalloc() in cat_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 21:20:43 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1

[PATCH 24/46] selinux: Return directly after a failed kzalloc() in cat_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 21:20:43 +0100 Return directly after a call of the function "kzalloc" failed at the beginning. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git

Re: Calling device_init_wakeup() on driver removal

2017-01-15 Thread Guenter Roeck
On 01/15/2017 06:49 AM, Rafael J. Wysocki wrote: On Saturday, January 14, 2017 08:46:05 PM Guenter Roeck wrote: Hi folks, Hi, while looking through driver initialization and removal functions, I noticed that many drivers call device_init_wakeup(dev, false) in the removal function. Given

Re: Calling device_init_wakeup() on driver removal

2017-01-15 Thread Guenter Roeck
On 01/15/2017 06:49 AM, Rafael J. Wysocki wrote: On Saturday, January 14, 2017 08:46:05 PM Guenter Roeck wrote: Hi folks, Hi, while looking through driver initialization and removal functions, I noticed that many drivers call device_init_wakeup(dev, false) in the removal function. Given

[PATCH 23/46] selinux: Move an assignment for a pointer in range_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 21:00:45 +0100 Move the assignment for the local variable "r" behind a call of the function "next_entry" at the beginning so that it will only be set after a successful call. Signed-off-by: Markus Elfring

[PATCH 23/46] selinux: Move an assignment for a pointer in range_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 21:00:45 +0100 Move the assignment for the local variable "r" behind a call of the function "next_entry" at the beginning so that it will only be set after a successful call. Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 ++-

[PATCH 22/46] selinux: Delete an unnecessary variable initialisation in range_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 20:40:12 +0100 The local variable "rt" will be set to an appropriate pointer a bit later. Thus omit the explicit initialisation at the beginning which became unnecessary with a previous update step. Signed-off-by: Markus

[PATCH 22/46] selinux: Delete an unnecessary variable initialisation in range_read()

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 20:40:12 +0100 The local variable "rt" will be set to an appropriate pointer a bit later. Thus omit the explicit initialisation at the beginning which became unnecessary with a previous update step. Signed-off-by: Markus Elfring ---

[PATCH 21/46] selinux: Two function calls less in range_read() after error detection

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 20:20:15 +0100 Adjust a jump target to avoid two calls of the function "kfree" at the end after a memory allocation failed for the local variable "rt". Signed-off-by: Markus Elfring ---

[PATCH 21/46] selinux: Two function calls less in range_read() after error detection

2017-01-15 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 14 Jan 2017 20:20:15 +0100 Adjust a jump target to avoid two calls of the function "kfree" at the end after a memory allocation failed for the local variable "rt". Signed-off-by: Markus Elfring --- security/selinux/ss/policydb.c | 3 ++- 1 file changed, 2

<    1   2   3   4   5   6   7   >