[Bug 13555] New: 64-bit warning fixes

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Thu Dec 6 20:24:34 PST 2007


http://bugs.freedesktop.org/show_bug.cgi?id=13555

           Summary: 64-bit warning fixes
           Product: xorg
           Version: git
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: medium
         Component: Driver/Radeon
        AssignedTo: xorg-driver-ati at lists.x.org
        ReportedBy: liswu at ati.com
         QAContact: xorg-team at lists.x.org
                CC: woliu at ati.com


in 64-bit Linux, we found radeon driver will print out absurdly-large in the
log messege
II) RADEON(0): PLL parameters: rf=2700 rd=12 min=20000 max=47996259547800;
xclk=20000

this problem is because print format mismatch data type. pll->max_pll_freq is
defined as a 32-bit unsigned integer -- type CARD32.  

To change the print format from ‘%ld’to "%u" will solve this problem.

we have noticed that git checkin 'd7230939f523610c57f92bdfc72966bdbc6f1070: 64
bit warning fixes' has attepmted to adjust printf format to match CARD32, but
it changes pll->max_pll_freq print format from '%ld' to '%d', we have concerns
that '%d' will still have problem to handle larger numbers. so we suggest do an
adjust to change '%d' to '%u'.


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


More information about the xorg-driver-ati mailing list