Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1374415

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

Show all headers | View raw

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

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v2 0/4] ext4: Improve parallel I/O performance on NVDIMM Waiman Long <Waiman.Long@hpe.com> - 2016-04-08 18:20 +0200
  [PATCH v2 3/4] ext4: Pass in DIO_SKIP_DIO_COUNT flag if inode_dio_begin() called Waiman Long <Waiman.Long@hpe.com> - 2016-04-08 18:20 +0200
  [PATCH v2 2/4] percpu_stats: Enable 64-bit counts in 32-bit architectures Waiman Long <Waiman.Long@hpe.com> - 2016-04-08 18:20 +0200
    Re: [PATCH v2 2/4] percpu_stats: Enable 64-bit counts in 32-bit  architectures Tejun Heo <tj@kernel.org> - 2016-04-08 18:50 +0200
      Re: [PATCH v2 2/4] percpu_stats: Enable 64-bit counts in 32-bit architectures Waiman Long <waiman.long@hpe.com> - 2016-04-08 19:40 +0200
        Re: [PATCH v2 2/4] percpu_stats: Enable 64-bit counts in 32-bit  architectures Tejun Heo <tj@kernel.org> - 2016-04-08 19:50 +0200
          Re: [PATCH v2 2/4] percpu_stats: Enable 64-bit counts in 32-bit architectures Waiman Long <waiman.long@hpe.com> - 2016-04-08 20:50 +0200
            Re: [PATCH v2 2/4] percpu_stats: Enable 64-bit counts in 32-bit  architectures Tejun Heo <tj@kernel.org> - 2016-04-12 00:20 +0200
              Re: [PATCH v2 2/4] percpu_stats: Enable 64-bit counts in 32-bit architectures Waiman Long <waiman.long@hpe.com> - 2016-04-12 20:20 +0200
  [PATCH v2 1/4] percpu_stats: Simple per-cpu statistics count helper functions Waiman Long <Waiman.Long@hpe.com> - 2016-04-08 18:20 +0200
    Re: [PATCH v2 1/4] percpu_stats: Simple per-cpu statistics count  helper functions kbuild test robot <lkp@intel.com> - 2016-04-08 18:50 +0200
    Re: [PATCH v2 1/4] percpu_stats: Simple per-cpu statistics count  helper functions kbuild test robot <lkp@intel.com> - 2016-04-08 19:50 +0200

csiph-web