Messages in this thread | | | Date | Wed, 13 Feb 2008 17:54:27 +0200 | From | Boaz Harrosh <> | Subject | Re: [BUGFIX 2/2] gdth: bugfix for the Timer at exit crash |
| |
On Wed, Feb 13 2008 at 17:44 +0200, James Bottomley <James.Bottomley@HansenPartnership.com> wrote: > On Tue, 2008-02-12 at 19:40 +0200, Boaz Harrosh wrote: >> - gdth_flush(ha); >> - > > This piece doesn't look right. gdth_flush() forces the internal cache > to disk backing. If you remove it, you're taking the chance that the > machine will be powered off without a writeback which can cause data > corruption. > > James > Yes. I have more problems reported, with exit, and am just sending one more patch that puts this back in. Which was tested.
So I will resend this one plus one new one.
Boaz
| |