Re: [Qemu-devel] [4367] Align file accesses with cache=off (Kevin Wolf, Laurent Vivier)

[Kevin Wolf]

Jamie Lokier schrieb:
> Kevin Wolf wrote:
> > Btw, it is quite interesting to see that a serious discussion of a patch 
> > happens only if it is already committed. This could have been discussed 
> > a week ago when we agreed to go in the apparently wrong direction. And 
> > the patch has been on the list much longer than this one week.
>
> IIRC, the same issues (recursive callback vs. queued callback,
> problems with direct and non-direct I/O in flight at the same time)
> were raised before it was committed, but it was committed anyway.

No, nobody mentioned the recursion problem. And you were talking about 
problem with two different file descriptors for one file, not about the 
fcntl solution. Ok, might also be that the hints were just not explicit 
enough for me. ;-)

But even if so, this is more of a general feeling about how patches are 
handled and not only related to this patch.

Kevin