Re: [2.6 patch make struct scsi_dh_devlist's static

From: Linus Torvalds
Date: Mon Aug 11 2008 - 14:01:19 EST




On Mon, 11 Aug 2008, Chandra Seetharaman wrote:
>
> This patch makes several needlessly global struct scsi_dh_devlist's
> static.
>
> Signed-off-by: Adrian Bunk <bunk@xxxxxxxxxx>
> Acked-by: Chandra Seetharaman <sekharan@xxxxxxxxxx>

This is not a valid sign-off chain.

Was the patch originally done by Adrian? If so, where is the

From: Adrian Bunk <bunk@xxxxxxxxxx>

at the top?

And since you're forwarding it, you should be signing off on it, not just
acking it. Otherwise, the next person in the chain can't sign off on it
and take it.

Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/