<br>
<br><font size=2><tt>crash-utility-bounces@redhat.com wrote on 03/09/2006
12:24:26 PM:<br>
<br>
> Dave Anderson wrote:<br>
> <br>
> > Sorry -- I see that NR_CPUS is 128 for ppc64 in crash...<br>
> ><br>
> > But the question remains, do you want to absolutely depend<br>
> > upon that always being the case in the kernel?<br>
> ><br>
> > Dave<br>
> <br>
> Actually, looking again at the paca code, it doesn't seem that<br>
> it is dependent upon the kernel NR_CPUS value.</tt></font>
<br>
<br><font size=2><tt>Dave, </tt></font>
<br>
<br><font size=2><tt>Since, we are iterating on nr_paca, though that it
may not be an issue even if the kernel changed NR_CPUS to < 128. The
only issue is we are reading NR_CPUS/8 for cpu_online_map - not using the
remaining part. </tt></font>
<br>
<br><font size=2><tt>Probably, it will be more clear if you the cpu_online_map
size is based on nr_paca. </tt></font>
<br>
<br><font size=2><tt>Like moving the following line  after calculated
nr_paca:</tt></font>
<br>
<br><font size=2 face="sans-serif">+        readmem(symbol_value("cpu_online_map"),
KVADDR, &cpu_online_map[0],</font>
<br><font size=2 face="sans-serif">+        
       nr_paca/8, "cpu_online_map",
FAULT_ON_ERROR);</font>
<br><font size=2 face="sans-serif">+</font>
<br>
<br><font size=2><tt>Can I go ahead and change it? - drawback of this approach
is if the NR_CPUS in the kernel is greater than 128.</tt></font>
<br>
<br><font size=2><tt>or do you prefer actual allocating the online_map
based on nr_paca? I will make changes based on your suggestions.</tt></font>
<br>
<br><font size=2><tt>Thanks</tt></font>
<br><font size=2><tt>Haren</tt></font>
<br><font size=2><tt><br>
> <br>
> So perhaps all my bitching is much ado about nothing?<br>
> <br>
> Dave<br>
> <br>
> <br>
> --<br>
> Crash-utility mailing list<br>
> Crash-utility@redhat.com<br>
> https://www.redhat.com/mailman/listinfo/crash-utility<br>
</tt></font>