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.

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

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

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

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?