Hi,
Wrong sizeof used, fixed in attached patch, and below.
>From 7f430a4104bee5f75929580a66128b7b0e2859d9 Mon Sep 17 00:00:00 2001
From: Vincent Penquerc'h
Date: Tue, 9 Oct 2012 17:18:33 +0100
Subject: [PATCH] drm/omap: fix allocation size for page addresses array
Signed-off-by: Rob Clark
---
Hi Linus,
just a bunch of nouveau fixes, Ben wants to get some alternate versions
into stable.
Dave.
The following changes since commit 1f31c69dac71bebc0f00bc8534a6345782045501:
Merge branch 'drm-intel-fixes' of
git://people.freedesktop.org/~danvet/drm-intel into drm-next (2012-10-07
21:1
On Tue, 09 Oct 2012, Ian Pilcher wrote:
> OK. I'm done.
Sincerely, if you want to get your stuff in the kernel, you need to not
give up so easily.
> I've literally been discussing these patches on this list for months, and
> you bring this up now?
Apologies, I have only been reading the list fo
nts/20121009/68edf543/attachment.html>
xt part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20121009/1adb211c/attachment.html>
ttp://lists.freedesktop.org/archives/dri-devel/attachments/20121009/7628903b/attachment.html>
On 08.10.12 21:50, Rob Clark wrote:
From: Rob Clark
Signed-off-by: Rob Clark
---
drivers/staging/omapdrm/omap_crtc.c | 31 ++-
1 file changed, 6 insertions(+), 25 deletions(-)
diff --git a/drivers/staging/omapdrm/omap_crtc.c
b/drivers/staging/omapdrm/omap_crt
Signed-off-by: Rob Clark
Signed-off-by: Vincent Penquerc'h
---
drivers/staging/omapdrm/omap_gem.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/omapdrm/omap_gem.c
b/drivers/staging/omapdrm/omap_gem.c
index c828743..4c1472c 100644
--- a/drivers/stagin
On Tue, 9 Oct 2012 14:50:46 -0300, Rodrigo Vivi
wrote:
> When cmd isn't IOC_IN | IOC_OUT a null "kdata" goes to "memset", which
> dereferences it.
>
usize should be 0 in that case, since the ioctl is neither copying data
in or out, for example I915_GEM_THROTTLE. To be on the safe side:
if (IOC
goto free_pages;
--
1.7.9.5
-- next part --
A non-text attachment was scrubbed...
Name: 0001-omap_gem-fix-incorrect-allocation-size-for-page-addr.patch
Type: text/x-patch
Size: 1153 bytes
Desc: not available
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments
Em Mon, 08 Oct 2012 19:45:14 +0200
Florian Fainelli escreveu:
> On Monday 08 October 2012 17:49:00 Hans Verkuil wrote:
> > On Mon October 8 2012 17:06:20 Florian Fainelli wrote:
> > > Hi Hans,
> > >
> > > On Thursday 27 September 2012 16:33:30 Hans Verkuil wrote:
> > > > Hi all,
> > > >
> > > >
When cmd isn't IOC_IN | IOC_OUT a null "kdata" goes to "memset", which
dereferences it.
v2: simpler version just using usize = 0 instead of allocating useless memory
Signed-off-by: Rodrigo Vivi
---
drivers/gpu/drm/drm_drv.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/
On Fri, 28 Sep 2012, Ian Pilcher wrote:
> Add the ability for users to define their own EDID quirks via a
> module parameter or sysfs attribute.
Hi Ian -
IMHO this patch should be chopped up to smaller pieces. For example,
change the edid_quirk_list format first (if you must), then add module
pa
When cmd isn't IOC_IN | IOC_OUT a null "kdata" goes to "memset", which
dereferences it.
Signed-off-by: Rodrigo Vivi
---
drivers/gpu/drm/drm_drv.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c
index 1490e76..59
|
--
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/20121009/718b92af/attachment.html>
<http://lists.freedesktop.org/archives/dri-devel/attachments/20121009/3848a219/attachment.html>
2012/10/9 Rob Clark :
> From: Rob Clark
>
> Add a helper fxn to send vblank event after pageflip, plus a handful
> of fixes for other issues that I noticed in the process.
>
patches 1, 5 and 6 are:
Reviewed-by: Inki Dae
https://bugs.freedesktop.org/show_bug.cgi?id=40211
pitam...@free.fr changed:
What|Removed |Added
Version|7.10|8.0
--- Comment #9 from pitam...@free.
https://bugs.freedesktop.org/show_bug.cgi?id=40211
pitam...@free.fr changed:
What|Removed |Added
Attachment #50399|0 |1
is obsolete|
On Die, 2012-10-09 at 13:31 +0200, Christian K?nig wrote:
> We want to use VMs without the IB pool in the future.
>
> v2: also remove it from radeon_vm_finish.
>
> Signed-off-by: Christian K?nig
Reviewed-by: Michel D?nzer
--
Earthling Michel D?nzer | http://www.
https://bugs.freedesktop.org/show_bug.cgi?id=55819
Alex Deucher changed:
What|Removed |Added
Assignee|xorg-driver-...@lists.x.org |dri-devel@lists.freedesktop
Make it possible to allocate a persistent page table.
Signed-off-by: Christian K?nig
Reviewed-by: Alex Deucher
---
drivers/gpu/drm/radeon/radeon.h |1 +
drivers/gpu/drm/radeon/radeon_cs.c |1 +
drivers/gpu/drm/radeon/radeon_gart.c | 20
3 files changed, 18
We want to use VMs without the IB pool in the future.
v2: also remove it from radeon_vm_finish.
Signed-off-by: Christian K?nig
---
drivers/gpu/drm/radeon/radeon.h |2 +-
drivers/gpu/drm/radeon/radeon_gart.c | 34 +++---
drivers/gpu/drm/radeon/radeon_kms.c
Based on Dmitries work, but splitting the code into page
directory and page table handling makes it far more
readable and (hopefully) more reliable.
Allocations of page tables are made from the SA on demand,
that should still work fine since all page tables are of
the same size.
Also using the fa
On Fri, 2012-10-05 at 12:14 -0400, Kristian H?gsberg wrote:
> On Fri, Oct 5, 2012 at 9:36 AM, Imre Deak wrote:
> > Needed by the upcoming DRM raw monotonic timestamp support.
>
> I just had a quick look at driver/input/evdev.c, since evdev devices
> did a similar change recently to allow evdev ti
When cmd isn't IOC_IN | IOC_OUT a null "kdata" goes to "memset", which
dereferences it.
v2: simpler version just using usize = 0 instead of allocating useless memory
Signed-off-by: Rodrigo Vivi
---
drivers/gpu/drm/drm_drv.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/
Signed-off-by: Rob Clark
---
drivers/staging/omapdrm/omap_gem.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/omapdrm/omap_gem.c
b/drivers/staging/omapdrm/omap_gem.c
index c828743..4c1472c 100644
--- a/drivers/staging/omapdrm/omap_gem.c
+++ b/drivers/
On Tue, 2012-10-09 at 12:35 +0300, Imre Deak wrote:
> On Mon, 2012-10-08 at 14:50 -0500, Rob Clark wrote:
> > From: Rob Clark
> >
> > Userspace might not request a vblank event. So it is not an error
> > for 'event' to be NULL, and we shouldn't use it to determine if
> > there is a pending flip
On Mon, 2012-10-08 at 14:50 -0500, Rob Clark wrote:
> From: Rob Clark
>
> Userspace might not request a vblank event. So it is not an error
> for 'event' to be NULL, and we shouldn't use it to determine if
> there is a pending flip already.
>
> Signed-off-by: Rob Clark
> ---
> drivers/staging
From: Damien Lespiau
As we parse the string given on the command line one char at a time, it
seems that we do want a break at every case.
Signed-off-by: Damien Lespiau
---
drivers/gpu/drm/drm_modes.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/gpu/drm/drm_m
From: Mika Kuoppala
Signed-off-by: Mika Kuoppala
---
drivers/gpu/drm/drm_crtc.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
index ef1b221..bb4c686 100644
--- a/drivers/gpu/drm/drm_crtc.c
+++ b/drivers/gpu/d
From: Mika Kuoppala
Possibility to disable polling to find connected displays was
introduced by commit [1]:
commit e58f637bb96d5a0ae0919b9998b891d1ba7e47c9
Author: Chris Wilson
Date: Fri Aug 20 09:13:36 2010 +0100
drm/kms: Add a module parameter to disable polling
Add description for th
Signed-off-by: Rob Clark
Signed-off-by: Vincent Penquerc'h
---
drivers/staging/omapdrm/omap_gem.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/omapdrm/omap_gem.c
b/drivers/staging/omapdrm/omap_gem.c
index c828743..4c1472c 100644
--- a/drivers/stagin
On Tue, Oct 09, 2012 at 01:01:08PM -0500, Vincent Penquerc'h wrote:
> Signed-off-by: Rob Clark
> ---
> drivers/staging/omapdrm/omap_gem.c |4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Having a patch "From:" someone else is fine, but please get a
signed-off-by from them as well.
On 10/8/12 7:22 PM, Ian Pilcher wrote:
> On 09/28/2012 02:03 AM, Ian Pilcher wrote:
>> As promised, this patch series does only the following:
>>
>> * Add user-defined EDID quirks -- via sysfs or a module parameter
>> * Add an EDID quirk flag to disable all HDMI functionality
>>(InfoFrames)
>>
On Tue, 9 Oct 2012 14:50:46 -0300, Rodrigo Vivi wrote:
> When cmd isn't IOC_IN | IOC_OUT a null "kdata" goes to "memset", which
> dereferences it.
>
usize should be 0 in that case, since the ioctl is neither copying data
in or out, for example I915_GEM_THROTTLE. To be on the safe side:
if (IOC_
Signed-off-by: Rob Clark
---
drivers/staging/omapdrm/omap_gem.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/omapdrm/omap_gem.c
b/drivers/staging/omapdrm/omap_gem.c
index c828743..4c1472c 100644
--- a/drivers/staging/omapdrm/omap_gem.c
+++ b/drivers/
When cmd isn't IOC_IN | IOC_OUT a null "kdata" goes to "memset", which
dereferences it.
Signed-off-by: Rodrigo Vivi
---
drivers/gpu/drm/drm_drv.c | 10 +-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_drv.c b/drivers/gpu/drm/drm_drv.c
index 1490e76..59
On 08.10.2012 18:25, alexdeucher at gmail.com wrote:
> From: Alex Deucher
>
> No need to emit them at VM flush as we no longer use
> variable sized page tables now that we support 2 level
> page tables. This matches the behavior of SI (which
> does not support variable sized page tables).
>
> Sig
From: Alex Deucher
If so, skip enabling it to save time.
v2: coding style fixes
Signed-off-by: Alex Deucher
---
drivers/gpu/drm/radeon/evergreen.c |7 ++-
drivers/gpu/drm/radeon/r600.c |6 ++
2 files changed, 12 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/
id->prod_code[0];
> > - eld[19] = edid->prod_code[1];
> > + *(u32 *)(&eld[16]) = edid->display_id.u;
> >
> > if (cea[1] >= 3)
> > for (db = cea + 4; db < cea + cea[2]; db += dbl + 1) {
> > dbl = db[0] & 0x1f;
> > -
> > +
> > switch ((db[0] & 0xe0) >> 5) {
> > case AUDIO_BLOCK:
> > /* Audio Data Block, contains SADs */
> > diff --git a/drivers/gpu/drm/drm_stub.c b/drivers/gpu/drm/drm_stub.c
> > index 21bcd4a..b939d51 100644
> > --- a/drivers/gpu/drm/drm_stub.c
> > +++ b/drivers/gpu/drm/drm_stub.c
> > @@ -46,16 +46,22 @@ EXPORT_SYMBOL(drm_vblank_offdelay);
> > unsigned int drm_timestamp_precision = 20; /* Default to 20 usecs. */
> > EXPORT_SYMBOL(drm_timestamp_precision);
> >
> > +char *drm_edid_quirks = NULL;
> > +EXPORT_SYMBOL(drm_edid_quirks);
> > +
> > MODULE_AUTHOR(CORE_AUTHOR);
> > MODULE_DESCRIPTION(CORE_DESC);
> > MODULE_LICENSE("GPL and additional rights");
> > MODULE_PARM_DESC(debug, "Enable debug output");
> > MODULE_PARM_DESC(vblankoffdelay, "Delay until vblank irq auto-disable
> [msecs]");
> > MODULE_PARM_DESC(timestamp_precision_usec, "Max. error on timestamps
> [usecs]");
> > +MODULE_PARM_DESC(edid_quirks, "MFG:prod:flags[,MFG:prod:flags[...]]\n"
> > + "(See Documentation/EDID/edid_quirks.txt)");
> >
> > module_param_named(debug, drm_debug, int, 0600);
> > module_param_named(vblankoffdelay, drm_vblank_offdelay, int, 0600);
> > module_param_named(timestamp_precision_usec, drm_timestamp_precision,
> int, 0600);
> > +module_param_named(edid_quirks, drm_edid_quirks, charp, 0400);
> >
> > struct idr drm_minors_idr;
> >
> > diff --git a/drivers/gpu/drm/drm_sysfs.c b/drivers/gpu/drm/drm_sysfs.c
> > index 45ac8d6..84dc365 100644
> > --- a/drivers/gpu/drm/drm_sysfs.c
> > +++ b/drivers/gpu/drm/drm_sysfs.c
> > @@ -84,6 +84,11 @@ static CLASS_ATTR_STRING(version, S_IRUGO,
> > __stringify(CORE_PATCHLEVEL) " "
> > CORE_DATE);
> >
> > +static CLASS_ATTR(edid_quirks_size, 0400, drm_edid_quirks_size_show, 0);
> > +
> > +static CLASS_ATTR(edid_quirks, 0600, drm_edid_quirks_show,
> > + drm_edid_quirks_store);
> > +
> > /**
> > * drm_sysfs_create - create a struct drm_sysfs_class structure
> > * @owner: pointer to the module that is to "own" this struct
> drm_sysfs_class
> > @@ -113,10 +118,22 @@ struct class *drm_sysfs_create(struct module
> *owner, char *name)
> > if (err)
> > goto err_out_class;
> >
> > + err = class_create_file(class, &class_attr_edid_quirks_size);
> > + if (err)
> > + goto err_out_version;
> > +
> > + err = class_create_file(class, &class_attr_edid_quirks);
> > + if (err)
> > + goto err_out_quirks_size;
> > +
> > class->devnode = drm_devnode;
> >
> > return class;
> >
> > +err_out_quirks_size:
> > + class_remove_file(class, &class_attr_edid_quirks_size);
> > +err_out_version:
> > + class_remove_file(class, &class_attr_version.attr);
> > err_out_class:
> > class_destroy(class);
> > err_out:
> > @@ -132,6 +149,8 @@ void drm_sysfs_destroy(void)
> > {
> > if ((drm_class == NULL) || (IS_ERR(drm_class)))
> > return;
> > + class_remove_file(drm_class, &class_attr_edid_quirks);
> > + class_remove_file(drm_class, &class_attr_edid_quirks_size);
> > class_remove_file(drm_class, &class_attr_version.attr);
> > class_destroy(drm_class);
> > drm_class = NULL;
> > diff --git a/include/drm/drmP.h b/include/drm/drmP.h
> > index d6b67bb..c947f3e 100644
> > --- a/include/drm/drmP.h
> > +++ b/include/drm/drmP.h
> > @@ -1501,6 +1501,7 @@ extern unsigned int drm_debug;
> >
> > extern unsigned int drm_vblank_offdelay;
> > extern unsigned int drm_timestamp_precision;
> > +extern char *drm_edid_quirks;
> >
> > extern struct class *drm_class;
> > extern struct proc_dir_entry *drm_proc_root;
> > @@ -1612,6 +1613,15 @@ void drm_gem_vm_open(struct vm_area_struct *vma);
> > void drm_gem_vm_close(struct vm_area_struct *vma);
> > int drm_gem_mmap(struct file *filp, struct vm_area_struct *vma);
> >
> > + /* EDID support (drm_edid.c) */
> > +void drm_edid_quirks_param_process(void);
> > +ssize_t drm_edid_quirks_size_show(struct class *class,
> > + struct class_attribute *attr, char *buf);
> > +ssize_t drm_edid_quirks_show(struct class *class, struct
> class_attribute *attr,
> > + char *buf);
> > +ssize_t drm_edid_quirks_store(struct class *class, struct
> class_attribute *attr,
> > + const char *buf, size_t count);
> > +
> > #include "drm_global.h"
> >
> > static inline void
> > diff --git a/include/drm/drm_edid.h b/include/drm/drm_edid.h
> > index 0cac551..713229b 100644
> > --- a/include/drm/drm_edid.h
> > +++ b/include/drm/drm_edid.h
> > @@ -202,11 +202,18 @@ struct detailed_timing {
> > #define DRM_EDID_FEATURE_PM_SUSPEND (1 << 6)
> > #define DRM_EDID_FEATURE_PM_STANDBY (1 << 7)
> >
> > +union edid_display_id {
> > + struct {
> > + __be16 mfg_id;
> > + __le16 prod_code;
> > + } __attribute__((packed)) s;
> > + u32 u;
> > +};
>
> I think adding this union is counterproductive. The u32 version is
> helpful in one comparison and one assignment, while making all the rest
> just slightly more confusing.
>
> > +
> > struct edid {
> > u8 header[8];
> > /* Vendor & product info */
> > - u8 mfg_id[2];
> > - u8 prod_code[2];
> > + union edid_display_id display_id;
> > u32 serial; /* FIXME: byte order */
> > u8 mfg_week;
> > u8 mfg_year;
> > @@ -242,8 +249,6 @@ struct edid {
> > u8 checksum;
> > } __attribute__((packed));
> >
> > -#define EDID_PRODUCT_ID(e) ((e)->prod_code[0] | ((e)->prod_code[1] <<
> 8))
> > -
> > struct drm_encoder;
> > struct drm_connector;
> > struct drm_display_mode;
> > --
> > 1.7.11.4
> >
> > ___
> > dri-devel mailing list
> > dri-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
-- next part --
An HTML attachment was scrubbed...
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20121009/7ec2da10/attachment-0001.html>
On Mon, Oct 08, 2012 at 02:50:40PM -0500, Rob Clark wrote:
> From: Rob Clark
>
> Signed-off-by: Rob Clark
Maybe note in the commit message that after this change pageflip events
will also fire the vblank_event_delivered tracepoint.
Reviewed-by: Daniel Vetter
> ---
> drivers/gpu/drm/i915/int
Hi Laurent,
On Mon, Oct 08, 2012 at 10:52:04PM +0200, Laurent Pinchart wrote:
> Hi Steffen,
>
> On Monday 08 October 2012 14:48:01 Steffen Trumtrar wrote:
> > On Mon, Oct 08, 2012 at 02:13:50PM +0200, Laurent Pinchart wrote:
> > > On Thursday 04 October 2012 19:59:20 Steffen Trumtrar wrote:
> > >
next part --
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 551 bytes
Desc: OpenPGP digital signature
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20121009/8197482c/attachment.pgp>
2012/10/8 :
> + if (speed_cntl & LC_CURRENT_DATA_RATE) {
> + DRM_INFO("PCIE gen 2 link speeds already enabled\n");
> + return;
> + } else
> + DRM_INFO("enabling PCIE gen 2 link speeds, disable with
> radeon.pcie_gen2=0\n");
> +
See documentat
On Tue, Oct 9, 2012 at 3:05 AM, Rafa? Mi?ecki wrote:
> 2012/10/8 :
>> + if (speed_cntl & LC_CURRENT_DATA_RATE) {
>> + DRM_INFO("PCIE gen 2 link speeds already enabled\n");
>> + return;
>> + } else
>> + DRM_INFO("enabling PCIE gen 2 link speeds
On Tue, Oct 9, 2012 at 2:26 AM, Paul Menzel
wrote:
> Am Montag, den 08.10.2012, 17:48 -0400 schrieb alexdeucher at gmail.com:
>> From: Alex Deucher
>>
>> If so, skip enabling it.
>
> Does that fix any error/problem or is it just for speed reasons?
It doesn't fix anything that I know of. Just sp
digitally signed message part
URL:
<http://lists.freedesktop.org/archives/dri-devel/attachments/20121009/7ee68b1c/attachment.pgp>
On 10/8/12 7:22 PM, Ian Pilcher wrote:
On 09/28/2012 02:03 AM, Ian Pilcher wrote:
As promised, this patch series does only the following:
* Add user-defined EDID quirks -- via sysfs or a module parameter
* Add an EDID quirk flag to disable all HDMI functionality
(InfoFrames)
* Add a quirk to
https://bugzilla.kernel.org/show_bug.cgi?id=47481
Anisse Astier changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
https://bugs.freedesktop.org/show_bug.cgi?id=55809
Rob Clark changed:
What|Removed |Added
Assignee|dri-devel@lists.freedesktop |r...@ti.com
|.org
https://bugs.freedesktop.org/show_bug.cgi?id=55809
Priority: medium
Bug ID: 55809
Assignee: dri-devel@lists.freedesktop.org
Summary: omap_drm: fix locking bug on error path
Severity: normal
Classification: Unclassified
OS: Li
From: Alex Deucher
If so, skip enabling it to save time.
v2: coding style fixes
Signed-off-by: Alex Deucher
---
drivers/gpu/drm/radeon/evergreen.c |7 ++-
drivers/gpu/drm/radeon/r600.c |6 ++
2 files changed, 12 insertions(+), 1 deletions(-)
diff --git a/drivers/gpu/drm/
On Mon, Oct 08, 2012 at 03:52:20PM +0300, Tapani P?lli wrote:
> From: Haitao Huang
>
> Export necessary header files used by other components for Android,
> such as libva intel-driver, gralloc, hwcomposer, etc.
>
> Signed-off-by: Haitao Huang
> Signed-off-by: Chad Versace
Reviewed-by: Oliver M
On Tue, Oct 9, 2012 at 3:05 AM, Rafał Miłecki wrote:
> 2012/10/8 :
>> + if (speed_cntl & LC_CURRENT_DATA_RATE) {
>> + DRM_INFO("PCIE gen 2 link speeds already enabled\n");
>> + return;
>> + } else
>> + DRM_INFO("enabling PCIE gen 2 link speeds
On Tue, Oct 9, 2012 at 2:26 AM, Paul Menzel
wrote:
> Am Montag, den 08.10.2012, 17:48 -0400 schrieb alexdeuc...@gmail.com:
>> From: Alex Deucher
>>
>> If so, skip enabling it.
>
> Does that fix any error/problem or is it just for speed reasons?
It doesn't fix anything that I know of. Just speed
On Fri, 28 Sep 2012, Ian Pilcher wrote:
> Add the ability for users to define their own EDID quirks via a
> module parameter or sysfs attribute.
Hi Ian -
IMHO this patch should be chopped up to smaller pieces. For example,
change the edid_quirk_list format first (if you must), then add module
pa
On Die, 2012-10-09 at 13:31 +0200, Christian König wrote:
> We want to use VMs without the IB pool in the future.
>
> v2: also remove it from radeon_vm_finish.
>
> Signed-off-by: Christian König
Reviewed-by: Michel Dänzer
--
Earthling Michel Dänzer | http://www.
Make it possible to allocate a persistent page table.
Signed-off-by: Christian König
Reviewed-by: Alex Deucher
---
drivers/gpu/drm/radeon/radeon.h |1 +
drivers/gpu/drm/radeon/radeon_cs.c |1 +
drivers/gpu/drm/radeon/radeon_gart.c | 20
3 files changed, 18
Based on Dmitries work, but splitting the code into page
directory and page table handling makes it far more
readable and (hopefully) more reliable.
Allocations of page tables are made from the SA on demand,
that should still work fine since all page tables are of
the same size.
Also using the fa
We want to use VMs without the IB pool in the future.
v2: also remove it from radeon_vm_finish.
Signed-off-by: Christian König
---
drivers/gpu/drm/radeon/radeon.h |2 +-
drivers/gpu/drm/radeon/radeon_gart.c | 34 +++---
drivers/gpu/drm/radeon/radeon_kms.c
From: Damien Lespiau
As we parse the string given on the command line one char at a time, it
seems that we do want a break at every case.
Signed-off-by: Damien Lespiau
---
drivers/gpu/drm/drm_modes.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/gpu/drm/drm_m
From: Mika Kuoppala
Signed-off-by: Mika Kuoppala
---
drivers/gpu/drm/drm_crtc.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
index ef1b221..bb4c686 100644
--- a/drivers/gpu/drm/drm_crtc.c
+++ b/drivers/gpu/d
From: Mika Kuoppala
Possibility to disable polling to find connected displays was
introduced by commit [1]:
commit e58f637bb96d5a0ae0919b9998b891d1ba7e47c9
Author: Chris Wilson
Date: Fri Aug 20 09:13:36 2010 +0100
drm/kms: Add a module parameter to disable polling
Add description for th
On Fri, 2012-10-05 at 12:14 -0400, Kristian Høgsberg wrote:
> On Fri, Oct 5, 2012 at 9:36 AM, Imre Deak wrote:
> > Needed by the upcoming DRM raw monotonic timestamp support.
>
> I just had a quick look at driver/input/evdev.c, since evdev devices
> did a similar change recently to allow evdev ti
On Tue, 2012-10-09 at 12:35 +0300, Imre Deak wrote:
> On Mon, 2012-10-08 at 14:50 -0500, Rob Clark wrote:
> > From: Rob Clark
> >
> > Userspace might not request a vblank event. So it is not an error
> > for 'event' to be NULL, and we shouldn't use it to determine if
> > there is a pending flip
On Mon, 2012-10-08 at 14:50 -0500, Rob Clark wrote:
> From: Rob Clark
>
> Userspace might not request a vblank event. So it is not an error
> for 'event' to be NULL, and we shouldn't use it to determine if
> there is a pending flip already.
>
> Signed-off-by: Rob Clark
> ---
> drivers/staging
On 08.10.2012 18:25, alexdeuc...@gmail.com wrote:
From: Alex Deucher
No need to emit them at VM flush as we no longer use
variable sized page tables now that we support 2 level
page tables. This matches the behavior of SI (which
does not support variable sized page tables).
Signed-off-by: Ale
On Mon, Oct 08, 2012 at 02:50:40PM -0500, Rob Clark wrote:
> From: Rob Clark
>
> Signed-off-by: Rob Clark
Maybe note in the commit message that after this change pageflip events
will also fire the vblank_event_delivered tracepoint.
Reviewed-by: Daniel Vetter
> ---
> drivers/gpu/drm/i915/int
On 08.10.12 13:35, Imre Deak wrote:
> On Sat, 2012-10-06 at 03:41 +0200, Mario Kleiner wrote:
>> [...]
>>
>> But then Psychtoolbox checks each timestamp it gets from somewhere
>> "outside" (OML_sync_control / INTEL_swap_events / ALSA audio timestamps,
>> network receive timestamps, evdev, x11, ...)
https://bugzilla.kernel.org/show_bug.cgi?id=47481
Anisse Astier changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|
Hi Laurent,
On Mon, Oct 08, 2012 at 10:52:04PM +0200, Laurent Pinchart wrote:
> Hi Steffen,
>
> On Monday 08 October 2012 14:48:01 Steffen Trumtrar wrote:
> > On Mon, Oct 08, 2012 at 02:13:50PM +0200, Laurent Pinchart wrote:
> > > On Thursday 04 October 2012 19:59:20 Steffen Trumtrar wrote:
> > >
On Mon, Oct 08, 2012 at 02:50:38PM -0500, Rob Clark wrote:
> From: Rob Clark
>
> Add a helper fxn to send vblank event after pageflip, plus a handful
> of fixes for other issues that I noticed in the process.
FWIW, patches 1-5 and 7 (with a fix to 1st) are:
Reviewed-by: Marcin Slusarz
>
> Oth
2012/10/8 :
> + if (speed_cntl & LC_CURRENT_DATA_RATE) {
> + DRM_INFO("PCIE gen 2 link speeds already enabled\n");
> + return;
> + } else
> + DRM_INFO("enabling PCIE gen 2 link speeds, disable with
> radeon.pcie_gen2=0\n");
> +
See documentat
74 matches
Mail list logo
