Re: [Resend PATCH v2] scsi_lib: rate-limit the error message from failing commands

["Love, Robert W" <robert.w.love@intel.com>]

On 05/25/2012 02:29 PM, Robert Love wrote:
> From: Yi Zou <yi.zou@intel.com>
>
> When performing a cable pull test w/ active stress I/O using fio over
> a dual port Intel 82599 FCoE CNA, w/ 256LUNs on one port and about 32LUNs
> on the other, it is observed that the system becomes not usable due to
> scsi-ml being busy printing the error messages for all the failing commands.
> I don't believe this problem is specific to FCoE and these commands are
> anyway failing due to link being down (DID_NO_CONNECT), just rate-limit
> the messages here to solve this issue.
>
> v2->v1: use __ratelimit() as Tomas Henzl mentioned as the proper way for
> rate-limit per function. However, in this case, the failed i/o gets to
> blk_end_request_err() and then blk_update_request(), which also has to
> be rate-limited, as added in the v2 of this patch.
>
> Signed-off-by: Yi Zou <yi.zou@intel.com>
> Acked-by: Tomas Henzl <thenzl@redhat.com>
> Signed-off-by: Robert Love <robert.w.love@intel.com>

Hi Jens,

   I think James may be waiting for an Ack from you before committing
this patch. The original patch submission wasn't sent to you. I'm just
resending so that it might get your attention.

Thanks, //Rob