Re: [PATCH] cryptoapi: Fix sleeping

From: Robert Love (rml_at_tech9.net)
Date: 08/14/03

  • Next message: Russell King: "Re: [PATCH] call drv->shutdown at rmmod" 
    To: Matt Mackall <mpm@selenic.com>
Date:	Thu, 14 Aug 2003 09:29:20 -0700

    On Wed, 2003-08-13 at 18:58, Matt Mackall wrote:

    > This is part of cryptoapi and given the large chunks of work you could
    > potentially hand to it, it's probably a good idea for it to work this
    > way. You hand it a long list of sg segments, it does the transform and
    > reschedules if it thinks it's safe. But its test of when it was safe
    > was not complete.

    Right. My concern is that you said sometimes it is called when
    preemption is disabled.

    I assume because it is accessing per-processor data.

    It is illegal to reschedule -- regardless of kernel preemption -- while
    in the middle of a critical section accessing per-processor data.

            Robert Love

    -
    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/

  • Next message: Russell King: "Re: [PATCH] call drv->shutdown at rmmod"

    Relevant Pages

    • Re: [PATCH] cryptoapi: Fix sleeping
      ... > per-processor data. ... potentially hand to it, it's probably a good idea for it to work this ... reschedules if it thinks it's safe. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)
    • Re: [PATCH] cryptoapi: Fix sleeping
      ... >> potentially hand to it, it's probably a good idea for it to work this ... >> reschedules if it thinks it's safe. ... person to try using cryptoapi with per_cpu. ... send the line "unsubscribe linux-kernel" in ...
      (Linux-Kernel)