RE: [PATCH 1/2] [v2] VT-d: Support multiple device assignment for KVM

2008-12-01 Thread Han, Weidong
It's fine. You only needs to change the APIs to generic APIs. I will update it soon. Regards, Weidong Joerg Roedel wrote: > Ok, I got them to apply. I also did the checkpatch cleanups. To speed > things up a bit I would suggest that I rebase my patchset on your > patches and send it out in a sin

Re: [PATCH 1/2] [v2] VT-d: Support multiple device assignment for KVM

2008-12-01 Thread Joerg Roedel
Ok, I got them to apply. I also did the checkpatch cleanups. To speed things up a bit I would suggest that I rebase my patchset on your patches and send it out in a single series. Any problems with this approach? Joerg On Mon, Dec 01, 2008 at 09:22:42PM +0800, Han, Weidong wrote: > Sorry, this pa

RE: [PATCH 1/2] [v2] VT-d: Support multiple device assignment for KVM

2008-12-01 Thread Han, Weidong
Sorry, this patch has style problem. I will update it and also split it to smaller patches for easy reviewing. Regards, Weidong 'Joerg Roedel' wrote: > Hmm, I get these errors using git-am: > > Applying VT-d: Support multiple device assignment for KVM > .dotest/patch:1344: space before tab in i

Re: [PATCH 1/2] [v2] VT-d: Support multiple device assignment for KVM

2008-12-01 Thread 'Joerg Roedel'
Hmm, I get these errors using git-am: Applying VT-d: Support multiple device assignment for KVM .dotest/patch:1344: space before tab in indent. clflush_cache_range(addr, size); .dotest/patch:1350: space before tab in indent. clflush_cache_ran

RE: [PATCH 1/2] [v2] VT-d: Support multiple device assignment for KVM

2008-11-30 Thread Han, Weidong
It's developed based on commit 0f7d3ee6 on avi/master, but it still can be applied on latest avi/master (commit 90755652). Regards, Weidong Joerg Roedel wrote: > Hmm, I tried to apply this patch against avi/master and linus/master > but get merge conflicts. Where do these patches apply cleanly?

Re: [PATCH 1/2] [v2] VT-d: Support multiple device assignment for KVM

2008-11-28 Thread Joerg Roedel
Hmm, I tried to apply this patch against avi/master and linus/master but get merge conflicts. Where do these patches apply cleanly? Joerg On Thu, Nov 27, 2008 at 09:49:04PM +0800, Han, Weidong wrote: > In order to support multiple device assignment for KVM, this patch does > following main chang