Subject: Re: [RFC PATCH 0/19] A kernel-level configfs enabled generic
 target engine for Linux v2.6.32
From: Daniel Walker <dwalker@fifo99.com>
To: "Nicholas A. Bellinger" <nab@linux-iscsi.org>
Cc: LKML <linux-kernel@vger.kernel.org>,
       linux-scsi <linux-scsi@vger.kernel.org>,
       Andrew Morton <akpm@linux-foundation.org>, Greg KH <greg@kroah.com>,
       Douglas Gilbert <dgilbert@interlog.com>,
       James Bottomley <James.Bottomley@HansenPartnership.com>,
       Hannes Reinecke <hare@suse.de>,
       FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>,
       Mike Christie <michaelc@cs.wisc.edu>,
       Joel Becker <joel.becker@oracle.com>,
       "Martin K. Petersen" <martin.petersen@oracle.com>,
       Christoph Hellwig <hch@lst.de>,
       Linus Torvalds <torvalds@linux-foundation.org>,
       Alan Stern <stern@rowland.harvard.edu>,
       Boaz Harrosh <bharrosh@panasas.com>,
       Florian Haas <florian.haas@linbit.com>,
       Philipp Reisner <philipp.reisner@linbit.com>
In-Reply-To: <1252724884.2067.251.camel@haakon2.linux-iscsi.org>
References: <1252720508.2067.196.camel@haakon2.linux-iscsi.org>
	 <1252723060.28368.48.camel@desktop>
	 <1252724884.2067.251.camel@haakon2.linux-iscsi.org>
Content-Type: text/plain
Date: Fri, 11 Sep 2009 20:17:48 -0700
Message-Id: <1252725468.28368.54.camel@desktop>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1530
Lines: 36

On Fri, 2009-09-11 at 20:08 -0700, Nicholas A. Bellinger wrote:


> 
> Ok, I will have a look at converting the HBA semaphore to a mutex in
> patch 5, and use some single use waitqueues in patch 16.  Btw, the ones
> using init_MUTEX_LOCKED() in the latter source file are being used when
> shutting down target_core_mod allocated kernel threads to sychronize
> between the stopping processing threads and configfs process context
> that waitqueues would work better for.  Thanks for pointing this out!

You could also use completions in patch 16 if the waitqueues turn out to
be not what your looking for. Completion can sometimes be a find&replace
substitute for the locked semaphores.


> I was thinking about going through the typedefs with a sed chainsaw, but
> I decided against for this TCM patch series because it makes keeping
> backward ports to <= v2.6.31 slightly easier to maintain for me for the
> current users of LIO kernel code.
> 
> However, I have been starting to not use typedef for the newest ALUA and
> pieces of PR code, and I would be happy to have another look at typedef
> style comments on this once the other larger interest items are on the
> way to being resolved.

It's up to you, but you should schedule the typedefs for removal at some
point.

Daniel

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