Re: [libvirt] [PATCH] docs: add dimm address document in docs

2015-05-14 Thread Peter Krempa
On Thu, May 14, 2015 at 17:08:53 +0800, Luyao Huang wrote:
 As we have a new device address type 'dimm', add
 some document and an example to our docs.
 
 Signed-off-by: Luyao Huang lhu...@redhat.com
 ---
  docs/formatdomain.html.in | 8 
  1 file changed, 8 insertions(+)
 
 diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
 index e0b6ba7..0e908a1 100644
 --- a/docs/formatdomain.html.in
 +++ b/docs/formatdomain.html.in
 @@ -2876,6 +2876,13 @@
  attributes: codeiobase/code and codeirq/code.
  span class=sinceSince 1.2.1/span
/dd
 +  dtcodetype='dimm'/code/dt
 +  ddDIMM addresses, for memory device, have the following additional
 +attributes: codeslot/code (a value between 0 and 4294967295,
 +inclusive), and codebase/code (a hex value between 0 and
 +0x, inclusive).
 +span class=sinceSince 1.2.14/span

The values for the DIMM address shouldn't really be documented. I'd
rather state that the values are determined automatically and the user
should not need to touch them.

Peter


signature.asc
Description: Digital signature
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH] docs: add dimm address document in docs

2015-05-14 Thread lhuang


On 05/14/2015 07:57 PM, Peter Krempa wrote:

On Thu, May 14, 2015 at 17:08:53 +0800, Luyao Huang wrote:

As we have a new device address type 'dimm', add
some document and an example to our docs.

Signed-off-by: Luyao Huang lhu...@redhat.com
---
  docs/formatdomain.html.in | 8 
  1 file changed, 8 insertions(+)

diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
index e0b6ba7..0e908a1 100644
--- a/docs/formatdomain.html.in
+++ b/docs/formatdomain.html.in
@@ -2876,6 +2876,13 @@
  attributes: codeiobase/code and codeirq/code.
  span class=sinceSince 1.2.1/span
/dd
+  dtcodetype='dimm'/code/dt
+  ddDIMM addresses, for memory device, have the following additional
+attributes: codeslot/code (a value between 0 and 4294967295,
+inclusive), and codebase/code (a hex value between 0 and
+0x, inclusive).
+span class=sinceSince 1.2.14/span

The values for the DIMM address shouldn't really be documented. I'd
rather state that the values are determined automatically and the user
should not need to touch them.


Oh, i see, i will remove the value for DIMM address and add some mention 
about the values.


Thanks for your quick review.



Peter


Luyao

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list


[libvirt] [PATCH] docs: add dimm address document in docs

2015-05-14 Thread Luyao Huang
As we have a new device address type 'dimm', add
some document and an example to our docs.

Signed-off-by: Luyao Huang lhu...@redhat.com
---
 docs/formatdomain.html.in | 8 
 1 file changed, 8 insertions(+)

diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
index e0b6ba7..0e908a1 100644
--- a/docs/formatdomain.html.in
+++ b/docs/formatdomain.html.in
@@ -2876,6 +2876,13 @@
 attributes: codeiobase/code and codeirq/code.
 span class=sinceSince 1.2.1/span
   /dd
+  dtcodetype='dimm'/code/dt
+  ddDIMM addresses, for memory device, have the following additional
+attributes: codeslot/code (a value between 0 and 4294967295,
+inclusive), and codebase/code (a hex value between 0 and
+0x, inclusive).
+span class=sinceSince 1.2.14/span
+  /dd
 /dl
 
 h4a name=elementsControllersControllers/a/h4
@@ -6038,6 +6045,7 @@ qemu-kvm -net nic,model=? /dev/null
 lt;size unit='KiB'gt;524287lt;/sizegt;
 lt;nodegt;0lt;/nodegt;
   lt;/targetgt;
+  lt;address type='dimm' slot='0' base='0x0'/gt;
 lt;/memorygt;
 lt;memory model='dimm'gt;
   lt;sourcegt;
-- 
1.8.3.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list