Messages in this thread | | | Date | Thu, 15 Jan 2004 21:52:30 -0800 | From | William Lee Irwin III <> | Subject | Re: seperator error in __mask_snprintf_len |
| |
Paul Jackson <pj@sgi.com> wrote: >> Should any of the other inline routines in include/bitmap.h >> be moved to your new file lib/bitmap.c?
On Thu, Jan 15, 2004 at 09:26:13PM -0800, Andrew Morton wrote: > Yup. The below patch will be in 2.6.1-mm4: > uninline bitmap functions > - A couple of them are using alloca (via DECLARE_BITMAP) and this generates > a cannot-inline warning with -Winline. > - These functions are too big to inline anwyay.
Uninlining is good (I originally wanted to avoid controversy by not having them compiled in unless they were called), but it's also possible to remove the stack allocations entirely with more sophisticated implementations. These are actually quite dumb as implementations of the bitmap operations, and meant to look simple as opposed to performing well or being well-behaved with respect to stackspace.
-- wli - 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/
| |