From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alan Cox Subject: Re: PATCH: switch ide-proc to use the ide_key functionality Date: Tue, 17 Aug 2004 18:15:25 -0400 Sender: linux-ide-owner@vger.kernel.org Message-ID: <20040817221525.GA29340@devserv.devel.redhat.com> References: <20040815150414.GA12181@devserv.devel.redhat.com> <200408170231.25725.bzolnier@elka.pw.edu.pl> <20040817010533.GB32628@devserv.devel.redhat.com> <200408171248.12235.bzolnier@elka.pw.edu.pl> <20040817120622.GF3204@devserv.devel.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mx1.redhat.com ([66.187.233.31]:7082 "EHLO mx1.redhat.com") by vger.kernel.org with ESMTP id S268479AbUHQWQT (ORCPT ); Tue, 17 Aug 2004 18:16:19 -0400 Content-Disposition: inline In-Reply-To: <20040817120622.GF3204@devserv.devel.redhat.com> List-Id: linux-ide@vger.kernel.org To: Alan Cox Cc: Bartlomiej Zolnierkiewicz , linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org On Tue, Aug 17, 2004 at 08:06:22AM -0400, Alan Cox wrote: > I'll have a look at what occurs if we make the ->key functions ref count > and add "put" functions. I think that can be made to work cleanly without > changing the rest of the code to refcounts at the same time. It'll still need > some locking because of the memset. We would still have keys but we'd > refcount usage off them as a starting point. Ok fixed this by using the cfg_sem. refcounting breaks the non refcounted code and its assumptions (good bad or otherwise). I've dropped the locking in in such a way as switching to refcounts later is easier. Alan