Re: cvs commit: src/sbin/atacontrol atacontrol.c

2007-12-18 Thread Bernd Walter
On Mon, Dec 17, 2007 at 02:55:31PM +, Poul-Henning Kamp wrote: > In message <[EMAIL PROTECTED]>, "Bruce M. Simpson" writes: > >Poul-Henning Kamp wrote: > >> I have the attached patch in sos@ mailbox for approval, that adds > >> BIO_DELETE support for the ata driver. > >> > >> I also want to imp

Re: cvs commit: src/sbin/atacontrol atacontrol.c

2007-12-17 Thread Poul-Henning Kamp
In message <[EMAIL PROTECTED]>, Bruce M Simpson writes: >Ah, let me rephrase, I meant: > >* NAND Flash embedded ATA controllers should not erase sectors >containing data unless absolutely necessary, to implement wear levelling. That's part of the job description, but it is more complex than that

Re: cvs commit: src/sbin/atacontrol atacontrol.c

2007-12-17 Thread Poul-Henning Kamp
In message <[EMAIL PROTECTED]>, "M. Warner Losh" writes: >In message: <[EMAIL PROTECTED]> >"Poul-Henning Kamp" <[EMAIL PROTECTED]> writes: >: > * NAND Flash devices should not have their sectors erased unless >: >absolutely necessary, to implement wear levelling. >: >: Wrong, almost e

Re: cvs commit: src/sbin/atacontrol atacontrol.c

2007-12-17 Thread Maxim Sobolev
Poul-Henning Kamp wrote: And finally the big item: msdosfs and ufs support to issue BIO_DELETE when files are deleted. UFS is nasty because of soft-updates. Cool! This will also benefit malloc(9) disk. -Maxim ___ cvs-all@freebsd.org mailing list htt

Re: cvs commit: src/sbin/atacontrol atacontrol.c

2007-12-17 Thread M. Warner Losh
In message: <[EMAIL PROTECTED]> "Poul-Henning Kamp" <[EMAIL PROTECTED]> writes: : > * NAND Flash devices should not have their sectors erased unless : >absolutely necessary, to implement wear levelling. : : Wrong, almost exactly the opposite in fact: : : Flash devices using wear-leve

Re: cvs commit: src/sbin/atacontrol atacontrol.c

2007-12-17 Thread Dag-Erling Smørgrav
Bruce M Simpson <[EMAIL PROTECTED]> writes: > The feedback I've seen in FreeBSD forums regarding ATA write caching > tends to back up your original assertion. It wouldn't be the first case of uninformed people reinforcing eachother's belief in incorrect information. > I wonder if vendor neutral,

Re: cvs commit: src/sbin/atacontrol atacontrol.c

2007-12-17 Thread Bruce M Simpson
Poul-Henning Kamp wrote: Aha, I understand now. CFA and SATA vendors have gone off in two separate directions: * PATA and SATA drives, for a few years now, have tended to rewrite one cylinder at a time, which implies erasing the data on that cylinder. Everybody denies this in the stongest

Re: cvs commit: src/sbin/atacontrol atacontrol.c

2007-12-17 Thread Bruce M. Simpson
Poul-Henning Kamp wrote: I have the attached patch in sos@ mailbox for approval, that adds BIO_DELETE support for the ata driver. I also want to implement a -E option to fsck(8) to erase all unallocated blocks. And finally the big item: msdosfs and ufs support to issue BIO_DELETE when files are

Re: cvs commit: src/sbin/atacontrol atacontrol.c

2007-12-17 Thread Poul-Henning Kamp
In message <[EMAIL PROTECTED]>, "Bruce M. Simpson" writes: >Poul-Henning Kamp wrote: >> I have the attached patch in sos@ mailbox for approval, that adds >> BIO_DELETE support for the ata driver. >> >> I also want to implement a -E option to fsck(8) to erase all >> unallocated blocks. >> >> And fin

Re: cvs commit: src/sbin/atacontrol atacontrol.c

2007-12-16 Thread Poul-Henning Kamp
In message <[EMAIL PROTECTED]>, "Bruce M. Simpson" writes: >> Modified files: >> sbin/atacontrol atacontrol.c >> Log: >> Recognize CFA devices using either identification method. > >Does this give the ata driver a means of identifying that a disk device >is actually a CompactFlash

Re: cvs commit: src/sbin/atacontrol atacontrol.c

2007-12-16 Thread Bruce M. Simpson
Poul-Henning Kamp wrote: phk 2007-12-16 21:19:07 UTC FreeBSD src repository Modified files: sbin/atacontrol atacontrol.c Log: Recognize CFA devices using either identification method. Does this give the ata driver a means of identifying that a disk device is act