View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0014821ParaView(No Category)public2014-06-24 11:252015-01-11 08:16
ReporterVadim SANDLER 
Assigned ToTristan Coulange 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version4.1 
Target VersionFixed in Version4.2 
Summary0014821: Long compilation of ParaView with early versions of gcc 4.4
DescriptionThe compilation of ParaView takes very long time on some Linux distributions.

The analysis has shown that this problem takes place with some versions of gcc compiler. For example, this problem is reproduced on:
- Mandriva 2010.0 64bits: gcc version 4.4.1.
- CentOS 5.5 64bits: gcc version 4.4.0.

The 'make' process freezes at compilation of ParaViewCore/ServerManager/SMApplication/vtkInitializationHelper.cxx file.
The compilation, if not stopped, might take more than a week!!! :)
In the end, the compilation of this file is successfully finished and rest of build procedure is done as usual.
During the compilation of this file, CPU shows ~100% of loading (on multi-core systems only one core is loaded).

The analysis has shown that the problem is caused by ParaView_build/ParaViewCore/ServerManager/SMApplication/vtkSMGeneratedModules.h file that is generated by kwProcessXML tool. Namely, this (very big) file operates with huge hard-coded string variables, e.g.

...
// From file /data/mnt/vsr/tmp/ParaView-4.1.0_SRC/ParaViewCore/ServerManager/SMApplication/Resources/filters.xml
static const char* const vtkSMDefaultModulesfiltersInterface0 =
"<ServerManagerConfiguration>\n"
" <ProxyGroup name=\"internal_filters\">\n"
...

This problem can be fixed by patching of the Utilities/ProcessXML/ProcessXML.cxx file, in "void PrintHeader(const char* title, const char* file)" function, by replacing (in line 53)

      << "static const char* " << this->Prefix.c_str() << title << this->Suffix.c_str() << this->Count

with

      << "static const char* const " << this->Prefix.c_str() << title << this->Suffix.c_str() << this->Count

With this patch, the compilation of mentioned file takes just a second. No any negative side effect has been found after applying of this patch.
TagsNo tags attached.
ProjectParaViS
Topic Name
Typeusability
Attached Files

 Relationships

  Notes
(0033107)
Tristan Coulange (developer)
2014-07-28 12:54

The change was approved on gerrit :

http://review.source.kitware.com/#/t/4445/ [^]

Fix slow compilation via processxml

The compilation was very slow (days) with some linux and gcc versions:
- Mandriva 2010.0 64bits: gcc version 4.4.1.
- CentOS 5.5 64bits: gcc version 4.4.0.
This change fixed the problem. See issue 0014821.

Change-Id: I4b54f17140a56ae22b8e1af8532723e7c8a755b0
(0033165)
Utkarsh Ayachit (administrator)
2014-07-31 09:00

SUMMARY
---------------------------------------------
Topics merged into master:
(VTK) 14599_fix_double_range_issues
        14789_fix_border
        14831_log_scaling_crash
        14848_fix_mutlselection
        appdata-file
        catalyst-dependencies
        catalyst-kits
        compilation_issue_processxml
        excess-moc-wrapping
        fix-cache-doc
        fix-catalyst-python
        kit-libraries
(VTK) newest-xdmf
        pass_connectid_to_autompi
        remove-mpi4py
(VTK) update-vtk-20140729


---------------------------------------------
Topics reverted from next:
  14813_fix_mode_shapes_range
  fix_proxy_widget_dialog_apply
  pvweb-fixes
(0033261)
Alan Scott (manager)
2014-08-19 18:13

Trusting that Vadim and Tristan tested.

 Issue History
Date Modified Username Field Change
2014-06-24 11:25 Vadim SANDLER New Issue
2014-07-17 03:37 Tristan Coulange Assigned To => Tristan Coulange
2014-07-25 04:46 Tristan Coulange Status backlog => active development
2014-07-28 12:54 Tristan Coulange Note Added: 0033107
2014-07-28 12:54 Tristan Coulange Status active development => gatekeeper review
2014-07-31 08:59 Utkarsh Ayachit Fixed in Version => git-master
2014-07-31 09:00 Utkarsh Ayachit Status gatekeeper review => customer review
2014-07-31 09:00 Utkarsh Ayachit Note Added: 0033165
2014-08-19 18:13 Alan Scott Note Added: 0033261
2014-08-19 18:13 Alan Scott Status customer review => closed
2014-08-19 18:13 Alan Scott Resolution open => fixed
2014-08-29 10:59 Utkarsh Ayachit Fixed in Version git-master => 4.2
2015-01-11 08:16 Tristan Coulange Source_changeset_attached => ParaView master 245c75c8


Copyright © 2000 - 2018 MantisBT Team