On 07/11/2017 11:15 PM, Cyril Novikov wrote: > Hi, all! > > The product that I work on supports physical device assignment (aka > "pass-through") to virtual machines, which means we need to take > portions of the physical FDT and include them in the guest FDT. This > needs to happen automatically in software. > > The problem is phandles, because we cannot identify them in the blob and > therefore can't find any dependent devices/nodes that also need to be > included in the guest FDT. So the process cannot be fully automated. We > can't even advise the user what other devices should be assigned to the > VM. The hypervisor runs or bare metal, so having to parse the source DTS > files for this is very inconvenient. > > Would it be possible to add metadata properties to the binary FDT > format, which would identify other property cells that are in fact > phandles? It could be a per-node property or a single root node > property, up to you guys. DTC would then automatically generate the > metadata property along with the phandle property when compiling the DTS. phandles are already per-node properties, they are not quite different from normal properties actually. If a node is referred to by other nodes, the DTC compiler would add a phandle = <N> (and maybe even linux,phandle = <N> depending on options passed to it) where N is a global integer that keeps being incremented every time a new phandle is generated. When you strip or create new nodes from a base blob for your virtual machine, either the node is still existing, in which case its phandle property (if existing) is still valid and can still be referenced to, or it is a new node and then you can control how to allocate new phandle integers. I guess I am just not clear on what problem you are seeing with phandles based on your description of what you want to do? -- Florian -- To unsubscribe from this list: send the line "unsubscribe devicetree-spec" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
