On Mon, 2006-06-12 at 00:21 -0400, Jeff Garzik wrote: > Jeff Garzik wrote: > > zhao, forrest wrote: > >> This patch makes libata "Snoop SET FEATURES - WRITE CACHE ENABLE/DISABLE > >> command" and clean the things up(e.g. revalidate and rescan). > >> > >> Signed-off-by: Forrest Zhao <forrest.zhao@xxxxxxxxx> > > > > applied > > Um... please make sure all your patches build! > > I had to apply the attached patch to get it to work. > > Jeff > > > > Plain text document attachment (patch) > diff --git a/include/linux/libata.h b/include/linux/libata.h > index 39e6b77..61eea57 100644 > --- a/include/linux/libata.h > +++ b/include/linux/libata.h > @@ -522,6 +522,7 @@ struct ata_port { > > struct work_struct port_task; > struct work_struct hotplug_task; > + struct work_struct scsi_rescan_task; > > unsigned int hsm_task_state; Oh, sorry for that. I had tested the patch before sending it out. But don't know why this line is missing. Do I need to re-send the whole patch? BTW. the disk of my develop machine for SATA was corrupted 10 minutes ago after I applied the AHCI suspend/resume patch against #upstream, I had to recover it painfully. Haven't expected that it can cause such bad damage :( Forrest - : send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html