Re: [Xen-devel] BUG - xen-netback stats interface limited to 32-bit values on 64 bit systems

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

On Wed, 2015-02-18 at 18:57 +0100, Atom2 wrote:
> Am 18.02.15 um 16:24 schrieb Ian Campbell:
> > create ^
> > thanks
> >
> > On Thu, 2015-02-12 at 20:47 +0100, Atom2 wrote:
> >> Hi guys,
> >> I am forwarding this message after initially having confirmed with Ian
> >> Campbell on the user list that there's really an issue - please see
> >> further below.
> >>
> >> I am currently running xen-4.3.3 on gentoo (dom0 is based on kernel
> >> 3.17.7) and I am happy to help out by applying and testing patches on my
> >> version.
> >
> > FWIW based on e00f85bec0a9 this seems like it would be a reasonably
> > straight forward introductory kernel hacking exercise (mostly copying
> > e00f85bec0a9) if you are interested I could help guide/mentor you
> > through it.
> >
> > Ian.
> Ian,
> thanks for your initiative in moving this forward. Based on your comment 
> and your offer to help/guide me in the process, I am happy to give this 
> a try. In the end this excercise might be more work for you than just 
> fixing it yourself, but it might pay off in the long run.

I'm unlikely to make the time to do it myself, but I'm always willing to
make time for helping others do things, usually in the long term / on
average it's worthwhile to help others get up to speed even if short
term it takes a bit more effort.

> I did actually have a look at the relevant source for the xen netback 
> code already a couple of days ago and think that I have already 
> identified (at least parts of) the code that requires change. 
> Unfortunately I failed miserable in finding the commit e00f85bec0a9 you 
> were/are referring to in your mails (I used google to search, but 
> nothing came up) and therefore thought it being too risky to just change 
> what I thought needs changing - albeit that's part of the kernel and 
> could just bring down a system if changed inappropriately.
> 
> So if you could provide me with a link to that commit, I have something 
> to start from and make myself familiar with the required changes.

The full commit id is:

        commit e00f85bec0a9924eb517ccd126ddbb9787068f53
        Author: stephen hemminger <shemminger@xxxxxxxxxx>
        Date:   Tue Jun 21 05:35:31 2011 +0000
        
            xen: convert to 64 bit stats interface
            
            Convert xen driver to 64 bit statistics interface.
            Use stats_sync to ensure that 64 bit update is read atomically on 
            Put hot statistics into per-cpu table.
            
            Signed-off-by: Stephen Hemminger <shemminger@xxxxxxxxxx>
            Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
            Signed-off-by: David S. Miller <davem@xxxxxxxxxxxxx>
        
and its in the mainline (i.e. Linus') Linux tree, see
http://git.kernel.org/linus/e00f85bec0a9924eb517ccd126ddbb9787068f53

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel