Re: [PATCH 04/18] csum_and_copy_..._user(): pass 0xffffffff instead of 0 as initial sum

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

 



On Wed, Jul 22, 2020 at 03:22:45PM +0000, David Laight wrote:

> > And the benefit of that would be...?  It wouldn't be any simpler,
> > it almost certainly would not even be a valid microoptimization
> > (nevermind that this is an arch-independent code)...
> 
> It ought to give a minor improvement because it saves the extra
> csum_fold() when the checksum from a buffer is added to the
> previous total.
> 

Sigh...  _WHAT_ csum_fold()?

static inline __wsum
csum_block_add(__wsum csum, __wsum csum2, int offset)
{
        u32 sum = (__force u32)csum2;

        /* rotate sum to align it with a 16b boundary */
        if (offset & 1)
                sum = ror32(sum, 8);

        return csum_add(csum, (__force __wsum)sum);
}

David, do you *ever* bother to RTFS?  I mean, competent supercilious twits
are annoying, but at least with those you can generally assume that what
they say makes sense and has some relation to reality.  You, OTOH, keep
spewing utter bollocks, without ever lowering yourself to checking if your
guesses have anything to do with the reality.  With supercilious twit part
proudly on the display - you do speak with confidence, and the way you
dispense the oh-so-valuable advice to everyone around...



[Index of Archives]     [Linux Kernel]     [Kernel Newbies]     [x86 Platform Driver]     [Netdev]     [Linux Wireless]     [Netfilter]     [Bugtraq]     [Linux Filesystems]     [Yosemite Discussion]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Device Mapper]

  Powered by Linux