tech-kern archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
re: Implement mode_select for atapi tape drives - round 4
> + u_int select_len;
> + struct select {
> + struct scsi_mode_parameter_header_6 header;
> + struct scsi_general_block_descriptor blk_desc;
> + u_char sense_data[MAX_PAGE_0_SIZE];
> + } select;
> + struct scsipi_periph *periph = st->sc_periph;
> +
> + select_len = sizeof(select.header) + sizeof(select.blk_desc) +
> + st->page_0_size;
>
> should select be __packed?
Yes, it should probably be. I know most hardware structures are not
declared __packed in the scsipi code, we've been luky this didn't cause
issues for now ...
are these structures used heavily? __packed will significantly
slow down accesses to and from them.
.mrg.
Home |
Main Index |
Thread Index |
Old Index