Re: Fix memory leak in swsusp

From: Dave Jones
Date: Sat Jun 12 2004 - 06:16:55 EST


On Fri, Jun 11, 2004 at 09:00:59PM -0700, Andrew Morton wrote:
> Nigel Cunningham <ncunningham@xxxxxxxxxxxxx> wrote:
> >
> > We were avoiding the use of memcpy because it messes up the preempt count with 3DNow, and
> > potentially as other unseen side effects. The preempt could possibly simply be reset at resume time,
> > but the point remains.
>
> eh? memcpy just copies memory. Maybe your meant copy_*_user()?

See arch/i386/lib/memcpy.c The 3dnow routine does kernel_fpu_begin()/..end()
which futzes with preempt count. What I'm missing though is that the count
afterwards should be the same as it was before the memcpy. Why is this
a problem for the suspend folks?

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