Humm, I like that idea better and that won't break anything. Thanks Dave!<br><br><div class="gmail_quote">On Tue, Jan 25, 2011 at 2:41 PM, David Cole <span dir="ltr"><<a href="mailto:david.cole@kitware.com">david.cole@kitware.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">I'm not going to object, but .... vtkMultiThreader contains *several*<br>
arrays that are based on the value here. Do a quick sanity test of<br>
creating a vtkMultiThreader object and inspecting its size. Just make<br>
sure it's not ridiculously large.<br>
<br>
Perhaps adding a CMake value that can be tweaked to taste (and then<br>
configured into vtkConfigure.h) with a default value of 32-ish would<br>
be more reasonable.<br>
<br>
<br>
On Tue, Jan 25, 2011 at 2:11 PM, David Partyka<br>
<div><div></div><div class="h5"><<a href="mailto:david.partyka@kitware.com">david.partyka@kitware.com</a>> wrote:<br>
> I am going to change it to 1024 unless someone objects for the immediate<br>
> term. And then from there switch it to be dynamic and deprecate<br>
> VTK_MAX_THREAD entirely.<br>
><br>
> On Tue, Jan 25, 2011 at 8:27 AM, David Partyka <<a href="mailto:david.partyka@kitware.com">david.partyka@kitware.com</a>><br>
> wrote:<br>
>><br>
>> I definitely agree but this looks like it should be given<br>
>> a deprecation cycle as it involves removing API, there are set/get macros<br>
>> and VTK_MAX_THREAD is a const in a global header. Though, nearly all of the<br>
>> usage of VTK_MAX_THREAD is in vtkMultiThreader and it definitely would not<br>
>> be hard to switch it to dynamically store threads.<br>
>> That said, I can check in a much larger value for VTK_MAX_THREAD right<br>
>> now. How about 128? 256? 1024? INT_MAX?<br>
>> After that, myself or someone else can switch it to use a vector instead<br>
>> of a fixed size array and then add the deprecation macros so people know<br>
>> it's going away. Does this sound reasonable? Or too cautious? :-p<br>
>><br>
>> On Tue, Jan 25, 2011 at 8:04 AM, Will Schroeder<br>
>> <<a href="mailto:will.schroeder@kitware.com">will.schroeder@kitware.com</a>> wrote:<br>
>>><br>
>>> I agree with Ken, IMO we should remove hardcoded limits on threads. In<br>
>>> fact I can see creating some specialized code to determine the "optimal"<br>
>>> number of threads (with provisions for manual override). After all in the<br>
>>> Nat'l Lab world that Ken/Sandia resides in, I've heard the dudes talk about<br>
>>> hundreds or thousands of processes, etc....it's evolving fast and we don't<br>
>>> want to have to keep changing this.<br>
>>> W<br>
>>><br>
>>><br>
>>><br>
>>> On Mon, Jan 24, 2011 at 11:14 AM, Moreland, Kenneth <<a href="mailto:kmorel@sandia.gov">kmorel@sandia.gov</a>><br>
>>> wrote:<br>
>>>><br>
>>>> Why is there a hardcoded maximum in the first place?  It seems to me<br>
>>>> that whatever number you choose now will be excessively high and/or obsolete<br>
>>>> in the coming years as the number of cores per processor grows.<br>
>>>><br>
>>>> -Ken<br>
>>>><br>
>>>><br>
>>>> On 1/24/11 8:04 AM, "Bill Lorensen" <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>> wrote:<br>
>>>><br>
>>>> ITK has had the max set to 128 for quite a while.<br>
>>>><br>
>>>> On Mon, Jan 24, 2011 at 9:53 AM, David Partyka<br>
>>>> <<a href="mailto:david.partyka@kitware.com">david.partyka@kitware.com</a>> wrote:<br>
>>>> > Sure thing, especially considering one can get a machine with dual<br>
>>>> > Hexa-cores + Hyper Threading resulting in 24 available cores.<br>
>>>> ><br>
>>>> > On Mon, Jan 24, 2011 at 9:39 AM, David Cole <<a href="mailto:david.cole@kitware.com">david.cole@kitware.com</a>><br>
>>>> > wrote:<br>
>>>> >><br>
>>>> >> I think it's just old and leftover, and nobody has yet found a<br>
>>>> >> compelling<br>
>>>> >> need for more than 8 threads on Windows. It should be fine to<br>
>>>> >> increase this<br>
>>>> >> number, especially if you actually have a Windows box with more than<br>
>>>> >> 8<br>
>>>> >> processors on it.<br>
>>>> >><br>
>>>> >> Feel free to change your build locally to increase that value.<br>
>>>> >> Perhaps<br>
>>>> >> somebody here on the VTK developers list is also contemplating more<br>
>>>> >> powerful<br>
>>>> >> Window machines these days... Anybody here want to bump this up and<br>
>>>> >> commit<br>
>>>> >> the change?<br>
>>>> >><br>
>>>> >><br>
>>>> >> On Mon, Jan 24, 2011 at 5:11 AM, Pavel Pokutnev<br>
>>>> >> <<a href="mailto:pavel.pokutnev@googlemail.com">pavel.pokutnev@googlemail.com</a>> wrote:<br>
>>>> >>><br>
>>>> >>> Hello vtk-devs,<br>
>>>> >>><br>
>>>> >>> sorry, but I would like to repeat my question.<br>
>>>> >>> There is a define VTK_MAX_THREADS in "Common/vtkSystemIncludes.h".<br>
>>>> >>> It<br>
>>>> >>> is defined to 8 for Win32 threads, but for all other platforms to<br>
>>>> >>> 32.<br>
>>>> >>> What is the cause for this non-conformity?<br>
>>>> >>><br>
>>>> >>> Best regards,<br>
>>>> >>> Pavel<br>
>>>> >>> _______________________________________________<br>
>>>> >>> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>>>> >>><br>
>>>> >>> Visit other Kitware open-source projects at<br>
>>>> >>> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
>>>> >>><br>
>>>> >>> Follow this link to subscribe/unsubscribe:<br>
>>>> >>> <a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
>>>> >>><br>
>>>> >><br>
>>>> >><br>
>>>> >> _______________________________________________<br>
>>>> >> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>>>> >><br>
>>>> >> Visit other Kitware open-source projects at<br>
>>>> >> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
>>>> >><br>
>>>> >> Follow this link to subscribe/unsubscribe:<br>
>>>> >> <a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
>>>> >><br>
>>>> >><br>
>>>> ><br>
>>>> ><br>
>>>> > _______________________________________________<br>
>>>> > Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>>>> ><br>
>>>> > Visit other Kitware open-source projects at<br>
>>>> > <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
>>>> ><br>
>>>> > Follow this link to subscribe/unsubscribe:<br>
>>>> > <a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
>>>> ><br>
>>>> ><br>
>>>> ><br>
>>>> _______________________________________________<br>
>>>> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>>>><br>
>>>> Visit other Kitware open-source projects at<br>
>>>> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
>>>><br>
>>>> Follow this link to subscribe/unsubscribe:<br>
>>>> <a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
>>>><br>
>>>><br>
>>>><br>
>>>><br>
>>>><br>
>>>>    ****      Kenneth Moreland<br>
>>>>     ***      Sandia National Laboratories<br>
>>>> ***********<br>
>>>> *** *** ***  email: <a href="mailto:kmorel@sandia.gov">kmorel@sandia.gov</a><br>
>>>> **  ***  **  phone: (505) 844-8919<br>
>>>>     ***      web:   <a href="http://www.cs.unm.edu/~kmorel" target="_blank">http://www.cs.unm.edu/~kmorel</a><br>
>>>><br>
>>>><br>
>>>> _______________________________________________<br>
>>>> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
>>>><br>
>>>> Visit other Kitware open-source projects at<br>
>>>> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
>>>><br>
>>>> Follow this link to subscribe/unsubscribe:<br>
>>>> <a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
>>>><br>
>>>><br>
>>><br>
>>><br>
>>><br>
>>> --<br>
>>> William J. Schroeder, PhD<br>
>>> Kitware, Inc.<br>
>>> 28 Corporate Drive<br>
>>> Clifton Park, NY 12065<br>
>>> <a href="mailto:will.schroeder@kitware.com">will.schroeder@kitware.com</a><br>
>>> <a href="http://www.kitware.com" target="_blank">http://www.kitware.com</a><br>
>>> (518) 881-4902<br>
>><br>
><br>
><br>
> _______________________________________________<br>
> Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
><br>
> Visit other Kitware open-source projects at<br>
> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
><br>
> Follow this link to subscribe/unsubscribe:<br>
> <a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
><br>
><br>
><br>
</div></div></blockquote></div><br>