Bug 770739

Summary: RPS/RFS: Invalid rxhash handling
Product: Red Hat Enterprise Linux 6 Reporter: Marcelo Ricardo Leitner <mleitner>
Component: kernelAssignee: Jiri Benc <jbenc>
Status: CLOSED DUPLICATE QA Contact: Red Hat Kernel QE team <kernel-qe>
Severity: medium Docs Contact:
Priority: medium    
Version: 6.1CC: nhorman
Target Milestone: rcKeywords: Patch
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-02-24 18:04:24 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Comment 2 Marcelo Ricardo Leitner 2011-12-29 17:05:01 UTC
Test kernel available at http://people.redhat.com/~mleitner/bz770739/

Comment 3 Jiri Benc 2012-01-04 13:06:52 UTC
The reasoning looks valid to me. Neil, commit cd045e31 contains

-u16 tcpu = map->cpus[((u32) (skb->rxhash * map->len)) >> 16];
+tcpu = map->cpus[((u64) skb->rxhash * map->len) >> 32];

Could you confirm this is just an overlook? Thanks!

Comment 4 Neil Horman 2012-01-04 15:02:09 UTC
Jiri, yes, thats correct, Due to ABI contraints we have to shift by 16 instead of 32 and cast to a u32

Comment 8 Jiri Benc 2012-02-24 18:04:24 UTC

*** This bug has been marked as a duplicate of bug 757040 ***