[PATCH review for 4.9 06/50] scsi: megaraid_sas: Do not set fp_possible if TM capable for non-RW syspdIO, change fp_possible to bool

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Shivasharan S [ Upstream commit 1d6dbd1752fb4347a4a5db06c8f5cd35dd1919f4 ] FIX - firmware wants non-RW SYS PD IOs to avoid FastPath for better tracking and other functionalities if the device is task management capable. Signed-off-by: Kashyap

[PATCH review for 4.9 06/50] scsi: megaraid_sas: Do not set fp_possible if TM capable for non-RW syspdIO, change fp_possible to bool

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Shivasharan S [ Upstream commit 1d6dbd1752fb4347a4a5db06c8f5cd35dd1919f4 ] FIX - firmware wants non-RW SYS PD IOs to avoid FastPath for better tracking and other functionalities if the device is task management capable. Signed-off-by: Kashyap Desai Signed-off-by: Shivasharan S

[PATCH review for 4.9 03/50] staging: fsl-mc: Add missing header

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Bogdan Purcareata [ Upstream commit 07e9ef146071adf316312b3dd4abfe41c9a45a7d ] Compiling the fsl-mc bus driver will yield a couple of static analysis errors: warning: symbol 'fsl_mc_msi_domain_alloc_irqs' was not declared warning: symbol

[PATCH review for 4.9 11/50] staging: rtl8712u: Fix endian settings for structs describing network packets

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Larry Finger [ Upstream commit 221c46d28957bd6e2158abc2179ce4a8c9ce07d3 ] The headers describing a number of network packets do not have the correct endian settings for several types of data. Signed-off-by: Larry Finger

[PATCH review for 4.4 01/24] PM / wakeirq: report a wakeup_event on dedicated wekup irq

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Grygorii Strashko [ Upstream commit 09bb6e93956ae5175b96905b723ec879c3ca0765 ] There are two reasons for reporting wakeup event when dedicated wakeup IRQ is triggered: - wakeup events accounting, so proper statistical data will be displayed in sysfs and

[PATCH review for 4.9 03/50] staging: fsl-mc: Add missing header

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Bogdan Purcareata [ Upstream commit 07e9ef146071adf316312b3dd4abfe41c9a45a7d ] Compiling the fsl-mc bus driver will yield a couple of static analysis errors: warning: symbol 'fsl_mc_msi_domain_alloc_irqs' was not declared warning: symbol 'fsl_mc_msi_domain_free_irqs' was not declared.

[PATCH review for 4.9 11/50] staging: rtl8712u: Fix endian settings for structs describing network packets

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Larry Finger [ Upstream commit 221c46d28957bd6e2158abc2179ce4a8c9ce07d3 ] The headers describing a number of network packets do not have the correct endian settings for several types of data. Signed-off-by: Larry Finger Signed-off-by: Greg Kroah-Hartman Signed-off-by: Sasha Levin ---

[PATCH review for 4.4 01/24] PM / wakeirq: report a wakeup_event on dedicated wekup irq

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Grygorii Strashko [ Upstream commit 09bb6e93956ae5175b96905b723ec879c3ca0765 ] There are two reasons for reporting wakeup event when dedicated wakeup IRQ is triggered: - wakeup events accounting, so proper statistical data will be displayed in sysfs and debugfs; - there are small

[PATCH review for 4.9 10/50] bnxt_en: Added PCI IDs for BCM57452 and BCM57454 ASICs

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Deepak Khungar [ Upstream commit 32b40798c1b40343641f04cdfd09652af70ea0e9 ] Signed-off-by: Deepak Khungar Signed-off-by: Michael Chan Signed-off-by: David S. Miller Signed-off-by:

[PATCH review for 4.9 10/50] bnxt_en: Added PCI IDs for BCM57452 and BCM57454 ASICs

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Deepak Khungar [ Upstream commit 32b40798c1b40343641f04cdfd09652af70ea0e9 ] Signed-off-by: Deepak Khungar Signed-off-by: Michael Chan Signed-off-by: David S. Miller Signed-off-by: Sasha Levin --- drivers/net/ethernet/broadcom/bnxt/bnxt.c | 6 ++ 1 file changed, 6 insertions(+)

[PATCH review for 4.9 13/50] net: mvneta: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Russell King [ Upstream commit 9303ab2b3402b60f6c39abfdbfa4ce00fce8bee4 ] drivers/net/ethernet/marvell/mvneta.c:2694:26: error: storage size of 'status' isn't known drivers/net/ethernet/marvell/mvneta.c:2695:26: error: storage size of 'changed' isn't known

[PATCH review for 4.9 13/50] net: mvneta: fix build errors when linux/phy*.h is removed from net/dsa.h

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Russell King [ Upstream commit 9303ab2b3402b60f6c39abfdbfa4ce00fce8bee4 ] drivers/net/ethernet/marvell/mvneta.c:2694:26: error: storage size of 'status' isn't known drivers/net/ethernet/marvell/mvneta.c:2695:26: error: storage size of 'changed' isn't known

[PATCH review for 4.9 14/50] ext4: fix stripe-unaligned allocations

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Jan Kara [ Upstream commit d9b22cf9f5466a057f2a4f1e642b469fa9d73117 ] When a filesystem is created using: mkfs.ext4 -b 4096 -E stride=512 and we try to allocate 64MB extent, we will end up directly in ext4_mb_complex_scan_group(). This is because the request is

[PATCH review for 4.9 14/50] ext4: fix stripe-unaligned allocations

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Jan Kara [ Upstream commit d9b22cf9f5466a057f2a4f1e642b469fa9d73117 ] When a filesystem is created using: mkfs.ext4 -b 4096 -E stride=512 and we try to allocate 64MB extent, we will end up directly in ext4_mb_complex_scan_group(). This is because the request is detected as

[PATCH review for 4.9 12/50] PCI/MSI: Return failure when msix_setup_entries() fails

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Christophe JAILLET [ Upstream commit 3adfb572f2978a980b250a9e1a56f84f3a031001 ] If alloc_msi_entry() fails, we free resources and set ret = -ENOMEM. However, msix_setup_entries() returns 0 unconditionally. Return the error code instead. Fixes:

[PATCH review for 4.9 12/50] PCI/MSI: Return failure when msix_setup_entries() fails

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Christophe JAILLET [ Upstream commit 3adfb572f2978a980b250a9e1a56f84f3a031001 ] If alloc_msi_entry() fails, we free resources and set ret = -ENOMEM. However, msix_setup_entries() returns 0 unconditionally. Return the error code instead. Fixes: e75eafb9b039 ("genirq/msi: Switch to new

[PATCH review for 4.9 04/50] IB/mlx5: Assign DSCP for R-RoCE QPs Address Path

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Majd Dibbiny [ Upstream commit ed88451e1f2d400fd6a743d0a481631cf9f97550 ] For Routable RoCE QPs, the DSCP should be set in the QP's address path. The DSCP's value is derived from the traffic class. Fixes: 2811ba51b049 ("IB/mlx5: Add RoCE fields to Address Vector") Cc:

[PATCH review for 4.9 04/50] IB/mlx5: Assign DSCP for R-RoCE QPs Address Path

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Majd Dibbiny [ Upstream commit ed88451e1f2d400fd6a743d0a481631cf9f97550 ] For Routable RoCE QPs, the DSCP should be set in the QP's address path. The DSCP's value is derived from the traffic class. Fixes: 2811ba51b049 ("IB/mlx5: Add RoCE fields to Address Vector") Cc: Achiad Shochat

[PATCH review for 4.9 18/50] drm/amdgpu: when dpm disabled, also need to stop/start vce.

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Rex Zhu [ Upstream commit 28ed5504ab4b211a4e589e648e5ebd1e0caa7a6a ] Signed-off-by: Rex Zhu Reviewed-by: Alex Deucher Acked-by: Christian König Signed-off-by: Alex Deucher

[PATCH review for 4.9 17/50] i2c: riic: correctly finish transfers

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Chris Brandt [ Upstream commit 71ccea095ea1d4efd004dab971be6d599e06fc3f ] This fixes the condition where the controller has not fully completed its final transfer and leaves the bus and controller in a undesirable state. At the end of the last transmitted byte,

[PATCH review for 4.9 18/50] drm/amdgpu: when dpm disabled, also need to stop/start vce.

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Rex Zhu [ Upstream commit 28ed5504ab4b211a4e589e648e5ebd1e0caa7a6a ] Signed-off-by: Rex Zhu Reviewed-by: Alex Deucher Acked-by: Christian König Signed-off-by: Alex Deucher Signed-off-by: Sasha Levin --- drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c | 9 + 1 file changed, 9

[PATCH review for 4.9 17/50] i2c: riic: correctly finish transfers

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Chris Brandt [ Upstream commit 71ccea095ea1d4efd004dab971be6d599e06fc3f ] This fixes the condition where the controller has not fully completed its final transfer and leaves the bus and controller in a undesirable state. At the end of the last transmitted byte, the existing driver would

[PATCH review for 4.9 16/50] net/ena: change driver's default timeouts

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Netanel Belgazal [ Upstream commit 7102a18ac3f323805e3cd8f3dc64907644608c1e ] The timeouts were too agressive and sometimes cause false alarms. Signed-off-by: Netanel Belgazal Signed-off-by: David S. Miller

[PATCH review for 4.9 20/50] iwlwifi: mvm: use the PROBE_RESP_QUEUE to send deauth to unknown station

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Emmanuel Grumbach [ Upstream commit d45cb20e123c5d7d6cd56301bc98f0bfd725cd77 ] When we send a deauth to a station we don't know about, we need to use the PROBE_RESP queue. This can happen when we send a deauth to a station that is not associated to us.

[PATCH review for 4.9 16/50] net/ena: change driver's default timeouts

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Netanel Belgazal [ Upstream commit 7102a18ac3f323805e3cd8f3dc64907644608c1e ] The timeouts were too agressive and sometimes cause false alarms. Signed-off-by: Netanel Belgazal Signed-off-by: David S. Miller Signed-off-by: Sasha Levin --- drivers/net/ethernet/amazon/ena/ena_com.c|

[PATCH review for 4.9 20/50] iwlwifi: mvm: use the PROBE_RESP_QUEUE to send deauth to unknown station

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Emmanuel Grumbach [ Upstream commit d45cb20e123c5d7d6cd56301bc98f0bfd725cd77 ] When we send a deauth to a station we don't know about, we need to use the PROBE_RESP queue. This can happen when we send a deauth to a station that is not associated to us. Signed-off-by: Emmanuel Grumbach

[PATCH review for 4.9 23/50] net: phy: dp83867: Recover from "port mirroring" N/A MODE4

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Lukasz Majewski [ Upstream commit ac6e058b75be71208e98a5808453aae9a17be480 ] The DP83867 when not properly bootstrapped - especially with LED_0 pin - can enter N/A MODE4 for "port mirroring" feature. To provide normal operation of the PHY, one needs not only to explicitly

[PATCH review for 4.9 21/50] drm/fsl-dcu: check for clk_prepare_enable() error

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Fabio Estevam [ Upstream commit ef15d36154cc741d7ded4ae4fa0cf7987354e313 ] clk_prepare_enable() may fail, so we should better check its return value. Also place the of_node_put() function right after clk_prepare_enable(), in order to avoid calling of_node_put()

[PATCH review for 4.9 23/50] net: phy: dp83867: Recover from "port mirroring" N/A MODE4

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Lukasz Majewski [ Upstream commit ac6e058b75be71208e98a5808453aae9a17be480 ] The DP83867 when not properly bootstrapped - especially with LED_0 pin - can enter N/A MODE4 for "port mirroring" feature. To provide normal operation of the PHY, one needs not only to explicitly disable the

[PATCH review for 4.9 21/50] drm/fsl-dcu: check for clk_prepare_enable() error

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Fabio Estevam [ Upstream commit ef15d36154cc741d7ded4ae4fa0cf7987354e313 ] clk_prepare_enable() may fail, so we should better check its return value. Also place the of_node_put() function right after clk_prepare_enable(), in order to avoid calling of_node_put() twice in case

[PATCH review for 4.9 22/50] clocksource/drivers/arm_arch_timer: Add dt binding for hisilicon-161010101 erratum

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Ding Tianhong [ Upstream commit 729e55225b1f6225ee7a2a358d5141a3264627c4 ] This erratum describes a bug in logic outside the core, so MIDR can't be used to identify its presence, and reading an SoC-specific revision register from common arch timer code would be

[PATCH review for 4.9 22/50] clocksource/drivers/arm_arch_timer: Add dt binding for hisilicon-161010101 erratum

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Ding Tianhong [ Upstream commit 729e55225b1f6225ee7a2a358d5141a3264627c4 ] This erratum describes a bug in logic outside the core, so MIDR can't be used to identify its presence, and reading an SoC-specific revision register from common arch timer code would be awkward. So, describe it

[PATCH review for 4.9 27/50] drm/msm/dsi: Set msm_dsi->encoders before initializing bridge

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Archit Taneja [ Upstream commit 0bb70b82c2f91e4667f3c617505235efd6d77e46 ] The commit "drm: bridge: Link encoder and bridge in core code" updated the drm_bridge_attach() API to also include the drm_encoder pointer the bridge attaches to. The func

[PATCH review for 4.4 09/24] i2c: riic: correctly finish transfers

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Chris Brandt [ Upstream commit 71ccea095ea1d4efd004dab971be6d599e06fc3f ] This fixes the condition where the controller has not fully completed its final transfer and leaves the bus and controller in a undesirable state. At the end of the last transmitted byte,

[PATCH review for 4.9 27/50] drm/msm/dsi: Set msm_dsi->encoders before initializing bridge

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Archit Taneja [ Upstream commit 0bb70b82c2f91e4667f3c617505235efd6d77e46 ] The commit "drm: bridge: Link encoder and bridge in core code" updated the drm_bridge_attach() API to also include the drm_encoder pointer the bridge attaches to. The func msm_dsi_manager_bridge_init() now relies

[PATCH review for 4.4 09/24] i2c: riic: correctly finish transfers

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Chris Brandt [ Upstream commit 71ccea095ea1d4efd004dab971be6d599e06fc3f ] This fixes the condition where the controller has not fully completed its final transfer and leaves the bus and controller in a undesirable state. At the end of the last transmitted byte, the existing driver would

[PATCH review for 4.9 24/50] [media] cx231xx: Fix I2C on Internal Master 3 Bus

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Oleh Kravchenko [ Upstream commit 6c5da8031a3abfad259190d35f83d89568b72ee2 ] Internal Master 3 Bus can send and receive only 4 bytes per time. Signed-off-by: Oleh Kravchenko Signed-off-by: Mauro Carvalho Chehab Signed-off-by:

[PATCH review for 4.9 31/50] xen/manage: correct return value check on xenbus_scanf()

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Jan Beulich [ Upstream commit 4fed1b125eb6252bde478665fc05d4819f774fa8 ] A negative return value indicates an error; in fact the function at present won't ever return zero. Signed-off-by: Jan Beulich Reviewed-by: Juergen Gross

[PATCH review for 4.9 24/50] [media] cx231xx: Fix I2C on Internal Master 3 Bus

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Oleh Kravchenko [ Upstream commit 6c5da8031a3abfad259190d35f83d89568b72ee2 ] Internal Master 3 Bus can send and receive only 4 bytes per time. Signed-off-by: Oleh Kravchenko Signed-off-by: Mauro Carvalho Chehab Signed-off-by: Sasha Levin --- drivers/media/usb/cx231xx/cx231xx-core.c |

[PATCH review for 4.9 31/50] xen/manage: correct return value check on xenbus_scanf()

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Jan Beulich [ Upstream commit 4fed1b125eb6252bde478665fc05d4819f774fa8 ] A negative return value indicates an error; in fact the function at present won't ever return zero. Signed-off-by: Jan Beulich Reviewed-by: Juergen Gross Signed-off-by: Boris Ostrovsky Signed-off-by: Sasha Levin

[PATCH review for 4.9 29/50] dmaengine: sun6i: allow build on ARM64 platforms (sun50i)

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Icenowy Zheng [ Upstream commit c429ceb1e18252122ba96b52e689dcf87103c186 ] As 64-bit Allwinner H5 SoC has the same DMA engine with H3, the DMA driver should be allowed to be built for ARM64, in order to make it work on H5. Signed-off-by: Icenowy Zheng

[PATCH review for 4.9 29/50] dmaengine: sun6i: allow build on ARM64 platforms (sun50i)

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Icenowy Zheng [ Upstream commit c429ceb1e18252122ba96b52e689dcf87103c186 ] As 64-bit Allwinner H5 SoC has the same DMA engine with H3, the DMA driver should be allowed to be built for ARM64, in order to make it work on H5. Signed-off-by: Icenowy Zheng Acked-by: Maxime Ripard Acked-by:

[PATCH review for 4.9 28/50] mtd: nand: sunxi: Fix the non-polling case in sunxi_nfc_wait_events()

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Boris Brezillon [ Upstream commit 19649e2c16fbc94b664f7074ec4fa9f15292fdce ] wait_for_completion_timeout() returns 0 if a timeout occurred, 1 otherwise. Fix the sunxi_nfc_wait_events() accordingly. Signed-off-by: Boris Brezillon

[PATCH review for 4.9 30/50] gpio: mcp23s08: Select REGMAP/REGMAP_I2C to fix build error

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Axel Lin [ Upstream commit e8e1a5b5679b1ae1ff03a3883b011b84e7226171 ] This driver now using devm_regmap_init/devm_regmap_init_i2c, so it needs to select REGMAP/REGMAP_I2C accordingly. Fixes: ("3d84fdb3f0b5 gpio: mcp23s08: use regmap") Signed-off-by: Axel Lin

[PATCH review for 4.9 28/50] mtd: nand: sunxi: Fix the non-polling case in sunxi_nfc_wait_events()

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Boris Brezillon [ Upstream commit 19649e2c16fbc94b664f7074ec4fa9f15292fdce ] wait_for_completion_timeout() returns 0 if a timeout occurred, 1 otherwise. Fix the sunxi_nfc_wait_events() accordingly. Signed-off-by: Boris Brezillon Signed-off-by: Sasha Levin ---

[PATCH review for 4.9 30/50] gpio: mcp23s08: Select REGMAP/REGMAP_I2C to fix build error

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Axel Lin [ Upstream commit e8e1a5b5679b1ae1ff03a3883b011b84e7226171 ] This driver now using devm_regmap_init/devm_regmap_init_i2c, so it needs to select REGMAP/REGMAP_I2C accordingly. Fixes: ("3d84fdb3f0b5 gpio: mcp23s08: use regmap") Signed-off-by: Axel Lin Acked-By: Sebastian Reichel

[PATCH review for 4.9 35/50] staging: lustre: hsm: stack overrun in hai_dump_data_field

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: frank zago [ Upstream commit 22aadb91c0a0055935109c175f5446abfb130702 ] The function hai_dump_data_field will do a stack buffer overrun when cat'ing /sys/fs/lustre/.../hsm/actions if an action has some data in it. hai_dump_data_field uses snprintf. But there is no check

[PATCH review for 4.9 40/50] vfs: open() with O_CREAT should not create inodes with unknown ids

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Seth Forshee [ Upstream commit 1328c727004d432bbdfba0ffa02a166df04c7305 ] may_create() rejects creation of inodes with ids which lack a mapping into s_user_ns. However for O_CREAT may_o_create() is is used instead. Add a similar check there. Fixes:

[PATCH review for 4.9 35/50] staging: lustre: hsm: stack overrun in hai_dump_data_field

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: frank zago [ Upstream commit 22aadb91c0a0055935109c175f5446abfb130702 ] The function hai_dump_data_field will do a stack buffer overrun when cat'ing /sys/fs/lustre/.../hsm/actions if an action has some data in it. hai_dump_data_field uses snprintf. But there is no check for truncation,

[PATCH review for 4.9 40/50] vfs: open() with O_CREAT should not create inodes with unknown ids

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Seth Forshee [ Upstream commit 1328c727004d432bbdfba0ffa02a166df04c7305 ] may_create() rejects creation of inodes with ids which lack a mapping into s_user_ns. However for O_CREAT may_o_create() is is used instead. Add a similar check there. Fixes: 036d523641c6 ("vfs: Don't create inodes

[PATCH review for 4.9 39/50] ASoC: omap-mcbsp: Add PM QoS support for McBSP to prevent glitches

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Matt Ranostay [ Upstream commit 9834ffd1ecc3a401d0ce64c2d4235a726da6d4f9 ] We can get audio errors if hitting deeper idle states on omaps: [alsa.c:230] error: Fatal problem with alsa output, error -5. [audio.c:614] error: Error in writing audio (Input/output

[PATCH review for 4.9 39/50] ASoC: omap-mcbsp: Add PM QoS support for McBSP to prevent glitches

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Matt Ranostay [ Upstream commit 9834ffd1ecc3a401d0ce64c2d4235a726da6d4f9 ] We can get audio errors if hitting deeper idle states on omaps: [alsa.c:230] error: Fatal problem with alsa output, error -5. [audio.c:614] error: Error in writing audio (Input/output error?)! This seems to

[PATCH review for 4.9 37/50] staging: lustre: lmv: Error not handled for lmv_find_target

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Ulka Vaze [ Upstream commit 30af99db7ab3483f5ce83ccb890533c9378c2ced ] This issue is found by smatch; has been reported as- Unchecked usage of potential ERR_PTR result in lmv_hsm_req_count and lmv_hsm_req_build. Added ERR_PTR in both functions and also return value

[PATCH review for 4.9 37/50] staging: lustre: lmv: Error not handled for lmv_find_target

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Ulka Vaze [ Upstream commit 30af99db7ab3483f5ce83ccb890533c9378c2ced ] This issue is found by smatch; has been reported as- Unchecked usage of potential ERR_PTR result in lmv_hsm_req_count and lmv_hsm_req_build. Added ERR_PTR in both functions and also return value check added.

[PATCH review for 4.9 19/50] perf tools: Only increase index if perf_evsel__new_idx() succeeds

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Taeung Song [ Upstream commit 75fc5ae5cc53fff71041ecadeb3354a2b4c9fe42 ] Signed-off-by: Taeung Song Cc: Jiri Olsa Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Wang Nan

[PATCH review for 4.9 19/50] perf tools: Only increase index if perf_evsel__new_idx() succeeds

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Taeung Song [ Upstream commit 75fc5ae5cc53fff71041ecadeb3354a2b4c9fe42 ] Signed-off-by: Taeung Song Cc: Jiri Olsa Cc: Namhyung Kim Cc: Peter Zijlstra Cc: Wang Nan Link: http://lkml.kernel.org/r/1485952447-7013-2-git-send-email-treeze.tae...@gmail.com Signed-off-by: Arnaldo Carvalho

[PATCH review for 4.4 16/24] scsi: aacraid: Process Error for response I/O

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Raghava Aditya Renukunta [ Upstream commit 4ec57fb4edaec523f0f78a0449a3b063749ac58b ] Make sure that the driver processes error conditions even in the fast response path for response from the adapter. Signed-off-by: Raghava Aditya Renukunta

[PATCH review for 4.4 18/24] staging: lustre: llite: don't invoke direct_IO for the EOF case

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Yang Sheng [ Upstream commit 77759771fb95420d23876cb104ab65c022613325 ] The function generic_file_read_iter() does not check EOF before invoke direct_IO callback. So we have to check it ourselves. Signed-off-by: Yang Sheng Intel-bug-id:

[PATCH review for 4.4 16/24] scsi: aacraid: Process Error for response I/O

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Raghava Aditya Renukunta [ Upstream commit 4ec57fb4edaec523f0f78a0449a3b063749ac58b ] Make sure that the driver processes error conditions even in the fast response path for response from the adapter. Signed-off-by: Raghava Aditya Renukunta Signed-off-by: Dave Carroll Reviewed-by:

[PATCH review for 4.4 18/24] staging: lustre: llite: don't invoke direct_IO for the EOF case

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Yang Sheng [ Upstream commit 77759771fb95420d23876cb104ab65c022613325 ] The function generic_file_read_iter() does not check EOF before invoke direct_IO callback. So we have to check it ourselves. Signed-off-by: Yang Sheng Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-8969

[PATCH review for 4.4 21/24] [media] exynos4-is: fimc-is: Unmap region obtained by of_iomap()

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Arvind Yadav [ Upstream commit 4742575cde1f3cee0ea6b41af42781672315b04b ] Free memory mapping, if fimc_is_probe is not successful. Signed-off-by: Arvind Yadav Signed-off-by: Sylwester Nawrocki Signed-off-by:

[PATCH review for 4.4 21/24] [media] exynos4-is: fimc-is: Unmap region obtained by of_iomap()

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Arvind Yadav [ Upstream commit 4742575cde1f3cee0ea6b41af42781672315b04b ] Free memory mapping, if fimc_is_probe is not successful. Signed-off-by: Arvind Yadav Signed-off-by: Sylwester Nawrocki Signed-off-by: Mauro Carvalho Chehab Signed-off-by: Sasha Levin ---

[PATCH review for 4.4 22/24] mei: return error on notification request to a disconnected client

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Alexander Usyskin [ Upstream commit 7c47d2ca0feca767479329da23523ed798acb854 ] Request for a notification from a disconnected client will be ignored silently by the FW but the caller should know that the operation hasn't succeeded. Signed-off-by: Alexander

[PATCH review for 4.4 22/24] mei: return error on notification request to a disconnected client

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Alexander Usyskin [ Upstream commit 7c47d2ca0feca767479329da23523ed798acb854 ] Request for a notification from a disconnected client will be ignored silently by the FW but the caller should know that the operation hasn't succeeded. Signed-off-by: Alexander Usyskin Signed-off-by: Tomas

[PATCH review for 4.4 20/24] staging: lustre: ptlrpc: skip lock if export failed

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Alexander Boyko [ Upstream commit 4c43c27ddc461d8473cedd70f2549614641dfbc7 ] This patch resolves IO vs eviction race. After eviction failed export stayed at stale list, a client had IO processing and reconnected during it. A client sent brw rpc with last lock

[PATCH review for 4.4 20/24] staging: lustre: ptlrpc: skip lock if export failed

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Alexander Boyko [ Upstream commit 4c43c27ddc461d8473cedd70f2549614641dfbc7 ] This patch resolves IO vs eviction race. After eviction failed export stayed at stale list, a client had IO processing and reconnected during it. A client sent brw rpc with last lock cookie and new connection.

[PATCH review for 4.4 23/24] s390/dasd: check for device error pointer within state change interrupts

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Stefan Haberland [ Upstream commit 2202134e48a3b50320aeb9e3dd1186833e9d7e66 ] Check if the device pointer is valid. Just a sanity check since we already are in the int handler of the device. Signed-off-by: Stefan Haberland Signed-off-by:

[PATCH review for 3.18 01/12] mmc: s3cmci: include linux/interrupt.h for tasklet_struct

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Arnd Bergmann [ Upstream commit e1c6ec26b853e9062f0b3daaf695c546d0702953 ] I got this new build error on today's linux-next drivers/mmc/host/s3cmci.h:69:24: error: field 'pio_tasklet' has incomplete type struct tasklet_struct pio_tasklet; drivers/mmc/host/s3cmci.c: In

[PATCH review for 4.4 23/24] s390/dasd: check for device error pointer within state change interrupts

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Stefan Haberland [ Upstream commit 2202134e48a3b50320aeb9e3dd1186833e9d7e66 ] Check if the device pointer is valid. Just a sanity check since we already are in the int handler of the device. Signed-off-by: Stefan Haberland Signed-off-by: Martin Schwidefsky Signed-off-by: Sasha Levin

[PATCH review for 3.18 01/12] mmc: s3cmci: include linux/interrupt.h for tasklet_struct

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Arnd Bergmann [ Upstream commit e1c6ec26b853e9062f0b3daaf695c546d0702953 ] I got this new build error on today's linux-next drivers/mmc/host/s3cmci.h:69:24: error: field 'pio_tasklet' has incomplete type struct tasklet_struct pio_tasklet; drivers/mmc/host/s3cmci.c: In function

[PATCH review for 3.18 08/12] platform/x86: intel_mid_thermal: Fix module autoload

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Javier Martinez Canillas [ Upstream commit a93151a72061e944a4915458b1b1d6d505c03bbf ] If the driver is built as a module, autoload won't work because the module alias information is not filled. So user-space can't match the registered device with the corresponding

[PATCH review for 3.18 07/12] xen/manage: correct return value check on xenbus_scanf()

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Jan Beulich [ Upstream commit 4fed1b125eb6252bde478665fc05d4819f774fa8 ] A negative return value indicates an error; in fact the function at present won't ever return zero. Signed-off-by: Jan Beulich Reviewed-by: Juergen Gross

[PATCH review for 4.4 13/24] drm/msm/dsi: Set msm_dsi->encoders before initializing bridge

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Archit Taneja [ Upstream commit 0bb70b82c2f91e4667f3c617505235efd6d77e46 ] The commit "drm: bridge: Link encoder and bridge in core code" updated the drm_bridge_attach() API to also include the drm_encoder pointer the bridge attaches to. The func

[PATCH review for 3.18 08/12] platform/x86: intel_mid_thermal: Fix module autoload

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Javier Martinez Canillas [ Upstream commit a93151a72061e944a4915458b1b1d6d505c03bbf ] If the driver is built as a module, autoload won't work because the module alias information is not filled. So user-space can't match the registered device with the corresponding module. Export the

[PATCH review for 3.18 07/12] xen/manage: correct return value check on xenbus_scanf()

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Jan Beulich [ Upstream commit 4fed1b125eb6252bde478665fc05d4819f774fa8 ] A negative return value indicates an error; in fact the function at present won't ever return zero. Signed-off-by: Jan Beulich Reviewed-by: Juergen Gross Signed-off-by: Boris Ostrovsky Signed-off-by: Sasha Levin

[PATCH review for 4.4 13/24] drm/msm/dsi: Set msm_dsi->encoders before initializing bridge

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Archit Taneja [ Upstream commit 0bb70b82c2f91e4667f3c617505235efd6d77e46 ] The commit "drm: bridge: Link encoder and bridge in core code" updated the drm_bridge_attach() API to also include the drm_encoder pointer the bridge attaches to. The func msm_dsi_manager_bridge_init() now relies

[PATCH review for 3.18 03/12] ext4: fix stripe-unaligned allocations

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Jan Kara [ Upstream commit d9b22cf9f5466a057f2a4f1e642b469fa9d73117 ] When a filesystem is created using: mkfs.ext4 -b 4096 -E stride=512 and we try to allocate 64MB extent, we will end up directly in ext4_mb_complex_scan_group(). This is because the request is

[PATCH review for 3.18 03/12] ext4: fix stripe-unaligned allocations

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Jan Kara [ Upstream commit d9b22cf9f5466a057f2a4f1e642b469fa9d73117 ] When a filesystem is created using: mkfs.ext4 -b 4096 -E stride=512 and we try to allocate 64MB extent, we will end up directly in ext4_mb_complex_scan_group(). This is because the request is detected as

[PATCH review for 4.4 15/24] xen/manage: correct return value check on xenbus_scanf()

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Jan Beulich [ Upstream commit 4fed1b125eb6252bde478665fc05d4819f774fa8 ] A negative return value indicates an error; in fact the function at present won't ever return zero. Signed-off-by: Jan Beulich Reviewed-by: Juergen Gross

[PATCH review for 4.4 15/24] xen/manage: correct return value check on xenbus_scanf()

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Jan Beulich [ Upstream commit 4fed1b125eb6252bde478665fc05d4819f774fa8 ] A negative return value indicates an error; in fact the function at present won't ever return zero. Signed-off-by: Jan Beulich Reviewed-by: Juergen Gross Signed-off-by: Boris Ostrovsky Signed-off-by: Sasha Levin

[PATCH review for 3.18 02/12] staging: rtl8712u: Fix endian settings for structs describing network packets

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Larry Finger [ Upstream commit 221c46d28957bd6e2158abc2179ce4a8c9ce07d3 ] The headers describing a number of network packets do not have the correct endian settings for several types of data. Signed-off-by: Larry Finger

[PATCH review for 4.4 24/24] [media] bt8xx: fix memory leak

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Sudip Mukherjee [ Upstream commit 6792eb0cf9310ec240b7e7c9bfa86dff4c758c68 ] If dvb_attach() fails then we were just printing an error message and exiting but the memory allocated to state was not released. Signed-off-by: Sudip Mukherjee

[PATCH review for 3.18 06/12] [media] cx231xx: Fix I2C on Internal Master 3 Bus

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Oleh Kravchenko [ Upstream commit 6c5da8031a3abfad259190d35f83d89568b72ee2 ] Internal Master 3 Bus can send and receive only 4 bytes per time. Signed-off-by: Oleh Kravchenko Signed-off-by: Mauro Carvalho Chehab Signed-off-by:

[PATCH review for 4.4 24/24] [media] bt8xx: fix memory leak

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Sudip Mukherjee [ Upstream commit 6792eb0cf9310ec240b7e7c9bfa86dff4c758c68 ] If dvb_attach() fails then we were just printing an error message and exiting but the memory allocated to state was not released. Signed-off-by: Sudip Mukherjee Signed-off-by: Hans Verkuil Signed-off-by: Mauro

[PATCH review for 3.18 06/12] [media] cx231xx: Fix I2C on Internal Master 3 Bus

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Oleh Kravchenko [ Upstream commit 6c5da8031a3abfad259190d35f83d89568b72ee2 ] Internal Master 3 Bus can send and receive only 4 bytes per time. Signed-off-by: Oleh Kravchenko Signed-off-by: Mauro Carvalho Chehab Signed-off-by: Sasha Levin --- drivers/media/usb/cx231xx/cx231xx-core.c |

[PATCH review for 3.18 02/12] staging: rtl8712u: Fix endian settings for structs describing network packets

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Larry Finger [ Upstream commit 221c46d28957bd6e2158abc2179ce4a8c9ce07d3 ] The headers describing a number of network packets do not have the correct endian settings for several types of data. Signed-off-by: Larry Finger Signed-off-by: Greg Kroah-Hartman Signed-off-by: Sasha Levin ---

[PATCH review for 3.18 04/12] ext4: do not use stripe_width if it is not set

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Jan Kara [ Upstream commit 5469d7c3087ecaf760f54b447f11af6061b7c897 ] Avoid using stripe_width for sbi->s_stripe value if it is not actually set. It prevents using the stride for sbi->s_stripe. Signed-off-by: Jan Kara Signed-off-by: Theodore Ts'o

[PATCH review for 3.18 04/12] ext4: do not use stripe_width if it is not set

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Jan Kara [ Upstream commit 5469d7c3087ecaf760f54b447f11af6061b7c897 ] Avoid using stripe_width for sbi->s_stripe value if it is not actually set. It prevents using the stride for sbi->s_stripe. Signed-off-by: Jan Kara Signed-off-by: Theodore Ts'o Signed-off-by: Sasha Levin ---

[PATCH review for 3.18 05/12] i2c: riic: correctly finish transfers

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Chris Brandt [ Upstream commit 71ccea095ea1d4efd004dab971be6d599e06fc3f ] This fixes the condition where the controller has not fully completed its final transfer and leaves the bus and controller in a undesirable state. At the end of the last transmitted byte,

[PATCH review for 3.18 05/12] i2c: riic: correctly finish transfers

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Chris Brandt [ Upstream commit 71ccea095ea1d4efd004dab971be6d599e06fc3f ] This fixes the condition where the controller has not fully completed its final transfer and leaves the bus and controller in a undesirable state. At the end of the last transmitted byte, the existing driver would

[PATCH review for 3.18 09/12] staging: lustre: hsm: stack overrun in hai_dump_data_field

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: frank zago [ Upstream commit 22aadb91c0a0055935109c175f5446abfb130702 ] The function hai_dump_data_field will do a stack buffer overrun when cat'ing /sys/fs/lustre/.../hsm/actions if an action has some data in it. hai_dump_data_field uses snprintf. But there is no check

[PATCH review for 3.18 11/12] s390/dasd: check for device error pointer within state change interrupts

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Stefan Haberland [ Upstream commit 2202134e48a3b50320aeb9e3dd1186833e9d7e66 ] Check if the device pointer is valid. Just a sanity check since we already are in the int handler of the device. Signed-off-by: Stefan Haberland Signed-off-by:

[PATCH review for 4.4 12/24] [media] cx231xx: Fix I2C on Internal Master 3 Bus

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Oleh Kravchenko [ Upstream commit 6c5da8031a3abfad259190d35f83d89568b72ee2 ] Internal Master 3 Bus can send and receive only 4 bytes per time. Signed-off-by: Oleh Kravchenko Signed-off-by: Mauro Carvalho Chehab Signed-off-by:

[PATCH review for 3.18 09/12] staging: lustre: hsm: stack overrun in hai_dump_data_field

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: frank zago [ Upstream commit 22aadb91c0a0055935109c175f5446abfb130702 ] The function hai_dump_data_field will do a stack buffer overrun when cat'ing /sys/fs/lustre/.../hsm/actions if an action has some data in it. hai_dump_data_field uses snprintf. But there is no check for truncation,

[PATCH review for 3.18 11/12] s390/dasd: check for device error pointer within state change interrupts

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Stefan Haberland [ Upstream commit 2202134e48a3b50320aeb9e3dd1186833e9d7e66 ] Check if the device pointer is valid. Just a sanity check since we already are in the int handler of the device. Signed-off-by: Stefan Haberland Signed-off-by: Martin Schwidefsky Signed-off-by: Sasha Levin

[PATCH review for 4.4 12/24] [media] cx231xx: Fix I2C on Internal Master 3 Bus

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Oleh Kravchenko [ Upstream commit 6c5da8031a3abfad259190d35f83d89568b72ee2 ] Internal Master 3 Bus can send and receive only 4 bytes per time. Signed-off-by: Oleh Kravchenko Signed-off-by: Mauro Carvalho Chehab Signed-off-by: Sasha Levin --- drivers/media/usb/cx231xx/cx231xx-core.c |

[PATCH review for 3.18 10/12] staging: lustre: ptlrpc: skip lock if export failed

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Alexander Boyko [ Upstream commit 4c43c27ddc461d8473cedd70f2549614641dfbc7 ] This patch resolves IO vs eviction race. After eviction failed export stayed at stale list, a client had IO processing and reconnected during it. A client sent brw rpc with last lock

[PATCH review for 3.18 10/12] staging: lustre: ptlrpc: skip lock if export failed

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Alexander Boyko [ Upstream commit 4c43c27ddc461d8473cedd70f2549614641dfbc7 ] This patch resolves IO vs eviction race. After eviction failed export stayed at stale list, a client had IO processing and reconnected during it. A client sent brw rpc with last lock cookie and new connection.

[PATCH review for 4.4 14/24] dmaengine: sun6i: allow build on ARM64 platforms (sun50i)

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Icenowy Zheng [ Upstream commit c429ceb1e18252122ba96b52e689dcf87103c186 ] As 64-bit Allwinner H5 SoC has the same DMA engine with H3, the DMA driver should be allowed to be built for ARM64, in order to make it work on H5. Signed-off-by: Icenowy Zheng

[PATCH review for 4.4 17/24] platform/x86: intel_mid_thermal: Fix module autoload

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Javier Martinez Canillas [ Upstream commit a93151a72061e944a4915458b1b1d6d505c03bbf ] If the driver is built as a module, autoload won't work because the module alias information is not filled. So user-space can't match the registered device with the corresponding

<    3   4   5   6   7   8   9   10   11   12   >