Hi, On Sunday 27 January 2008, Borislav Petkov wrote: > Hi Bart, > > after a lot of hammering ide-tape got pimped pretty considerably (ca. 600 > lines > shorter and slicker :)). I'm sure there's more to be done like, e.g. replacing
Good work. :) > the BKL in idetape_write_release() with finer-grained locking etc, probably > also > some pipeline improvements, removal of OnStream support, etc. but that'll come > later. On-Stream support has been long gone but it seems that deprecation warning etc. managed to survive. w.r.t. to the pipeline-mode: it should be pipelined into /dev/null rationale: - it is _very_ complex - causes errors to be deferred till the next user-space access - direct I/O using blk_rq_map_user() will offer superior performance the only question is whether to remove it... > Documentation/ide/ChangeLog.ide-tape.1995-2002 | 405 +++ > drivers/ide/Kconfig | 3 +- > drivers/ide/ide-tape.c | 4146 > +++++++++--------------- > 3 files changed, 1991 insertions(+), 2563 deletions(-) applied #1-6, #8-9, #11-20, #29, #31 #10, #24 and #25 are also fine but since they depend on other patches I couldn't merge them immediately #7 and #21 need some recasting #22 should be deferred for now #26-28, #30 and #32 are still to be reviewed BTW what happend to patch #23? Thanks, Bart - To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
