View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0012385VTK(No Category)public2011-07-20 19:582014-10-02 09:40
Reporterdomibel 
Assigned To 
PriorityurgentSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Summary0012385: Please don't exclude Intel GPU's from VolumeRendering/vtkOpenGLGPUVolumeRayCastMapper.cxx
DescriptionHi,

Intel GPU's contain as usual the substring "Mesa" in the OpenGL renderer string.

e.g.: Mesa DRI Mobile Intel® GM45 Express Chipset GEM 20091221 2009Q4 x86/MMX/SSE2

The following code excludes all those GPU's. I think this check should be removed. It is better to inform the user what OpenGL feature is missing.


in VolumeRendering/vtkOpenGLGPUVolumeRayCastMapper.cxx:2160

  const char *gl_version=reinterpret_cast<const char *>(glGetString(GL_VERSION));
  if(strstr(gl_version,"Mesa")!=0)
    {
    // - GL_VENDOR cannot be used because it can be "Brian Paul" or
    // "Mesa project"
    // - GL_RENDERER cannot be used because it can be "Software Rasterizer" or
    // "Mesa X11"
    // - GL_VERSION is more robust. It has things like "2.0 Mesa 7.0.4" or
    // "2.1 Mesa 7.2" or "2.1 Mesa 7.3-devel"
    // Mesa does not work with multiple draw buffers:
    // "framebuffer has bad draw buffer"
    // "render clipped 1 ERROR (x506) invalid framebuffer operation ext"
    this->LoadExtensionsSucceeded=0;
    return;
    }


Thanks
Dominique
TagsNo tags attached.
ProjectTBD
Typeusability
Attached Files

 Relationships

  Notes
(0028419)
domibel (reporter)
2012-04-01 19:18

 can this be addressed in vtk 5.10?
(0030901)
Jean-Christophe Fillion-Robin (manager)
2013-06-05 14:17

 The changes has been integrated in 06385b7 and reverted in b24567. Do you happen to know what motivated the revert ?
(0030903)
domibel (reporter)
2013-06-05 14:34

 http://www.vtk.org/pipermail/vtk-developers/2011-July/010169.html [^]

might be fixed in the meanwhile...
(0030904)
Jean-Christophe Fillion-Robin (manager)
2013-06-05 14:39

 For reference:

After some internal discussion we have decided to revert this change. The
reason that check was implemented was because often older versions of Mesa
report that they support various OpenGL extensions falsely which then result
in crashes. If further refinement of this check can be developed that would
be preferred. Specifically if we can pinpoint a minimum version of Mesa
that vtkOpenGLGPUVolumeRayCastMapper for sure works with. Until then, in our
effort to minimize crashes the check will have to remain in place.

Source: http://www.vtk.org/pipermail/vtk-developers/2011-July/010169.html [^]
(0030905)
Jean-Christophe Fillion-Robin (manager)
2013-06-05 14:43

 Unit we get more info or somebody re-open the issue, is it okay to close ?
(0030965)
domibel (reporter)
2013-06-06 09:43
edited on: 2013-06-06 09:46

 Please don't close it.

Recent Intel cards come up with lspci:

"Intel Corporation Ivy Bridge Graphics Controller"


But The opengl info still contains

 GL_RENDERER Mesa DRI Intel(R) Ivybridge Mobile
(0030969)
Jean-Christophe Fillion-Robin (manager)
2013-06-06 18:55

 After checking it works, do you think you could submit patch that would exclude all Mesa card expect the Intel one ?
See http://www.vtk.org/Wiki/VTK/Git/Develop [^]
(0033423)
Ben Boeckel (Kitware) (developer)
2014-10-02 09:40

 I don't see such a check anymore, so I'm closing this.

 Issue History
Date Modified Username Field Change
2011-07-20 19:58 domibel New Issue
2011-08-02 14:06 David Partyka Source_changeset_attached => VTK master 06385b75
2011-08-02 14:06 Dominique Belhachemi Source_changeset_attached => VTK master b7dde9bb
2012-04-01 19:18 domibel Note Added: 0028419
2013-06-05 14:17 Jean-Christophe Fillion-Robin Note Added: 0030901
2013-06-05 14:34 domibel Note Added: 0030903
2013-06-05 14:39 Jean-Christophe Fillion-Robin Note Added: 0030904
2013-06-05 14:43 Jean-Christophe Fillion-Robin Note Added: 0030905
2013-06-06 09:43 domibel Note Added: 0030965
2013-06-06 09:46 domibel Note Edited: 0030965
2013-06-06 18:55 Jean-Christophe Fillion-Robin Note Added: 0030969
2014-10-02 09:40 Ben Boeckel (Kitware) Note Added: 0033423
2014-10-02 09:40 Ben Boeckel (Kitware) Status backlog => closed
2014-10-02 09:40 Ben Boeckel (Kitware) Resolution open => fixed

Copyright © 2000 - 2018 MantisBT Team