Re: [patch 3/8] zfcp: fix mempool usage for status_read requests

[Christof Schmitt <christof.schmitt@de.ibm.com>]

On Wed, Nov 05, 2008 at 12:46:12PM -0500, James Bottomley wrote:
> On Tue, 2008-11-04 at 16:35 +0100, Christof Schmitt wrote:
> > plain text document attachment (fix-status-read-alloc.diff)
> > From: Christof Schmitt <christof.schmitt@de.ibm.com>
> > 
> > When allocating fsf requests without qtcb, store the pointer to the
> > mempool in the fsf requests for later call to mempool_free. This
> > codepath is only used by the status_read requests.
> > 
> > Signed-off-by: Christof Schmitt <christof.schmitt@de.ibm.com>
> > Signed-off-by: Swen Schillig <swen@vnet.ibm.com>
> 
> I put this one in, but can you explain this last signoff better?  You
> sent me the patch, so your signoff should appear last.  If Swen altered
> the patch, his signoff should appear before yours (with a small
> explanation of the alteration in square brackets).  If he's just
> approving it, it should be Acked-by: not Signed-off-by:

I wrote the patch, Swen reviewed it and i sent it as part of the patch
series. It looks like this did not follow the exact guidelines. For
future patches, i will document the review with the "Acked-by:".

Christof