Page 1 of 5 123 ... LastLast
Results 1 to 10 of 49

Thread: GLAMOR Radeon Shows 2D Acceleration Promise

  1. #1
    Join Date
    Jan 2007
    Posts
    13,474

    Default GLAMOR Radeon Shows 2D Acceleration Promise

    Phoronix: GLAMOR Radeon Shows 2D Acceleration Promise

    It looks like, thanks in part to an existing shoddy EXA 2D acceleration implementation, that the GLAMOR-based Radeon acceleration support for xf86-video-ati may work out quite well...

    http://www.phoronix.com/vr.php?view=MTEzODc

  2. #2
    Join Date
    Nov 2008
    Location
    Old Europe
    Posts
    888

    Default

    Please AMD, entice Chris Wilson away!

  3. #3
    Join Date
    Jul 2012
    Posts
    1

    Default (Nicaragua is in America)

    There is no need to return...

    Quote Originally Posted by phoronix View Post
    Phoronix: GLAMOR Radeon Shows 2D Acceleration Promise

    It looks like, thanks in part to an existing shoddy EXA 2D acceleration implementation, that the GLAMOR-based Radeon acceleration support for xf86-video-ati may work out quite well...

    http://www.phoronix.com/vr.php?view=MTEzODc

  4. #4

    Default

    If GLAMOR is really that much better, it should be extended for all chips within the r600g driver at some point. Not just SI.

    Thanks Chris!

  5. #5
    Join Date
    Nov 2008
    Location
    Madison, WI, USA
    Posts
    843

    Default

    Quote Originally Posted by halfmanhalfamazing View Post
    If GLAMOR is really that much better, it should be extended for all chips within the r600g driver at some point. Not just SI.

    Thanks Chris!
    According to what's on his blog, he was already testing on a Radeon HD5770. The patches were to the xf86-video-ati driver, and they don't seem chip-specific.

    From radeon_accel.c:
    Code:
    if (info->directRenderingEnabled) {
    	if (info->use_glamor) {
    	    if (!radeon_glamor_init(pScreen)) {
    		info->use_glamor = FALSE;
    		return FALSE;
    	    }
    	} else if (info->ChipFamily >= CHIP_FAMILY_CEDAR) {
    	    if (!EVERGREENDrawInit(pScreen))
    		return FALSE;
    	} else
    	    if (info->ChipFamily >= CHIP_FAMILY_R600) {
    		if (!R600DrawInit(pScreen))
    		    return FALSE;
    	    } else {
    		if (!RADEONDrawInit(pScreen))
    		    return FALSE;
    	    }
        }
    From what I was reading in the patches, you just need glamour >= 0.3.0, and a new git pull of the xf86-video-ati driver (xorg-edgers will probably be along shortly...).
    Last edited by Veerappan; 07-12-2012 at 09:05 AM.

  6. #6
    Join Date
    Dec 2007
    Posts
    2,279

    Default

    Quote Originally Posted by halfmanhalfamazing View Post
    If GLAMOR is really that much better, it should be extended for all chips within the r600g driver at some point. Not just SI.
    It already works with all radeon chips with a 3D driver that supports shaders (basically R300+).

  7. #7
    Join Date
    Nov 2009
    Posts
    92

    Default uhm

    So Intel has come and done some work to make their competitor's GPU perform better in Linux? I realize that It's not really Intel, but just a single Intel dev's hobby project... but I can't help but wonder at AMD's relative rate of progress at this point .

  8. #8
    Join Date
    Sep 2008
    Location
    Vilnius, Lithuania
    Posts
    2,406

    Default

    Quote Originally Posted by Rodrigo View Post
    There is no need to return...
    I saw what you did there.

    Quote Originally Posted by hiryu View Post
    So Intel has come and done some work to make their competitor's GPU perform better in Linux? I realize that It's not really Intel, but just a single Intel dev's hobby project... but I can't help but wonder at AMD's relative rate of progress at this point .
    Hey, cooperation is always good. Intel has acceleration that turned out to be not very spectacular, but why not give it to everyone, including AMD, if they have even worse acceleration. Why AMD has such poor acceleration in the first place is another matter, however...

  9. #9
    Join Date
    Jun 2010
    Location
    ฿ 16LDJ6Hrd1oN3nCoFL7BypHSEYL84ca1JR
    Posts
    969

    Default

    Quote Originally Posted by Veerappan View Post
    From what I was reading in the patches, you just need glamour >= 0.3.0, and a new git pull of the xf86-video-ati driver (xorg-edgers will probably be along shortly...).
    xf86-video-ati has a new configure flag: --enable-glamor

    But you can't use xorg git as of now:
    Code:
    make  all-recursive
    make[1]: Entering directory `/home/chris/build/glamor-git/src/glamor-build'
    Making all in src
    make[2]: Entering directory `/home/chris/build/glamor-git/src/glamor-build/src'
      CC       glamor.lo
      CC       glamor_copyarea.lo
      CC       glamor_copywindow.lo
      CC       glamor_core.lo
      CC       glamor_fill.lo
    In file included from glamor_copywindow.c:24:0:
    glamor_priv.h:551:11: warning: redundant redeclaration of 'glamor_create_pixmap' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor_copywindow.c:24:
    glamor.h:144:28: note: previous declaration of 'glamor_create_pixmap' was here
    In file included from glamor_copywindow.c:24:0:
    glamor_priv.h:570:6: warning: redundant redeclaration of 'glamor_init_pixmap_fbo' [-Wredundant-decls]
    glamor_priv.h:566:6: note: previous declaration of 'glamor_init_pixmap_fbo' was here
    glamor_priv.h:571:6: warning: redundant redeclaration of 'glamor_fini_pixmap_fbo' [-Wredundant-decls]
    glamor_priv.h:567:6: note: previous declaration of 'glamor_fini_pixmap_fbo' was here
    glamor_priv.h:605:6: warning: redundant redeclaration of 'glamor_create_gc' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor_copywindow.c:24:
    glamor.h:233:22: note: previous declaration of 'glamor_create_gc' was here
    In file included from glamor_copywindow.c:24:0:
    glamor_priv.h:685:6: warning: redundant redeclaration of 'glamor_glyph_unrealize' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor_copywindow.c:24:
    glamor.h:300:23: note: previous declaration of 'glamor_glyph_unrealize' was here
    In file included from glamor_copywindow.c:24:0:
    glamor_priv.h:950:2: warning: redundant redeclaration of 'glamor_upload_picture_to_texture' [-Wredundant-decls]
    glamor_priv.h:924:2: note: previous declaration of 'glamor_upload_picture_to_texture' was here
    glamor_priv.h:954:1: warning: redundant redeclaration of 'glamor_fixup_pixmap_priv' [-Wredundant-decls]
    glamor_priv.h:578:6: note: previous declaration of 'glamor_fixup_pixmap_priv' was here
    In file included from glamor_priv.h:1007:0,
                     from glamor_copywindow.c:24:
    glamor_utils.h:801:0: warning: "ARRAY_SIZE" redefined [enabled by default]
    In file included from /usr/include/xorg/privates.h:152:0,
                     from /usr/include/xorg/cursor.h:53,
                     from /usr/include/xorg/scrnintstr.h:53,
                     from glamor.h:32,
                     from glamor_priv.h:40,
                     from glamor_copywindow.c:24:
    /usr/include/xorg/dix.h:67:0: note: this is the location of the previous definition
    In file included from glamor_core.c:36:0:
    glamor_priv.h:551:11: warning: redundant redeclaration of 'glamor_create_pixmap' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor_core.c:36:
    glamor.h:144:28: note: previous declaration of 'glamor_create_pixmap' was here
    In file included from glamor_core.c:36:0:
    glamor_priv.h:570:6: warning: redundant redeclaration of 'glamor_init_pixmap_fbo' [-Wredundant-decls]
    glamor_priv.h:566:6: note: previous declaration of 'glamor_init_pixmap_fbo' was here
    glamor_priv.h:571:6: warning: redundant redeclaration of 'glamor_fini_pixmap_fbo' [-Wredundant-decls]
    glamor_priv.h:567:6: note: previous declaration of 'glamor_fini_pixmap_fbo' was here
    glamor_priv.h:605:6: warning: redundant redeclaration of 'glamor_create_gc' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor_core.c:36:
    glamor.h:233:22: note: previous declaration of 'glamor_create_gc' was here
    In file included from glamor_core.c:36:0:
    glamor_priv.h:685:6: warning: redundant redeclaration of 'glamor_glyph_unrealize' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor_core.c:36:
    glamor.h:300:23: note: previous declaration of 'glamor_glyph_unrealize' was here
    In file included from glamor_core.c:36:0:
    glamor_priv.h:950:2: warning: redundant redeclaration of 'glamor_upload_picture_to_texture' [-Wredundant-decls]
    glamor_priv.h:924:2: note: previous declaration of 'glamor_upload_picture_to_texture' was here
    glamor_priv.h:954:1: warning: redundant redeclaration of 'glamor_fixup_pixmap_priv' [-Wredundant-decls]
    glamor_priv.h:578:6: note: previous declaration of 'glamor_fixup_pixmap_priv' was here
    In file included from glamor_priv.h:1007:0,
                     from glamor_core.c:36:
    glamor_utils.h:801:0: warning: "ARRAY_SIZE" redefined [enabled by default]
    In file included from /usr/include/xorg/privates.h:152:0,
                     from /usr/include/xorg/cursor.h:53,
                     from /usr/include/xorg/scrnintstr.h:53,
                     from glamor.h:32,
                     from glamor_priv.h:40,
                     from glamor_core.c:36:
    /usr/include/xorg/dix.h:67:0: note: this is the location of the previous definition
    In file included from glamor_fill.c:28:0:
    glamor_priv.h:551:11: warning: redundant redeclaration of 'glamor_create_pixmap' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor_fill.c:28:
    glamor.h:144:28: note: previous declaration of 'glamor_create_pixmap' was here
    In file included from glamor_fill.c:28:0:
    glamor_priv.h:570:6: warning: redundant redeclaration of 'glamor_init_pixmap_fbo' [-Wredundant-decls]
    glamor_priv.h:566:6: note: previous declaration of 'glamor_init_pixmap_fbo' was here
    glamor_priv.h:571:6: warning: redundant redeclaration of 'glamor_fini_pixmap_fbo' [-Wredundant-decls]
    glamor_priv.h:567:6: note: previous declaration of 'glamor_fini_pixmap_fbo' was here
    glamor_priv.h:605:6: warning: redundant redeclaration of 'glamor_create_gc' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor_fill.c:28:
    glamor.h:233:22: note: previous declaration of 'glamor_create_gc' was here
    In file included from glamor_fill.c:28:0:
    glamor_priv.h:685:6: warning: redundant redeclaration of 'glamor_glyph_unrealize' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor_fill.c:28:
    glamor.h:300:23: note: previous declaration of 'glamor_glyph_unrealize' was here
    In file included from glamor_fill.c:28:0:
    glamor_priv.h:950:2: warning: redundant redeclaration of 'glamor_upload_picture_to_texture' [-Wredundant-decls]
    glamor_priv.h:924:2: note: previous declaration of 'glamor_upload_picture_to_texture' was here
    glamor_priv.h:954:1: warning: redundant redeclaration of 'glamor_fixup_pixmap_priv' [-Wredundant-decls]
    glamor_priv.h:578:6: note: previous declaration of 'glamor_fixup_pixmap_priv' was here
    In file included from glamor_priv.h:1007:0,
                     from glamor_fill.c:28:
    glamor_utils.h:801:0: warning: "ARRAY_SIZE" redefined [enabled by default]
    In file included from /usr/include/xorg/privates.h:152:0,
                     from /usr/include/xorg/cursor.h:53,
                     from /usr/include/xorg/scrnintstr.h:53,
                     from glamor.h:32,
                     from glamor_priv.h:40,
                     from glamor_fill.c:28:
    /usr/include/xorg/dix.h:67:0: note: this is the location of the previous definition
    glamor_core.c: In function 'glamor_gl_has_extension':
    glamor_core.c:577:9: warning: cast discards '__attribute__((const))' qualifier from pointer target type [-Wcast-qual]
    In file included from glamor_copyarea.c:28:0:
    glamor_priv.h:551:11: warning: redundant redeclaration of 'glamor_create_pixmap' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor_copyarea.c:28:
    glamor.h:144:28: note: previous declaration of 'glamor_create_pixmap' was here
    In file included from glamor_copyarea.c:28:0:
    glamor_priv.h:570:6: warning: redundant redeclaration of 'glamor_init_pixmap_fbo' [-Wredundant-decls]
    glamor_priv.h:566:6: note: previous declaration of 'glamor_init_pixmap_fbo' was here
    glamor_priv.h:571:6: warning: redundant redeclaration of 'glamor_fini_pixmap_fbo' [-Wredundant-decls]
    glamor_priv.h:567:6: note: previous declaration of 'glamor_fini_pixmap_fbo' was here
    glamor_priv.h:605:6: warning: redundant redeclaration of 'glamor_create_gc' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor_copyarea.c:28:
    glamor.h:233:22: note: previous declaration of 'glamor_create_gc' was here
    In file included from glamor_copyarea.c:28:0:
    glamor_priv.h:685:6: warning: redundant redeclaration of 'glamor_glyph_unrealize' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor_copyarea.c:28:
    glamor.h:300:23: note: previous declaration of 'glamor_glyph_unrealize' was here
    In file included from glamor_copyarea.c:28:0:
    glamor_priv.h:950:2: warning: redundant redeclaration of 'glamor_upload_picture_to_texture' [-Wredundant-decls]
    glamor_priv.h:924:2: note: previous declaration of 'glamor_upload_picture_to_texture' was here
    glamor_priv.h:954:1: warning: redundant redeclaration of 'glamor_fixup_pixmap_priv' [-Wredundant-decls]
    glamor_priv.h:578:6: note: previous declaration of 'glamor_fixup_pixmap_priv' was here
    In file included from glamor_priv.h:1007:0,
                     from glamor_copyarea.c:28:
    glamor_utils.h:801:0: warning: "ARRAY_SIZE" redefined [enabled by default]
    In file included from /usr/include/xorg/privates.h:152:0,
                     from /usr/include/xorg/cursor.h:53,
                     from /usr/include/xorg/scrnintstr.h:53,
                     from glamor.h:32,
                     from glamor_priv.h:40,
                     from glamor_copyarea.c:28:
    /usr/include/xorg/dix.h:67:0: note: this is the location of the previous definition
    glamor_copyarea.c: In function '_glamor_copy_n_to_n':
    glamor_copyarea.c:402:12: warning: variable 'screen' set but not used [-Wunused-but-set-variable]
    glamor_copyarea.c:395:18: warning: variable 'dst_access' set but not used [-Wunused-but-set-variable]
    In file included from glamor.c:36:0:
    glamor_priv.h:551:11: warning: redundant redeclaration of 'glamor_create_pixmap' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor.c:36:
    glamor.h:144:28: note: previous declaration of 'glamor_create_pixmap' was here
    In file included from glamor.c:36:0:
    glamor_priv.h:570:6: warning: redundant redeclaration of 'glamor_init_pixmap_fbo' [-Wredundant-decls]
    glamor_priv.h:566:6: note: previous declaration of 'glamor_init_pixmap_fbo' was here
    glamor_priv.h:571:6: warning: redundant redeclaration of 'glamor_fini_pixmap_fbo' [-Wredundant-decls]
    glamor_priv.h:567:6: note: previous declaration of 'glamor_fini_pixmap_fbo' was here
    glamor_priv.h:605:6: warning: redundant redeclaration of 'glamor_create_gc' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor.c:36:
    glamor.h:233:22: note: previous declaration of 'glamor_create_gc' was here
    In file included from glamor.c:36:0:
    glamor_priv.h:685:6: warning: redundant redeclaration of 'glamor_glyph_unrealize' [-Wredundant-decls]
    In file included from glamor_priv.h:40:0,
                     from glamor.c:36:
    glamor.h:300:23: note: previous declaration of 'glamor_glyph_unrealize' was here
    In file included from glamor.c:36:0:
    glamor_priv.h:950:2: warning: redundant redeclaration of 'glamor_upload_picture_to_texture' [-Wredundant-decls]
    glamor_priv.h:924:2: note: previous declaration of 'glamor_upload_picture_to_texture' was here
    glamor_priv.h:954:1: warning: redundant redeclaration of 'glamor_fixup_pixmap_priv' [-Wredundant-decls]
    glamor_priv.h:578:6: note: previous declaration of 'glamor_fixup_pixmap_priv' was here
    In file included from glamor_priv.h:1007:0,
                     from glamor.c:36:
    glamor_utils.h:801:0: warning: "ARRAY_SIZE" redefined [enabled by default]
    In file included from /usr/include/xorg/privates.h:152:0,
                     from /usr/include/xorg/cursor.h:53,
                     from /usr/include/xorg/scrnintstr.h:53,
                     from glamor.h:32,
                     from glamor_priv.h:40,
                     from glamor.c:36:
    /usr/include/xorg/dix.h:67:0: note: this is the location of the previous definition
    glamor.c: In function 'glamor_init':
    glamor.c:328:6: warning: passing argument 1 of 'glamor_gl_has_extension' discards 'const' qualifier from pointer target type [enabled by default]
    In file included from glamor.c:36:0:
    glamor_priv.h:638:6: note: expected 'char *' but argument is of type 'const char *'
    glamor.c:330:6: warning: passing argument 1 of 'glamor_gl_has_extension' discards 'const' qualifier from pointer target type [enabled by default]
    In file included from glamor.c:36:0:
    glamor_priv.h:638:6: note: expected 'char *' but argument is of type 'const char *'
    glamor.c:350:22: warning: assignment from incompatible pointer type [enabled by default]
    glamor.c: In function 'glamor_close_screen':
    glamor.c:536:2: warning: passing argument 1 of 'screen->CloseScreen' makes pointer from integer without a cast [enabled by default]
    glamor.c:536:2: note: expected 'ScreenPtr' but argument is of type 'int'
    glamor.c:536:2: error: too many arguments to function 'screen->CloseScreen'
    make[2]: *** [glamor.lo] Error 1
    make[2]: *** Waiting for unfinished jobs....
    make[2]: Leaving directory `/home/chris/build/glamor-git/src/glamor-build/src'
    make[1]: *** [all-recursive] Error 1
    make[1]: Leaving directory `/home/chris/build/glamor-git/src/glamor-build'
    make: *** [all] Error 2

  10. #10
    Join Date
    Nov 2009
    Posts
    92

    Default

    Quote Originally Posted by GreatEmerald View Post
    Hey, cooperation is always good. Intel has acceleration that turned out to be not very spectacular, but why not give it to everyone, including AMD, if they have even worse acceleration. Why AMD has such poor acceleration in the first place is another matter, however...
    Oh definitely! I was just pointing out the irony as well as the fact that a non-AMD dev was able to quickly get results in a little of their spare time. Hopefully with that China deal, AMD will be able to justify putting more resources towards the open radeon drivers.

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •