[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v3 15/24] xen/dts: Provide an helper to get a DT node from a path provided by a guest



On Mon, 2015-02-23 at 16:01 +0000, Julien Grall wrote:
> Hi Ian,
> 
> On 23/02/15 15:30, Ian Campbell wrote:
> > On Tue, 2015-01-13 at 14:25 +0000, Julien Grall wrote:
> > 
> >> +/* This limit is used by the hypercalls to restrict the size of the path 
> >> */
> >> +#define DEVICE_TREE_MAX_PATHLEN 1024
> > 
> > Is this something you've made up or derived from the DT spec/ePAPR etc?
> 
> I didn't find a such requirements on the spec.

I vaguely recall a limit on the length of a single node name, and a
limit on the depth which they may nest, which can be multiplied. It's
probably an unhelpfully large number though, so...

> I chose this number based on the linux pathlen because the path is also
> used in the sysfs (/sys/firmware/devicetree).

...that's a good as anything I suppose!

> > Apart from this the patch seems fine, clarifying the genesis of that
> > number in the commit log would be sufficient for me.
> 
> I will update the commit message.

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.