Re: [PATCHSET block#for-2.6.36-post] block: replace barrier with sequenced flush

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Tejun Heo
Date: Monday, August 23, 2010 - 5:36 am

Hello,

On 08/20/2010 03:22 PM, Christoph Hellwig wrote:

A bit nit picky but flash devices can also have writeback caches and
the term physical medium sounds a bit off for those cases.  Maybe just
saying "non-volatile media" is better?


It may be worthwhile to explain the sequence of operations when
REQ_FLUSH + data + REQ_FUA is executed.  It can be extrapolated from
the previous two descriptions but I think giving examples of different
sequences depending on FLUSH/FUA configuration would be helpful to
help understanding the overall picture of things.

Other than those, looks good to me.

Thanks.

-- 
tejun
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 02/11] block: kill QUEUE_ORDERED_BY_TAG, Tejun Heo, (Thu Aug 12, 5:41 am)
[PATCH 05/11] block: misc cleanups in barrier code, Tejun Heo, (Thu Aug 12, 5:41 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Christoph Hellwig, (Fri Aug 13, 4:48 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Vladislav Bolkhovitin, (Fri Aug 13, 5:55 am)
Re: [PATCH 02/11] block: kill QUEUE_ORDERED_BY_TAG, Vladislav Bolkhovitin, (Fri Aug 13, 5:56 am)
Re: [PATCH 02/11] block: kill QUEUE_ORDERED_BY_TAG, Christoph Hellwig, (Fri Aug 13, 6:06 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Christoph Hellwig, (Fri Aug 13, 6:17 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Christoph Hellwig, (Fri Aug 13, 7:38 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Christoph Hellwig, (Sat Aug 14, 3:36 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Christoph Hellwig, (Tue Aug 17, 6:19 am)
Re: [PATCH 08/11] block: rename barrier/ordered to flush, Christoph Hellwig, (Tue Aug 17, 6:26 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Christoph Hellwig, (Tue Aug 17, 9:59 am)
Re: [PATCH 08/11] block: rename barrier/ordered to flush, Christoph Hellwig, (Tue Aug 17, 10:08 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Christoph Hellwig, (Wed Aug 18, 2:46 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Vladislav Bolkhovitin, (Wed Aug 18, 12:29 pm)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Vladislav Bolkhovitin, (Wed Aug 18, 12:30 pm)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Christoph Hellwig, (Thu Aug 19, 3:20 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Christoph Hellwig, (Fri Aug 20, 6:22 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Tejun Heo, (Mon Aug 23, 5:36 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Christoph Hellwig, (Mon Aug 23, 5:48 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Christoph Hellwig, (Mon Aug 23, 7:05 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Christoph Hellwig, (Mon Aug 23, 7:08 am)
[PATCH] block: simplify queue_next_fseq, Christoph Hellwig, (Mon Aug 23, 7:15 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Christoph Hellwig, (Mon Aug 23, 7:19 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Vladislav Bolkhovitin, (Tue Aug 24, 10:11 am)
Re: [PATCHSET block#for-2.6.36-post] block: replace barrie ..., Vladislav Bolkhovitin, (Mon Aug 30, 1:34 pm)