Messages in this thread | | | Date | Thu, 31 Jan 2013 04:25:35 -0600 | From | Rob Landley <> | Subject | Re: [PATCH v2 1/2] Fix wrong EOF compare |
| |
On 01/13/2013 12:15:41 PM, Randy Dunlap wrote: > On 01/13/13 03:44, Rob Landley wrote: > > On 01/11/2013 08:21:55 AM, Michal Nazarewicz wrote: > >> On Fri, Jan 11 2013, Minchan Kim <minchan@kernel.org> wrote: > >> > The C standards allows the character type char to be singed or > unsinged, > >> > depending on the platform and compiler. Most of systems uses > signed char, > >> > but those based on PowerPC and ARM processors typically use > unsigned char. > >> > This can lead to unexpected results when the variable is used to > compare > >> > with EOF(-1). It happens my ARM system and this patch fixes it. > >> > > >> > Cc: Mel Gorman <mgorman@suse.de> > >> > Cc: Andy Whitcroft <apw@shadowen.org> > >> > Cc: Alexander Nyberg <alexn@dsv.su.se> > >> > Cc: Michal Nazarewicz <mina86@mina86.com> > >> > >> Acked-by: Michal Nazarewicz <mina86@mina86.com> > >> > >> > Cc: Randy Dunlap <rdunlap@infradead.org> > >> > Signed-off-by: Minchan Kim <minchan@kernel.org> > >> > --- > >> > Documentation/page_owner.c | 7 ++++--- > >> > 1 file changed, 4 insertions(+), 3 deletions(-) > > > > My kernel tree doesn't have Documentation/page_owner.c, where do I > find this file? > > It's in -mm (mmotm), so Andrew can/should merge this ...
Actually, why is a .c source file at the top level of Documentation?
Example code is nice and all, but this name doesn't say "test" or "example" or anything like that, and isn't collated into a subdirectory with any kind of explanatory files like all the others are.
Rob
| |