From: Zach Brown <zach.brown@oracle.com>
To: suparna@in.ibm.com
Cc: linux-aio@kvack.org, linux-kernel@vger.kernel.org,
Benjamin LaHaise <bcrl@kvack.org>, Andrew Morton <akpm@osdl.org>,
mason@suse.com
Subject: Re: [PATCH] [AIO] remove unused aio_run_iocbs()
Date: Tue, 01 Aug 2006 09:14:19 -0700 [thread overview]
Message-ID: <44CF7DDB.3070705@oracle.com> (raw)
In-Reply-To: <20060801072731.GA20484@in.ibm.com>
> Chris Mason's aio pipe patches used these to reduce the large
> number of context switches he was observing when running pipetest.
> Of course aio pipe support hasn't been merged into mainline so far, and hence
> you could argue that we put these back in if/when we hit that problem.
Yeah, and that's trivial.
> But why
> not just put in a comment there for now to ease the confusion ... generally
> I'd rather go a little slow in removing apparently unused code at this
> point when we are churning things up again.
The only thing slower than not removing it after *years* of not being
used would be to never remove it :)
So I don't see any value in keeping it, but I won't make a fight of it
either.
- z
next prev parent reply other threads:[~2006-08-01 16:14 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-07-31 22:12 [PATCH] [AIO] remove unused aio_run_iocbs() Zach Brown
2006-08-01 7:27 ` Suparna Bhattacharya
2006-08-01 16:14 ` Zach Brown [this message]
2006-08-02 8:09 ` Suparna Bhattacharya
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=44CF7DDB.3070705@oracle.com \
--to=zach.brown@oracle.com \
--cc=akpm@osdl.org \
--cc=bcrl@kvack.org \
--cc=linux-aio@kvack.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mason@suse.com \
--cc=suparna@in.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.