Flash 11.2 content displays in shades of green and purple only, and in a horizontally compressed space

2013-09-28 Thread John Hupp
=00, secondary=01, subordinate=01, sec-latency=32
 I/O behind bridge: d000-dfff
 Memory behind bridge: fe90-feaf
 Prefetchable memory behind bridge: fff0-000f
 Secondary status: 66MHz- FastB2B+ ParErr- DEVSEL=medium >TAbort-
Reset- FastB2B-
 PriDiscTmr- SecDiscTmr- DiscTmrStat- DiscTmrSERREn-

00:1f.0 ISA bridge: Intel Corporation 82801DB/DBL (ICH4/ICH4-L) LPC
Interface Bridge (rev 01)
 Control: I/O+ Mem+ BusMaster+ SpecCycle+ MemWINV- VGASnoop- ParErr-
Stepping- SERR+ FastB2B- DisINTx-
 Status: Cap- 66MHz- UDF- FastB2B+ ParErr- DEVSEL=medium >TAbort-
SERR- TAbort-
SERR- TAbort-
SERR- TAbort-
SERR- TAbort-
SERR- TAbort+
SERR- http://lists.freedesktop.org/archives/dri-devel/attachments/20130928/7d3b3c96/attachment-0001.html>


[patch] drm/radeon/dpm: off by one in si_set_mc_special_registers()

2013-09-28 Thread Dan Carpenter
These checks should be = instead of .  j is used as an offset into
the table-mc_reg_address[] array and that has
SMC_SISLANDS_MC_REGISTER_ARRAY_SIZE (16) elements.

Signed-off-by: Dan Carpenter dan.carpen...@oracle.com

diff --git a/drivers/gpu/drm/radeon/si_dpm.c b/drivers/gpu/drm/radeon/si_dpm.c
index cfe5d4d..4a1220c 100644
--- a/drivers/gpu/drm/radeon/si_dpm.c
+++ b/drivers/gpu/drm/radeon/si_dpm.c
@@ -5184,7 +5184,7 @@ static int si_set_mc_special_registers(struct 
radeon_device *rdev,
table-mc_reg_table_entry[k].mc_data[j] 
|= 0x100;
}
j++;
-   if (j  SMC_SISLANDS_MC_REGISTER_ARRAY_SIZE)
+   if (j = SMC_SISLANDS_MC_REGISTER_ARRAY_SIZE)
return -EINVAL;
 
if (!pi-mem_gddr5) {
@@ -5194,7 +5194,7 @@ static int si_set_mc_special_registers(struct 
radeon_device *rdev,
table-mc_reg_table_entry[k].mc_data[j] 
=

(table-mc_reg_table_entry[k].mc_data[i]  0x)  16;
j++;
-   if (j  SMC_SISLANDS_MC_REGISTER_ARRAY_SIZE)
+   if (j = SMC_SISLANDS_MC_REGISTER_ARRAY_SIZE)
return -EINVAL;
}
break;
@@ -5207,7 +5207,7 @@ static int si_set_mc_special_registers(struct 
radeon_device *rdev,
(temp_reg  0x) |

(table-mc_reg_table_entry[k].mc_data[i]  0x);
j++;
-   if (j  SMC_SISLANDS_MC_REGISTER_ARRAY_SIZE)
+   if (j = SMC_SISLANDS_MC_REGISTER_ARRAY_SIZE)
return -EINVAL;
break;
default:
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[pull] drm/msm: fixes for 3.12-rc2

2013-09-28 Thread Rob Clark
Hi Dave,

A small fix + deal with fallout of iommu changes + use new
drm_gem_dumb_destroy helper.

The following changes since commit d8524ae9d6f492a9c6db9f4d89c5f9b8782fa2d5:

  Merge branch 'drm-fixes' of
git://people.freedesktop.org/~airlied/linux (2013-09-22 19:51:49
-0700)

are available in the git repository at:


  git://people.freedesktop.org/~robclark/linux msm-fixes-3.12-rc2

for you to fetch changes up to 30600a9092dc44694fbb4d0c3fa796e977fd5c96:

  drm/msm: use drm_gem_dumb_destroy helper (2013-09-28 10:14:06 -0400)


Joerg Roedel (1):
  drm/msm: Remove iommu include from mdp4_kms.c

Rob Clark (2):
  drm/msm: deal with mach/iommu.h removal
  drm/msm: use drm_gem_dumb_destroy helper

Thomas Meyer (1):
  drm/msm: Odd PTR_ERR usage

 drivers/gpu/drm/msm/mdp4/mdp4_kms.c | 2 --
 drivers/gpu/drm/msm/msm_drv.c   | 8 
 drivers/gpu/drm/msm/msm_gem.c   | 7 ---
 3 files changed, 4 insertions(+), 13 deletions(-)
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[PATCH 1/5] drm: Remove unused variable in drm_global_item_ref()

2013-09-28 Thread Damien Lespiau
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
 drivers/gpu/drm/drm_global.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/gpu/drm/drm_global.c b/drivers/gpu/drm/drm_global.c
index f731116..3d2e91c 100644
--- a/drivers/gpu/drm/drm_global.c
+++ b/drivers/gpu/drm/drm_global.c
@@ -67,7 +67,6 @@ int drm_global_item_ref(struct drm_global_reference *ref)
 {
int ret;
struct drm_global_item *item = glob[ref-global_type];
-   void *object;
 
mutex_lock(item-mutex);
if (item-refcount == 0) {
@@ -85,7 +84,6 @@ int drm_global_item_ref(struct drm_global_reference *ref)
}
++item-refcount;
ref-object = item-object;
-   object = item-object;
mutex_unlock(item-mutex);
return 0;
 out_err:
-- 
1.8.3.1

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[PATCH 2/5] drm: Remove unused variable in drm_prime_sg_to_page_addr_arrays()

2013-09-28 Thread Damien Lespiau
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
 drivers/gpu/drm/drm_prime.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/drm_prime.c b/drivers/gpu/drm/drm_prime.c
index 276d470..56805c3 100644
--- a/drivers/gpu/drm/drm_prime.c
+++ b/drivers/gpu/drm/drm_prime.c
@@ -637,14 +637,13 @@ int drm_prime_sg_to_page_addr_arrays(struct sg_table 
*sgt, struct page **pages,
unsigned count;
struct scatterlist *sg;
struct page *page;
-   u32 len, offset;
+   u32 len;
int pg_index;
dma_addr_t addr;
 
pg_index = 0;
for_each_sg(sgt-sgl, sg, sgt-nents, count) {
len = sg-length;
-   offset = sg-offset;
page = sg_page(sg);
addr = sg_dma_address(sg);
 
-- 
1.8.3.1

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[PATCH 3/5] drm: Remove unused variable in drm_fb_helper_pan_display()

2013-09-28 Thread Damien Lespiau
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
 drivers/gpu/drm/drm_fb_helper.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c
index f6f6cc7..e69d26b 100644
--- a/drivers/gpu/drm/drm_fb_helper.c
+++ b/drivers/gpu/drm/drm_fb_helper.c
@@ -852,7 +852,6 @@ int drm_fb_helper_pan_display(struct fb_var_screeninfo *var,
struct drm_fb_helper *fb_helper = info-par;
struct drm_device *dev = fb_helper-dev;
struct drm_mode_set *modeset;
-   struct drm_crtc *crtc;
int ret = 0;
int i;
 
@@ -863,8 +862,6 @@ int drm_fb_helper_pan_display(struct fb_var_screeninfo *var,
}
 
for (i = 0; i  fb_helper-crtc_count; i++) {
-   crtc = fb_helper-crtc_info[i].mode_set.crtc;
-
modeset = fb_helper-crtc_info[i].mode_set;
 
modeset-x = var-xoffset;
-- 
1.8.3.1

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[PATCH 4/5] drm: Remove unused variable in drm_pick_crtcs()

2013-09-28 Thread Damien Lespiau
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
 drivers/gpu/drm/drm_fb_helper.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c
index e69d26b..f61ac23 100644
--- a/drivers/gpu/drm/drm_fb_helper.c
+++ b/drivers/gpu/drm/drm_fb_helper.c
@@ -1357,7 +1357,6 @@ static int drm_pick_crtcs(struct drm_fb_helper *fb_helper,
struct drm_connector *connector;
struct drm_connector_helper_funcs *connector_funcs;
struct drm_encoder *encoder;
-   struct drm_fb_helper_crtc *best_crtc;
int my_score, best_score, score;
struct drm_fb_helper_crtc **crtcs, *crtc;
struct drm_fb_helper_connector *fb_helper_conn;
@@ -1369,7 +1368,6 @@ static int drm_pick_crtcs(struct drm_fb_helper *fb_helper,
connector = fb_helper_conn-connector;
 
best_crtcs[n] = NULL;
-   best_crtc = NULL;
best_score = drm_pick_crtcs(fb_helper, best_crtcs, modes, n+1, width, 
height);
if (modes[n] == NULL)
return best_score;
@@ -1418,7 +1416,6 @@ static int drm_pick_crtcs(struct drm_fb_helper *fb_helper,
score = my_score + drm_pick_crtcs(fb_helper, crtcs, modes, n + 
1,
  width, height);
if (score  best_score) {
-   best_crtc = crtc;
best_score = score;
memcpy(best_crtcs, crtcs,
   dev-mode_config.num_connector *
-- 
1.8.3.1

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[PATCH 5/5] drm: Remove unused variables in drm_fb_helper_hotplug_event()

2013-09-28 Thread Damien Lespiau
Signed-off-by: Damien Lespiau damien.lesp...@intel.com
---
 drivers/gpu/drm/drm_fb_helper.c | 7 ++-
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c
index f61ac23..21742a8 100644
--- a/drivers/gpu/drm/drm_fb_helper.c
+++ b/drivers/gpu/drm/drm_fb_helper.c
@@ -1582,8 +1582,7 @@ EXPORT_SYMBOL(drm_fb_helper_initial_config);
 int drm_fb_helper_hotplug_event(struct drm_fb_helper *fb_helper)
 {
struct drm_device *dev = fb_helper-dev;
-   int count = 0;
-   u32 max_width, max_height, bpp_sel;
+   u32 max_width, max_height;
 
if (!fb_helper-fb)
return 0;
@@ -1598,10 +1597,8 @@ int drm_fb_helper_hotplug_event(struct drm_fb_helper 
*fb_helper)
 
max_width = fb_helper-fb-width;
max_height = fb_helper-fb-height;
-   bpp_sel = fb_helper-fb-bits_per_pixel;
 
-   count = drm_fb_helper_probe_connector_modes(fb_helper, max_width,
-   max_height);
+   drm_fb_helper_probe_connector_modes(fb_helper, max_width, max_height);
mutex_unlock(fb_helper-dev-mode_config.mutex);
 
drm_modeset_lock_all(dev);
-- 
1.8.3.1

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 65611] UVD accelerated decoding causes hangs (ARUBA - HD 7540D)

2013-09-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=65611

--- Comment #4 from Pierre Ossman pierre-bugzi...@ossman.eu ---
(In reply to comment #3)
 
 Yet my WMV3 file is still going through VDPAU. So either my file is advanced
 profile and that is broken, or xbmc not fully respecting the list?

Turned out that the UVD symbols get duplicated both into the VDPAU driver and
the DRI driver. So you have to replace both to get your changes to go through.

Still didn't work properly though as XBMC seems to not bother asking the driver
what it can do and just blindly assumes that every format is available...

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH 1/7] drm/exynos: move hdmiphy related code to hdmiphy driver

2013-09-28 Thread Inki Dae
2013/9/27 Rahul Sharma r.sh.o...@gmail.com

 On 16 September 2013 18:10, Inki Dae inki@samsung.com wrote:
  CCing devicetree,
 
  -Original Message-
  From: Rahul Sharma [mailto:r.sh.o...@gmail.com]
  Sent: Tuesday, September 10, 2013 5:28 PM
  To: Sean Paul
  Cc: Inki Dae; Rahul Sharma; linux-samsung-soc; dri-devel; kgene.kim;
  sw0312.kim; Lucas Stach; Tomasz Figa; Sylwester Nawrocki; sunil joshi;
  Shirish S
  Subject: Re: [PATCH 1/7] drm/exynos: move hdmiphy related code to
 hdmiphy
  driver
 
  On 6 September 2013 19:21, Sean Paul seanp...@chromium.org wrote:
   On Thu, Sep 5, 2013 at 11:37 PM, Rahul Sharma r.sh.o...@gmail.com
  wrote:
   On 5 September 2013 19:20, Inki Dae inki@samsung.com wrote:
  
  
   -Original Message-
   From: Sean Paul [mailto:seanp...@chromium.org]
   Sent: Thursday, September 05, 2013 10:20 PM
   To: Inki Dae
   Cc: Rahul Sharma; Rahul Sharma; linux-samsung-soc; dri-devel;
  kgene.kim;
   sw0312.kim; Lucas Stach; Tomasz Figa; Sylwester Nawrocki; sunil
  joshi;
   Shirish S
   Subject: Re: [PATCH 1/7] drm/exynos: move hdmiphy related code to
  hdmiphy
   driver
  
   On Thu, Sep 5, 2013 at 2:19 AM, Inki Dae inki@samsung.com
  wrote:
   
   
-Original Message-
From: linux-samsung-soc-ow...@vger.kernel.org [mailto:linux-
  samsung-
   soc-
ow...@vger.kernel.org] On Behalf Of Rahul Sharma
Sent: Thursday, September 05, 2013 3:04 PM
To: Inki Dae
Cc: Sean Paul; Rahul Sharma; linux-samsung-soc; dri-devel;
  kgene.kim;
sw0312.kim; Lucas Stach; Tomasz Figa; Sylwester Nawrocki; sunil
  joshi;
Shirish S
Subject: Re: [PATCH 1/7] drm/exynos: move hdmiphy related code
 to
   hdmiphy
driver
   
On 5 September 2013 10:52, Inki Dae inki@samsung.com
 wrote:
+static struct hdmiphy_config
 hdmiphy_4210_configs[] =
  {
+   {
+   .pixel_clock = 2700,
+   .conf = {
+   0x01, 0x05, 0x00, 0xD8,
 0x10,
  0x1C,
0x30,
  0x40,
+   0x6B, 0x10, 0x02, 0x51,
 0xDF,
  0xF2,
0x54,
  0x87,
+   0x84, 0x00, 0x30, 0x38,
 0x00,
  0x08,
0x10,
  0xE0,
+   0x22, 0x40, 0xE3, 0x26,
 0x00,
  0x00,
0x00,
  0x00,
+   },
+   },
+   {
+   .pixel_clock = 27027000,
+   .conf = {
+   0x01, 0x05, 0x00, 0xD4,
 0x10,
  0x9C,
0x09,
  0x64,
+   0x6B, 0x10, 0x02, 0x51,
 0xDF,
  0xF2,
0x54,
  0x87,
+   0x84, 0x00, 0x30, 0x38,
 0x00,
  0x08,
0x10,
  0xE0,
+   0x22, 0x40, 0xE3, 0x26,
 0x00,
  0x00,
0x00,
  0x00,
+   },
+   },
+   {
+   .pixel_clock = 74176000,
+   .conf = {
+   0x01, 0x05, 0x00, 0xD8,
 0x10,
  0x9C,
0xef,
  0x5B,
+   0x6D, 0x10, 0x01, 0x51,
 0xef,
  0xF3,
0x54,
  0xb9,
+   0x84, 0x00, 0x30, 0x38,
 0x00,
  0x08,
0x10,
  0xE0,
+   0x22, 0x40, 0xa5, 0x26,
 0x01,
  0x00,
0x00,
  0x00,
+   },
+   },
+   {
+   .pixel_clock = 7425,
+   .conf = {
+   0x01, 0x05, 0x00, 0xd8,
 0x10,
  0x9c,
0xf8,
  0x40,
+   0x6a, 0x10, 0x01, 0x51,
 0xff,
  0xf1,
0x54,
  0xba,
+   0x84, 0x00, 0x10, 0x38,
 0x00,
  0x08,
0x10,
  0xe0,
+   0x22, 0x40, 0xa4, 0x26,
 0x01,
  0x00,
0x00,
  0x00,
+   },
+   },
+   {
+   .pixel_clock = 14850,
+   .conf = {
+   0x01, 0x05, 0x00, 0xD8,
 0x10,
  0x9C,
0xf8,
  0x40,
+   0x6A, 0x18, 0x00, 0x51,
 0xff,
  0xF1,
0x54,
  0xba,
+   0x84, 0x00, 0x10, 0x38,
 0x00,
  0x08,
0x10,
  0xE0,
+   0x22, 0x40, 0xa4, 0x26,
 0x02,
  0x00,
0x00,
  0x00,
+   },
+   },
+};
+
   
Are you aware of the effort to move these to dt?
 Since
  these
are
board-specific values, it seems incorrect to apply
 them
 universally.
Shirish has uploaded a patch to the chromium review
  site to
push
  these
into dt
   (https://chromium-review.googlesource.com/#/c/65581).
 Maybe
you can work that into your patch set?
   
  
   Are these really board-specific values?
 
  According to your hardware people:
 
  If the signal 

Re: [RFC] deadlock in drm/exynos: fix wrong pointer access at vm close

2013-09-28 Thread Inki Dae
Thanks for your comments.

Thank,
Inki Dae

2013/9/26 Al Viro v...@zeniv.linux.org.uk

 On Tue, Sep 24, 2013 at 01:41:00PM +0900, Inki Dae wrote:

  I can't see to hold -mmap_sem when it calls find_vma() anywhere else.

 Er...  What, in your opinion, would protect the result of find_vma(), if
 not that?  E.g. if another thread does munmap() on that area...


Right. the vm_area_struct object could be removed from rb tree;
mm-mm_rb-rb_node.


  vma isn't
 refcounted; there are only two things that can prevent its freeing -
 mmap_sem being held and the lack of anybody else seeing the address of
 mm_struct it belongs to (basically, when we are killing mm_struct off
 or when we are populating a fresh mm_struct; in the latter case the
 parent is locked, but the child doesn't need to).


Ok, will add down_write(mm-mmap_sem) and up_write(mm-mmap_sem) between
and after find_vma() call.



 Look at page fault handlers - they grab mmap_sem (shared) before looking
 for
 vma.


Yes, and do_munmap also.


  And anything modifying the list of vmas (vm_mmap(), etc.) grabs it
 exclusive...

   to caller) and clones it manually, regardless of whether that vma
 allows
   to clone itself or not.  Quite a few drivers rely on that not
 happening...
  
 
  I think that has no any problem because exynos_gem_get_vma() takes a
  reference to vma, and also v4l2 side is using same way. I and v4l2 guys
  might be missing something what you are concerning. So Could you give us
  more comments?

 vb2_get_vma()/vb2_put_userptr() is also buggy.  At the very least, it
 should refuse to handle ones with VM_DONTCOPY in flags.  Again, there
 are drivers that seriously rely on VM_DONTCOPY being honoured.


Got it. will check the VM_DONTCOPY flag before copying the vma.



 BTW, what do you expect exynos_gem_get_pages_from_userptr() to do if
 the area has been unmapped in the meanwhile?


Yes, that function would try to access a invaild vma.


  Or, for that matter,
 if that has been followed by mmap() of something with VM_IO on the
 same range of addresses...  -open() does *NOT* prevent any of that.
 ___
 dri-devel mailing list
 dri-devel@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/dri-devel

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [PATCH 1/5] drm: Remove unused variable in drm_global_item_ref()

2013-09-28 Thread Alex Deucher
On Sat, Sep 28, 2013 at 11:24 AM, Damien Lespiau
damien.lesp...@intel.com wrote:
 Signed-off-by: Damien Lespiau damien.lesp...@intel.com

For the series:

Reviewed-by: Alex Deucher alexander.deuc...@amd.com

 ---
  drivers/gpu/drm/drm_global.c | 2 --
  1 file changed, 2 deletions(-)

 diff --git a/drivers/gpu/drm/drm_global.c b/drivers/gpu/drm/drm_global.c
 index f731116..3d2e91c 100644
 --- a/drivers/gpu/drm/drm_global.c
 +++ b/drivers/gpu/drm/drm_global.c
 @@ -67,7 +67,6 @@ int drm_global_item_ref(struct drm_global_reference *ref)
  {
 int ret;
 struct drm_global_item *item = glob[ref-global_type];
 -   void *object;

 mutex_lock(item-mutex);
 if (item-refcount == 0) {
 @@ -85,7 +84,6 @@ int drm_global_item_ref(struct drm_global_reference *ref)
 }
 ++item-refcount;
 ref-object = item-object;
 -   object = item-object;
 mutex_unlock(item-mutex);
 return 0;
  out_err:
 --
 1.8.3.1

 ___
 dri-devel mailing list
 dri-devel@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/dri-devel
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [RFC] deadlock in drm/exynos: fix wrong pointer access at vm close

2013-09-28 Thread Inki Dae
2013/9/26 Al Viro v...@zeniv.linux.org.uk

 On Wed, Sep 25, 2013 at 01:34:30PM +0900, Inki Dae wrote:

  It seems that we can use a new anon file instead of using drm file to
  resolve the issue.

 Could you describe what are you trying to achieve with that ioctl() and
 what semantics do you want from normal mmap()?


Ok, The reason that we are using ioctl mmap is that I thought it doesn't
need demand paging mechanism for drm driver for ARM SoCs: they have no
iommu so need physically continuous memory for DMA.

And another reason is that dumb* interface - it seems that you say this
interface is a normal mmap but this is a fake mmap - has more steps against
ioctl mmap. So to map some user space with physical pages, first,
application should request a gem allocation, and then request a hash key of
the allcated gem, and then fake mmap. This fake mmap system call doesn't
map a given user space with the physical pages. And then the user space
will be mapped with the physical pages by page fault handler when cpu tries
to access the user space. On the other hand, the ioctl mmap call will
map the user space with physical pages, and I thought this way would have
better performance than normal mmap at least in case of ARM based SoC.

And last, the reason I had implemented the normal mmap is that dumb*
interfaces are required as default. i.e. modetest app of libdrm uses only
dumb interfaces for buffer management.

Anyway,we have faced with some issues by using ioctl mmap and normal mmap
together, and these issues should be fixed.


Thanks,
Inki Dae


 ___
 dri-devel mailing list
 dri-devel@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/dri-devel

___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


Re: [RFC] deadlock in drm/exynos: fix wrong pointer access at vm close

2013-09-28 Thread Inki Dae
2013/9/29 Inki Dae inki@samsung.com



 2013/9/26 Al Viro v...@zeniv.linux.org.uk

 On Wed, Sep 25, 2013 at 01:34:30PM +0900, Inki Dae wrote:

  It seems that we can use a new anon file instead of using drm file to
  resolve the issue.

 Could you describe what are you trying to achieve with that ioctl() and
 what semantics do you want from normal mmap()?


 Ok, The reason that we are using ioctl mmap is that I thought it doesn't
 need demand paging mechanism for drm driver for ARM SoCs: they have no
 iommu so need physically continuous memory for DMA.


Sorry, there was a wrong comment. it's not all ARM SoC have no iommu.
Exynos also has the iommu, and maybe recently most SoCs.



 And another reason is that dumb* interface - it seems that you say this
 interface is a normal mmap but this is a fake mmap - has more steps against
 ioctl mmap. So to map some user space with physical pages, first,
 application should request a gem allocation, and then request a hash key of
 the allcated gem, and then fake mmap. This fake mmap system call doesn't
 map a given user space with the physical pages. And then the user space
 will be mapped with the physical pages by page fault handler when cpu tries
 to access the user space. On the other hand, the ioctl mmap call will
 map the user space with physical pages, and I thought this way would have
 better performance than normal mmap at least in case of ARM based SoC.

 And last, the reason I had implemented the normal mmap is that dumb*
 interfaces are required as default. i.e. modetest app of libdrm uses only
 dumb interfaces for buffer management.

 Anyway,we have faced with some issues by using ioctl mmap and normal mmap
 together, and these issues should be fixed.


 Thanks,
 Inki Dae


 ___
 dri-devel mailing list
 dri-devel@lists.freedesktop.org
 http://lists.freedesktop.org/mailman/listinfo/dri-devel



___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 67800] Computer freezes after several hours (trinity)

2013-09-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=67800

--- Comment #14 from Kertesz Laszlo laszlo.kert...@gmail.com ---
Well interesting news. 
I played around with glamor acceleration, and to my surprise, my computer ran
for 2 days and 16 hours or so without problems (did just about everything i do
usually - browse, watch movies with vdpau, play games).
After that restarted X with exa rendering and the computer froze hard within
minutes. Will test this further.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 69728] Radeon Redwood (5670) GPU Lockup

2013-09-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=69728

--- Comment #7 from pablow.1...@gmail.com ---
I've done what you suggested with Mesa 9.3.0-devel (git-a64d3dd), the GPU
lockup still occurs. Is there anything more that I could try?

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Bug 69723] Computer freezes with kernel 3.11.0 / 3.12-rc1 (with bug 68235's patches applied) when dpm=1 on r600g (Cayman)

2013-09-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=69723

--- Comment #7 from Alexandre Demers alexandre.f.dem...@gmail.com ---
Alex, using debugfs, should I see the maxed values (sclk and mclk) or the
theorical value from the table? For now, even if I have confirmation in dmesg
the values were maxed out, looking in debugfs gives me:
uvdvclk: 0 dclk: 0
power level 2sclk: 83000 mclk: 13 vddc: 1060 vddci: 1150
(it should be sclk: 8 mclk: 125000...)

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[git pull] drm fixes

2013-09-28 Thread Dave Airlie
Hi Linus,

nothing too major, radeon still has some dpm changes for off by default.

radeon, intel, msm:
radeon a few more dpm fixes (still off by default), uvd fixes,
i915 runtime warn backtrace and regression fix
msm: iommu changes fallout,

Dave.

The following changes since commit 4a10c2ac2f368583138b774ca41fac4207911983:

  Linux 3.12-rc2 (2013-09-23 15:41:09 -0700)

are available in the git repository at:

  git://people.freedesktop.org/~airlied/linux drm-fixes

for you to fetch changes up to 665441791f553221901f679cb47236798f9a649c:

  Merge branch 'msm-fixes-3.12-rc2' of
git://people.freedesktop.org/~robclark/linux into drm-fixes
(2013-09-29 10:06:28 +1000)



Alex Deucher (13):
  drm/radeon: avoid UVD corruption on AGP cards using GPU gart
  drm/radeon: additional gcc fixes for radeon_atombios.c
  drm/radeon: fix missed variable sized access
  drm/radeon/dpm: fetch the max clk from voltage dep tables helper
  drm/radeon/dpm/btc: filter clocks based on voltage/clk dep tables
  drm/radeon/dpm/ni: filter clocks based on voltage/clk dep tables
  drm/radeon/dpm/si: filter clocks based on voltage/clk dep tables
  drm/radeon/dpm/ci: filter clocks based on voltage/clk dep tables
  drm/radeon: don't set default clocks for SI when DPM is disabled
  drm/radeon: disable tests/benchmarks if accel is disabled
  drm/radeon: add missing hdmi callbacks for rv6xx
  drm/radeon/cik: fix overflow in vram fetch
  drm/radeon: fix hdmi audio on DCE3.0/3.1 asics

Alex Ivanov (1):
  drm/radeon: Make r100_cp_ring_info() and radeon_ring_gfx() safe (v2)

Chris Wilson (1):
  drm/i915: Use a temporary va_list for two-pass string handling

Christian König (1):
  drm/radeon/uvd: lower msgfb buffer requirements on UVD3

Daniel Vetter (3):
  drm/i915/tv: clear adjusted_mode.flags
  drm/i915: preserve pipe A quirk in i9xx_set_pipeconf
  drm/i915: Fix up usage of SHRINK_STOP

Dave Airlie (3):
  Merge tag 'drm-intel-fixes-2013-09-26' of
git://people.freedesktop.org/~danvet/drm-intel into drm-fixes
  Merge branch 'drm-fixes-3.12' of
git://people.freedesktop.org/~agd5f/linux into drm-fixes
  Merge branch 'msm-fixes-3.12-rc2' of
git://people.freedesktop.org/~robclark/linux into drm-fixes

Jani Nikula (1):
  drm/i915/dp: increase i2c-over-aux retry interval on AUX DEFER

Joerg Roedel (1):
  drm/msm: Remove iommu include from mdp4_kms.c

Michel Dänzer (3):
  drm/radeon/cik: Fix printing of client name on VM protection fault
  drm/radeon/cik: Fix encoding of number of banks in tiling
configuration info
  drm/radeon/cik: Add tiling mode index for 1D tiled depth/stencil surfaces

Rob Clark (2):
  drm/msm: deal with mach/iommu.h removal
  drm/msm: use drm_gem_dumb_destroy helper

Thomas Meyer (1):
  drm/msm: Odd PTR_ERR usage

 drivers/gpu/drm/i915/i915_gem.c  |  8 ++--
 drivers/gpu/drm/i915/i915_gpu_error.c|  6 ++-
 drivers/gpu/drm/i915/intel_display.c |  4 ++
 drivers/gpu/drm/i915/intel_dp.c  | 13 ++-
 drivers/gpu/drm/i915/intel_tv.c  |  8 
 drivers/gpu/drm/msm/mdp4/mdp4_kms.c  |  2 -
 drivers/gpu/drm/msm/msm_drv.c|  8 ++--
 drivers/gpu/drm/msm/msm_gem.c|  7 
 drivers/gpu/drm/radeon/btc_dpm.c | 51 
 drivers/gpu/drm/radeon/btc_dpm.h |  2 +
 drivers/gpu/drm/radeon/ci_dpm.c  | 26 +
 drivers/gpu/drm/radeon/cik.c | 17 
 drivers/gpu/drm/radeon/ni_dpm.c  | 24 
 drivers/gpu/drm/radeon/r100.c|  8 ++--
 drivers/gpu/drm/radeon/r600_dpm.c|  2 +-
 drivers/gpu/drm/radeon/r600_hdmi.c   | 20 +++---
 drivers/gpu/drm/radeon/radeon_asic.c |  2 +
 drivers/gpu/drm/radeon/radeon_atombios.c | 66 +---
 drivers/gpu/drm/radeon/radeon_cs.c   |  5 ++-
 drivers/gpu/drm/radeon/radeon_device.c   | 15 ++--
 drivers/gpu/drm/radeon/radeon_pm.c   |  8 ++--
 drivers/gpu/drm/radeon/radeon_ring.c |  8 ++--
 drivers/gpu/drm/radeon/radeon_uvd.c  |  3 +-
 drivers/gpu/drm/radeon/si_dpm.c  | 24 
 drivers/gpu/drm/radeon/uvd_v1_0.c|  4 +-
 include/uapi/drm/radeon_drm.h|  2 +
 26 files changed, 266 insertions(+), 77 deletions(-)
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel