Curious about corner case in btrfs code

Mandeep Sandhu mandeepsandhu.chd at gmail.com
Tue Aug 26 18:58:18 EDT 2014


If it's a corner case, it won't be hit often enough right? And if it
was hit often enough, it wouldn't be corner case!? :)

These 2 are mutually exclusive!


On Tue, Aug 26, 2014 at 3:47 PM, Nick <xerofoify at gmail.com> wrote:
> After reading through the code in inode.c today , I am curious about the comment and the following code I will paste
> below. I am curious if this corner case is hit often enough for me to write a patch to improve the speed of this
> corner case. Furthermore , compress_file_range is the function name, in case you can't guess by the pasted code.
> Regards Nick
> 411     /*
> 412      * we don't want to send crud past the end of i_size through
> 413      * compression, that's just a waste of CPU time.  So, if the
> 414      * end of the file is before the start of our current
> 415      * requested range of bytes, we bail out to the uncompressed
> 416      * cleanup code that can deal with all of this.
> 417      *
> 418      * It isn't really the fastest way to fix things, but this is a
> 419      * very uncommon corner.
> 420      */
> 421     if (actual_end <= start)
> 422             goto cleanup_and_bail_uncompressed;
>
> _______________________________________________
> Kernelnewbies mailing list
> Kernelnewbies at kernelnewbies.org
> http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies



More information about the Kernelnewbies mailing list