Re: [PATCH v2 2/4] percpu_stats: Enable 64-bit counts in 32-bit architectures

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 04/08/2016 01:46 PM, Tejun Heo wrote:
Hello,

On Fri, Apr 08, 2016 at 01:32:52PM -0400, Waiman Long wrote:
Yes, I think it will be more efficient to use percpu_counter in this case.
The preempt_disable/enable() calls are pretty cheap. Once in a while, you
need to take the lock and update the global count. How about I change the
2nd patch to use percpu_counter internally when 64-bit counts are needed in
32-bit archs, but use the regular percpu counts on 64-bit archs? If you are
OK with that, I can update the patch accordingly.
Does having percpu_stats as a separate construct make sense after
that?  Just use percpu_counter directly?  You end up wasting a bit
more space that way but most of space overhead for these things are in
percpu part anyway, so in proportion it shouldn't make that much of a
difference.

Thanks.


The percpu_stats construct allows minimal overhead in maintaining statistics counts. The percpu_counter construct, on the other hand, has a higher performance overhead and a bit more complex to set up and tear down when more than one statistics counts are needed. In fact, my first draft of the ext4 patch used percpu_counter for that purpose. However, I just feel that using percpu_counter is kind of an overkill if what we just want is to keep some counts that we want to have their sums returned when requested. That are the main reasons for creating a separate percpu_stats.

Of course, if you think we don't need a separate percpu_stats construct. I am fine with that and I can rework the ext4 patch to use percpu_counter instead.

Cheers,
Longman




--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Reiser Filesystem Development]     [Ceph FS]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite National Park]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]     [Linux Media]

  Powered by Linux