Messages in this thread | | | Date | Tue, 4 Sep 2007 00:19:22 -0700 | From | "Paul Menage" <> | Subject | Re: [-mm PATCH] Memory controller improve user interface (v3) |
| |
On 9/3/07, Balbir Singh <balbir@linux.vnet.ibm.com> wrote: > Paul Menage wrote: > > On 9/2/07, Balbir Singh <balbir@linux.vnet.ibm.com> wrote: > >> - s += sprintf(s, "%lu\n", *val); > >> + if (read_strategy) > >> + s += read_strategy(*val, s); > >> + else > >> + s += sprintf(s, "%lu\n", *val); > > > > This would be better as %llu > > > > Hi, Paul, > > This does not need fixing, since the other counters like failcnt are > still unsigned long >
But val is an unsigned long long*. So printing *val with %lu will break (at least a warning, and maybe corruption if you had other parameters) on 32-bit archs.
Paul - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |