[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CALZtONBra8HJxP-KM5FtMZThfJXe-M0TNqmFq7rJJUzUKA=KEg@mail.gmail.com>
Date: Tue, 24 Jun 2014 11:41:33 -0400
From: Dan Streetman <ddstreet@...e.org>
To: Andrew Morton <akpm@...ux-foundation.org>
Cc: Seth Jennings <sjennings@...iantweb.net>,
Minchan Kim <minchan@...nel.org>,
Weijie Yang <weijie.yang@...sung.com>,
Nitin Gupta <ngupta@...are.org>, Bob Liu <bob.liu@...cle.com>,
Hugh Dickins <hughd@...gle.com>, Mel Gorman <mgorman@...e.de>,
Rik van Riel <riel@...hat.com>,
Johannes Weiner <hannes@...xchg.org>,
Sergey Senozhatsky <sergey.senozhatsky@...il.com>,
Linux-MM <linux-mm@...ck.org>,
linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCHv2 6/6] mm/zpool: prevent zbud/zsmalloc from unloading when used
On Mon, Jun 23, 2014 at 5:48 PM, Andrew Morton
<akpm@...ux-foundation.org> wrote:
> On Mon, 2 Jun 2014 18:19:46 -0400 Dan Streetman <ddstreet@...e.org> wrote:
>
>> Add try_module_get() to zpool_create_pool(), and module_put() to
>> zpool_destroy_pool(). Without module usage counting, the driver module(s)
>> could be unloaded while their pool(s) were active, resulting in an oops
>> when zpool tried to access them.
>
> Was wondering about that ;) We may as well fold
> this fix into "mm/zpool: implement common zpool api to zbud/zsmalloc"?
Yes. Sorry, I had this pulled out of that because I was trying to
keep the patches logically separated. But they do need to be
together, to be safe ;-)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists