Re: [PATCH vfs 1/2] lib: implement ptrset

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

 



Hello,

On Tue, Nov 18, 2014 at 05:19:18PM +0800, Lai Jiangshan wrote:
> Is it too ugly?

What is "it"?  The whole thing?  percpu preloading?  I'm just gonna
continue assuming that you're talking about preloading.  If you think
it's ugly, please go ahead and explain why you think it is.

It's almost impossible to respond to your "review".  It's not clear
what your subject matter or opinion on it is.  Might as well just bang
on the keyboard randomly.  When reviewing (or communicating in
general), please try to properly form and elaborate your points.
Other people can't know what's going on in your brain and have to
speculate what you could have meant.

This implementation of preloading an evolution of a design pattern
which, IIRC, first started with the radix tree.  The non-failing
aspect was introduced while the pattern was being applied to idr.  I
think it's one of the better ways to implement preloading.

> What will be the most important result it achieve?

This is the same as other preloading.  It allows pulling allocation
out of critical section so that it can be done with more generous
allocation mask (ie. GFP_KERNEL instead of GPF_NOWAIT).  It's a common
pattern found in data structures which may allocate memory internally
such as radix tree or idr.

Thanks.

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




[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux