On Fri, 14 Apr 2006, James Bottomley wrote:
> On Sun, 2006-04-02 at 21:57 +0200, Guennadi Liakhovetski wrote:
> > + if (fc == 0x40 && (srb->dcb->sync_period & WIDE_SYNC)) {
> > + /* Read the last byte ... */
> > + if (srb->total_xfer_length > 0) {
> > + u8 byte = DC395x_read8(acb, TRM_S1040_SCSI_FIFO);
> > +
> > + *virt++ = byte;
> > + srb->total_xfer_length--;
> > + if (debug_enabled(DBG_PIO))
> > + printk(" % 02x", byte);
> > + }
> > +
> > + DC395x_write8(acb, TRM_S1040_SCSI_CONFIG2, 0);
> > + }
> > +
> > + if (srb->cmd->use_sg) {
> > + scsi_kunmap_atomic_sg(base);
>
> I put this in, but just a warning here. You're modifying a kmapped
> page, so you need to call flush_kernel_dcache_page() before unmapping it
> otherwise user space may not see the change on VIPT or VIVT CPU (the
> data may be in a non congruent cache line).
Thanks! Ok, sure, I'll try to cook an incremental patch, perhaps, next
week. At the same time I'll clean-up comments as suggested by Randy
Dunlap.
Guennadi
---
Guennadi Liakhovetski
-
: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
