Re: [PATCH 1/2] drm/ttm: remove some bo-mutex remains

2010-03-12 Thread Maarten Maathuis
Ping?

On Mon, Mar 1, 2010 at 9:55 PM, Jerome Glisse gli...@freedesktop.org wrote:
 On Mon, Mar 01, 2010 at 07:34:39PM +0100, Maarten Maathuis wrote:
 - A few comments existed here and there that referred to a bo-mutex.

 Signed-off-by: Maarten Maathuis madman2...@gmail.com

 Reviewed-by: Jerome Glisse jgli...@redhat.com

 ---
  drivers/gpu/drm/ttm/ttm_bo.c    |    6 +-
  drivers/gpu/drm/ttm/ttm_bo_vm.c |    2 +-
  include/drm/ttm/ttm_bo_api.h    |    3 +--
  include/drm/ttm/ttm_bo_driver.h |    1 +
  4 files changed, 4 insertions(+), 8 deletions(-)

 diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c
 index 2920f9a..f5333d9 100644
 --- a/drivers/gpu/drm/ttm/ttm_bo.c
 +++ b/drivers/gpu/drm/ttm/ttm_bo.c
 @@ -46,7 +46,6 @@
  #include linux/file.h
  #include linux/module.h

 -#define TTM_ASSERT_LOCKED(param)
  #define TTM_DEBUG(fmt, arg...)
  #define TTM_BO_HASH_ORDER 13

 @@ -306,9 +305,6 @@ void ttm_bo_unreserve(struct ttm_buffer_object *bo)
  }
  EXPORT_SYMBOL(ttm_bo_unreserve);

 -/*
 - * Call bo-mutex locked.
 - */
  static int ttm_bo_add_ttm(struct ttm_buffer_object *bo, bool zero_alloc)
  {
       struct ttm_bo_device *bdev = bo-bdev;
 @@ -316,7 +312,7 @@ static int ttm_bo_add_ttm(struct ttm_buffer_object *bo, 
 bool zero_alloc)
       int ret = 0;
       uint32_t page_flags = 0;

 -     TTM_ASSERT_LOCKED(bo-mutex);
 +     BUG_ON(!atomic_read(bo-reserved));
       bo-ttm = NULL;

       if (bdev-need_dma32)
 diff --git a/drivers/gpu/drm/ttm/ttm_bo_vm.c 
 b/drivers/gpu/drm/ttm/ttm_bo_vm.c
 index 668dbe8..41b0c1e 100644
 --- a/drivers/gpu/drm/ttm/ttm_bo_vm.c
 +++ b/drivers/gpu/drm/ttm/ttm_bo_vm.c
 @@ -146,7 +146,7 @@ static int ttm_bo_vm_fault(struct vm_area_struct *vma, 
 struct vm_fault *vmf)
        * since the mmap_sem is only held in read mode. However, we
        * modify only the caching bits of vma-vm_page_prot and
        * consider those bits protected by
 -      * the bo-mutex, as we should be the only writers.
 +      * bo-reserved, as we should be the only writers.
        * There shouldn't really be any readers of these bits except
        * within vm_insert_mixed()? fork?
        *
 diff --git a/include/drm/ttm/ttm_bo_api.h b/include/drm/ttm/ttm_bo_api.h
 index 81eb9f4..c1093ae 100644
 --- a/include/drm/ttm/ttm_bo_api.h
 +++ b/include/drm/ttm/ttm_bo_api.h
 @@ -35,7 +35,6 @@
  #include linux/kref.h
  #include linux/list.h
  #include linux/wait.h
 -#include linux/mutex.h
  #include linux/mm.h
  #include linux/rbtree.h
  #include linux/bitmap.h
 @@ -298,7 +297,7 @@ ttm_bo_reference(struct ttm_buffer_object *bo)
   * @interruptible:  Use interruptible wait.
   * @no_wait:  Return immediately if buffer is busy.
   *
 - * This function must be called with the bo::mutex held, and makes
 + * This function must be called with bo-reserved held, and makes
   * sure any previous rendering to the buffer is completed.
   * Note: It might be necessary to block validations before the
   * wait by reserving the buffer.
 diff --git a/include/drm/ttm/ttm_bo_driver.h 
 b/include/drm/ttm/ttm_bo_driver.h
 index ff7664e..d3fc5f8 100644
 --- a/include/drm/ttm/ttm_bo_driver.h
 +++ b/include/drm/ttm/ttm_bo_driver.h
 @@ -37,6 +37,7 @@
  #include linux/workqueue.h
  #include linux/fs.h
  #include linux/spinlock.h
 +#include linux/mutex.h

  struct ttm_backend;

 --
 1.7.0


 --
 Download Intel#174; Parallel Studio Eval
 Try the new software tools for yourself. Speed compiling, find bugs
 proactively, and fine-tune applications for parallel performance.
 See why Intel Parallel Studio got high marks during beta.
 http://p.sf.net/sfu/intel-sw-dev
 --
 ___
 Dri-devel mailing list
 Dri-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/dri-devel



--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 23532] Qt apps using opengl graphicssystem break with KMS

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=23532





--- Comment #8 from Marc marvi...@gmx.de  2010-03-12 00:39:58 PST ---
try git show addedd09 with current mesa/master. Unfortunately, I cannot test
this anymore as my system was upgraded.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27010] mesa git build failure regarding winsys

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27010


Michel Dänzer mic...@daenzer.net changed:

   What|Removed |Added

 CC||wol...@onsneteindhoven.nl




--- Comment #3 from Michel Dänzer mic...@daenzer.net  2010-03-12 01:15:12 PST 
---
*** Bug 27037 has been marked as a duplicate of this bug. ***


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27010] mesa git build failure regarding winsys

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27010





--- Comment #4 from Michel Dänzer mic...@daenzer.net  2010-03-12 01:18:42 PST 
---
Keith just pushed a fix which might help for this, please give it a try.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27010] mesa git build failure regarding winsys

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27010





--- Comment #5 from Lars Lindley lind...@coyote.org  2010-03-12 01:22:33 PST 
---
Yes that fixed the build for me..Thanks!


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH 0/5] clean up radeon_asic.h v2

2010-03-12 Thread Jerome Glisse
On Thu, Mar 11, 2010 at 10:19:13PM +0100, Daniel Vetter wrote:
 Hi all,
 
 All new patch pile to make radeon_asic.h into a real header file. Now all
 the asic structs are gathered in the new radeon_asic.c file.
 
 Tested on my rv570.
 
 I've also added a new patch that gathers all r100 specific declarations
 into radeon_asic.h (at least where it makes sense). This is just an example
 to convince Jerome that radeon_asic.h might not be totally useless ;)
 
 Again, comments higly welcome.
 
 Yours, Daniel
 

I would merge patch 1  2 into a single patch, also i think you
include radeon_asic.h at top of radeon.h so everyfile would also
include radeon_asic.h that would be simplier than adding include
to file and reduce the likelyhood to forget doing so in the future.
(you might need to add forward declaration like struct radeon; at
top of radeon_asic.h no biggy thought).

Cheers,
Jerome

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27010] mesa git build failure regarding winsys

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27010





--- Comment #6 from Lars Lindley lind...@coyote.org  2010-03-12 01:36:43 PST 
---
And it works great too! Always 60 fps @ 1680x1050 in extreme tuxracer..


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH 0/5] clean up radeon_asic.h v2

2010-03-12 Thread Daniel Vetter
On Fri, Mar 12, 2010 at 10:25:56AM +0100, Jerome Glisse wrote:
 I would merge patch 1  2 into a single patch,
I've split this up to make patch-reading easier. And it's fully
bisectable.

 ... also i think you
 include radeon_asic.h at top of radeon.h so everyfile would also
 include radeon_asic.h that would be simplier than adding include
 to file and reduce the likelyhood to forget doing so in the future.
 (you might need to add forward declaration like struct radeon; at
 top of radeon_asic.h no biggy thought).

I disagree. It's not quite there yet, but when all the asic specific stuff
is gathered in radeon_asic.h (like I've done for r100 in my last patch),
radeon_asic.h is the private header for asic stuff. And radeon.h is the
public interface for radeon_asic. Then adding #include radeon_asic.h
in random places would serve as a warning sign that there's likely a
layering violation ahead. After all, generic code should not muck around
in the asic private stuff.

Unconditionally including radeon_asic.h therefore runs counter to the
bigger idea behind my patches.

Cheers, Daniel
-- 
Daniel Vetter
Mail: dan...@ffwll.ch
Mobile: +41 (0)79 365 57 48

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27010] mesa git build failure regarding winsys

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27010





--- Comment #7 from Jos van Wolput wol...@onsneteindhoven.nl  2010-03-12 
02:04:52 PST ---
Fixed, thanks!


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27010] mesa git build failure regarding winsys

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27010


Michel Dänzer mic...@daenzer.net changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Comment #8 from Michel Dänzer mic...@daenzer.net  2010-03-12 02:16:15 PST 
---
BTW, you can resolve bugs yourselves. :)


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27043] New: [bisected DRI1] drmRadeonCmdBuffer: -22. Kernel failed to parse or rejected command stream. See dmesg for more info.

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27043

   Summary: [bisected DRI1] drmRadeonCmdBuffer: -22. Kernel failed
to parse or rejected command stream. See dmesg for more
info.
   Product: Mesa
   Version: git
  Platform: x86 (IA32)
OS/Version: Linux (All)
Status: NEW
  Severity: major
  Priority: medium
 Component: Drivers/DRI/r300
AssignedTo: dri-devel@lists.sourceforge.net
ReportedBy: fabio@libero.it
CC: m.cenc...@gmail.com


This commit:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=67108b5d12e0526ebedcdf2dbeeadfdbd0782161
make sauerbraten (at least on map douze) crash on UMS/DRI1 with:
drmRadeonCmdBuffer: -22. Kernel failed to parse or rejected command stream. See
dmesg for more info.

Note: commit e5c691f445e1c02e6e2f75b817b13d7024f7a3a6 have to be reverted,
however bug #26954 is hit.

Card:
GL_RENDERER   = Mesa DRI R300 (RV530 71C5) 20090101 x86/MMX/SSE2 TCL


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27010] mesa git build failure regarding winsys

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27010





--- Comment #9 from Nikolay Rysev mad.f...@gmail.com  2010-03-12 03:52:08 PST 
---
Fixed for me too, thanks.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCH 2/3] libdrm_radeon: Optimize cs_gem_reloc to do less looping. (V2)

2010-03-12 Thread Pauli Nieminen
bo-referenced_in_cs is checked if bo is already in cs. Adding and removing
reference in bo is done with atomic operations to allow parallel access to a
bo from multiple contexts.

cs-id generation code quarentees there is not duplicated ids which limits
number of cs-ids to 32. If there is more cs objects rest will get id 0.

V2:
 - Fix configure to check for atomics operations if libdrm_radeon is selected 
and libdrm_intel is not.
 - Make atomic operations private to libdrm_radeon.
 - Add warning messages if disabling automaticaly selected libdrm_(radeon|intel)

This optimization decreases cs_write_reloc share of torcs profiling from 4.3%
to 2.6%.

Signed-off-by: Pauli Nieminen suok...@gmail.com
---
 configure.ac   |   27 ---
 radeon/radeon_bo_gem.c |9 
 radeon/radeon_bo_gem.h |1 +
 radeon/radeon_cs.c |6 ++
 radeon/radeon_cs.h |2 +-
 radeon/radeon_cs_gem.c |  123 
 radeon/radeon_cs_int.h |1 +
 xf86atomic.h   |6 ++
 8 files changed, 136 insertions(+), 39 deletions(-)

diff --git a/configure.ac b/configure.ac
index 953a758..374beb3 100644
--- a/configure.ac
+++ b/configure.ac
@@ -56,8 +56,8 @@ AC_ARG_ENABLE(intel,
 
 AC_ARG_ENABLE(radeon,
  AS_HELP_STRING([--disable-radeon],
- [Enable support for radeon's KMS API (default: enabled)]),
- [RADEON=$enableval], [RADEON=yes])
+ [Enable support for radeon's KMS API (default: auto)]),
+ [RADEON=$enableval], [RADEON=auto])
 
 AC_ARG_ENABLE(vmwgfx-experimental-api,
  AS_HELP_STRING([--enable-vmwgfx-experimental-api],
@@ -173,7 +173,7 @@ if test x$HAVE_LIBUDEV = xyes; then
 fi
 AM_CONDITIONAL(HAVE_LIBUDEV, [test x$HAVE_LIBUDEV = xyes])
 
-if test x$INTEL != xno; then
+if test x$INTEL != xno -o x$RADEON != xno; then
 # Check for atomic intrinsics
 AC_CACHE_CHECK([for native atomic primitives], drm_cv_atomic_primitives,
 [
@@ -206,13 +206,26 @@ if test x$INTEL != xno; then
 fi
 
 if test x$drm_cv_atomic_primitives = xnone; then
-   if test x$INTEL != xauto; then
-   AC_MSG_ERROR([libdrm_intel depends upon atomic operations, 
which were not found for your compiler/cpu. Try compiling with -march=native, 
or install the libatomics-op-dev package, or, failing both of those, disable 
support for Intel GPUs by passing --disable-intel to ./configure])
-  else
+   if test x$INTEL != xauto; then
+AC_MSG_ERROR([libdrm_intel depends upon atomic operations, 
which were not found for your compiler/cpu. Try compiling with -march=native, 
or install the libatomics-op-dev package, or, failing both of those, disable 
support for Intel GPUs by passing --disable-intel to ./configure])
+   else
+AC_MSG_WARN([Disabling libdrm_intel. It depends on atomic 
operations, which were not found for your compiler/cpu. Try compiling with 
-march=native, or install the libatomics-op-dev package.])
INTEL=no
   fi
+   if test x$RADEON != xauto; then
+   AC_MSG_ERROR([libdrm_radeon depends upon atomic operations, 
which were not found for your compiler/cpu. Try compiling with -march=native, 
or install the libatomics-op-dev package, or, failing both of those, disable 
support for Radeon support by passing --disable-radeon to ./configure])
+  else
+AC_MSG_WARN([Disabling libdrm_radeon. It depends on atomic 
operations, which were not found for your compiler/cpu. Try compiling with 
-march=native, or install the libatomics-op-dev package.])
+   RADEON=no
+  fi
+
 else
-  INTEL=yes
+   if test x$INTEL != xno; then
+   INTEL=yes
+   fi
+   if test x$RADEON != xno; then
+   RADEON=yes
+   fi
 fi
 fi
 
diff --git a/radeon/radeon_bo_gem.c b/radeon/radeon_bo_gem.c
index bc8058d..081ccb9 100644
--- a/radeon/radeon_bo_gem.c
+++ b/radeon/radeon_bo_gem.c
@@ -39,6 +39,7 @@
 #include sys/mman.h
 #include errno.h
 #include xf86drm.h
+#include xf86atomic.h
 #include drm.h
 #include radeon_drm.h
 #include radeon_bo.h
@@ -49,6 +50,7 @@ struct radeon_bo_gem {
 struct radeon_bo_int base;
 uint32_tname;
 int map_count;
+atomic_treloc_in_cs;
 void *priv_ptr;
 };
 
@@ -80,6 +82,7 @@ static struct radeon_bo *bo_open(struct radeon_bo_manager 
*bom,
 bo-base.domains = domains;
 bo-base.flags = flags;
 bo-base.ptr = NULL;
+atomic_set(bo-reloc_in_cs, 0);
 bo-map_count = 0;
 if (handle) {
 struct drm_gem_open open_arg;
@@ -309,6 +312,12 @@ uint32_t radeon_gem_name_bo(struct radeon_bo *bo)
 return bo_gem-name;
 }
 
+void *radeon_gem_get_reloc_in_cs(struct radeon_bo *bo)
+{
+struct radeon_bo_gem *bo_gem = (struct radeon_bo_gem*)bo;
+return bo_gem-reloc_in_cs;
+}
+
 int 

[PATCH 1/3] libdrm: Move intel_atomic.h to libdrm core for sharing. (V3)

2010-03-12 Thread Pauli Nieminen
intel_atomic.h includes very usefull atomic operations for
lock free parrallel access of variables. Moving these to
core libdrm for code sharing with radeon.

V2: Fix remaining references to intel_atomic.h and libdrm-intel.

V3: Remove useless wrapper header intel_atomic.h

Signed-off-by: Pauli Nieminen suok...@gmail.com
---
 Makefile.am  |3 +
 configure.ac |2 +-
 intel/Makefile.am|1 -
 intel/intel_atomic.h |   93 --
 intel/intel_bufmgr_gem.c |2 +-
 xf86atomic.h |   93 ++
 6 files changed, 98 insertions(+), 96 deletions(-)
 delete mode 100644 intel/intel_atomic.h
 create mode 100644 xf86atomic.h

diff --git a/Makefile.am b/Makefile.am
index ee3ccc7..e434e1d 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -61,6 +61,9 @@ libdrm_la_SOURCES =   \
 libdrmincludedir = ${includedir}
 libdrminclude_HEADERS = xf86drm.h xf86drmMode.h
 
+noinst_HEADERS = xf86atomic.h
+
+
 EXTRA_DIST = libdrm.pc.in include/drm/*
 
 copy-headers :
diff --git a/configure.ac b/configure.ac
index aaa8efa..953a758 100644
--- a/configure.ac
+++ b/configure.ac
@@ -198,7 +198,7 @@ if test x$INTEL != xno; then
 
 ])
 if test x$drm_cv_atomic_primitives = xIntel; then
-   AC_DEFINE(HAVE_INTEL_ATOMIC_PRIMITIVES, 1,
+   AC_DEFINE(HAVE_LIBDRM_ATOMIC_PRIMITIVES, 1,
  [Enable if your compiler supports the Intel __sync_* 
atomic primitives])
 fi
 if test x$drm_cv_atomic_primitives = xlibatomic-ops; then
diff --git a/intel/Makefile.am b/intel/Makefile.am
index 8bb2c6e..9add505 100644
--- a/intel/Makefile.am
+++ b/intel/Makefile.am
@@ -35,7 +35,6 @@ libdrm_intel_la_LDFLAGS = -version-number 1:0:0 -no-undefined
 libdrm_intel_la_LIBADD = ../libdrm.la @PTHREADSTUBS_LIBS@ @CLOCK_LIB@
 
 libdrm_intel_la_SOURCES = \
-   intel_atomic.h \
intel_bufmgr.c \
intel_bufmgr_priv.h \
intel_bufmgr_fake.c \
diff --git a/intel/intel_atomic.h b/intel/intel_atomic.h
deleted file mode 100644
index 12bb96b..000
--- a/intel/intel_atomic.h
+++ /dev/null
@@ -1,93 +0,0 @@
-/*
- * Copyright © 2009 Intel Corporation
- *
- * Permission is hereby granted, free of charge, to any person obtaining a
- * copy of this software and associated documentation files (the Software),
- * to deal in the Software without restriction, including without limitation
- * the rights to use, copy, modify, merge, publish, distribute, sublicense,
- * and/or sell copies of the Software, and to permit persons to whom the
- * Software is furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice (including the next
- * paragraph) shall be included in all copies or substantial portions of the
- * Software.
- *
- * THE SOFTWARE IS PROVIDED AS IS, WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
- * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
- * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
- * IN THE SOFTWARE.
- *
- * Authors:
- *Chris Wilson ch...@chris-wilson.co.uk
- *
- */
-
-/**
- * @file intel_atomics.h
- *
- * Private definitions for atomic operations
- */
-
-#ifndef INTEL_ATOMICS_H
-#define INTEL_ATOMICS_H
-
-#ifdef HAVE_CONFIG_H
-#include config.h
-#endif
-
-#if HAVE_INTEL_ATOMIC_PRIMITIVES
-
-#define HAS_ATOMIC_OPS 1
-
-typedef struct {
-   int atomic;
-} atomic_t;
-
-# define atomic_read(x) ((x)-atomic)
-# define atomic_set(x, val) ((x)-atomic = (val))
-# define atomic_inc(x) ((void) __sync_fetch_and_add ((x)-atomic, 1))
-# define atomic_dec_and_test(x) (__sync_fetch_and_add ((x)-atomic, -1) == 1)
-# define atomic_cmpxchg(x, oldv, newv) __sync_val_compare_and_swap 
((x)-atomic, oldv, newv)
-
-#endif
-
-#if HAVE_LIB_ATOMIC_OPS
-#include atomic_ops.h
-
-#define HAS_ATOMIC_OPS 1
-
-typedef struct {
-   AO_t atomic;
-} atomic_t;
-
-# define atomic_read(x) AO_load_full((x)-atomic)
-# define atomic_set(x, val) AO_store_full((x)-atomic, (val))
-# define atomic_inc(x) ((void) AO_fetch_and_add1_full((x)-atomic))
-# define atomic_dec_and_test(x) (AO_fetch_and_sub1_full((x)-atomic) == 1)
-# define atomic_cmpxchg(x, oldv, newv) AO_compare_and_swap_full((x)-atomic, 
oldv, newv)
-
-#endif
-
-#if defined(__sun)  !defined(HAS_ATOMIC_OPS)  /* Solaris  OpenSolaris */
-
-#include sys/atomic.h
-#define HAS_ATOMIC_OPS 1
-
-typedef struct { uint_t atomic; } atomic_t;
-
-# define atomic_read(x) (int) ((x)-atomic)
-# define atomic_set(x, val) ((x)-atomic = (uint_t)(val))
-# define atomic_inc(x) (atomic_inc_uint ((x)-atomic))
-# define atomic_dec_and_test(x) (atomic_dec_uint_nv((x)-atomic) == 1)
-# define 

[PATCH 3/3] libdrm: Move all noninstalled headers to noinst_HEADERS.

2010-03-12 Thread Pauli Nieminen
Cleanup make system so that all noninstalled headers are put
to noinst_HEADERS. This quarentees that header will be present
in tar ball but not installed with make install.

CC: nouv...@lists.freedesktop.org
Signed-off-by: Pauli Nieminen suok...@gmail.com
---
 Makefile.am |6 +++---
 intel/Makefile.am   |   10 ++
 nouveau/Makefile.am |4 +++-
 3 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/Makefile.am b/Makefile.am
index e434e1d..61da186 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -55,13 +55,13 @@ libdrm_la_SOURCES = \
xf86drmHash.c   \
xf86drmRandom.c \
xf86drmSL.c \
-   xf86drmMode.c   \
-   libdrm_lists.h
+   xf86drmMode.c
 
 libdrmincludedir = ${includedir}
 libdrminclude_HEADERS = xf86drm.h xf86drmMode.h
 
-noinst_HEADERS = xf86atomic.h
+noinst_HEADERS = xf86atomic.h \
+   libdrm_lists.h
 
 
 EXTRA_DIST = libdrm.pc.in include/drm/*
diff --git a/intel/Makefile.am b/intel/Makefile.am
index 9add505..d56b03a 100644
--- a/intel/Makefile.am
+++ b/intel/Makefile.am
@@ -36,14 +36,16 @@ libdrm_intel_la_LIBADD = ../libdrm.la @PTHREADSTUBS_LIBS@ 
@CLOCK_LIB@
 
 libdrm_intel_la_SOURCES = \
intel_bufmgr.c \
-   intel_bufmgr_priv.h \
intel_bufmgr_fake.c \
intel_bufmgr_gem.c \
-   intel_chipset.h \
-   mm.c \
-   mm.h
+   mm.c
 
 libdrm_intelincludedir = ${includedir}
 libdrm_intelinclude_HEADERS = intel_bufmgr.h
 
+noinst_HEADERS = \
+   intel_bufmgr_priv.h \
+   intel_chipset.h \
+   mm.h
+
 pkgconfig_DATA = libdrm_intel.pc
diff --git a/nouveau/Makefile.am b/nouveau/Makefile.am
index 5d759c5..53099eb 100644
--- a/nouveau/Makefile.am
+++ b/nouveau/Makefile.am
@@ -18,7 +18,6 @@ libdrm_nouveau_la_SOURCES = \
nouveau_notifier.c \
nouveau_bo.c \
nouveau_resource.c \
-   nouveau_private.h \
nouveau_reloc.c
 
 libdrm_nouveaucommonincludedir = ${includedir}/nouveau
@@ -38,5 +37,8 @@ libdrm_nouveauincludedir = ${includedir}/drm
 libdrm_nouveauinclude_HEADERS = \
nouveau_drmif.h
 
+noinst_HEADERS = \
+ nouveau_private.h
+
 pkgconfigdir = @pkgconfigdir@
 pkgconfig_DATA = libdrm_nouveau.pc
-- 
1.6.3.3


--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [Patch RFC] ttm: nouveau accelerated on Xen pv-ops kernel

2010-03-12 Thread Arvind R
On Thu, Mar 11, 2010 at 4:32 PM, Pekka Paalanen p...@iki.fi wrote:
 I'm adding dri-devel@ to CC, since this suggested patch touches
 TTM code, and none of the Nouveau code. TTM patches go via
 dri-de...@.

 Thanks.


 On Wed, 10 Mar 2010 18:51:21 +0530
 Arvind R arvin...@gmail.com wrote:

 Hi,
 Following is a simple patch that is needed in nouveau to get
 accelerated X on a Xen dom0 pv_ops kernel. The kernel is jeremy's
 2.6.31.6 as of 20100222. The whole gpu tree of nouveau (which is
 almost the mainline merge), was substituted into the kernel-tree.
 All components of X (mesa, Xorg-server-7.5, xf86-nouveau, libdrm)
 used of the same day.

 Patch:
 diff -Naur nouveau-kernel.orig/drivers/gpu/drm/ttm/ttm_bo_vm.c
 nouveau-kernel.new/drivers/gpu/drm/ttm/ttm_bo_vm.c
 --- nouveau-kernel.orig/drivers/gpu/drm/ttm/ttm_bo_vm.c 2010-01-27
 10:19:28.0 +0530
 +++ nouveau-kernel.new/drivers/gpu/drm/ttm/ttm_bo_vm.c  2010-03-10
 17:28:59.0 +0530
 @@ -271,7 +271,10 @@
          */

         vma-vm_private_data = bo;
 -       vma-vm_flags |= VM_RESERVED | VM_IO | VM_MIXEDMAP |
 VM_DONTEXPAND;
 +       vma-vm_flags |= VM_RESERVED | VM_MIXEDMAP |
 VM_DONTEXPAND;
 +       if (!((bo-mem.placement  TTM_PL_MASK_MEM) 
 TTM_PL_FLAG_TT))
 +               vma-vm_flags |= VM_IO;
 +       vma-vm_page_prot = vma_get_vm_prot(vma-vm_flags);
         return 0;
  out_unref:
         ttm_bo_unref(bo);

sorry for the typo and other procedural errors.
the last added line should be
+vma-vm_page_prot = vm_get_page_prot(vma-vm_flags)

 This patch is necessary because, in Xen, PFN of a page is
 virtualised. So physical addresses
 for DMA programming needs to use the MFN. Xen transparently does
 the correct translation
 using the _PAGE_IOMEM prot-bit in the PTE. If the bit is set,
 then Xen assumes that the backing
 memory is in the IOMEM space, and PFN equals MFN. If not set,
 page_to_pfn() returns MFN.

 The patch enables the ttm_bo_vm_fault() handler to behave
 correctly under Xen, and has no
 side-effects on normal (not under Xen) operations. The use of
 TTM_PL_FLAG_TT in the
 check assumes that all other placements are backed by device
 memory or IO. If there are
 any other placements that use system memory, that flag has to be
 OR'ed into the check.

 The above patch has no implications on a normal kernel or a Xen
 pv_ops kernel booted without
 the Xen hypervisor. My testing is on a debian-lenny environment
 on a Core2 processor with
 nVidia GeForce 9400 GT.

Efficacy of patch:
successful flightgear run on dom0 AND bareboot!

Arvind R.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27043] [bisected DRI1] drmRadeonCmdBuffer: -22. Kernel failed to parse or rejected command stream. See dmesg for more info.

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27043





--- Comment #1 from Fabio Pedretti fabio@libero.it  2010-03-12 06:13:30 
PST ---
This is what dmesg shows:
[drm:r300_do_cp_cmdbuf] *ERROR* bad cmd_type 148 at e666f878


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [Xen-devel] Re: [Patch RFC] ttm: nouveau accelerated on Xen pv-ops kernel

2010-03-12 Thread Michael D Labriola
xen-devel-boun...@lists.xensource.com wrote on 03/12/2010 07:45:57 AM:

 On Thu, Mar 11, 2010 at 4:32 PM, Pekka Paalanen p...@iki.fi wrote:
*snip*
 Efficacy of patch:
 successful flightgear run on dom0 AND bareboot!
 
 Arvind R.
 

If you boot your system up w/out a monitor plugged in, then plug one in, 
do you ever get signal?  I've been playing with nouveau (just bare-metal 
up until now!) for a few weeks as well, and my biggest problem with it is 
that if you have a bunch of PCs on a cheap KVM, they all have to have the 
monitor/keyboard/mouse at boot or you never get any TTY or X logins... 
Maybe I'm just doing something wrong?

-Mike

---
Michael D Labriola
Electric Boat
mlabr...@gdeb.com
401-848-8871 (desk)
401-848-8513 (lab)
401-316-9844 (cell)



--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 25776] [regression] crash on loading radeon module on 2.6.33-rc1 vanilla

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=25776


Alex Deucher ag...@yahoo.com changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Comment #4 from Alex Deucher ag...@yahoo.com  2010-03-12 06:35:05 PST ---
This should be fixed.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [Intel-gfx] [PATCH 3/3] libdrm: Move all noninstalled headers to noinst_HEADERS.

2010-03-12 Thread Dan Nicholson
On Fri, Mar 12, 2010 at 4:50 AM, Pauli Nieminen suok...@gmail.com wrote:
 Cleanup make system so that all noninstalled headers are put
 to noinst_HEADERS. This quarentees that header will be present
 in tar ball but not installed with make install.

 CC: nouv...@lists.freedesktop.org
 Signed-off-by: Pauli Nieminen suok...@gmail.com
 ---
  Makefile.am         |    6 +++---
  intel/Makefile.am   |   10 ++
  nouveau/Makefile.am |    4 +++-
  3 files changed, 12 insertions(+), 8 deletions(-)

 diff --git a/Makefile.am b/Makefile.am
 index e434e1d..61da186 100644
 --- a/Makefile.am
 +++ b/Makefile.am
 @@ -55,13 +55,13 @@ libdrm_la_SOURCES =                         \
        xf86drmHash.c                           \
        xf86drmRandom.c                         \
        xf86drmSL.c                             \
 -       xf86drmMode.c                           \
 -       libdrm_lists.h
 +       xf86drmMode.c

  libdrmincludedir = ${includedir}
  libdrminclude_HEADERS = xf86drm.h xf86drmMode.h

 -noinst_HEADERS = xf86atomic.h
 +noinst_HEADERS = xf86atomic.h \
 +       libdrm_lists.h


  EXTRA_DIST = libdrm.pc.in include/drm/*
 diff --git a/intel/Makefile.am b/intel/Makefile.am
 index 9add505..d56b03a 100644
 --- a/intel/Makefile.am
 +++ b/intel/Makefile.am
 @@ -36,14 +36,16 @@ libdrm_intel_la_LIBADD = ../libdrm.la @PTHREADSTUBS_LIBS@ 
 @CLOCK_LIB@

  libdrm_intel_la_SOURCES = \
        intel_bufmgr.c \
 -       intel_bufmgr_priv.h \
        intel_bufmgr_fake.c \
        intel_bufmgr_gem.c \
 -       intel_chipset.h \
 -       mm.c \
 -       mm.h
 +       mm.c

  libdrm_intelincludedir = ${includedir}
  libdrm_intelinclude_HEADERS = intel_bufmgr.h

 +noinst_HEADERS = \
 +       intel_bufmgr_priv.h \
 +       intel_chipset.h \
 +       mm.h
 +
  pkgconfig_DATA = libdrm_intel.pc
 diff --git a/nouveau/Makefile.am b/nouveau/Makefile.am
 index 5d759c5..53099eb 100644
 --- a/nouveau/Makefile.am
 +++ b/nouveau/Makefile.am
 @@ -18,7 +18,6 @@ libdrm_nouveau_la_SOURCES = \
                            nouveau_notifier.c \
                            nouveau_bo.c \
                            nouveau_resource.c \
 -                           nouveau_private.h \
                            nouveau_reloc.c

  libdrm_nouveaucommonincludedir = ${includedir}/nouveau
 @@ -38,5 +37,8 @@ libdrm_nouveauincludedir = ${includedir}/drm
  libdrm_nouveauinclude_HEADERS = \
                                nouveau_drmif.h

 +noinst_HEADERS = \
 +                 nouveau_private.h
 +
  pkgconfigdir = @pkgconfigdir@
  pkgconfig_DATA = libdrm_nouveau.pc

I don't think this is the best strategy. Having the headers under
_SOURCES already means they'll get distributed. The only reason for
the separate _HEADERS declarations is if you need to install them.

http://www.gnu.org/software/automake/manual/automake.html#Headers

--
Dan

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [Linux-fbdev-devel] drm_fb_helper: Impossible to change video mode

2010-03-12 Thread James Simmons

On Thu, 11 Mar 2010, Michal Suchanek wrote:

 On 11 March 2010 16:17, James Simmons jsimm...@infradead.org wrote:
 
   It would be nice to find a way to reclaim the console memory for X,
   but I'm not sure that can be done and still provide a good way to
   provide oops support.
  
   What do you think the average user will care about more?
  
        * Seeing kernel oops/panic output about once in a lifetime.
        * Being able to start/use X in the first place and enabling it to
          use all of VRAM.
  
   Personally, I've never even seen any kernel oops/panic output despite
   numerous opportunities for that in the couple of months I've been using
   KMS. But I have spent considerable time and effort trying to get rid of
   the pinned fbcon BO. If the oops/panic output is the only thing
   preventing that, maybe that should only be enabled via some module
   option for developers.
 
  I'm all for it!
 
  I'm looking into the details for this. It will require some changes to
  internal apis to make it to work.
 
 
 Can't it print the oops on whatever is currently displayed?
 
 It need not be a dedicated buffer as long as there is always some buffer.
 
 But perhaps this is more complex than that.

Yes it is very complex. Reading the code and drm specs you come to
realize buffer handling is done with GEM, TTM, or for older drivers drm_maps.
Drivers often handle a combine of those, meaning no real wrapper from one 
api to another :-( From the code it appears GEM is the main userland interface
when using KMS. Some how TTM is also usable from userland but I never found a
clear example of how that is done. So to the average userland app writer it is
a mystery. As for hardware that has a static front buffer I can see how to 
use drm_maps or TTM but I don't see a easy way to map it to the GEM api. 
Also their exist ioctl for gem but it appears no one actually uses them 
but instead write their own :-( So you can see the confusion here. 
Outside of what I described above the drm_framebuffer handling is 
a mess. From what I can see with the code you can only create a 
drm_framebuffer with the GEM api. With this case the two most important 
functions to provide are

dev-mode_config.funcs-fb_create(dev, file_priv, r)

and

fb-funcs-create_handle(fb, file_priv, r-handle);

As you can see if the functions they depend on a handle and a drm_file. To 
make it possible to create a framebuffer internally using a common code we 
would remove those requirements. 
This gets me to point of where to go from here. We have two choices.
The first being we could just make the drm_framebuffer code totally gem 
dependent thus we could cleanup the drivers code up by moving gem code 
there. The second option is to make the drm_framebuffer code agnostic to the 
gem 
layer. So I have been pondering on how to make the second option work. 
There is one thing that all these layers do share in common. That is they 
have some sort of drm_hash with a object lookup. Still pondering how that 
would be done.--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH 2/3] libdrm_radeon: Optimize cs_gem_reloc to do less looping. (V2)

2010-03-12 Thread Michel Dänzer
On Fri, 2010-03-12 at 14:50 +0200, Pauli Nieminen wrote: 
 bo-referenced_in_cs is checked if bo is already in cs. Adding and removing
 reference in bo is done with atomic operations to allow parallel access to a
 bo from multiple contexts.
 
 cs-id generation code quarentees there is not duplicated ids which limits
 number of cs-ids to 32. If there is more cs objects rest will get id 0.
 
 V2:
  - Fix configure to check for atomics operations if libdrm_radeon is selected 
 and libdrm_intel is not.
  - Make atomic operations private to libdrm_radeon.
  - Add warning messages if disabling automaticaly selected 
 libdrm_(radeon|intel)
 
 This optimization decreases cs_write_reloc share of torcs profiling from 4.3%
 to 2.6%.
 
 Signed-off-by: Pauli Nieminen suok...@gmail.com

Tested-by: Michel Dänzer mic...@daenzer.net


-- 
Earthling Michel Dänzer   |http://www.vmware.com
Libre software enthusiast |  Debian, X and DRI developer

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27043] [bisected DRI1] drmRadeonCmdBuffer: -22. Kernel failed to parse or rejected command stream. See dmesg for more info.

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27043





--- Comment #2 from Alex Deucher ag...@yahoo.com  2010-03-12 08:23:29 PST ---
should be fixed in d1ca1599f0319f5c99852ce24420aa592e806db0


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27043] [bisected DRI1] drmRadeonCmdBuffer: -22. Kernel failed to parse or rejected command stream. See dmesg for more info.

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27043





--- Comment #3 from Fabio Pedretti fabio@libero.it  2010-03-12 08:43:09 
PST ---
(In reply to comment #2)
 should be fixed in d1ca1599f0319f5c99852ce24420aa592e806db0
 

It's fixed, but I noticed that it would work also with:

functions-CopyPixels = _mesa_meta_CopyPixels;
functions-DrawPixels = _mesa_meta_DrawPixels;
if (radeon-radeonScreen-kernel_mm) {
functions-ReadPixels = radeonReadPixels;
}

Is this better than current?

if (radeon-radeonScreen-kernel_mm) {
functions-CopyPixels = _mesa_meta_CopyPixels;
functions-DrawPixels = _mesa_meta_DrawPixels;
functions-ReadPixels = radeonReadPixels;
}


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


blit framebeuffer to other framebuffer

2010-03-12 Thread Peter Hanzel
Hello.

I have an app that uses drmModeSetCtrc to display frontbuffer (f_fb_id) and 
next again drmModeSetCrtc to display backbuffer (f_bb_id).

I have tested this on vmwgfx, but it doesn't work.
vmwgfx says:
vmw_ldu_crtc_set_config: DRM_ERROR(Multiple framebuffers not supported\n);

So my question is how to blit f_bb_id to f_fb_id to display it?

Thanks.


--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26430] [KMS] Hw i2c patch doesn't work fully on rv280

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26430





--- Comment #13 from Alex Deucher ag...@yahoo.com  2010-03-12 10:01:04 PST ---
Created an attachment (id=34005)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=34005)
fix i2c prescale calc

This patch should hopefully do the trick.  Can you try it with and without the
previous patch from this bug?


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26430] [KMS] Hw i2c patch doesn't work fully on rv280

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26430





--- Comment #14 from Alex Deucher ag...@yahoo.com  2010-03-12 10:02:01 PST ---
You might also try setting i2c_clock to 50 if you are still having problems.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[PATCHES] more radeon drm kms hw i2c fixes

2010-03-12 Thread Alex Deucher
This patch series gets hw i2c working on my r1xx card and should fix
the hw i2c issues some others have run into on some r2xx and r3xx
cards.

Alex


0001-drm-radeon-kms-fix-for-hw-i2c.patch
Description: application/mbox


0002-drm-radeon-kms-fix-i2c-prescale-calc-on-older-radeo.patch
Description: application/mbox


0003-drm-radeon-kms-r1xx-enable-hw-i2c.patch
Description: application/mbox
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27043] [bisected DRI1] drmRadeonCmdBuffer: -22. Kernel failed to parse or rejected command stream. See dmesg for more info.

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27043





--- Comment #4 from Alex Deucher ag...@yahoo.com  2010-03-12 10:53:26 PST ---
(In reply to comment #3)
 (In reply to comment #2)
  should be fixed in d1ca1599f0319f5c99852ce24420aa592e806db0
  
 
 It's fixed, but I noticed that it would work also with:
 
 functions-CopyPixels = _mesa_meta_CopyPixels;
 functions-DrawPixels = _mesa_meta_DrawPixels;
 if (radeon-radeonScreen-kernel_mm) {
 functions-ReadPixels = radeonReadPixels;
 }
 
 Is this better than current?
 
 if (radeon-radeonScreen-kernel_mm) {
 functions-CopyPixels = _mesa_meta_CopyPixels;
 functions-DrawPixels = _mesa_meta_DrawPixels;
 functions-ReadPixels = radeonReadPixels;
 }
 

Should be fine.  Care to create a patch?


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27043] [bisected DRI1] drmRadeonCmdBuffer: -22. Kernel failed to parse or rejected command stream. See dmesg for more info.

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27043





--- Comment #5 from Michel Dänzer mic...@daenzer.net  2010-03-12 10:58:05 PST 
---
(In reply to comment #3)
 functions-CopyPixels = _mesa_meta_CopyPixels;
 functions-DrawPixels = _mesa_meta_DrawPixels;
 if (radeon-radeonScreen-kernel_mm) {
 functions-ReadPixels = radeonReadPixels;
 }

Did you actually test glReadPixels() with that?


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27043] [bisected DRI1] drmRadeonCmdBuffer: -22. Kernel failed to parse or rejected command stream. See dmesg for more info.

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27043





--- Comment #6 from Michel Dänzer mic...@daenzer.net  2010-03-12 11:02:17 PST 
---
(In reply to comment #5)
 Did you actually test glReadPixels() with that?

Err, I mean glCopyPixels and glDrawPixels.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 15276] latest git kernel: general protection fault: 0000 [#1]

2010-03-12 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=15276





--- Comment #55 from Michał Witkowski ne...@o2.pl  2010-03-12 19:34:40 ---
Created an attachment (id=25492)
 -- (http://bugzilla.kernel.org/attachment.cgi?id=25492)
message.log after applying the patch

I tried 2.6.34-rc1, KDE 4.4.1, mesa/libdrm/xf86-video-ati from 20100306. Tested
with the Present Windows testcase.

Observations:
After a few seconds of moving the mouse around the presented windows, the
screen goes out for a second but later comes back. Please note: there is _no
more_ permanent freeze after this. Each mini freeze is noted in messages.log
(given in attachment).

The above patches are a huge improvement :) They don't eliminate the bug, yet,
but make the freezes not permament :) Awesome work Jerome!

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are watching the assignee of the bug.
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26999] _mesa_reference_renderbuffer: Assertion `oldRb-Magic == 0xaabbccdd' failed

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26999


Julien Cristau jcris...@debian.org changed:

   What|Removed |Added

 AssignedTo|xorg-t...@lists.x.org   |dri-
   ||de...@lists.sourceforge.net
  Component|Driver/Via  |Drivers/DRI/Unichrome
   Keywords|NEEDINFO|
Product|xorg|Mesa
  QAContact|xorg-t...@lists.x.org   |




-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27043] [bisected DRI1] drmRadeonCmdBuffer: -22. Kernel failed to parse or rejected command stream. See dmesg for more info.

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27043





--- Comment #7 from Alex Deucher ag...@yahoo.com  2010-03-12 12:09:02 PST ---
Created an attachment (id=34007)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=34007)
enable meta ops for draw/coopy pixels

Seems to work ok here.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26887] fence errors with rs785 and kernel 2.6.33

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26887





--- Comment #7 from Marc marvi...@gmx.de  2010-03-12 12:11:32 PST ---
tried with nosmp, mem=2G (out of 4) and NO_HZ, NO_PREEMPT - no change. below is
the log with glisse drm-radeon-next tree (grr - again slow chip clock default):


[7.940041] [drm] Initialized drm 1.1.0 20060810
[8.479532] [drm] radeon defaulting to kernel modesetting.
[8.482583] [drm] radeon kernel modesetting enabled.
[8.492371] radeon :01:05.0: PCI INT A - Link[LNKC] - GSI 10 (level,
low) - IRQ 10
[8.495381] radeon :01:05.0: setting latency timer to 64
[8.496445] [drm] radeon: Initializing kernel modesetting.
[8.499493] [drm] register mmio base: 0xFE9F
[8.502432] [drm] register mmio size: 65536
[8.505857] ATOM BIOS: 113
[8.508694] [drm] Clocks initialized !
[8.511485] [drm] 3 Power State(s)
[8.514250] [drm] State 0 Default (default)
[8.517008] [drm]1 Clock Mode(s)
[8.519743] [drm]0 engine: 30
[8.522456] [drm] State 1 Performance 
[8.525133] [drm]1 Clock Mode(s)
[8.527734] [drm]0 engine: 20
[8.530262] [drm] State 2 Default 
[8.532758] [drm]1 Clock Mode(s)
[8.535239] [drm]0 engine: 50
[8.537706] [drm] radeon: power management initialized
[8.540196] radeon :01:05.0: VRAM: 128M 0xC000 - 0xC7FF (128M
used)
[8.542724] radeon :01:05.0: GTT: 512M 0xA000 - 0xBFFF
[8.545807] [drm] Detected VRAM RAM=128M, BAR=128M
[8.546508] [drm] RAM width 32bits DDR
[8.550749] [TTM] Zone  kernel: Available graphics memory: 1029488 kiB.
[8.551439] [drm] radeon: 128M of VRAM memory ready
[8.552113] [drm] radeon: 512M of GTT memory ready.
[8.552785] [drm] radeon: irq initialized.
[8.553447] [drm] GART: num cpu pages 131072, num gpu pages 131072
[8.554599] [drm] Loading RS780 Microcode
[8.555252] platform radeon_cp.0: firmware: requesting radeon/RS780_pfp.bin
[8.615296] platform radeon_cp.0: firmware: requesting radeon/RS780_me.bin
[8.643581] platform radeon_cp.0: firmware: requesting radeon/R600_rlc.bin
[8.688448] [drm] ring test succeeded in 1 usecs
[8.689138] [drm] radeon: ib pool ready.
[   14.190109] radeon :01:05.0: GPU lockup CP stall for more than 1000msec
[   14.190738] [ cut here ]
[   14.191396] WARNING: at drivers/gpu/drm/radeon/radeon_fence.c:234
radeon_fence_wait+0x35d/0x3c0 [radeon]()
[   14.192044] Hardware name: System Product Name
[   14.192671] GPU lockup (waiting for 0x0001 last fence id 0x)
[   14.193314] Modules linked in: snd_hda_intel(+) radeon(+) snd_emu10k1
snd_rawmidi ttm snd_hda_codec snd_ac97_codec ac97_bus drm_kms_helper snd_pcm
snd_seq_device drm snd_util_mem amd64_edac_mod emu10k1_gp snd_timer
i2c_algo_bit snd_hwdep firewire_ohci snd kobil_sct edac_core firewire_core
shpchp gameport crc_itu_t asus_atk0110 pcspkr soundcore snd_page_alloc button
usbserial k10temp edac_mce_amd i2c_piix4 pci_hotplug sr_mod sg cdrom sd_mod
ahci fan processor pata_atiixp libata scsi_mod thermal thermal_sys
[   14.196212] Pid: 691, comm: work_for_cpu Not tainted 2.6.33 #3
[   14.196888] Call Trace:
[   14.197575]  [810466a8] warn_slowpath_common+0x78/0xb0
[   14.198260]  [8104673c] warn_slowpath_fmt+0x3c/0x40
[   14.198937]  [a033a5fd] radeon_fence_wait+0x35d/0x3c0 [radeon]
[   14.199616]  [81064070] ? autoremove_wake_function+0x0/0x40
[   14.200299]  [a0375569] r600_ib_test+0x189/0x300 [radeon]
[   14.200961]  [a037d6e0] r600_init+0x2e0/0x360 [radeon]
[   14.201627]  [a03293ad] radeon_device_init+0x29d/0x370 [radeon]
[   14.202297]  [a032a1ee] radeon_driver_load_kms+0x9e/0x1d0 [radeon]
[   14.202945]  [a020140e] drm_get_dev+0x34e/0x560 [drm]
[   14.203593]  [8103c86d] ? default_wake_function+0xd/0x10
[   14.204227]  [8105f7f0] ? do_work_for_cpu+0x0/0x30
[   14.204851]  [a0397012] radeon_pci_probe+0x10/0x270 [radeon]
[   14.205479]  [81225d72] local_pci_probe+0x12/0x20
[   14.206100]  [8105f803] do_work_for_cpu+0x13/0x30
[   14.206704]  [81063b7e] kthread+0x8e/0xa0
[   14.207314]  [81003b94] kernel_thread_helper+0x4/0x10
[   14.207902]  [81063af0] ? kthread+0x0/0xa0
[   14.208503]  [81003b90] ? kernel_thread_helper+0x0/0x10
[   14.209100] ---[ end trace 48fab13bc7a5b259 ]---
[   14.209681] [drm] Disabling audio support
[   14.209708] radeon :01:05.0: GPU softreset 
[   14.210855] radeon :01:05.0:   R_008010_GRBM_STATUS=0xA0003030
[   14.211443] radeon :01:05.0:   R_008014_GRBM_STATUS2=0x0003
[   14.212028] radeon :01:05.0:   R_000E50_SRBM_STATUS=0x20002040
[   14.339731] radeon :01:05.0: Wait for MC idle timedout !
[   14.340318] radeon :01:05.0:   R_008020_GRBM_SOFT_RESET=0x7FEE
[   14.355896] radeon :01:05.0: R_008020_GRBM_SOFT_RESET=0x0001
[   

[Bug 26887] fence errors with rs785 and kernel 2.6.33

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26887





--- Comment #8 from Marc marvi...@gmx.de  2010-03-12 12:40:44 PST ---
ok - turned out that the oops where pm related. When started with
radeon.{dynpm,dynclks}=0 everything works fine!
Unfortunately, I cannot test the GPU reset patches alone, as they do not apply
to 2.6.33. Jérôme, could you please supply something relative to 2.6.33?

Thanks!


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [Linux-fbdev-devel] drm_fb_helper: Impossible to change video mode

2010-03-12 Thread Dave Airlie
 
   It would be nice to find a way to reclaim the console memory for X,
   but I'm not sure that can be done and still provide a good way to
   provide oops support.
  
   What do you think the average user will care about more?
  
        * Seeing kernel oops/panic output about once in a lifetime.
        * Being able to start/use X in the first place and enabling it to
          use all of VRAM.
  
   Personally, I've never even seen any kernel oops/panic output despite
   numerous opportunities for that in the couple of months I've been using
   KMS. But I have spent considerable time and effort trying to get rid of
   the pinned fbcon BO. If the oops/panic output is the only thing
   preventing that, maybe that should only be enabled via some module
   option for developers.
 
  I'm all for it!
 
  I'm looking into the details for this. It will require some changes to
  internal apis to make it to work.
 

 Can't it print the oops on whatever is currently displayed?

 It need not be a dedicated buffer as long as there is always some buffer.

 But perhaps this is more complex than that.

        Yes it is very complex. Reading the code and drm specs you come to
 realize buffer handling is done with GEM, TTM, or for older drivers drm_maps.
 Drivers often handle a combine of those, meaning no real wrapper from one
 api to another :-( From the code it appears GEM is the main userland interface
 when using KMS. Some how TTM is also usable from userland but I never found a
 clear example of how that is done. So to the average userland app writer it is
 a mystery. As for hardware that has a static front buffer I can see how to
 use drm_maps or TTM but I don't see a easy way to map it to the GEM api.
 Also their exist ioctl for gem but it appears no one actually uses them
 but instead write their own :-( So you can see the confusion here.

Userspace buffer management interfaces are pre-driver, the only requirement
if that they have a 32-bit handle to identify buffers uniquely. Pre-KMS drivers
don't exist for the purposes of fb interaction, so drm_maps are ignorable from
that pov.

        Outside of what I described above the drm_framebuffer handling is
 a mess. From what I can see with the code you can only create a
 drm_framebuffer with the GEM api. With this case the two most important
 functions to provide are

This isn't correct. You get a drm_file and a handle, the driver then uses
these to do whatever it wants to do. This means lookup a GEM object or
whatever but there is no reliance on GEM or any other memory manager
outside the driver. Again a handle a file priv are in no way GEM specific.


 dev-mode_config.funcs-fb_create(dev, file_priv, r)

 and

 fb-funcs-create_handle(fb, file_priv, r-handle);

 As you can see if the functions they depend on a handle and a drm_file. To
 make it possible to create a framebuffer internally using a common code we
 would remove those requirements.

We already have an internal framebuffer creation for fbdev, there is
an fb_create
callback that does this, its not up to dynamic fbdev creation.

        This gets me to point of where to go from here. We have two choices.
 The first being we could just make the drm_framebuffer code totally gem
 dependent thus we could cleanup the drivers code up by moving gem code
 there. The second option is to make the drm_framebuffer code agnostic to the 
 gem
 layer. So I have been pondering on how to make the second option work.
 There is one thing that all these layers do share in common. That is they
 have some sort of drm_hash with a object lookup. Still pondering how that
 would be done.

I'm not sure either of these makes sense, can you clearly state the
goal and maybe
we can work out what you need.

Dave.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27052] New: [KMS] We should Recalculate I2C clock on engine clock setting

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27052

   Summary: [KMS] We should Recalculate I2C clock on engine clock
setting
   Product: DRI
   Version: unspecified
  Platform: Other
OS/Version: All
Status: NEW
  Severity: normal
  Priority: medium
 Component: DRM/Radeon
AssignedTo: dri-devel@lists.sourceforge.net
ReportedBy: zaj...@gmail.com


http://cgit.freedesktop.org/xorg/driver/xf86-video-radeonhd/commit/?id=42a81085042606cd812732a13b66527d37fc625c

pm: Recalculate I2C clock on engine clock setting.
On pre-R6xx the DDC clock is derived from the engine clock, thus changing the
engine clock implies that the DDC clock is to be updated as well.

Same should be done for radeon KMS.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 15276] latest git kernel: general protection fault: 0000 [#1]

2010-03-12 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=15276





--- Comment #56 from Andreas Wallberg andreas.wallb...@gmail.com  2010-03-12 
20:56:28 ---
(In reply to comment #55)
 Created an attachment (id=25492)
 -- (http://bugzilla.kernel.org/attachment.cgi?id=25492) [details]
 message.log after applying the patch
 
 I tried 2.6.34-rc1, KDE 4.4.1, mesa/libdrm/xf86-video-ati from 20100306. 
 Tested
 with the Present Windows testcase.
 
 Observations:
 After a few seconds of moving the mouse around the presented windows, the
 screen goes out for a second but later comes back. Please note: there is _no
 more_ permanent freeze after this. Each mini freeze is noted in messages.log
 (given in attachment).
 
 The above patches are a huge improvement :) They don't eliminate the bug, yet,
 but make the freezes not permament :) Awesome work Jerome!

Wow, that sounds absolutely fantastic. Just for the record, could you please
provide the commands to get this patched easily. We're both using Arch. Did you
hack the PKGBUILD? Yes, I am git n00b, and lazy.

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are watching the assignee of the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27052] [KMS] We should Recalculate I2C clock on engine clock setting

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27052


Alex Deucher ag...@yahoo.com changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||NOTABUG




--- Comment #1 from Alex Deucher ag...@yahoo.com  2010-03-12 13:08:13 PST ---
It's already done for kms.  If we use the hw engines we take the pm mutex and
call get_engine_clock() to get the current sclk.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 27052] [KMS] We should Recalculate I2C clock on engine clock setting

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=27052





--- Comment #2 from Rafał Miłecki zaj...@gmail.com  2010-03-12 13:12:55 PST 
---
Oh, OK, thanks.

Browsing ooold e-mails I noticed last time we didn't implement that. I thought
it was forgotten.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: [PATCH 1/2] libdrm: Move intel_atomic.h to libdrm core for sharing.

2010-03-12 Thread Eric Anholt
On Wed, 10 Mar 2010 18:20:42 +0200, Pauli Nieminen suok...@gmail.com wrote:
 intel_atomic.h includes very usefull atomic operations for
 lock free parrallel access of variables. Moving these to
 core libdrm for code sharing with radeon.

s/xf86/libdrm/ but other than that, cool.


pgp9PLwqTrOT9.pgp
Description: PGP signature
--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 15276] latest git kernel: general protection fault: 0000 [#1]

2010-03-12 Thread bugzilla-daemon
http://bugzilla.kernel.org/show_bug.cgi?id=15276





--- Comment #57 from Andreas Wallberg andreas.wallb...@gmail.com  2010-03-13 
00:59:46 ---
Created an attachment (id=25493)
 -- (http://bugzilla.kernel.org/attachment.cgi?id=25493)
dmesg output regarding radeon_fence.c

This appears in log as video output goes black for about a second using patches
applied to Linus' git tree as of 2009-03-13.

-- 
Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are watching the assignee of the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: Regression in i915 on 2.6.34-rc1

2010-03-12 Thread Pete Zaitcev
On Thu, 11 Mar 2010 00:33:58 -0700, Pete Zaitcev zait...@redhat.com wrote:

I apologise for answering to myself, but while there was no answer,
git bisect found the offending commit and I verified that it was
the culprit. Also, I am adding Bjorn and Jesse to cc:.

 I seem to hit a sudden regression in 2.6.34-rc1: the modeset fails.
 On this box it also means, no way to start X, which is unfortunate.
 
 Here's a quote from bad dmesg (truncated front and back for brievity):
 
 Linux agpgart interface v0.103
 agpgart-intel :00:00.0: Intel HD Graphics Chipset
 agpgart-intel :00:00.0: detected 131068K stolen memory
 agpgart-intel :00:00.0: AGP aperture is 256M @ 0xd000
 tpm_tis 00:09: 1.2 TPM (device-id 0xB, rev-id 16)
 Serial: 8250/16550 driver, 4 ports, IRQ sharing enabled
 
 udev: starting version 151
 [drm] Initialized drm 1.1.0 20060810
 i915 :00:02.0: PCI INT A - GSI 16 (level, low) - IRQ 16
 i915 :00:02.0: setting latency timer to 64
   alloc irq_desc for 33 on node -1
   alloc kstat_irqs on node -1
 i915 :00:02.0: irq 33 for MSI/MSI-X
 [drm] set up 127M of stolen space
 [drm:i915_gem_init_ringbuffer] *ERROR* Ring head not reset to zero ctl 
  head  tail  start 
 [drm:i915_gem_init_ringbuffer] *ERROR* Ring head forced to zero ctl  
 head  tail  start 
 [drm:i915_gem_init_ringbuffer] *ERROR* Ring initialization failed ctl 
  head  tail  start 
 [drm:i915_driver_load] *ERROR* failed to init modeset
 i915: probe of :00:02.0 failed with error -5
 dracut: Starting plymouth daemon
 
 Here's old one from 2.6.33:
 
 Linux agpgart interface v0.103
 agpgart-intel :00:00.0: Intel Ironlake/D Chipset
 agpgart-intel :00:00.0: detected 131068K stolen memory
 agpgart-intel :00:00.0: AGP aperture is 256M @ 0xd000
 [drm] Initialized drm 1.1.0 20060810
 Serial: 8250/16550 driver, 4 ports, IRQ sharing enabled
 ..
 ACPI: Power Button [PWRF]
 i915 :00:02.0: PCI INT A - GSI 16 (level, low) - IRQ 16
 i915 :00:02.0: setting latency timer to 64
 i915 :00:02.0: irq 31 for MSI/MSI-X
 [drm] set up 127M of stolen space
 Console: switching to colour frame buffer device 210x65
 fb0: inteldrmfb frame buffer device
 registered panic notifier
 [Firmware Bug]: ACPI: ACPI brightness control misses _BQC function
 acpi device:1d: registered as cooling_device5
 input: Video Bus as 
 /devices/LNXSYSTM:00/LNXSYBUS:00/PNP0A08:00/LNXVIDEO:00/input/input3
 ACPI: Video Device [GFX0] (multi-head: yes  rom: no  post: no)
 [drm] Initialized i915 1.6.0 20080730 for :00:02.0 on minor 0
 dracut: Starting plymouth daemon

The commit follows appended.

It's possible that the BIOS on this motherboard is not up to snuff,
but the 2.6.33-rc8 worked fine, so clearly Linux can do it... right?

Cheers,
-- Pete

commit 7bc5e3f2be32ae6fb0c74cd0f707f986b3a01a26
Author: Bjorn Helgaas bjorn.helg...@hp.com
Date:   Tue Feb 23 10:24:41 2010 -0700

x86/PCI: use host bridge _CRS info by default on 2008 and newer machines

The main benefit of using ACPI host bridge window information is that
we can do better resource allocation in systems with multiple host bridges,
e.g., http://bugzilla.kernel.org/show_bug.cgi?id=14183

Sometimes we need _CRS information even if we only have one host bridge,
e.g., https://bugs.launchpad.net/ubuntu/+source/linux/+bug/341681

Most of these systems are relatively new, so this patch turns on
pci=use_crs only on machines with a BIOS date of 2008 or newer.

Signed-off-by: Bjorn Helgaas bjorn.helg...@hp.com
Signed-off-by: Jesse Barnes jbar...@virtuousgeek.org

diff --git a/Documentation/kernel-parameters.txt 
b/Documentation/kernel-parameters.txt
index 516225a..3e69c1c 100644
--- a/Documentation/kernel-parameters.txt
+++ b/Documentation/kernel-parameters.txt
@@ -1948,8 +1948,12 @@ and is between 256 and 4096 characters. It is defined in 
the file
IRQ routing is enabled.
noacpi  [X86] Do not use ACPI for IRQ routing
or for PCI scanning.
-   use_crs [X86] Use _CRS for PCI resource
-   allocation.
+   use_crs [X86] Use PCI host bridge window information
+   from ACPI.  On BIOSes from 2008 or later, this
+   is enabled by default.  If you need to use this,
+   please report a bug.
+   nocrs   [X86] Ignore PCI host bridge windows from ACPI.
+   If you need to use this, please report a bug.
routeirqDo IRQ routing for all PCI devices.
This is normally done in pci_enable_device(),
so this option is a temporary workaround
diff --git 

[Bug 26430] [KMS] Hw i2c patch doesn't work fully on rv280

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26430





--- Comment #15 from Andrew Randrianasulu rand...@mail.ru  2010-03-12 
22:24:20 PST ---
(In reply to comment #13)
 Created an attachment (id=34005)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=34005) [details]
 fix i2c prescale calc
 
 This patch should hopefully do the trick.  Can you try it with and without the
 previous patch from this bug?
 

Tried with and without previous patch, with i2c_clock = 60 and i2c_clock = 50
(and also 40). No good results, slightly different errors about EDID still
around.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26430] [KMS] Hw i2c patch doesn't work fully on rv280

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26430


Andrew Randrianasulu rand...@mail.ru changed:

   What|Removed |Added

  Attachment #33953|0   |1
is obsolete||
  Attachment #33973|0   |1
is obsolete||




--- Comment #16 from Andrew Randrianasulu rand...@mail.ru  2010-03-12 
22:26:01 PST ---
Created an attachment (id=34014)
 -- (http://bugs.freedesktop.org/attachment.cgi?id=34014)
last two attempt - 60, before - 50


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


[Bug 26430] [KMS] Hw i2c patch doesn't work fully on rv280

2010-03-12 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26430


Andrew Randrianasulu rand...@mail.ru changed:

   What|Removed |Added

  Attachment #34014|application/octet-stream|text/plain
  mime type||




-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug.

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel


Re: Regression in i915 on 2.6.34-rc1

2010-03-12 Thread Pete Zaitcev
On Fri, 12 Mar 2010 22:37:56 -0700
Bjorn Helgaas bjorn.helg...@hp.com wrote:

 Thanks for the report.  Would you mind posting the entire dmesg
 log, /proc/iomem contents, and lspci -vv output somewhere (maybe in
 bugzilla)?
 
 The quote below isn't enough for me to see the problem, but
 http://bugzilla.kernel.org/show_bug.cgi?id=15480 is another regression
 related to this commit. []

Do you prefer me to attach the requested data to Yanko's bug, or
file a new one for now?

-- Pete

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel