> From: David Gwynne <[email protected]> > Date: Mon, 31 Oct 2016 10:38:49 +1000 > > > On 31 Oct 2016, at 09:18, Stefan Fritsch <[email protected]> wrote: > > > > On Sun, 30 Oct 2016, Stefan Fritsch wrote: > >> I agree with all your comments (and should have reviewed the initial patch > >> better). Here is an updated diff. > > > > The FWRITE check was missing in wdioctl(). Next try: > > the diff looks fine to me. > > whats the use of the ioctl though? why do we need extra sync cache things?
Right. This smells like papering over real bugs.
