incubator-kato-spec mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stuart Monteith <>
Subject Andrew Johnson's comments
Date Wed, 09 Dec 2009 18:51:54 GMT
     Here are Andrew Johnson's comments (adjusted for context). The 
grammar and spelling errors have been corrected already.
I'll correct the doc when there are undistib

I'll do this by page. Comments to follow in separate email.
Each number is listed like (1)., (2)., etc.

Chapter 3

Page 7



Instead of:
     List<ManagedRuntime> ImageProcess.getRuntimes()
Do we need:
     List<? extends ManagedRuntime>
- would allow List<PHPJavaRuntime> getRuntimes()



Does List equality hold?

E.g. heap.getObjects().equals(heap.getObjects()


Which methods returning lists throw exceptions?



Predictable order via different VMs?

Chapter 6
Page 24


Please have method as page header/footer.


Walk GC graph using JSR 326.

References:   \/---------------------------\
     JavaObject        \                    |
     JavaClass         |                    |
     JavaStackFrame    }- getReferences() getTarget()
     JavaThread?       |
     JavaClassLoader?  / <-----------------getSource()

Have interface for classes having getReferences.

Page 25

re. `services directory'
. This text file should contain a single line which is the package 
qualified name of the ImageFactory....

Multiple factories?
How would multiple factories work in one jar?


re. Table 6.3 FactoryRegistry Methods

public Iterator FactoryRegistry.iterator()
Iterator<ImageFactory> or List?


public ImageFactory getFactories()

should be:

public ImageFactory[] getFactories()



Returns the factories in the registry as an array


(I presume Andrew means that the array is mutable without ill effect)


public Image getImage(File file)

IllegalArgumentException if file is null

Why not throw NPE?


List<ImageFactory>   sniff file type!

get suggested factories for a file or files:

/ getFactories(File file)
\ getFactories(File file, File file)

Page 27

Choose a better order to list the classes.

e.g. ImageFactory, Image, ImageAddressSpace, ImagePointer, ImageSection


Table 6.4. Interface Summary

Image - This class represents an entire operating system image (e.g. ??

The text after "(e.g." is completely missing.


Table 6.5. Class Summary

DataUnavailable should be DataUnavailableException

DU extends DUE?

Page 28


interface ImageModule
--> running header

(I guess Andrew means that the header for the page should have the name 
of the interface.

Table 6.6
     public List getSymbols()

should be

     public List<ImageSymbol> getSymbols()

(The API has generics added now to the Image API. However, the document 
generation needs to be modified.
  I won't add any further comments about it.)

Page 29

Table 6.7. ImageMethods


     Also segmented arch?

Page 30


Returns the number of CPUs running in the system on which the image was 

     System = partition?

(I guess Andrew is suggesting "System" should be described as a 
"partition" in a virtual machine, z/Arch LPAR, pSeries LPAR, etc.?)

I'll continue from page 31 soon.


Stuart Monteith

View raw message