On Tue, Mar 27, 2018 at 01:30:04PM +0200, Michael Grzeschik wrote:
> On Mon, Mar 26, 2018 at 06:17:44PM +0200, Michael Grzeschik wrote:
> > We move drm_kms_helper_poll_init behind the drm_fbdev_cma_init so the
> > set_par will be called and fb will be active.
> >
>
> As this commit message is not
On Mon, Mar 26, 2018 at 11:24:42PM +0200, Peter Rosin wrote:
> Hi!
>
> [I got to v2 sooner than expected]
>
> I have an Atmel sama5d31 hooked up to an lvds encoder and then
> on to an lvds panel. Which seems like something that has been
> done one or two times before...
>
> The problem is that t
On Wed, Mar 28, 2018 at 02:24:48AM +0100, Emil Velikov wrote:
> From: Deepak Sharma
>
> Modify vgem_init to take platform dev as parent in drm_dev_init.
> This will make drm device available at "/sys/devices/platform/vgem"
> in x86 chromebook.
>
> Cc: Daniel Vetter
> Signed-off-by: Deepak Sharm
On Wed, Mar 28, 2018 at 02:24:49AM +0100, Emil Velikov wrote:
> From: Emil Velikov
>
> Previous commit removed the only reason why we were allowing NULL as
> a parent device. With that resolved, we can enforce nobody else does
> that mistake.
>
> With that we can drop the ugly drm_dev_set_unique
On Wed, Mar 28, 2018 at 02:24:51AM +0100, Emil Velikov wrote:
> From: Emil Velikov
>
> As of last commit we hide this from the drivers.
>
> Effectively reverts commit a6b5fac59cb216ac906f02300d3630c24520d9ef.
>
> Cc: Daniel Vetter
> Signed-off-by: Emil Velikov
Yay!
Reviewed-by: Daniel Vette
On Wed, Mar 28, 2018 at 02:24:50AM +0100, Emil Velikov wrote:
> From: Emil Velikov
>
> Ealier commit a325725633c26aa66ab940f762a6b0778edf76c0 did not attribute
> that virtio can be either PCI or a platform device and removed the
> .set_busid hook. Whereas only the "platform" instance should have
On Wed, Mar 28, 2018 at 09:47:40AM +0300, Oleksandr Andrushchenko wrote:
> From: Noralf Trønnes
>
> Use srcu to protect drm_device.unplugged in a race free manner.
> Drivers can use drm_dev_enter()/drm_dev_exit() to protect and mark
> sections preventing access to device resources that are not av
Hi Peter,
On Mon, 26 Mar 2018 09:35:02 +0200
Peter Rosin wrote:
> I have an sama5d31-based system with 64MB of memory and a 1920x1080
> LVDS display wired for 16-bpp. When I enable legacy fbdev support,
> the contiguous memory allocator invariably fails with the order-11
> allocation for a 1920x
From: Noralf Trønnes
Use srcu to protect drm_device.unplugged in a race free manner.
Drivers can use drm_dev_enter()/drm_dev_exit() to protect and mark
sections preventing access to device resources that are not available
after the device is gone.
Suggested-by: Daniel Vetter
Signed-off-by: Nora
On Wed, Mar 28, 2018 at 09:47:41AM +0300, Oleksandr Andrushchenko wrote:
> From: Oleksandr Andrushchenko
>
> Add support for Xen para-virtualized frontend display driver.
> Accompanying backend [1] is implemented as a user-space application
> and its helper library [2], capable of running as a We
On 03/28/2018 10:42 AM, Daniel Vetter wrote:
kms side looks good now too.
Reviewed-by: Daniel Vetter
Thank you
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
https://bugzilla.kernel.org/show_bug.cgi?id=199123
--- Comment #16 from david becerra (davidbecerrapor...@gmail.com) ---
the new issue is https://bugzilla.kernel.org/show_bug.cgi?id=199229
--
You are receiving this mail because:
You are watching the assignee of the bug.
_
Am Mittwoch, den 28.03.2018, 09:02 +0200 schrieb Daniel Vetter:
> On Tue, Mar 27, 2018 at 01:30:04PM +0200, Michael Grzeschik wrote:
> > On Mon, Mar 26, 2018 at 06:17:44PM +0200, Michael Grzeschik wrote:
> > > We move drm_kms_helper_poll_init behind the drm_fbdev_cma_init so
> > > the
> > > set_par
https://bugs.freedesktop.org/show_bug.cgi?id=104064
--- Comment #51 from taij...@posteo.de ---
(In reply to Bjorn from comment #50)
> That did not help. However, looking closer at my dmesg output on the
> 4.17-wip branch, I see that there's actually some error loading the intel
> graphics firmware
Hi,
On 06/02/18 14:05, Peter Ujfalusi wrote:
> Hi,
>
> Changes since v3:
> - Moved the new normalize_zpos bool to be around another bools
> - Extended the commit message for sti that the drm_atomic_helper_check() is
> going to ask for async_update due to the legacy cursor usage.
>
> Changes si
Hi,
On 27/03/18 11:23, Daniel Vetter wrote:
> The only thing that omap_gem_free_object does that might need the
> magic protection of struct_mutex (of keeping all objects alive if that
> lock is held, even if the last reference is gone) is the mm_list
> manipulation.
>
> But that is already prote
On Wed, Mar 28, 2018 at 11:52 AM, Tomi Valkeinen wrote:
> Hi,
>
> On 27/03/18 11:23, Daniel Vetter wrote:
>> The only thing that omap_gem_free_object does that might need the
>> magic protection of struct_mutex (of keeping all objects alive if that
>> lock is held, even if the last reference is go
On Wed, Mar 28, 2018 at 11:08 AM, Lucas Stach wrote:
> Am Mittwoch, den 28.03.2018, 09:02 +0200 schrieb Daniel Vetter:
>> On Tue, Mar 27, 2018 at 01:30:04PM +0200, Michael Grzeschik wrote:
>> > On Mon, Mar 26, 2018 at 06:17:44PM +0200, Michael Grzeschik wrote:
>> > > We move drm_kms_helper_poll_in
On 03/28/18 03:24, Emil Velikov wrote:
> From: Emil Velikov
>
> Ealier commit a325725633c26aa66ab940f762a6b0778edf76c0 did not attribute
> that virtio can be either PCI or a platform device and removed the
> .set_busid hook. Whereas only the "platform" instance should have been
> removed.
>
> Si
Hi, Daniel!
I just noticed I have missed one change in the patch:
the below must be static.
On 03/28/2018 10:42 AM, Daniel Vetter wrote:
+enum drm_mode_status display_mode_valid(struct drm_crtc *crtc,
+ const struct drm_display_mode *mode)
+{
+ struct xen_drm_front_drm_pipel
https://bugs.freedesktop.org/show_bug.cgi?id=100979
--- Comment #13 from Przemek ---
Using the opportunity that I was working on another bug report with AMD DC
kernel driver, and have git cloned agd5f drm-next-4.17-wip branch, I have tried
to debug the situation on hibernate process with this exp
On 28/03/18 13:12, Daniel Vetter wrote:
>> This triggers WARN_ON(!mutex_is_locked(&dev->struct_mutex)), we have a
>> few of those. I need to reverse engineer the omap_gem locking a bit to
>> understand what's needed. And I need to figure out what exactly does
>> struct_mutext protect =).
>
> Oops
The only thing that omap_gem_free_object does that might need the
magic protection of struct_mutex (of keeping all objects alive if that
lock is held, even if the last reference is gone) is the mm_list
manipulation.
But that is already protected by the separate omapdrm->list_lock,
which means stru
On 2018-03-28 12:30, Tomi Valkeinen wrote:
> Hi,
>
> On 06/02/18 14:05, Peter Ujfalusi wrote:
>> Hi,
>>
>> Changes since v3:
>> - Moved the new normalize_zpos bool to be around another bools
>> - Extended the commit message for sti that the drm_atomic_helper_check() is
>> going to ask for asyn
On 28/03/18 15:10, Peter Ujfalusi wrote:
>> Thanks. I think this looks fine, I'll push via drm-misc.
>
> There is v5 on the list.
Yep, that's what I pushed. Somehow I managed to reply to this older
thread...
Tomi
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Bus
On Wed, Mar 28, 2018 at 09:34:54AM +0200, Boris Brezillon wrote:
> Hi Peter,
>
> On Mon, 26 Mar 2018 09:35:02 +0200
> Peter Rosin wrote:
>
> > I have an sama5d31-based system with 64MB of memory and a 1920x1080
> > LVDS display wired for 16-bpp. When I enable legacy fbdev support,
> > the contig
On Wed, 28 Mar 2018 14:22:36 +0200
Daniel Vetter wrote:
> On Wed, Mar 28, 2018 at 09:34:54AM +0200, Boris Brezillon wrote:
> > Hi Peter,
> >
> > On Mon, 26 Mar 2018 09:35:02 +0200
> > Peter Rosin wrote:
> >
> > > I have an sama5d31-based system with 64MB of memory and a 1920x1080
> > > LVDS
Hi Laurent,
Thank you for the patch.
On 26/02/18 21:45, Laurent Pinchart wrote:
> The entities in the pipeline are all started when the LIF is setup.
> Remove the outdated comment that state otherwise.
>
> Signed-off-by: Laurent Pinchart
I'll start with the easy ones :-)
Reviewed-by: Kieran B
Hi Laurent,
Thanks for the patch,
On 26/02/18 21:45, Laurent Pinchart wrote:
> The vsp1_drm_pipeline enabled field is set but never used. Remove it.
>
Indeed - not used.
> Signed-off-by: Laurent Pinchart
Reviewed-by: Kieran Bingham
> ---
> drivers/media/platform/vsp1/vsp1_drm.c | 4
>
On Sun, Mar 25, 2018 at 12:59:53PM +0200, Christian König wrote:
> Use this function to set an sg entry to point to device resources mapped
> using dma_map_resource(). The page pointer is set to NULL and only the DMA
> address, length and offset values are valid.
NAK. Please provide a higher leve
On Sun, Mar 25, 2018 at 12:59:54PM +0200, Christian König wrote:
> From: "wda...@nvidia.com"
>
> Add an interface to find the first device which is upstream of both
> devices.
Please work with Logan and base this on top of the outstanding peer
to peer patchset.
__
On Thursday, March 15, 2018 06:16:24 PM Colin King wrote:
> From: Colin Ian King
>
> There is a missing indentation following an if statement, fix this.
>
> Detected by Coccinelle:
> drivers/video/fbdev/aty/mach64_ct.c:183:2-15: code aligned with
> following code on line 184
>
> Signed-off-by:
On Thursday, March 22, 2018 03:13:32 PM Gustavo A. R. Silva wrote:
> Assign true or false to boolean variables instead of an integer value.
>
> This issue was detected with the help of Coccinelle.
>
> Signed-off-by: Gustavo A. R. Silva
Patch queued for 4.17, thanks.
Best regards,
--
Bartlomiej
On Sunday, January 07, 2018 12:56:45 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 Jan 2018 11:33:59 +0100
>
> Replace an error code for the indication of a memory allocation failure
> in this function.
>
> Fixes: 1da177e4c3f41524e886b7f1b8a0c1fc7321cac ("Linux-2.6.12-rc2: In
On Sunday, January 07, 2018 12:58:47 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 Jan 2018 12:34:22 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus Elfr
On Sunday, January 07, 2018 02:34:51 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 Jan 2018 14:02:36 +0100
>
> * Return an error code without storing it in an intermediate variable.
>
> * Delete the label "error" and local variable "retval"
> which became unnecessary with t
On Sunday, January 07, 2018 05:02:21 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 Jan 2018 15:56:11 +0100
>
> Up to four checks could be repeated by the ufx_usb_probe() function
> during error handling even if the relevant properties can be determined
> for the involved varia
Hi Laurent,
Thank you for the patch.
On 26/02/18 21:45, Laurent Pinchart wrote:
> Various types of objects subclassing vsp1_entity currently store a
> pointer to the pipeline. Move the pointer to vsp1_entity to simplify the
> code and avoid storing the pipeline in more entity subclasses later.
T
On Sunday, January 07, 2018 05:04:04 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 Jan 2018 16:07:11 +0100
>
> * Return an error code without storing it in an intermediate variable.
>
> * Delete the label "error" and local variable "retval"
> which became unnecessary with t
On Sunday, January 07, 2018 05:06:13 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 Jan 2018 16:12:40 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus Elfr
On Sunday, January 07, 2018 07:34:44 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 7 Jan 2018 19:27:28 +0100
>
> Replace the specification of a data structure by a pointer dereference
> as the parameter for the operator "sizeof" to make the corresponding size
> determination a b
On Monday, November 27, 2017 11:31:10 AM Timur Tabi wrote:
> On 11/27/17 3:00 AM, SF Markus Elfring wrote:
> > From: Markus Elfring
> > Date: Mon, 27 Nov 2017 09:56:09 +0100
> >
> > Omit an extra message for a memory allocation failure in this function.
> >
> > This issue was detected by using th
On Tue, Mar 27, 2018 at 10:23:53AM +0200, Daniel Vetter wrote:
> Just want to clean out all grep hits. gem_free_object is deprecated.
>
> Signed-off-by: Daniel Vetter
Reviewed-by: Sean Paul
> Cc: Sandy Huang
> Cc: "Heiko Stübner"
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: linux-rockc..
On Monday, November 27, 2017 01:06:30 PM Jingoo Han wrote:
> On Sunday, November 26, 2017 9:17 AM, SF Markus Elfring wrote:
> >
> > From: Markus Elfring
> > Date: Sun, 26 Nov 2017 15:00:16 +0100
> >
> > Omit an extra message for a memory allocation failure in this function.
> >
> > This issue w
On Monday, November 27, 2017 01:07:15 PM Jingoo Han wrote:
> On Sunday, November 26, 2017 9:18 AM, SF Markus Elfring wrote:
> >
> > From: Markus Elfring
> > Date: Sun, 26 Nov 2017 15:03:03 +0100
> >
> > Replace the specification of a data structure by a pointer dereference
> > as the parameter f
On Sunday, November 26, 2017 10:32:06 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Sun, 26 Nov 2017 22:13:55 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus El
Hi Laurent,
On 26/02/18 21:45, Laurent Pinchart wrote:
> The DRM pipeline handling code uses the entity's pipe list head to check
> whether the entity is already included in a pipeline. This method is a
> bit fragile in the sense that it uses list_empty() on a list_head that
> is a list member. Re
https://bugs.freedesktop.org/show_bug.cgi?id=104064
--- Comment #52 from Bjorn ---
(In reply to taijian from comment #51)
>
> OK, I'm not seeing that at all, and I also have a Kaby Lake chip. What
> version of Linux-Firmware are you using?
1.169.3.
I tried going back to an older kernel, and th
On Tuesday, December 05, 2017 06:12:30 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Tue, 5 Dec 2017 17:50:11 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus El
On Tuesday, December 05, 2017 06:13:33 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Tue, 5 Dec 2017 17:53:19 +0100
>
> Replace the specification of a data structure by a pointer dereference
> as the parameter for the operator "sizeof" to make the corresponding size
> determination a
On Tuesday, December 05, 2017 07:00:22 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Tue, 5 Dec 2017 18:45:42 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus El
On Monday, November 27, 2017 06:58:22 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Mon, 27 Nov 2017 18:14:41 +0100
>
> Omit an extra message for a memory allocation failure in this function.
>
> This issue was detected by using the Coccinelle software.
>
> Signed-off-by: Markus El
On 26/02/18 21:45, Laurent Pinchart wrote:
> Move the duplicated DRM pipeline configuration code to a function and
> call it from vsp1_du_setup_lif() and vsp1_du_atomic_flush().
>
> Signed-off-by: Laurent Pinchart
LGTM.
I thought I had a bit of deja-vu on this patch ... but I can't seem to fi
On Monday, November 27, 2017 06:59:57 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Mon, 27 Nov 2017 18:18:34 +0100
>
> Replace the specification of a data structure by a pointer dereference
> as the parameter for the operator "sizeof" to make the corresponding size
> determination a
On Monday, November 27, 2017 07:01:14 PM SF Markus Elfring wrote:
> From: Markus Elfring
> Date: Mon, 27 Nov 2017 18:40:23 +0100
>
> The local variable "fbdev" will be reassigned by a following statement.
> Thus omit the explicit initialisation at the beginning.
>
> Signed-off-by: Markus Elfring
On Wednesday, February 14, 2018 04:56:53 PM Jingoo Han wrote:
> On Wednesday, February 14, 2018 7:01 AM wrote:
> >
> > Exynos5, Exynos4 and S5PV210 platforms have been converted to
> > use Device Tree and Exynos DRM driver long time ago. Remove
> > dead platform code for these platforms and updat
Hi Laszlo,
On 28 March 2018 at 11:27, Laszlo Ersek wrote:
> On 03/28/18 03:24, Emil Velikov wrote:
>> From: Emil Velikov
>>
>> Ealier commit a325725633c26aa66ab940f762a6b0778edf76c0 did not attribute
>> that virtio can be either PCI or a platform device and removed the
>> .set_busid hook. Wherea
Hi Laurent,
On 26/02/18 21:45, Laurent Pinchart wrote:
> The DRM pipeline setup code used at atomic commit time is similar to the
> setup code used when enabling the pipeline. Move it to a separate
> function in order to share it.
>
> Signed-off-by: Laurent Pinchart
Assuming no hidden secret co
On 28 March 2018 at 08:16, Daniel Vetter wrote:
> On Wed, Mar 28, 2018 at 02:24:48AM +0100, Emil Velikov wrote:
>> From: Deepak Sharma
>>
>> Modify vgem_init to take platform dev as parent in drm_dev_init.
>> This will make drm device available at "/sys/devices/platform/vgem"
>> in x86 chromebook
Quoting Emil Velikov (2018-03-28 02:24:48)
> From: Deepak Sharma
>
> Modify vgem_init to take platform dev as parent in drm_dev_init.
> This will make drm device available at "/sys/devices/platform/vgem"
> in x86 chromebook.
>
> Cc: Daniel Vetter
> Signed-off-by: Deepak Sharma
> Reviewed-by: S
Hi Laurent,
On 26/02/18 21:45, Laurent Pinchart wrote:
> The vsp1_du_setup_lif() function setups the DRM pipeline input manually.
s/ setups the / sets up the /
> This duplicates the code from the vsp1_du_pipeline_setup_input()
> function. Replace the manual implementation by a call to the functi
Am 28.03.2018 um 14:38 schrieb Christoph Hellwig:
On Sun, Mar 25, 2018 at 12:59:54PM +0200, Christian König wrote:
From: "wda...@nvidia.com"
Add an interface to find the first device which is upstream of both
devices.
Please work with Logan and base this on top of the outstanding peer
to peer
On Tue, Mar 27, 2018 at 05:25:21PM +0200, Greg Kroah-Hartman wrote:
> On Tue, Mar 27, 2018 at 11:18:03AM +0200, Daniel Vetter wrote:
> > On Tue, Mar 27, 2018 at 10:34:07AM +0200, Greg Kroah-Hartman wrote:
> > > On Tue, Mar 27, 2018 at 10:23:52AM +0200, Daniel Vetter wrote:
> > > > vboxvideo doesn't
On Wed, Mar 28, 2018 at 10:38:35AM +0300, Oleksandr Andrushchenko wrote:
> From: Noralf Trønnes
>
> Use srcu to protect drm_device.unplugged in a race free manner.
> Drivers can use drm_dev_enter()/drm_dev_exit() to protect and mark
> sections preventing access to device resources that are not av
On 28 March 2018 at 15:49, Chris Wilson wrote:
> Quoting Emil Velikov (2018-03-28 02:24:48)
>> From: Deepak Sharma
>>
>> Modify vgem_init to take platform dev as parent in drm_dev_init.
>> This will make drm device available at "/sys/devices/platform/vgem"
>> in x86 chromebook.
>>
>> Cc: Daniel V
https://bugs.freedesktop.org/show_bug.cgi?id=105786
Bug ID: 105786
Summary: System freeze, amdgpu 4.17-WIP,
4.16.0-rc1-576e538e5fe6
Product: DRI
Version: DRI git
Hardware: Other
OS: All
Status: N
On Sat, Mar 17, 2018 at 2:33 PM, Stefan Schake wrote:
> Hey John,
>
> On Fri, Mar 16, 2018 at 10:48 PM, John Stultz wrote:
>> In trying to integrate the new gralloc_handle.h with the
>> drm_hwcomposer, I started seeing the following compilation
>> errors:
>>
>> In file included from external/drm_
https://bugs.freedesktop.org/show_bug.cgi?id=105733
--- Comment #5 from Emil Velikov ---
Hi Allan, just sharing some ideas - I'm not working on the AMD drivers
Make sure you're not using libdrm* 2.4.90 - it has some nasty bugs.
Afterwords, try to track down exactly what's causing the problem a
On Wed, Mar 28, 2018 at 10:19 AM, Rob Herring wrote:
> On Sat, Mar 17, 2018 at 2:33 PM, Stefan Schake wrote:
>> Hey John,
>>
>> On Fri, Mar 16, 2018 at 10:48 PM, John Stultz wrote:
>>> In trying to integrate the new gralloc_handle.h with the
>>> drm_hwcomposer, I started seeing the following com
Hi Dave,
The following changes since commit 192b4af6cd28cdad9b42fd79c21a90a2aeb0bec7:
drm/tegra: Shutdown on driver unbind (2018-03-19 09:57:32 +0100)
are available in the Git repository at:
git://anongit.freedesktop.org/tegra/linux tags/drm/tegra/for-4.16-fixes
for you to fetch changes up
https://bugs.freedesktop.org/show_bug.cgi?id=105788
Bug ID: 105788
Summary: Account request for drm-misc
Product: DRI
Version: DRI git
Hardware: Other
OS: All
Status: NEW
Severity: normal
Priorit
https://bugs.freedesktop.org/show_bug.cgi?id=105788
--- Comment #1 from Oleksandr Andrushchenko ---
Created attachment 138393
--> https://bugs.freedesktop.org/attachment.cgi?id=138393&action=edit
SSH public key
--
You are receiving this mail because:
You are the assignee for the bug._
https://bugs.freedesktop.org/show_bug.cgi?id=105788
--- Comment #2 from Oleksandr Andrushchenko ---
Created attachment 138394
--> https://bugs.freedesktop.org/attachment.cgi?id=138394&action=edit
GPG key
--
You are receiving this mail because:
You are the assignee for the bug.
Building amdkfd without MMU notifiers is broken:
In file included from drivers/gpu/drm/amd/amdkfd/kfd_mqd_manager_vi.c:28:
drivers/gpu/drm/amd/amdkfd/kfd_priv.h:584:22: error: field 'mmu_notifier' has
incomplete type
This adds the missing 'select MMU_NOTIFIER' line to make it build
cleanly all t
From: Emil Velikov
Currently the error pointer returned by msm_alloc_stolen_fb gets passed
to drm_framebuffer_remove. The latter handles only NULL pointers, thus
a nasty crash will occur.
Drop the unnecessary fail label and the associated checks - both err and
fb will be set at this stage.
Cc:
Am 28.03.2018 um 17:47 schrieb Logan Gunthorpe:
On 28/03/18 09:07 AM, Christian König wrote:
Am 28.03.2018 um 14:38 schrieb Christoph Hellwig:
On Sun, Mar 25, 2018 at 12:59:54PM +0200, Christian König wrote:
From: "wda...@nvidia.com"
Add an interface to find the first device which is upstre
Am 28.03.2018 um 17:53 schrieb Arnd Bergmann:
Building amdkfd without MMU notifiers is broken:
In file included from drivers/gpu/drm/amd/amdkfd/kfd_mqd_manager_vi.c:28:
drivers/gpu/drm/amd/amdkfd/kfd_priv.h:584:22: error: field 'mmu_notifier' has
incomplete type
This adds the missing 'select M
https://bugs.freedesktop.org/show_bug.cgi?id=105788
Daniel Vetter changed:
What|Removed |Added
Assignee|dri-devel@lists.freedesktop |sitewranglers@lists.freedes
https://bugs.freedesktop.org/show_bug.cgi?id=105788
--- Comment #3 from Daniel Vetter ---
Ack. Also acked by Gustavo and Sean on irc.
--
You are receiving this mail because:
You are the assignee for the bug.___
dri-devel mailing list
dri-devel@lists.f
On 27 March 2018 at 00:27, John Stultz wrote:
> On Tue, Mar 20, 2018 at 10:36 AM, Emil Velikov
> wrote:
>> From: Tomasz Figa
>>
>> This patch makes the code not rely anymore on libpciaccess when compiled
>> for Android to eliminate ioperm() and iopl() syscalls required by that
>> library. As a
We have seen a case of a bad reference count for vblanks with the
Rockchip VOP:
[ cut here ]
WARNING: CPU: 1 PID: 383 at drivers/gpu/drm/drm_irq.c:1198
drm_vblank_put+0x40/0xcc
Modules linked in: brcmfmac brcmutil
CPU: 1 PID: 383 Comm: kwork
On Fri, Mar 23, 2018 at 05:28:03PM +0100, Noralf Trønnes wrote:
>
> Den 23.03.2018 16.35, skrev Ville Syrjala:
> > From: Ville Syrjälä
> >
> > mipi_dbi_enable_flush() wants to call the fb->dirty() hook from the
> > bowels of the .atomic_enable() hook. That prevents us from taking the
> > plane mu
From: Emil Velikov
Currently the error pointer returned by msm_alloc_stolen_fb gets passed
to drm_framebuffer_remove. The latter handles only NULL pointers, thus
a nasty crash will occur.
Drop the unnecessary fail label and the associated checks - both err and
fb will be set at this stage.
Cc:
https://bugs.freedesktop.org/show_bug.cgi?id=101442
Emil Velikov changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
Am 28.03.2018 um 18:25 schrieb Logan Gunthorpe:
On 28/03/18 10:02 AM, Christian König wrote:
Yeah, that looks very similar to what I picked up from the older
patches, going to read up on that after my vacation.
Yeah, I was just reading through your patchset and there are a lot of
similarities.
From: Colin Ian King
Trivial fix to spelling mistake in warning message text
Signed-off-by: Colin Ian King
---
drivers/gpu/drm/nouveau/nvkm/subdev/bios/init.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/bios/init.c
b/drivers/gpu/drm/
Hi Laurent,
On 26/02/18 21:45, Laurent Pinchart wrote:
> To implement fully dynamic plane assignment to pipelines, we need to
> reassign the BRU and BRS to the DRM pipelines in the atomic commit
> handler. In preparation for this setup factor out the BRU source pad
> code and call it both at LIF s
On 28/03/18 13:27, Kieran Bingham wrote:
> Hi Laurent,
>
> Thank you for the patch.
>
> On 26/02/18 21:45, Laurent Pinchart wrote:
>> The entities in the pipeline are all started when the LIF is setup.
>> Remove the outdated comment that state otherwise.
>>
>> Signed-off-by: Laurent Pinchart
>
I noticed a couple of rough edges yesterday as I was running some tests,
so here's v2 which fixes them. Most notably, I was experiencing issues when
the crtc was being disabled with abandoned events and vblank timeouts.
Note that I've bundled the private_obj change into this set since it
shares th
Now that we have private state handled by the core, we can use those
instead of rolling our own swap_state for private data.
Originally posted here: https://patchwork.freedesktop.org/patch/211361/
Changes in v2:
- Use state->state in disp duplicate_state callback (Jeykumar)
Changes in v3:
- Upd
Factor out the commit_tail() portions of complete_commit() into a
separate function to facilitate moving to the atomic helpers in future
patches.
Changes in v2:
- None
Cc: Jeykumar Sankaran
Signed-off-by: Sean Paul
---
drivers/gpu/drm/msm/msm_atomic.c | 25 -
1 file cha
Don't leave the event != NULL once it's consumed, this is used a signal
to the atomic helpers that the event will be handled by the driver.
Changes in v2:
- None
Cc: Jeykumar Sankaran
Signed-off-by: Sean Paul
---
drivers/gpu/drm/msm/disp/mdp4/mdp4_crtc.c | 1 +
drivers/gpu/drm/msm/disp/mdp5/md
Ensure that any queued events are issued when disabling the crtc. This
avoids timeouts when we come back and wait for dependencies (like the
previous frame's flip_done).
Changes in v2:
- None
Signed-off-by: Sean Paul
---
drivers/gpu/drm/msm/disp/mdp5/mdp5_crtc.c | 9 +
1 file changed, 9
Moving further towards switching fully to the the atomic helpers, this
patch removes the hand-rolled worker nonblock commit code and uses the
atomic helpers commit_work model.
Changes in v2:
- Remove commit_destroy()
- Shuffle order of commit_tail calls to further serialize commits
- Use stall in
Now that all of the msm-specific goo is tucked safely away we can switch
over to using the atomic helper commit directly. \o/
Changes in v2:
- None
Cc: Abhinav Kumar
Signed-off-by: Sean Paul
---
drivers/gpu/drm/msm/msm_atomic.c | 120 +--
drivers/gpu/drm/msm/msm_drv
Hi Dave,
Here's a lonely fix for -next, please pull at your convenience.
drm-misc-next-fixes-2018-03-28:
UABI:
- Mask mode type garbage from userspace (Ville)
Cc: Ville Syrjälä
Cheers, Sean
The following changes since commit b4eec0fa537165efc3265cdbb4bac06e6bdaf596:
Merge tag 'drm-intel-
Am 28.03.2018 um 20:57 schrieb Logan Gunthorpe:
On 28/03/18 12:28 PM, Christian König wrote:
I'm just using amdgpu as blueprint because I'm the co-maintainer of it
and know it mostly inside out.
Ah, I see.
The resource addresses are translated using dma_map_resource(). As far
as I know that
On Mon, Mar 26, 2018 at 3:22 PM, Samuel Li wrote:
> To reduce some warnings.
>
> Signed-off-by: Samuel Li
Reviewed-by: Alex Deucher
And pushed.
Alex
> ---
> drivers/gpu/drm/drm_prime.c | 13 +
> 1 file changed, 13 insertions(+)
>
> diff --git a/drivers/gpu/drm/drm_prime.c b/driv
tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next
head: 6761ccd944db730dca6f9033781b4645a6eecfd4
commit: 8e62ef009a4bb61d25d391d0384f3314cc6d2059 [1046/1062] drm/amd/pp: Use
atombios api directly in powerplay
config: x86_64-randconfig-s5-03290316 (attached as .config)
co
https://bugs.freedesktop.org/show_bug.cgi?id=105463
erhar...@mailbox.org changed:
What|Removed |Added
Summary|[r600g] 4210 piglit |[r600g] 3047 piglit
1 - 100 of 128 matches
Mail list logo