Opened 10 years ago

Closed 6 years ago

#722 closed enhancement (fixed)

r.li.daemon: respect for WORKERS enviro var

Reported by: hamish Owned by: grass-dev@…
Priority: normal Milestone: 6.4.4
Component: Raster Version: svn-develbranch6
Keywords: r.li, smp Cc:
CPU: All Platform: All

Description

Hi,

it would be nice if r.li.daemon respected the WORKERS environment variable for setting the number of threads to launch.

AFAICS most of the code can be borrowed from trunk/lib/gis/worker.c. (some doxygen comments would help)

I wonder if both the pthreads and r.li.daemon fork() code should use the same "GRASS_WORKERS" enviro var instead of the more generic "WORKERS" name? or is that name some pthreads standard?

or for that matter, if r.li.daemon should be adapted to use the grass7 pthreads support instead of doing its own thing?

Hamish

Change History (3)

comment:1 Changed 6 years ago by hamish

done in devbr6 with r56730. Glynn has already rewritten that part of the code in trunk with r40696.

comment:2 Changed 6 years ago by hamish

Milestone: 7.0.06.4.4
Version: svn-trunksvn-develbranch6

comment:3 Changed 6 years ago by neteler

Resolution: fixed
Status: newclosed

The WORKERS had to be removed in the backport r59304 of the rewrite of r.li.* in trunk.

Note: See TracTickets for help on using tickets.