Re: [patch-2.4.0-test6-pre8] fork.c FIXME removed (UPDATE)

From: Bill Wendling (wendling@ganymede.isdn.uiuc.edu)
Date: Tue Aug 08 2000 - 11:33:27 EST


Also sprach Matthew Wilcox:
} On Tue, Aug 08, 2000 at 03:49:08AM -0500, Bill Wendling wrote:
} > - /* Kill me slowly. UGLY! FIXME! */
} > - memcpy(&mm->start_code, &current->mm->start_code, 15*sizeof(unsigned long));
}
} why isn't the fix simply:
}
} memcpy(&mm->start_code, &current->mm->start_code, sizeof(struct mm_struct));
}
} ? looks a lot less ugly to me.
}
That code wouldn't work since start_code isn't the first element in the
structure. It's also a bit less efficient then just copying the fields
you are interested in IMHO :).

-- 
|| Bill Wendling			wendling@ganymede.isdn.uiuc.edu

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Tue Aug 15 2000 - 21:00:17 EST