On Mon, 2015-02-09 at 14:28 +0800, Chen, Tiejun wrote: > What about this?
I've not read the code in detail,since I'm travelling but from a quick glance it looks to be implementing the sort of thing I meant, thanks. A couple of higher level comments: I'd suggest to put the code for reading the vid/did into a helper function so it can be reused. You might like to optionally consider add a forcing option somehow so that people with new devices not in the list can control things without the need to recompile (e.g. gfx_passthru_kind_override?). Perhaps that isn't needed for a first cut though and it would be a libxl API so thought required. I think it should probably log something at a lowish level when it has autodetected IGD. Ian.
