MantisBT - VTK
View Issue Details
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 
ProjectTBD
Typeusability
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
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2011-07-20 19:58domibelNew Issue
2011-08-02 14:06David PartykaSource_changeset_attached => VTK master 06385b75
2011-08-02 14:06Dominique BelhachemiSource_changeset_attached => VTK master b7dde9bb
2012-04-01 19:18domibelNote Added: 0028419
2013-06-05 14:17Jean-Christophe Fillion-RobinNote Added: 0030901
2013-06-05 14:34domibelNote Added: 0030903
2013-06-05 14:39Jean-Christophe Fillion-RobinNote Added: 0030904
2013-06-05 14:43Jean-Christophe Fillion-RobinNote Added: 0030905
2013-06-06 09:43domibelNote Added: 0030965
2013-06-06 09:46domibelNote Edited: 0030965bug_revision_view_page.php?bugnote_id=30965#r618
2013-06-06 18:55Jean-Christophe Fillion-RobinNote Added: 0030969
2014-10-02 09:40Ben Boeckel (Kitware)Note Added: 0033423
2014-10-02 09:40Ben Boeckel (Kitware)Statusbacklog => closed
2014-10-02 09:40Ben Boeckel (Kitware)Resolutionopen => fixed

Notes
(0028419)
domibel   
2012-04-01 19:18   
can this be addressed in vtk 5.10?
(0030901)
Jean-Christophe Fillion-Robin   
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   
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   
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   
2013-06-05 14:43   
Unit we get more info or somebody re-open the issue, is it okay to close ?
(0030965)
domibel   
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   
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)   
2014-10-02 09:40   
I don't see such a check anymore, so I'm closing this.