linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Hannes Reinecke <hare@suse.de>
To: Christoph Hellwig <hch@lst.de>,
	James Bottomley <James.Bottomley@HansenPartnership.com>
Cc: "Martin K. Petersen" <martin.petersen@oracle.com>,
	linux-scsi@vger.kernel.org
Subject: Re: [PATCH 10/10] scsi: mark scsi_setup_blk_pc_cmnd static
Date: Fri, 11 Jul 2014 14:33:57 +0200	[thread overview]
Message-ID: <53BFD9B5.6080407@suse.de> (raw)
In-Reply-To: <1404048881-19526-11-git-send-email-hch@lst.de>

On 06/29/2014 03:34 PM, Christoph Hellwig wrote:
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---
>   drivers/scsi/scsi_lib.c    |    3 +--
>   include/scsi/scsi_driver.h |    2 --
>   2 files changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c
> index ea23860..f0a3ef1 100644
> --- a/drivers/scsi/scsi_lib.c
> +++ b/drivers/scsi/scsi_lib.c
> @@ -1080,7 +1080,7 @@ static struct scsi_cmnd *scsi_get_cmd_from_req(struct scsi_device *sdev,
>   	return cmd;
>   }
>
> -int scsi_setup_blk_pc_cmnd(struct scsi_device *sdev, struct request *req)
> +static int scsi_setup_blk_pc_cmnd(struct scsi_device *sdev, struct request *req)
>   {
>   	struct scsi_cmnd *cmd = req->special;
>
> @@ -1105,7 +1105,6 @@ int scsi_setup_blk_pc_cmnd(struct scsi_device *sdev, struct request *req)
>   	cmd->allowed = req->retries;
>   	return BLKPREP_OK;
>   }
> -EXPORT_SYMBOL(scsi_setup_blk_pc_cmnd);
>
>   /*
>    * Setup a REQ_TYPE_FS command.  These are simple request from filesystems
> diff --git a/include/scsi/scsi_driver.h b/include/scsi/scsi_driver.h
> index 009d2ae..c2b7598 100644
> --- a/include/scsi/scsi_driver.h
> +++ b/include/scsi/scsi_driver.h
> @@ -29,6 +29,4 @@ extern int scsi_register_interface(struct class_interface *);
>   #define scsi_unregister_interface(intf) \
>   	class_interface_unregister(intf)
>
> -int scsi_setup_blk_pc_cmnd(struct scsi_device *sdev, struct request *req);
> -
>   #endif /* _SCSI_SCSI_DRIVER_H */
>
Reviewed-by: Hannes Reinecke <hare@suse.de>

Cheers,

Hannes
-- 
Dr. Hannes Reinecke		      zSeries & Storage
hare@suse.de			      +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2014-07-11 12:33 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-29 13:34 RFC: clean up command setup Christoph Hellwig
2014-06-29 13:34 ` [PATCH 01/10] scsi: move the nr_phys_segments assert into scsi_init_io Christoph Hellwig
2014-07-11 12:17   ` Hannes Reinecke
2014-07-13 14:03   ` Martin K. Petersen
2014-06-29 13:34 ` [PATCH 02/10] scsi: restructure command initialization for TYPE_FS requests Christoph Hellwig
2014-07-11 12:18   ` Hannes Reinecke
2014-07-13 14:04   ` Martin K. Petersen
2014-06-29 13:34 ` [PATCH 03/10] scsi: set sc_data_direction in common code Christoph Hellwig
2014-07-11 12:19   ` Hannes Reinecke
2014-07-13 14:06   ` Martin K. Petersen
2014-06-29 13:34 ` [PATCH 04/10] sd: don't use scsi_setup_blk_pc_cmnd for flush requests Christoph Hellwig
2014-07-11 12:20   ` Hannes Reinecke
2014-07-13 14:07   ` Martin K. Petersen
2014-06-29 13:34 ` [PATCH 05/10] sd: don't use scsi_setup_blk_pc_cmnd for write same requests Christoph Hellwig
2014-07-11 12:25   ` Hannes Reinecke
2014-07-11 15:15     ` Christoph Hellwig
2014-07-13 14:14   ` Martin K. Petersen
2014-07-17 15:29   ` Christoph Hellwig
2014-06-29 13:34 ` [PATCH 06/10] sd: don't use scsi_setup_blk_pc_cmnd for discard requests Christoph Hellwig
2014-07-07  0:01   ` Elliott, Robert (Server Storage)
2014-07-07  2:01     ` Elliott, Robert (Server Storage)
2014-07-07  9:24     ` Christoph Hellwig
2014-07-11 12:26   ` Hannes Reinecke
2014-07-11 15:15     ` Christoph Hellwig
2014-07-13 14:35   ` Martin K. Petersen
2014-07-13 14:52     ` Douglas Gilbert
2014-07-13 14:56       ` Christoph Hellwig
2014-07-13 15:03         ` Martin K. Petersen
2014-06-29 13:34 ` [PATCH 07/10] sd: retry write same commands Christoph Hellwig
2014-07-11 12:26   ` Hannes Reinecke
2014-07-13 14:36   ` Martin K. Petersen
2014-06-29 13:34 ` [PATCH 08/10] sd: retry discard commands Christoph Hellwig
2014-07-11 12:27   ` Hannes Reinecke
2014-07-13 14:36   ` Martin K. Petersen
2014-06-29 13:34 ` [PATCH 09/10] sd: split sd_init_command Christoph Hellwig
2014-07-11 12:33   ` Hannes Reinecke
2014-07-13 14:37   ` Martin K. Petersen
2014-06-29 13:34 ` [PATCH 10/10] scsi: mark scsi_setup_blk_pc_cmnd static Christoph Hellwig
2014-07-11 12:33   ` Hannes Reinecke [this message]
2014-07-13 14:38   ` Martin K. Petersen
2014-07-11  9:16 ` RFC: clean up command setup Christoph Hellwig

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53BFD9B5.6080407@suse.de \
    --to=hare@suse.de \
    --cc=James.Bottomley@HansenPartnership.com \
    --cc=hch@lst.de \
    --cc=linux-scsi@vger.kernel.org \
    --cc=martin.petersen@oracle.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).