[PATCH for 4.9 13/39] hugetlbfs: initialize shared policy as part of inode allocation

2017-09-17 Thread Levin, Alexander (Sasha Levin)
From: Mike Kravetz [ Upstream commit 4742a35d9de745e867405b4311e1aac412f0ace1 ] Any time after inode allocation, destroy_inode can be called. The hugetlbfs inode contains a shared_policy structure, and mpol_free_shared_policy is unconditionally called as part of

[PATCH for 4.9 03/39] team: fix memory leaks

2017-09-17 Thread Levin, Alexander (Sasha Levin)
From: Pan Bian [ Upstream commit 72ec0bc64b9a5d8e0efcb717abfc757746b101b7 ] In functions team_nl_send_port_list_get() and team_nl_send_options_get(), pointer skb keeps the return value of nlmsg_new(). When the call to genlmsg_put() fails, the memory is not freed(). This

[PATCH for 4.9 11/39] Btrfs: fix potential use-after-free for cloned bio

2017-09-17 Thread Levin, Alexander (Sasha Levin)
From: Liu Bo [ Upstream commit a967efb30b3afa3d858edd6a17f544f9e9e46eea ] KASAN reports that there is a use-after-free case of bio in btrfs_map_bio. If we need to submit IOs to several disks at a time, the original bio would get cloned and mapped to the destination disk,

[PATCH for 4.9 14/39] kasan: do not sanitize kexec purgatory

2017-09-17 Thread Levin, Alexander (Sasha Levin)
From: Mike Galbraith [ Upstream commit 13a6798e4a03096b11bf402a063786a7be55d426 ] Fixes this: kexec: Undefined symbol: __asan_load8_noabort kexec-bzImage64: Loading purgatory failed Link: http://lkml.kernel.org/r/1489672155.4458.7.ca...@gmx.de Signed-off-by: Mike Galbraith

[PATCH for 4.9 08/39] mmc: sdio: fix alignment issue in struct sdio_func

2017-09-17 Thread Levin, Alexander (Sasha Levin)
From: Heiner Kallweit [ Upstream commit 5ef1ecf060f28ecef313b5723f1fd39bf5a35f56 ] Certain 64-bit systems (e.g. Amlogic Meson GX) require buffers to be used for DMA to be 8-byte-aligned. struct sdio_func has an embedded small DMA buffer not meeting this requirement. When

[PATCH for 4.9 23/39] parisc: perf: Fix potential NULL pointer dereference

2017-09-17 Thread Levin, Alexander (Sasha Levin)
From: Arvind Yadav [ Upstream commit 74e3f6e63da6c8e8246fba1689e040bc926b4a1a ] Fix potential NULL pointer dereference and clean up coding style errors (code indent, trailing whitespaces). Signed-off-by: Arvind Yadav Signed-off-by: Helge

[PATCH for 4.9 22/39] netfilter: nfnl_cthelper: fix incorrect helper->expect_class_max

2017-09-17 Thread Levin, Alexander (Sasha Levin)
From: Liping Zhang [ Upstream commit ae5c682113f9f94cc5e76f92cf041ee624c173ee ] The helper->expect_class_max must be set to the total number of expect_policy minus 1, since we will use the statement "if (class > helper->expect_class_max)" to validate the CTA_EXPECT_CLASS

[PATCH for 4.9 28/39] iommu/io-pgtable-arm: Check for leaf entry before dereferencing it

2017-09-17 Thread Levin, Alexander (Sasha Levin)
From: Oleksandr Tyshchenko [ Upstream commit ed46e66cc1b3d684042f92dfa2ab15ee917b4cac ] Do a check for already installed leaf entry at the current level before dereferencing it in order to avoid walking the page table down with wrong pointer to the next level.

[PATCH for 4.9 33/39] netfilter: nf_tables: set pktinfo->thoff at AH header if found

2017-09-17 Thread Levin, Alexander (Sasha Levin)
From: Pablo Neira Ayuso [ Upstream commit 568af6de058cb2b0c5b98d98ffcf37cdc6bc38a7 ] Phil Sutter reports that IPv6 AH header matching is broken. From userspace, nft generates bytecode that expects to find the AH header at NFT_PAYLOAD_TRANSPORT_HEADER both for IPv4 and IPv6.

[PATCH for 4.9 21/39] nvme-rdma: handle cpu unplug when re-establishing the controller

2017-09-17 Thread Levin, Alexander (Sasha Levin)
From: Sagi Grimberg [ Upstream commit c248c64387fac5a6b31b343d9acb78f478e8619c ] If a cpu unplug event has occured, we need to take the minimum of the provided nr_io_queues and the number of online cpus, otherwise we won't be able to connect them as blk-mq mapping won't

[PATCH for 4.9 26/39] cpufreq: intel_pstate: Update pid_params.sample_rate_ns in pid_param_set()

2017-09-17 Thread Levin, Alexander (Sasha Levin)
From: "Rafael J. Wysocki" [ Upstream commit 6e7408acd04d06c04981c0c0fb5a2462b16fae4f ] Fix the debugfs interface for PID tuning to actually update pid_params.sample_rate_ns on PID parameters updates, as changing pid_params.sample_rate_ms via debugfs has no effect

Re: [PATCH for 4.9 11/59] MIPS: fix mem=X@Y commandline processing

2017-09-14 Thread Levin, Alexander (Sasha Levin)
On Thu, Sep 14, 2017 at 08:59:05PM +0200, Mathieu Malaterre wrote: >On Thu, Sep 14, 2017 at 5:51 PM, Levin, Alexander (Sasha Levin) ><alexander.le...@verizon.com> wrote: >> From: Marcin Nowakowski <marcin.nowakow...@imgtec.com> >> >> [ Upstream commit 73fb

[PATCH review for 3.18 25/30] netfilter: nfnl_cthelper: fix incorrect helper->expect_class_max

2017-09-19 Thread Levin, Alexander (Sasha Levin)
From: Liping Zhang [ Upstream commit ae5c682113f9f94cc5e76f92cf041ee624c173ee ] The helper->expect_class_max must be set to the total number of expect_policy minus 1, since we will use the statement "if (class > helper->expect_class_max)" to validate the CTA_EXPECT_CLASS

[PATCH review for 3.18 19/30] net/packet: check length in getsockopt() called with PACKET_HDRLEN

2017-09-19 Thread Levin, Alexander (Sasha Levin)
From: Alexander Potapenko [ Upstream commit fd2c83b35752f0a8236b976978ad4658df14a59f ] In the case getsockopt() is called with PACKET_HDRLEN and optlen < 4 |val| remains uninitialized and the syscall may behave differently depending on its value, and even copy garbage to

[PATCH review for 3.18 20/30] team: fix memory leaks

2017-09-19 Thread Levin, Alexander (Sasha Levin)
From: Pan Bian [ Upstream commit 72ec0bc64b9a5d8e0efcb717abfc757746b101b7 ] In functions team_nl_send_port_list_get() and team_nl_send_options_get(), pointer skb keeps the return value of nlmsg_new(). When the call to genlmsg_put() fails, the memory is not freed(). This

[PATCH review for 3.18 16/30] partitions/efi: Fix integer overflow in GPT size calculation

2017-09-19 Thread Levin, Alexander (Sasha Levin)
From: Alden Tondettar [ Upstream commit c5082b70adfe8e1ea1cf4a8eff92c9f260e364d2 ] If a GUID Partition Table claims to have more than 2**25 entries, the calculation of the partition table size in alloc_read_gpt_entries() will overflow a 32-bit integer and not enough

[PATCH review for 3.18 29/30] xfs: remove kmem_zalloc_greedy

2017-09-19 Thread Levin, Alexander (Sasha Levin)
From: "Darrick J. Wong" [ Upstream commit 08b005f1333154ae5b404ca28766e0ffb9f1c150 ] The sole remaining caller of kmem_zalloc_greedy is bulkstat, which uses it to grab 1-4 pages for staging of inobt records. The infinite loop in the greedy allocation function is

[PATCH review for 3.18 17/30] audit: log 32-bit socketcalls

2017-09-19 Thread Levin, Alexander (Sasha Levin)
From: Richard Guy Briggs [ Upstream commit 62bc306e2083436675e33b5bdeb6a77907d35971 ] 32-bit socketcalls were not being logged by audit on x86_64 systems. Log them. This is basically a duplicate of the call from net/socket.c:sys_socketcall(), but it addresses the impedance

[PATCH review for 3.18 24/30] [media] exynos-gsc: Do not swap cb/cr for semi planar formats

2017-09-19 Thread Levin, Alexander (Sasha Levin)
From: Thibault Saunier [ Upstream commit d7f3e33df4fbdc9855fb151f4a328ec46447e3ba ] In the case of semi planar formats cb and cr are in the same plane in memory, meaning that will be set to 'cb' whatever the format is, and whatever the (packed) order of those

[PATCH review for 4.4 47/47] libata: transport: Remove circular dependency at free time

2017-09-19 Thread Levin, Alexander (Sasha Levin)
From: Gwendal Grignou [ Upstream commit d85fc67dd11e9a32966140677d4d6429ca540b25 ] Without this patch, failed probe would not free resources like irq. ata port tdev object currently hold a reference to the ata port object. Therefore the ata port object release function

[PATCH review for 4.4 10/47] extcon: axp288: Use vbus-valid instead of -present to determine cable presence

2017-09-19 Thread Levin, Alexander (Sasha Levin)
From: Hans de Goede [ Upstream commit 5757aca10146061befd168dab37fb0db1ccd8f73 ] The vbus-present bit in the power status register also gets set to 1 when a usb-host cable (id-pin shorted to ground) is plugged in and a 5v boost converter is supplying 5v to the otg usb bus.

[PATCH review for 4.4 32/47] net/packet: check length in getsockopt() called with PACKET_HDRLEN

2017-09-19 Thread Levin, Alexander (Sasha Levin)
From: Alexander Potapenko [ Upstream commit fd2c83b35752f0a8236b976978ad4658df14a59f ] In the case getsockopt() is called with PACKET_HDRLEN and optlen < 4 |val| remains uninitialized and the syscall may behave differently depending on its value, and even copy garbage to

Re: [PATCH 4.4 22/50] pinctrl: mvebu: Use seq_puts() in mvebu_pinconf_group_dbg_show()

2017-10-06 Thread Levin, Alexander (Sasha Levin)
On Fri, Oct 06, 2017 at 11:20:11AM +0200, Greg Kroah-Hartman wrote: >On Fri, Oct 06, 2017 at 02:09:14AM -0700, Joe Perches wrote: >> On Fri, 2017-10-06 at 10:53 +0200, Greg Kroah-Hartman wrote: >> > 4.4-stable review patch. If anyone has any objections, please let me know. >> >> I hope this patch

[PATCH review for 4.4 07/24] 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.4 06/24] 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 04/24] mfd: ab8500-sysctrl: Handle probe deferral

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Linus Walleij [ Upstream commit 7e9c40c63933a643908d686bd89dfc2315e8c70a ] In the current boot, clients making use of the AB8500 sysctrl may be probed before the ab8500-sysctrl driver. This gives them -EINVAL, but should rather give -EPROBE_DEFER. Before this,

[PATCH review for 4.9 49/50] drm/exynos: g2d: prevent integer overflow in

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Joonyoung Shim [ Upstream commit e41456bfc811f12b5dcda6f2d6849bdff68f6c0a ] The size computations done in the ioctl function use an integer. If userspace submits a request with req->cmd_nr or req->cmd_buf_nr set to INT_MAX, the integer computations overflow later,

[PATCH review for 4.9 05/50] 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.4 08/24] 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 4.9 01/50] cpufreq: Do not clear real_cpus mask on policy init

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: "Rafael J. Wysocki" [ Upstream commit f451014692ae34e587b00de6745e16661cf734d8 ] If new_policy is set in cpufreq_online(), the policy object has just been created and its real_cpus mask has been zeroed on allocation, and the driver's ->init() callback should

[PATCH review for 4.4 03/24] ARM: pxa: Don't rely on public mmc header to include leds.h

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Ulf Hansson [ Upstream commit 40d727a2defa176b78159d445bcf7afcc2ed9021 ] Some of the pxa platforms, balloon3, colibri-pxa270-income, corgi, trizeps4, vpac270, zeus and zylonite depends on leds.h. Explicitly include it instead of relying on the public mmc header

[PATCH review for 4.9 07/50] 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 11/24] 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 34/50] 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.9 32/50] 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.9 26/50] clk: sunxi-ng: Check kzalloc() for errors and cleanup error path

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Stephen Boyd [ Upstream commit 5d806f9fc8e63d7a44e0fd1ef26a7c27efae0e51 ] This kzalloc() could fail. Let's bail out with -ENOMEM here instead of NULL dereferencing. That silences static checkers. We should also cleanup on the error path even though this function

[PATCH review for 4.9 15/50] 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 4.4 19/24] 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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.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 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 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 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 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 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.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 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.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.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 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.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 41/50] ASoC: Intel: boards: remove .pm_ops in all Atom/DPCM machine drivers

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Pierre-Louis Bossart [ Upstream commit 3639ac1cd5177685a5c8abb7230096b680e1d497 ] This patch corrects an omission in bytcr_rt5640 and bytcr_rt5651. All existing machine drivers shall not use .pm_ops to avoid a double suspend, as initially implemented

[PATCH review for 4.9 42/50] [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.9 44/50] 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.9 36/50] 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.9 09/50] mfd: axp20x: Fix axp288 PEK_DBR and PEK_DBF irqs being swapped

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Hans de Goede [ Upstream commit 1af468ebe45591651ec3bafc2e9ddc6fdef70ae0 ] The R in PEK_DBR stands for rising, so it should be mapped to AXP288_IRQ_POKP where the last P stands for positive edge. Likewise PEK_DBF should be mapped to the falling edge, aka the

[PATCH review for 4.9 45/50] s390/prng: Adjust generation of entropy to produce real 256 bits.

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Harald Freudenberger [ Upstream commit d34b1acb78af41b8b8d5c60972b6555ea19f7564 ] The generate_entropy function used a sha256 for compacting together 256 bits of entropy into 32 bytes hash. However, it is questionable if a sha256 can really be used here, as

[PATCH review for 4.9 33/50] 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 4.9 02/50] crypto: ccp - Set the AES size field for all modes

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Gary R Hook [ Upstream commit f7cc02b3c3a33a10dd5bb9e5dfd22e47e09503a2 ] Ensure that the size field is correctly populated for all AES modes. Signed-off-by: Gary R Hook Signed-off-by: Herbert Xu Signed-off-by: Sasha

[PATCH review for 4.9 43/50] 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.9 38/50] brcmfmac: check brcmf_bus_get_memdump result for error

2017-10-07 Thread Levin, Alexander (Sasha Levin)
From: Rafał Miłecki [ Upstream commit f4737a62033d7f3e0db740c449fc62119da7ab8a ] This method may be unsupported (see: USB bus) or may just fail (see: SDIO bus). While at it rework logic in brcmf_sdio_bus_get_memdump function to avoid too many conditional code nesting levels.

[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 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

[PATCH review for 3.18 12/12] [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 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 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 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 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:

<    1   2   3   4   5   6   7   8   9   10   >