Skip to content

Commit a63f9e9

Browse files
committed
Merge: SCSI updates for 9.7
MR: https://gitlab.com/redhat/centos-stream/src/kernel/centos-stream-9/-/merge_requests/6719 SCSI core updates for RHEL 9.7 to upstream 6.14 +/- JIRA: https://issues.redhat.com/browse/RHEL-86156 Signed-off-by: Ewan D. Milne <emilne@redhat.com> Approved-by: Chris Leech <cleech@redhat.com> Approved-by: John Meneghini <jmeneghi@redhat.com> Approved-by: Ming Lei <ming.lei@redhat.com> Approved-by: Maurizio Lombardi <mlombard@redhat.com> Approved-by: CKI KWF Bot <cki-ci-bot+kwf-gitlab-com@redhat.com> Merged-by: Augusto Caringi <acaringi@redhat.com>
2 parents 0ad5a0e + 2691277 commit a63f9e9

File tree

13 files changed

+349
-173
lines changed

13 files changed

+349
-173
lines changed

Documentation/scsi/scsi_mid_low_api.rst

Lines changed: 6 additions & 55 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ supplied functions" below.
101101
Those functions in group b) are listed in a section entitled "Interface
102102
functions" below. Their function pointers are placed in the members of
103103
"struct scsi_host_template", an instance of which is passed to
104-
scsi_host_alloc() [#]_. Those interface functions that the LLD does not
104+
scsi_host_alloc(). Those interface functions that the LLD does not
105105
wish to supply should have NULL placed in the corresponding member of
106106
struct scsi_host_template. Defining an instance of struct
107107
scsi_host_template at file scope will cause NULL to be placed in function
@@ -116,9 +116,6 @@ should be static. For example the slave_alloc() function in an LLD
116116
called "xxx" could be defined as
117117
``static int xxx_slave_alloc(struct scsi_device * sdev) { /* code */ }``
118118

119-
.. [#] the scsi_host_alloc() function is a replacement for the rather vaguely
120-
named scsi_register() function in most situations.
121-
122119

123120
Hotplug initialization model
124121
============================
@@ -302,14 +299,12 @@ Summary:
302299
- scsi_host_alloc - return a new scsi_host instance whose refcount==1
303300
- scsi_host_get - increments Scsi_Host instance's refcount
304301
- scsi_host_put - decrements Scsi_Host instance's refcount (free if 0)
305-
- scsi_register - create and register a scsi host adapter instance.
306302
- scsi_remove_device - detach and remove a SCSI device
307303
- scsi_remove_host - detach and remove all SCSI devices owned by host
308304
- scsi_report_bus_reset - report scsi _bus_ reset observed
309305
- scsi_scan_host - scan SCSI bus
310306
- scsi_track_queue_full - track successive QUEUE_FULL events
311307
- scsi_unblock_requests - allow further commands to be queued to given host
312-
- scsi_unregister - [calls scsi_host_put()]
313308

314309

315310
Details::
@@ -474,27 +469,6 @@ Details::
474469
void scsi_host_put(struct Scsi_Host *shost)
475470

476471

477-
/**
478-
* scsi_register - create and register a scsi host adapter instance.
479-
* @sht: pointer to scsi host template
480-
* @privsize: extra bytes to allocate in hostdata array (which is the
481-
* last member of the returned Scsi_Host instance)
482-
*
483-
* Returns pointer to new Scsi_Host instance or NULL on failure
484-
*
485-
* Might block: yes
486-
*
487-
* Notes: When this call returns to the LLD, the SCSI bus scan on
488-
* this host has _not_ yet been done.
489-
* The hostdata array (by default zero length) is a per host scratch
490-
* area for the LLD.
491-
*
492-
* Defined in: drivers/scsi/hosts.c .
493-
**/
494-
struct Scsi_Host * scsi_register(struct scsi_host_template * sht,
495-
int privsize)
496-
497-
498472
/**
499473
* scsi_remove_device - detach and remove a SCSI device
500474
* @sdev: a pointer to a scsi device instance
@@ -524,7 +498,7 @@ Details::
524498
*
525499
* Notes: Should only be invoked if the "hotplug initialization
526500
* model" is being used. It should be called _prior_ to
527-
* scsi_unregister().
501+
* calling scsi_host_put().
528502
*
529503
* Defined in: drivers/scsi/hosts.c .
530504
**/
@@ -601,31 +575,12 @@ Details::
601575
void scsi_unblock_requests(struct Scsi_Host * shost)
602576

603577

604-
/**
605-
* scsi_unregister - unregister and free memory used by host instance
606-
* @shp: pointer to scsi host instance to unregister.
607-
*
608-
* Returns nothing
609-
*
610-
* Might block: no
611-
*
612-
* Notes: Should not be invoked if the "hotplug initialization
613-
* model" is being used. Called internally by exit_this_scsi_driver()
614-
* in the "passive initialization model". Hence a LLD has no need to
615-
* call this function directly.
616-
*
617-
* Defined in: drivers/scsi/hosts.c .
618-
**/
619-
void scsi_unregister(struct Scsi_Host * shp)
620-
621-
622-
623578

624579
Interface Functions
625580
===================
626581
Interface functions are supplied (defined) by LLDs and their function
627582
pointers are placed in an instance of struct scsi_host_template which
628-
is passed to scsi_host_alloc() [or scsi_register() / init_this_scsi_driver()].
583+
is passed to scsi_host_alloc().
629584
Some are mandatory. Interface functions should be declared static. The
630585
accepted convention is that driver "xyz" will declare its slave_configure()
631586
function as::
@@ -636,8 +591,8 @@ and so forth for all interface functions listed below.
636591

637592
A pointer to this function should be placed in the 'slave_configure' member
638593
of a "struct scsi_host_template" instance. A pointer to such an instance
639-
should be passed to the mid level's scsi_host_alloc() [or scsi_register() /
640-
init_this_scsi_driver()].
594+
should be passed to the mid level's scsi_host_alloc().
595+
.
641596

642597
The interface functions are also described in the include/scsi/scsi_host.h
643598
file immediately above their definition point in "struct scsi_host_template".
@@ -817,10 +772,6 @@ Details::
817772
* The SCSI_IOCTL_PROBE_HOST ioctl yields the string returned by this
818773
* function (or struct Scsi_Host::name if this function is not
819774
* available).
820-
* In a similar manner, init_this_scsi_driver() outputs to the console
821-
* each host's "info" (or name) for the driver it is registering.
822-
* Also if proc_info() is not supplied, the output of this function
823-
* is used instead.
824775
*
825776
* Optionally defined in: LLD
826777
**/
@@ -1115,7 +1066,7 @@ of interest:
11151066
hostdata[0]
11161067
- area reserved for LLD at end of struct Scsi_Host. Size
11171068
is set by the second argument (named 'xtr_bytes') to
1118-
scsi_host_alloc() or scsi_register().
1069+
scsi_host_alloc().
11191070
vendor_id
11201071
- a unique value that identifies the vendor supplying
11211072
the LLD for the Scsi_Host. Used most often in validating

drivers/scsi/megaraid/megaraid_mbox.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -618,7 +618,7 @@ megaraid_io_attach(adapter_t *adapter)
618618
host = scsi_host_alloc(&megaraid_template_g, 8);
619619
if (!host) {
620620
con_log(CL_ANN, (KERN_WARNING
621-
"megaraid mbox: scsi_register failed\n"));
621+
"megaraid mbox: scsi_host_alloc failed\n"));
622622

623623
return -1;
624624
}

0 commit comments

Comments
 (0)