On 2012-07-16 15:07 NeilBrown [off-list ref] Wrote:

On Mon, 16 Jul 2012 14:42:54 +0800 majianpeng [off-list ref] wrote:

quoted

On 2012-07-16 13:40 NeilBrown [off-list ref] Wrote:

quoted

On Mon, 16 Jul 2012 09:31:55 +0800 majianpeng [off-list ref] wrote:

[snip]

quoted

quoted

Normal 'sync' requests use WRITE_SYNC which includes "REQ_NOIDLE" which means
  /* don't anticipate more IO after this one */
O_DIRECT request use WRITE_ODIRECT which does not include this flag.

quoted

Using REQ_NOIDEL to difference odirect and sync.Why not using:
 +	if (bi->bi_rw & WRITE_ODIRECT)
 +		bi->bi_rw &= ~REQ_SYNC;

Because that code is wrong.  WRITE_ODIRECT is not one flag, it is two flags
'or'ed together.  So this code does not do what you expect.

No, I used those code test and it's ok.
The code used & not &&.
Maybe I wrong?

quoted

[snip]