From: Chris Mason <chris.mason@oracle.com>
To: Or Gerlitz <ogerlitz@voltaire.com>
Cc: Andy Grover <andy.grover@oracle.com>,
linux-rdma@vger.kernel.org, netdev@vger.kernel.org,
RDS Devel <rds-devel@oss.oracle.com>,
Roland Dreier <rdreier@cisco.com>
Subject: Re: [rds-devel] net-next pull request: RDS
Date: Mon, 13 Sep 2010 13:14:28 -0400 [thread overview]
Message-ID: <20100913171428.GR21374@think> (raw)
In-Reply-To: <4C8C9422.2070306@Voltaire.com>
On Sun, Sep 12, 2010 at 10:49:38AM +0200, Or Gerlitz wrote:
> Hi Andy,
>
> Some clarifications/questions from whatever quick look one can have over 107 patches...
>
> Zach Brown's "RDS/IB: print IB event strings as well as their number" - commit
> 1bde04a63d532c2540d6fdee0a661530a62b1686 in net-next-2.6 looks perfect to reside as a helper function in the core IB stack which can be in use by other rdma drivers (e.g ipoib, iser, srp, etc).
>
> Chris Mason's "rds: recycle FMRs through lockless lists added net/rds/xlist.h" - 6fa70da6081bbcf948801fd5ee0be4d222298a43 adds net/rds/xlist.h - isn't this something that better be placed under include/linux/. etc?
Yes, I plan on expanding xlist.h a little and trying to use it in a few
other places. But since RDS is currently the only consumer it made sense
to me to keep it in rds until I had more code in line to take advantage
of it.
-chris
next prev parent reply other threads:[~2010-09-13 17:14 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-09 18:51 net-next pull request: RDS Andy Grover
[not found] ` <4C892C9C.4060304-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>
2010-09-09 21:58 ` David Miller
2010-09-12 8:49 ` Or Gerlitz
2010-09-13 17:14 ` Chris Mason [this message]
[not found] ` <4C8C9422.2070306-hKgKHo2Ms0FWk0Htik3J/w@public.gmane.org>
2010-09-14 18:35 ` [rds-devel] " Zach Brown
[not found] ` <20100914183514.GA3417-Mg0Rv1oTIzK+NQz5YmTpAAHTzkgPINtM@public.gmane.org>
2010-09-14 22:28 ` Andrew Grover
[not found] ` <AANLkTin69m0kK6bqUiwYeHRQT0Ehso16xA6RBtBFFgW3-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-09-14 23:10 ` Roland Dreier
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=20100913171428.GR21374@think \
--to=chris.mason@oracle.com \
--cc=andy.grover@oracle.com \
--cc=linux-rdma@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=ogerlitz@voltaire.com \
--cc=rdreier@cisco.com \
--cc=rds-devel@oss.oracle.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).