From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Jesse Barnes <jbarnes@virtuousgeek.org>
Cc: linux-pci@vger.kernel.org, xen-devel@lists.xensource.com,
linux-kernel@vger.kernel.org, konrad@kernel.org
Subject: Re: [Xen-devel] Re: [PATCH v6] xen pci backend driver.
Date: Fri, 15 Jul 2011 11:37:21 -0400 [thread overview]
Message-ID: <20110715153721.GA7512@dumpdata.com> (raw)
In-Reply-To: <20110531100040.60ade056@jbarnes-desktop>
> > Jesse,
> >
> > Xen has a drivers/xen directory. In the past we have put some of the
> > "backends" in there, such as the SWIOTLB one. Perhaps some of it (or all?)
> > of this driver should go in there? If you are OK with this I can do it
> > that way and start chasing folks to review this driver?
>
> Yeah, I'm fine with that. Each arch typically has a chunk of PCI code
> under arch/ anyway (some more than others), so doing it that way is ok
> with me.
>
> > > Figuring out some way to preserve the git history is probably a good
> > > idea, assuming it's not too much of a mess. If it is, a more
> >
> > I tried to keep it clean.. Most of the stuff is from me thought so I might
> > as well just squash it in one.
>
> Yeah that's up to you; presumably you just want to preserve the useful
> parts of the history, or maybe just make the initial changelog big
> enough to include all the important bits you've learned over the
> years. :)
Hey Jesse,
I reposted it (fixed a bug, added a feature, and moved them to drivers/xen)
here:
https://lkml.org/lkml/2011/7/12/299
And I've coalesced the git history a bit to clump all the cleanpatch, incremental
features, etc in one big piece. Made it from 50 down to 18 patches. They are
in
git://git.kernel.org/pub/scm/linux/kernel/git/konrad/xen.git devel/xen-pciback-0.6.2
What are your thoughts about it?
prev parent reply other threads:[~2011-07-15 15:39 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-20 21:07 [PATCH v6] xen pci backend driver Konrad Rzeszutek Wilk
2011-04-20 21:07 ` [PATCH] " Konrad Rzeszutek Wilk
2011-04-25 18:58 ` [PATCH v6] " Jesse Barnes
2011-04-26 3:05 ` [Xen-devel] " Konrad Rzeszutek Wilk
2011-05-27 15:22 ` Konrad Rzeszutek Wilk
2011-05-28 14:18 ` Fix wrong memory size of nested shadow_io_bitmap Dong, Eddie
2011-05-31 17:00 ` [Xen-devel] Re: [PATCH v6] xen pci backend driver Jesse Barnes
2011-07-15 15:37 ` Konrad Rzeszutek Wilk [this message]
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=20110715153721.GA7512@dumpdata.com \
--to=konrad.wilk@oracle.com \
--cc=jbarnes@virtuousgeek.org \
--cc=konrad@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=xen-devel@lists.xensource.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.