[PATCH] tree wide: Use kvfree() than conditional kfree()/vfree()

> ACK for the ACPI changes (and CCing Tony and Boris for the heads-up as they > are way more famailiar with the APEI code than I am). Sure. If kvfree() really is smart enough to figure it out then there it no point in the if (blah) kfree() else vfree(). The drivers/acpi/apei/erst.c code isn't

[PATCH] tree wide: Use kvfree() than conditional kfree()/vfree()

On Monday, November 09, 2015 08:56:10 PM Tetsuo Handa wrote: > There are many locations that do > > if (memory_was_allocated_by_vmalloc) > vfree(ptr); > else > kfree(ptr); > > but kvfree() can handle both kmalloc()ed memory and vmalloc()ed memory > using is_vmalloc_addr(). Unless

[PATCH i915 v2 1/2] i915: wait for fences in mmio_flip()

>> +/* For framebuffer backed by dmabuf, wait for fence */ >> +mutex_lock(>object_name_lock); >The lock here is unfortunate. I thought once a dmabuf as attached to an >object, it persists until the object is destroyed, so afaict the lock here is >unnecessary (as it only protects against

[PATCH i915 v2 2/2] i915: wait for fences in atomic commit

>Could you put the wait inside prepare_plane_fb? Yeah, that works, and it gets rid of the need to loop over the planes. I wait on the fence before taking out the struct_mutex lock like so: @@ -13369,6 +13345,13 @@ intel_prepare_plane_fb(struct drm_plane *plane, if (!obj)

[PATCH] tree wide: Use kvfree() than conditional kfree()/vfree()

There are many locations that do if (memory_was_allocated_by_vmalloc) vfree(ptr); else kfree(ptr); but kvfree() can handle both kmalloc()ed memory and vmalloc()ed memory using is_vmalloc_addr(). Unless callers have special reasons, we can replace this branch with kvfree(). Please

[PATCH 2/2] drm/rockchip: unset pgoff when mmap'ing gems

Commit 371f0f085f629 ("ARM: 8426/1: dma-mapping: add missing range check in dma_mmap()") introduced offset-checking for mappings, which collides with the fake-offset the drm sets for gems. Other drm-drivers set this offset to 0 before doing the mapping, so this looks like the correct way to go

[PATCH 1/2] drm/rockchip: create the fake mmap offset during gem creation

Similarly to the equivalent exynos-fix in commit 48cf53f4343a ("drm/exynos: create a fake mmap offset with gem creation") the rockchip drm driver also calls the drm_gem_create_mmap_offset fake-offset creation on every invocation of rockchip_gem_dumb_map_offset. Move that to the creation of the

[RFC PATCH v3 1/4] drm: Introduce generic probe function for component based masters.

Hi Liviu, Am Montag, 9. November 2015, 12:07:20 schrieb Liviu Dudau: > On Mon, Nov 09, 2015 at 12:03:35PM +, Russell King - ARM Linux wrote: > > On Mon, Nov 09, 2015 at 11:57:27AM +, Liviu Dudau wrote: > > > Meanwhile, what is your suggestion regarding the patchset. I've seen > > > David

[PATCH] tree wide: Use kvfree() than conditional kfree()/vfree()

On 2015/11/09, 04:56, "Tetsuo Handa" wrote: >There are many locations that do > > if (memory_was_allocated_by_vmalloc) >vfree(ptr); > else >kfree(ptr); > >but kvfree() can handle both kmalloc()ed memory and vmalloc()ed memory >using is_vmalloc_addr(). Unless callers have special

[PATCH 1/1] apple-gmux: Assign apple_gmux_data before registering

From: Matthew Garrett Registering the handler after both GPUs will trigger a DDC switch for connector reprobing. This will oops if apple_gmux_data hasn't already been assigned. Reorder the code to do that. Tested-by: Pierre Moreau [MBP 5,3 2009 nvidia MCP79 +

[PATCH 0/1] apple-gmux: Assign apple_gmux_data before registering

Hi Darren, the following patch is a useful fix for apple-gmux by Matthew Garrett which is well over a year old but unfortunately never got merged. The commit message makes it sound as if the fix is only needed for reprobing (in case apple-gmux registers after the DRM drivers). I'm not yet sure

[Bug 92865] MSI R7 370 Armor 2X dpm issues

|--- |INVALID -- You are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/44c2f6a2/attachment.html>

[Bug 92865] MSI R7 370 Armor 2X dpm issues

are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/2b3e8922/attachment.html>

[Bug 92865] MSI R7 370 Armor 2X dpm issues

m.c, quirk list is its place. -- You are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/19bfe8c8/attachment.html>

[Bug 76130] Radeon HD 4570 set dpm state fails after suspend

me (both *ERROR*s in kernel log disappear, performance returns to normal). -- You are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachme

[PATCH] drm/ttm: Set CPU caching mode to cached for BOs being swapped out

On 06.11.2015 22:02, Thomas Hellstrom wrote: > On 11/06/2015 03:33 AM, Michel Dänzer wrote: >> On 05.11.2015 17:47, Thomas Hellstrom wrote: >>> Hi, Michel, >>> >>> On 11/05/2015 09:08 AM, Michel Dänzer wrote: From: Michel Dänzer I ran into the BUG_ON in ttm_tt_swapout,

[Bug 92865] MSI R7 370 Armor 2X dpm issues

|issues and errors |issues -- You are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/e0ca5f98/attachment.html>

[Bug 92865] MSI R7 370 Armor 2X dpm issues and errors

bed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/df51b099/attachment.html>

[RFC PATCH v3 1/4] drm: Introduce generic probe function for component based masters.

On 2015年10月19日 23:07, Liviu Dudau wrote: > A lot of component based DRM drivers use a variant of the same code > as the probe function. They bind the crtc ports in the first iteration > and then scan through the child nodes and bind the encoders attached > to the remote endpoints. Factor the

[PATCH 4/4] drm/imx: Remove of_node assignment from ipuv3-crtc driver probe

The crtc child device driver shouldn't modify the of_node of its platform device in the probe function. Instead, since the previous patch, the IPU core driver sets the of_node when the platform device is created. Drop the now unused custom imx_drm_get_port_by_id function. Signed-off-by: Philipp

[PATCH 3/4] gpu: ipu-v3: Assign of_node of child platform devices to corresponding ports

The crtc child device driver shouldn't have to modify the of_node of its platform device in the probe function. Instead, let the IPU core driver set the of_node when the platform device is created. Also reorder the client_reg array so the elements are in port id order (CSIs first, then DIs).

[PATCH 2/4] gpu: ipu-v3: Remove reg_offset field

This is not used, so remove it. Signed-off-by: Philipp Zabel --- drivers/gpu/ipu-v3/ipu-common.c | 19 +++ 1 file changed, 3 insertions(+), 16 deletions(-) diff --git a/drivers/gpu/ipu-v3/ipu-common.c b/drivers/gpu/ipu-v3/ipu-common.c index ba47b30..97a36e3 100644 ---

[PATCH 1/4] gpu: ipu-v3: drop unused dmfc field from client platform data

This field is never used, drop it. Signed-off-by: Philipp Zabel --- include/video/imx-ipu-v3.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/video/imx-ipu-v3.h b/include/video/imx-ipu-v3.h index 85dedca..eeba753 100644 --- a/include/video/imx-ipu-v3.h +++

[PATCH 1/2] drm/dp: Add definition for Display Control DPCD Registers capability size

cc: Jani Nikula Signed-off-by: Yetunde Adebisi --- include/drm/drm_dp_helper.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h index 1252108..92d9a52 100644 --- a/include/drm/drm_dp_helper.h +++ b/include/drm/drm_dp_helper.h @@

[PATCH 4/4] drm/imx: Remove of_node assignment from ipuv3-crtc driver probe

On Mon, Nov 09, 2015 at 05:28:41PM +0100, Philipp Zabel wrote: > The crtc child device driver shouldn't modify the of_node of its platform > device in the probe function. Instead, since the previous patch, the IPU > core driver sets the of_node when the platform device is created. > > Drop the

[PATCH v4] tests/exynos: add fimg2d performance analysis

Hello, I think this patch should update .gitignore, not for adding the built binary to untracked file list. Also, I want to make clear about the purpose of this test program. What do you want to get after this test? This program runs G2D with randomly chosen number of pixel and shows the elapsed

[PATCH 10/13] tests/exynos: add test for g2d_move

Hello, I think this patch should update .gitignore, not for adding the built binary to untracked file list. But without it, it looks good to me, and I tested it on my Odroid U3 board. Tested-by: Hyungwon Hwang Reviewed-by: Hyungwon Hwang Best regards, Hyungwon Hwang On Tue, 22 Sep 2015

[PATCH 09/13] exynos/fimg2d: add g2d_move

Hello Tobias, I was in vacation last week, so I could run your code today. I found that what g2d_move() does is actually copying not moving, because the operation does not clear the previous area. Would it be possible to generalize g2d_copy() works better, so it could works well in case of the

[PATCH libdrm] intel: Allow some codenames in INTEL_DEVID_OVERRIDE

As well as allowing a hexadecimal PCI ID number, the INTEL_DEVID_OVERRIDE environment variable can now contain one of a few short codenames. The codenames are stored in a small table to map them to a corresponding PCI ID. This makes it easier to use without having to look up the PCI IDs manually.

[PATCH] tree wide: Use kvfree() than conditional kfree()/vfree()

On Mon, Nov 09, 2015 at 08:56:10PM +0900, Tetsuo Handa wrote: > There are many locations that do > > if (memory_was_allocated_by_vmalloc) > vfree(ptr); > else > kfree(ptr); > > but kvfree() can handle both kmalloc()ed memory and vmalloc()ed memory > using is_vmalloc_addr(). Unless

[PATCH] drm: Add aux plane verification in addFB2

On 11/5/2015 10:50 PM, Vandana Kannan wrote: > From: Daniel Vetter > > For render compression, userspace passes aux stride and offset values as an > additional entry in the fb structure. This should not be treated as garbage > and discarded as data belonging to no plane. > This patch introduces

[PATCH] drm/nouveau/abi16: add missing mutex_unlock()

adding missing mutex_unlock() Signed-off-by: Saurabh Sengar --- drivers/gpu/drm/nouveau/nouveau_abi16.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/nouveau/nouveau_abi16.c b/drivers/gpu/drm/nouveau/nouveau_abi16.c index d336c22..280bd8e 100644

[PATCH] drm/nouveau: Fix pre-nv50 pageflip events (v2)

On 11/09/2015 02:02 PM, Ilia Mirkin wrote: > On Mon, Nov 9, 2015 at 7:57 AM, Mario Kleiner > wrote: >> From: Daniel Vetter >> >> Apparently pre-nv50 pageflip events happen before the actual vblank >> period. Therefore that functionality got semi-disabled in >> >> commit

[PATCH] drm/nouveau: Fix pre-nv50 pageflip events

Hi, i just sent out a (v2) of Daniels patch, with my review comments and reviewed-by for the code already applied to the code for convenience. Interspersed below in the patch the review comments for a few small bugs. This and Daniels original patch is only compile tested. I still have that

[PATCH] drm/nouveau: Fix pre-nv50 pageflip events (v2)

From: Daniel Vetter Apparently pre-nv50 pageflip events happen before the actual vblank period. Therefore that functionality got semi-disabled in commit af4870e406126b7ac0ae7c7ce5751f25ebe60f28 Author: Mario Kleiner Date: Tue May 13 00:42:08 2014 +0200

[PATCH] tree wide: Use kvfree() than conditional kfree()/vfree()

On Mon 09-11-15 20:56:10, Tetsuo Handa wrote: > There are many locations that do > > if (memory_was_allocated_by_vmalloc) > vfree(ptr); > else > kfree(ptr); > > but kvfree() can handle both kmalloc()ed memory and vmalloc()ed memory > using is_vmalloc_addr(). Unless callers have

[PATCH 14/19] ARM: sun5i: dt: Add pll3 and pll7 clocks

On Fri, Oct 30, 2015 at 10:21 PM, Maxime Ripard wrote: > Enable the pll3 and pll7 clocks in the DT that are used to drive the > display-related clocks. > > Signed-off-by: Maxime Ripard Acked-by: Chen-Yu Tsai

[RFC PATCH v3 1/4] drm: Introduce generic probe function for component based masters.

Am Montag, den 09.11.2015, 10:53 + schrieb Liviu Dudau: > On Mon, Nov 09, 2015 at 05:39:25PM +0800, Mark yao wrote: > > On 2015年10月19日 23:07, Liviu Dudau wrote: > > >A lot of component based DRM drivers use a variant of the same code > > >as the probe function. They bind the crtc ports

[PATCH 05/19] clk: sunxi: add DRAM gates

On Fri, Oct 30, 2015 at 10:20 PM, Maxime Ripard wrote: > The Allwinner SoCs have a gate controller to gate the access to the DRAM > clock to the some devices that need to access the DRAM directly (mostly > display / image related IPs). > > Use a simple gates driver to support it. > >

[RFC PATCH v3 1/4] drm: Introduce generic probe function for component based masters.

On Mon, Nov 09, 2015 at 12:03:35PM +, Russell King - ARM Linux wrote: > On Mon, Nov 09, 2015 at 11:57:27AM +, Liviu Dudau wrote: > > Meanwhile, what is your suggestion regarding the patchset. I've seen David > > has > > sent Linus a pull request for 4.4-rc1 that includes it. Should we

[RFC PATCH v3 1/4] drm: Introduce generic probe function for component based masters.

On Mon, Nov 09, 2015 at 11:57:27AM +, Liviu Dudau wrote: > Meanwhile, what is your suggestion regarding the patchset. I've seen David has > sent Linus a pull request for 4.4-rc1 that includes it. Should we send a > revert for rockchip commit and then patch later the function? It definitely

[RFC PATCH v3 1/4] drm: Introduce generic probe function for component based masters.

On Mon, Nov 09, 2015 at 11:43:00AM +, Russell King - ARM Linux wrote: > On Mon, Nov 09, 2015 at 05:39:25PM +0800, Mark yao wrote: > > Hi Liviu > > Rockchip drm can't work with drm_of_component_probe function now, > > > > At drm_of_component_probe: > >

[PATCH 00/19] drm: Add Allwinner A10 display engine support

Hi Maxime, On Fri, Oct 30, 2015 at 10:20 PM, Maxime Ripard wrote: > Maxime Ripard (19): > clk: sunxi: Add display clock > clk: sunxi: Add PLL3 clock > clk: sunxi: Add TCON channel0 clock > clk: sunxi: Add TCON channel1 clock > clk: sunxi: add DRAM gates > clk: sunxi: Add Allwinner R8

[RFC PATCH v3 1/4] drm: Introduce generic probe function for component based masters.

On Mon, Nov 09, 2015 at 05:39:25PM +0800, Mark yao wrote: > Hi Liviu > Rockchip drm can't work with drm_of_component_probe function now, > > At drm_of_component_probe: > component_match_add(dev, , compare_of, port); > And original rockchip drm use: >

[PATCH 04/19] clk: sunxi: Add TCON channel1 clock

On Sat, Nov 7, 2015 at 8:01 AM, Maxime Ripard wrote: > Hi, > > On Sat, Oct 31, 2015 at 05:53:26PM +0800, Chen-Yu Tsai wrote: >> On Fri, Oct 30, 2015 at 10:20 PM, Maxime Ripard >> wrote: >> > The TCON is a controller generating the timings to output videos signals, >> > acting like both a CRTC

[PATCH 10/13] tests/exynos: add test for g2d_move

On 9 November 2015 at 09:47, Tobias Jakobi wrote: > Hello Hyungwon, > > Hyungwon Hwang wrote: >> Hello, >> >> I think this patch should update .gitignore, not for adding the built >> binary to untracked file list. > good point. I should do this for the event test as well I guess. > > Going to

[PATCH] tree wide: Use kvfree() than conditional kfree()/vfree()

From: Jan Kara Date: Mon, 9 Nov 2015 13:11:26 +0100 > You can add > > Acked-by: Jan Kara > > for the UDF and fs/xattr.c parts. Please do not quote and entire large patch just to give an ACK. Just quote the minimum necessary context, which is usually just the commit message.

[RFC PATCH v3 1/4] drm: Introduce generic probe function for component based masters.

On Mon, Oct 19, 2015 at 04:07:47PM +0100, Liviu Dudau wrote: > A lot of component based DRM drivers use a variant of the same code > as the probe function. They bind the crtc ports in the first iteration > and then scan through the child nodes and bind the encoders attached > to the remote

[RFC PATCH v3 1/4] drm: Introduce generic probe function for component based masters.

On Mon, Nov 09, 2015 at 05:39:25PM +0800, Mark yao wrote: > On 2015年10月19日 23:07, Liviu Dudau wrote: > >A lot of component based DRM drivers use a variant of the same code > >as the probe function. They bind the crtc ports in the first iteration > >and then scan through the child nodes and

[PATCH v4] tests/exynos: add fimg2d performance analysis

Hello Hyungwon, Hyungwon Hwang wrote: > Hello, > > I think this patch should update .gitignore, not for adding the built > binary to untracked file list. Thanks! > Also, I want to make clear about the purpose of this test program. What > do you want to get after this test? This program runs

[PATCH 10/13] tests/exynos: add test for g2d_move

Hello Hyungwon, Hyungwon Hwang wrote: > Hello, > > I think this patch should update .gitignore, not for adding the built > binary to untracked file list. good point. I should do this for the event test as well I guess. Going to respin the series. With best wishes, Tobias > But without it,

[PATCH 09/13] exynos/fimg2d: add g2d_move

Hello Hyungwon, Hyungwon Hwang wrote: > Hello Tobias, > > I was in vacation last week, so I could run your code today. I found > that what g2d_move() does is actually copying not moving, because the > operation does not clear the previous area. I choose g2d_move() because we also have memcpy()

[Bug 92865] MSI R7 370 Armor 2X dpm issues and errors

m.c-diff.html -- You are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/ed3fe90c/attachment-0001.html>

[Bug 92865] MSI R7 370 Armor 2X dpm issues and errors

was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/86a26682/attachment.html>

[Bug 92865] MSI R7 370 Armor 2X dpm issues and errors

are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/021b45fc/attachment.html>

[Bug 92865] MSI R7 370 Armor 2X dpm issues and errors

because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/5f459c26/attachment.html>

[Bug 92865] MSI R7 370 Armor 2X dpm issues and errors

because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/3cb86595/attachment.html>

[Bug 92865] MSI R7 370 Armor 2X dpm issues and errors

are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/337874a1/attachment-0001.html>

[Bug 76490] Hang during boot when DPM is on (R9 270X)

, if someone's interested: https://bugs.freedesktop.org/show_bug.cgi?id=92865 -- You are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/92e4d

[PATCH] drm/nouveau: Fix pre-nv50 pageflip events (v2)

On Mon, Nov 9, 2015 at 7:57 AM, Mario Kleiner wrote: > From: Daniel Vetter > > Apparently pre-nv50 pageflip events happen before the actual vblank > period. Therefore that functionality got semi-disabled in > > commit af4870e406126b7ac0ae7c7ce5751f25ebe60f28 > Author: Mario Kleiner > Date:

[Bug 92865] MSI R7 370 Armor 2X dpm issues and errors

was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/d863b905/attachment.html>

[Bug 92865] MSI R7 370 Armor 2X dpm issues and errors

ideas other than those above? -- You are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/c74c68ef/attachment.html>

[Bug 92860] [radeonsi][bisected] st/mesa: implement ARB_copy_image - Corruption in ARK Survival Evolved

are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/75248f64/attachment.html>

[Bug 92806] 1 second freezes during new effects UT4

was scrubbed... URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20151109/d7389ad1/attachment.html>

[Bug 104551] Unable to resume from suspend AMD Radeon HD 6570

https://bugzilla.kernel.org/show_bug.cgi?id=104551 --- Comment #7 from Ignacio Taranto --- I tried with and NVIDIA card (GTX 650) in the same board and the issue persist (using either nouveau driver the proprietary one). Using the integrated Intel graphics the suspend works. Hibernating to disk

[git pull] drm for 4.4-rc1

Hi Linus, I Was Almost Tempted To Capitalise Every Word, but then I decided I couldn't read it myself! There are some conflicts with your tree in MAINTAINERS, i915 and amdgpu, they are all fairly trivial, I've pushed a drm-next-merged branch if you want to check anything out. I've also got