RE: device assignemnt: updated patches

Han, Weidong
2008-07-26T09:24:11Z

Avi Kivity wrote:
> Ben-Ami Yassour wrote:
>> Following are the device assignment patches with the fixes of the
>> comments that were sent for the previous version.
>>
>> Here is the list of changes that were made with respect to the
>> previous version:
>> 1. Replace the interrupt ack hook patches with the notifiers list
>> patch by Avi.
>> 2. Remove code from ioapic - the ack notifier is called before the
>> checking the irr bit.
>> 3. Remove the interrupt ack work queue and handle it directly.
>> 4. Minimize the function for finding an assigned device.
>> 5. Remove the pt_lock, after making the changes above it is no
>> longer needed.
>> 6. Move declarations from kvm_para.h
>> 7. Add irq array to ioctl API. Note that this is only a change to the
>> API, currently only single irq is supported.
>> 8. Renaming: use "assigned device" and not "passthrough device"
>> 9. Fix device release error path
>> 10. Moving the assigned devices list pointer to the device struct
>> itself and remove the extra structure.
>>
>> Pending comment: shared guest interrputs are not tested.
>>
>>
>
> (and not implemented, either).
>
> Good progress; there are still many issues, but this is inevitable
> with
> such a large and complex patchset.
>
> How are we standing with merging the changes to the VT-d driver, which
> are a prerequisite?

You mean push the changes into VT-d driver first. But the VT-d driver
modification patch maybe need some changes during following development.
How about making it stable in KVM first, then push it into VT-d driver?

Randy (Weidong)

>
>
> --
> Do not meddle in the internals of kernels, for they are subtle and
> quick to panic.