Re: [REGRESSION] Linux 4.9-rc4: gfx glitches on Intel Sandybridge (was: Re: Linux 4.9-rc4)

2016-11-06 Thread Mihai Donțu
On Sun, 06 Nov 2016 15:48:36 +0100 Martin Steigerwald wrote: > Hi. > > Am Samstag, 5. November 2016, 16:46:33 CET schrieb Linus Torvalds: > > So it's once again a Saturday afternoon rather than Sunday, this time > > because I felt this rc was already big enough. > > With kernel 4.9-rc4 I saw gf

Re: [Intel-gfx] [Announcement] 2016-Q3 release of XenGT - a Mediated Graphics Passthrough Solution from Intel

2016-11-06 Thread Jike Song
Hi all, We are pleased to announce another update of Intel GVT-g for Xen. Intel GVT-g is a full GPU virtualization solution with mediated pass-through, starting from 4th generation Intel Core(TM) processors with Intel Graphics processors. A virtual GPU instance is maintained for each VM, with p

[REGRESSION] Linux 4.9-rc4: gfx glitches on Intel Sandybridge (was: Re: Linux 4.9-rc4)

2016-11-06 Thread Martin Steigerwald
Hi. Am Samstag, 5. November 2016, 16:46:33 CET schrieb Linus Torvalds: > So it's once again a Saturday afternoon rather than Sunday, this time > because I felt this rc was already big enough. With kernel 4.9-rc4 I saw gfx corruptions like https://martin-steigerwald.de/tmp/display-issues-with-ker

Re: [PATCH] leds: Add mutex protection in brightness_show()

2016-11-06 Thread Hans de Goede
Hi, On 04-11-16 17:46, Hans de Goede wrote: Hi, On 04-11-16 17:06, Jacek Anaszewski wrote: Hi Hans, On 11/04/2016 12:53 PM, Hans de Goede wrote: Hi, On 04-11-16 08:52, Jacek Anaszewski wrote: Initially the claim about no need for lock in brightness_show() was valid as the function was just

Linux 4.9: Reported regressions as of Sunday, 2016-11-06

2016-11-06 Thread Thorsten Leemhuis
Hi! Here is my third regression report for Linux 4.9. It lists 17 regressions I'm aware of. 6 of them are new; 3 got fixed since last weeks report (a fourth looks fixed as well). The console problem ("console: don't prefer first registered [...]") got reported to me multiple times, but the revert t

Re: [PATCH 0/3] iopmem : A block device for PCIe memory

2016-11-06 Thread Stephen Bates
On Tue, October 25, 2016 3:19 pm, Dave Chinner wrote: > On Tue, Oct 25, 2016 at 05:50:43AM -0600, Stephen Bates wrote: >> >> Dave are you saying that even for local mappings of files on a DAX >> capable system it is possible for the mappings to move on you unless the >> FS supports locking? >> > >

Re: [RFC PATCH] ARM: dts: add panel and tcon nodes to Allwinner A33 Q8 tablet dts

2016-11-06 Thread Hans de Goede
Hi, On 06-11-16 12:11, Icenowy Zheng wrote: All A33 Q8 tablets features a LCD panel, with a resolution of either 800x480 or 1024x600. Add "bone" device nodes to the device tree. Bone ? Signed-off-by: Icenowy Zheng He, as discussed in the other thread since sun8i-a33-q8-tablet.dts is use

Re: [PATCH 7/9] staging: iio: tsl2583: fix issue with changes to calibscale and int_time not being set on the chip

2016-11-06 Thread Brian Masney
On Sun, Nov 06, 2016 at 12:03:53PM +, Jonathan Cameron wrote: > On 03/11/16 12:56, Brian Masney wrote: > > When updating the in_illuminance_calibscale and > > in_illuminance_integration_time sysfs attributes, these values were not > > actually written to the chip. The chip would continue to use

[PATCH] Staging: speakup: Remove unnecessary space after cast

2016-11-06 Thread Shiva Kerdel
The affected files have been modified to remove redundant spaces after casts to solve checkpatch.pl checks. Signed-off-by: Shiva Kerdel --- drivers/staging/speakup/main.c | 42 ++--- drivers/staging/speakup/selection.c | 2 +- drivers/staging/speakup/seriali

[PATCH v2] net: 3com: typhoon: fix typhoon_get_link_ksettings

2016-11-06 Thread Philippe Reynes
When moving from typhoon_get_settings to typhoon_getlink_ksettings in the commit f7a5537cd2a5 ("net: 3com: typhoon: use new api ethtool_{get|set}_link_ksettings"), we use a local variable supported but we forgot to update the struct ethtool_link_ksettings with this value. We also initialize advert

[PATCH] net: xgbe: use new api ethtool_{get|set}_link_ksettings

2016-11-06 Thread Philippe Reynes
The ethtool api {get|set}_settings is deprecated. We move this driver to new api {get|set}_link_ksettings. Signed-off-by: Philippe Reynes --- drivers/net/ethernet/amd/xgbe/xgbe-ethtool.c | 68 ++ 1 files changed, 37 insertions(+), 31 deletions(-) diff --git a/drivers/n

Re: [PATCH 2/2] leds: core: Remove delayed_set_value property from struct led_classdev

2016-11-06 Thread Jacek Anaszewski
Withdrawing this patch as it breaks restoring brightness on resume. Thanks, Jacek Anaszewski On 10/05/2016 12:03 PM, Jacek Anaszewski wrote: delayed_set_value property was introduced in the commit d23a22a74fde ("leds: delay led_set_brightness if stopping soft-blink"). Its aim was to allow calli

Re: [PATCH v3 1/1] x86/platform/intel-mid: Retrofit pci_platform_pm_ops ->get_state hook

2016-11-06 Thread Thorsten Leemhuis
Hi! On 26.10.2016 17:01, Andy Shevchenko wrote: > On Wed, 2016-10-26 at 15:06 +0100, Bryan O'Donoghue wrote: >> On Tue, 2016-10-25 at 08:19 +0200, Lukas Wunner wrote: > >> So the conclusion is to apply this patch now and go and look further @ >> locking in a separate series right ? There's not much

Re: [PATCH v2 1/1] ARM: dmaengine: sun6i: share the dma driver with sun50i

2016-11-06 Thread Maxime Ripard
On Sat, Nov 05, 2016 at 04:05:48PM +0800, Hao Zhang wrote: > Add soc a64 dma support. > > Signed-off-by: Hao Zhang > --- > drivers/dma/sun6i-dma.c | 19 +++ > 1 file changed, 19 insertions(+) > > diff --git a/drivers/dma/sun6i-dma.c b/drivers/dma/sun6i-dma.c > index 8346199..00f

Re: [PATCH v2 0/1] ARM: dmaengine: sun6i: share the dma driver with sun50i

2016-11-06 Thread Maxime Ripard
Hi, On Sat, Nov 05, 2016 at 04:05:23PM +0800, hao zhang wrote: > Hi, > > This patch add the allwinner soc a64 dma support on already done driver > sun6i-dma. > > A64 is a 64bit SOC, it has 8 channel DMA which flexible > data width of 8/16/32/64-bit. Detailed info about it is on > Allwinner_A64

Re: Linux 4.9: Reported regressions as of Sunday, 2016-10-30

2016-11-06 Thread Thorsten Leemhuis
Lo! On 01.11.2016 09:18, Paul Bolle wrote: > On Sun, 2016-10-30 at 14:20 +0100, Thorsten Leemhuis wrote: >> As always: Are you aware of any other regressions? Then please let me >> know (simply CC regressi...@leemhuis.info). > Do build regressions count? That's a good question. > Because I was tr

Re: [PATCH] drm: move allocation out of drm_get_format_name()

2016-11-06 Thread Rob Clark
On Sun, Nov 6, 2016 at 4:47 AM, Christian König wrote: > Am 05.11.2016 um 17:49 schrieb Rob Clark: >> >> On Sat, Nov 5, 2016 at 12:38 PM, Eric Engestrom wrote: >>> >>> On Saturday, 2016-11-05 13:11:36 +0100, Christian König wrote: Am 05.11.2016 um 02:33 schrieb Eric Engestrom: > >>>

[PATCH] x86/efi: Fix EFI memmap pointer size warning

2016-11-06 Thread Borislav Petkov
Hi Matt, please doublecheck me on this but I think we're fine using an unsigned long. Thanks. --- From: Borislav Petkov Date: Sun, 6 Nov 2016 13:49:10 +0100 Subject: [PATCH] x86/efi: Fix EFI memmap pointer size warning MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-E

Re: [BUG][REGRESSION] mangled display since -rc1

2016-11-06 Thread Martin Kepplinger
Am 2016-11-06 um 12:43 schrieb Martin Kepplinger: > Am 2016-11-06 um 12:21 schrieb Thorsten Leemhuis: >> On 01.11.2016 12:47, Jani Nikula wrote: >>> On Tue, 01 Nov 2016, Martin Kepplinger wrote: I'll come up with a nouveau system example and it was quite easy to bisect. To quote the mer

[PATCH RFC] rtlwifi: btcoexist: fix port assignment

2016-11-06 Thread Nicholas Mc Guire
The port assignment in the if case should be to AUX not MAIN. Fixes: commit baa170229095 ("rtlwifi: btcoexist: Implement antenna selection") Signed-off-by: Nicholas Mc Guire --- problem located by coccinelle in: drivers/net/wireless/realtek/rtlwifi/btcoexist/halbtcoutsrc.c:exhalbtc_set_ant_num()

[PATCH 1/2] VMCI: drop condition with no effect

2016-11-06 Thread Nicholas Mc Guire
The condition calls the same code, further the code called does not differentiate between old/new-style VMX anyway so the if/else here is simply not needed. Fixes: 06164d2b72aa ("VMCI: queue pairs implementation.") Signed-off-by: Nicholas Mc Guire --- problem found by coccinelle drivers/misc/

[PATCH 2/2] VMCI: drop unused variable

2016-11-06 Thread Nicholas Mc Guire
The boolean vmci_page_files is used to differentiate between old style VMX and new-style, but the code actually only used this variable at one point in an if/else that had no effect so it can be removed. Fixes: 06164d2b72aa ("VMCI: queue pairs implementation.") Signed-off-by: Nicholas Mc Guire -

[PATCH] This is a patch to the capi.c file that fixes one instance of the following error: ERROR: do not use assignment in if condition

2016-11-06 Thread Naeil ZOUEIDI
Staging: i4l: act200: Signed-off-by: Naeil Zoueidi --- drivers/staging/i4l/act2000/capi.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/i4l/act2000/capi.c b/drivers/staging/i4l/act2000/capi.c index 62f5629..b494917 100644 --- a/drivers/staging/i4l/act2000

Re: [REGRESSION] "console: don't prefer first registered if DT specifies stdout-path" breaks console on video outputs of various ARM boards

2016-11-06 Thread Thorsten Leemhuis
JFYI: I added below report to the list of regressions for Linux 4.9. I'll watch this thread for further updates on this issue to document progress in my weekly reports. Please let me know via regressi...@leemhuis.info in case the discussion moves to a different place (bugzilla or another mail threa

Re: [PATCH 7/9] staging: iio: tsl2583: fix issue with changes to calibscale and int_time not being set on the chip

2016-11-06 Thread Jonathan Cameron
On 03/11/16 12:56, Brian Masney wrote: > When updating the in_illuminance_calibscale and > in_illuminance_integration_time sysfs attributes, these values were not > actually written to the chip. The chip would continue to use the old > parameters. Extracted out tsl2583_set_als_gain() and > tsl2583_

Re: [PATCH 9/9] staging: iio: tsl2583: remove redundant write to the control register in taos_probe()

2016-11-06 Thread Jonathan Cameron
On 03/11/16 12:56, Brian Masney wrote: > taos_probe() calls i2c_smbus_write_byte() to select the control > register, however there is no subsequent calls to i2c_smbus_read_byte(). > The write call is unnecessary and is removed by this patch. > > Verified that the driver still functions correctly u

Re: [PATCH 8/9] staging: iio: tsl2583: check if chip is suspended in in_illuminance_calibrate_store

2016-11-06 Thread Jonathan Cameron
On 03/11/16 12:56, Brian Masney wrote: > in_illuminance_calibrate_store() did not check to see if the chip is > suspended. This patch adds the proper check. The return value from > taos_als_calibrate() was also not checked in this function, so the > proper check was also added while changes are bei

Re: [PATCH 6/9] staging: iio: tsl2583: remove redudant i2c call in taos_als_calibrate()

2016-11-06 Thread Jonathan Cameron
On 03/11/16 12:56, Brian Masney wrote: > taos_als_calibrate() queries the control register to determine if the > unit is powered on and has the ADC enabled. It then queries the same > register a second time to determine if the ADC reading is valid. This > patch removes the redundant i2c_smbus_read_

Re: [PATCH 5/9] staging: iio: tsl2583: don't shutdown chip when updating the lux table

2016-11-06 Thread Jonathan Cameron
On 03/11/16 12:56, Brian Masney wrote: > in_illuminance_lux_table_store() shuts down the chip, updates the > contents of the lux table, and then turns the chip back on. The values > in lux table are not used by the chip and are only used internally by > the driver. It is not necessary to change the

Re: [PATCH 4/9] staging: iio: tsl2583: remove redundant power off sequence in taos_chip_on()

2016-11-06 Thread Jonathan Cameron
On 03/11/16 12:56, Brian Masney wrote: > taos_chip_on() explicitly turns the sensor power on and then writes the > 8 registers that are stored in taos_config. The first register in > taos_config is the CONTROL register and the configuration is set to > turn the power off. The existing state sequenc

Re: [PATCH 3/9] staging: iio: tsl2583: fixed ordering of comments

2016-11-06 Thread Jonathan Cameron
On 03/11/16 12:56, Brian Masney wrote: > in taos_defaults() > > The comments in taos_defaults() appear after the line of code > that they apply to. This patch moves the comments so that they appear > before the code. Some of the comments were updated to be more > informative. > > Signed-off-by: B

Re: [PATCH 2/9] staging: iio: tsl2583: removed unused code from device probing

2016-11-06 Thread Jonathan Cameron
On 03/11/16 12:56, Brian Masney wrote: > taos_probe() queries the all of the sensor's registers and loads all of > the values into a buffer stored on the stack. Only the chip ID register > was actually used. Change the probe function to just query the chip ID > register on the device. > > Verified

Re: [BUG][REGRESSION] mangled display since -rc1

2016-11-06 Thread Martin Kepplinger
Am 2016-11-06 um 12:21 schrieb Thorsten Leemhuis: > On 01.11.2016 12:47, Jani Nikula wrote: >> On Tue, 01 Nov 2016, Martin Kepplinger wrote: >>> I'll come up with a nouveau system example and it was quite easy to bisect. >>> To quote the merge commit msg: >>> This also required some changes outsi

Re: [PATCH 1/9] staging: iio: tsl2583: i2c_smbus_write_byte() / i2c_smbus_read_byte() migration

2016-11-06 Thread Jonathan Cameron
On 03/11/16 12:56, Brian Masney wrote: > There were several places where the driver would first call > i2c_smbus_write_byte() to select the register on the device, and then > call i2c_smbus_read_byte() to get the contents of that register. The > code would look roughly like: > > /* Select register

Re: [PATCH 1/9] staging: iio: tsl2583: i2c_smbus_write_byte() / i2c_smbus_read_byte() migration

2016-11-06 Thread Jonathan Cameron
On 03/11/16 12:56, Brian Masney wrote: > There were several places where the driver would first call > i2c_smbus_write_byte() to select the register on the device, and then > call i2c_smbus_read_byte() to get the contents of that register. The > code would look roughly like: > > /* Select register

Re: Regression in kernel 4.9-rcX - bisected to commit a481daa88fd4

2016-11-06 Thread Thorsten Leemhuis
On 31.10.2016 00:32, Larry Finger wrote: > With the 4.9-rcX kernel, my Dell Latitude D600 laptop has a kernel panic on > shutdown in routine radeon_connector_unregister(). This problem has been > bisected to show that commit a481daa88fd4 ("drm/radeon: always apply pci > shutdown callbacks") is a

Re: [BUG][REGRESSION] mangled display since -rc1

2016-11-06 Thread Thorsten Leemhuis
On 01.11.2016 12:47, Jani Nikula wrote: > On Tue, 01 Nov 2016, Martin Kepplinger wrote: >> I'll come up with a nouveau system example and it was quite easy to bisect. >> To quote the merge commit msg: >> This also required some changes outside of the IOMMU code, but these are >> acked by the res

Re: [REGRESSION] "console: don't prefer first registered if DT specifies stdout-path" breaks console on video outputs of various ARM boards

2016-11-06 Thread Hans de Goede
Hi, On 05-11-16 11:40, Paul Burton wrote: On Friday, 4 November 2016 14:22:17 GMT Hans de Goede wrote: Hi, On 04-11-16 13:30, Paul Burton wrote: Hi Hans, On Friday, 4 November 2016 13:11:34 GMT Hans de Goede wrote: Hi All, While booting 4.9-rc# for the first time on an Allwinner A33 tablet

[PATCH 2/2] Adding a missing parenthesis to an assignment and fixing my first patch This patch is a fix to the error ERROR: do not use assignment in if condition found by the checkpatch tool

2016-11-06 Thread Naeil ZOUEIDI
Staging: i4l: act2000: Signed-off-by: Naeil ZOUEIDI --- drivers/staging/i4l/act2000/capi.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/i4l/act2000/capi.c b/drivers/staging/i4l/act2000/capi.c index b494917..62dee39 100644 --- a/drivers/staging/i4l/act2000/

Re: [PATCH 2/6] pinctrl: rockchip: add support for rk1108

2016-11-06 Thread Linus Walleij
On Thu, Nov 3, 2016 at 1:34 PM, Andy Yan wrote: > Add basic support for rk1108 soc > > Signed-off-by: Andy Yan I only see this patch from the series, I guess this is the only patch affecting pin control so thanks for not spamming :) Please resend with Heiko's requested fixes and his Reviewed-b

Re: [PATCH] net: 3com: typhoon: fix typhoon_get_link_ksettings

2016-11-06 Thread Sergei Shtylyov
Hello. On 11/5/2016 6:11 PM, Philippe Reynes wrote: When moving from typhoon_get_settings to typhoon_getlink_ksettings in the commit commit f7a5537cd2a5 ("net: 3com: typhoon: use new api One "commit" is enough. :-) ethtool_{get|set}_link_ksettings"), we use a local variable supported but

Re: [PATCH] drm: move allocation out of drm_get_format_name()

2016-11-06 Thread Christian König
Am 05.11.2016 um 17:49 schrieb Rob Clark: On Sat, Nov 5, 2016 at 12:38 PM, Eric Engestrom wrote: On Saturday, 2016-11-05 13:11:36 +0100, Christian König wrote: Am 05.11.2016 um 02:33 schrieb Eric Engestrom: +typedef char drm_format_name_buf[32]; Please don't use a typedef for this, just defi

Re: [PATCH/RFC] z3fold: use per-page read/write lock

2016-11-06 Thread Vitaly Wool
On Sun, Nov 6, 2016 at 12:38 AM, Andi Kleen wrote: > Vitaly Wool writes: > >> Most of z3fold operations are in-page, such as modifying z3fold >> page header or moving z3fold objects within a page. Taking >> per-pool spinlock to protect per-page objects is therefore >> suboptimal, and the idea of

Re: [PATCH] This is a patch to the capi.c file that fixes one instance of the following error: ERROR: do not use assignment in if condition

2016-11-06 Thread kbuild test robot
-the-capi-c-file-that-fixes-one-instance-of-the-following-error-ERROR-do-not-use-assignment-in-if-condition/20161106-165820 config: i386-randconfig-x077-201645 (attached as .config) compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 reproduce: # save the attached .config to linux build tree

Re: scripts/gcc-plugins/gcc-common.h:4:22: fatal error: bversion.h: No such file or directory

2016-11-06 Thread Emese Revfy
On Sun, 6 Nov 2016 07:09:57 +0800 kbuild test robot wrote: > Hi Emese, > > FYI, the error/warning still remains. > >++(scripts/gcc-plugin.sh:12): main(): gcc-4.6 -E -x c++ - -o /dev/null > -Iscripts/gcc-plugins -Iplugin/include >scripts/Makefile.gcc-plugins:30: warning: your gcc install

[Resend][PATCH] cpufreq: conservative: Decrease frequency faster when the timer deferred

2016-11-06 Thread Stratos Karafotis
Conservative governor changes the CPU frequency in steps. That means that if a CPU runs at max frequency, it will need several sampling periods to return at min frequency when the workload is finished. If the timer that calculates the load and target frequency is deferred, the governor might need

[PATCH v6] mm: Add memory allocation watchdog kernel thread.

2016-11-06 Thread Tetsuo Handa
Commit 63f53dea0c9866e9 ("mm: warn about allocations which stall for too long") was a great step for reducing possibility of silent hang up problem caused by memory allocation stalls [1]. But it became clear that Michal Hocko is not going to make warn_alloc() bullet proof [2]. Therefore, I again pr

arch/xtensa/include/asm/initialize_mmu.h:41: Error: invalid register 'atomctl' for 'wsr' instruction

2016-11-06 Thread kbuild test robot
Hi Pete, FYI, the error/warning still remains. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: bc33b0ca11e3df46a4fa7639ba488c9d4911 commit: d0b73b488c55df905ea8faaad079f8535629ed26 xtensa: Add config files for Diamond 233L - Rev C processor variant

<    1   2   3