| 1 | TODO:
|
|---|
| 2 |
|
|---|
| 3 | * Ensure that we can have our 'non-threaded' thread be any thread.
|
|---|
| 4 |
|
|---|
| 5 | * Turn off the global 'fast_calls' variable, to knobble
|
|---|
| 6 | old stubs, and go with the new ABI.
|
|---|
| 7 |
|
|---|
| 8 | * Cross thread Object references
|
|---|
| 9 | + I think the way to fix this is by using a different
|
|---|
| 10 | interface->adaptor_type: ORBIT_ADAPTOR_POA_THREADED,
|
|---|
| 11 | such that the fast case will always drop through to
|
|---|
| 12 | small_invoke_stub_n, where we can get far more creative.
|
|---|
| 13 |
|
|---|
| 14 | * Threading
|
|---|
| 15 | + GIOPMessageQueueEntry needs to hold a cnx ref.
|
|---|
| 16 | + orb->current_invocations needs to be per-thread ...
|
|---|
| 17 | also, we should push a stack-data pointer.
|
|---|
| 18 | + Need to emit 'broken' connection - via the listen/unlisten
|
|---|
| 19 | stuff in the main thread ... [ and garentee the same
|
|---|
| 20 | invariants as previously ? cnx status etc. ? ]
|
|---|
| 21 |
|
|---|
| 22 | + Create a per-object policy 'PROCESS_REENTERANTLY', using our
|
|---|
| 23 | VPVID space; takes a boolean ... set it on the object ...
|
|---|
| 24 | [ make some libbonobo wrappers to improve the situation ]
|
|---|
| 25 | + CORBA_Object_non_existant - needs to do a blocking
|
|---|
| 26 | check for CONNECTED / DISCONNETED.
|
|---|
| 27 | + bin redundant forwards in poa.c
|
|---|
| 28 | + giop_recv_thread_fn - more work needed
|
|---|
| 29 | + ** ERROR **: Need to handle zapped cnx' async callbacks
|
|---|
| 30 |
|
|---|
| 31 |
|
|---|
| 32 | * User friendliness
|
|---|
| 33 | + tolerate NULL CORBA_Environments in stubs
|
|---|
| 34 | + 'sequence from GList' API.
|
|---|
| 35 | + always return initialized memory on exception
|
|---|
| 36 | + add an X like 'event' processing API for incoming bits (?)
|
|---|
| 37 |
|
|---|
| 38 | * GIOP fragments
|
|---|
| 39 | / + need to handle a cancel request in the middle of a fragment stream
|
|---|
| 40 |
|
|---|
| 41 | * It seems the local case optimization can go wrong when
|
|---|
| 42 | invoking bogus methods from a different interface on an Object ref.
|
|---|
| 43 | + this is to do with
|
|---|
| 44 | a) not checking the vepv offset we get
|
|---|
| 45 | b) not expanding the lookup of the vepv offset at all.
|
|---|
| 46 |
|
|---|
| 47 | * Thread safety
|
|---|
| 48 | + For independent clients, we need some way to
|
|---|
| 49 | arbitrate the giop_recv_buffer_get method - so
|
|---|
| 50 | that either: a single thread does the poll and the
|
|---|
| 51 | rest block, or we all poll.
|
|---|
| 52 |
|
|---|
| 53 | + apparently g_object_ref / unref are not thread safe
|
|---|
| 54 | we need linc_object_ref / unref instead that does
|
|---|
| 55 | a LINC_MUTEX_LOCK on the global object lifecycle lock
|
|---|
| 56 | that needs to move into linc ... sigh !
|
|---|
| 57 |
|
|---|
| 58 | + we need to ensure that linc does no signal emissions,
|
|---|
| 59 | since these are horribly slow and again not safe.
|
|---|
| 60 |
|
|---|
| 61 | * Optimization
|
|---|
| 62 | + we need 2 de-marshalling routines, 1 with (slow)
|
|---|
| 63 | byteswap and 1 without it, save scads and scads of
|
|---|
| 64 | branch cache misses; easy to do too; make it a virtual
|
|---|
| 65 | method on the recv buffer (?) [ for easy chaining ]
|
|---|
| 66 |
|
|---|
| 67 | + we need some concept of align == native align flagged
|
|---|
| 68 | on TypeCodes, so we can blat things quickly with memcpy
|
|---|
| 69 | in a generic way - eg. sequences of octets.
|
|---|
| 70 |
|
|---|
| 71 | + we need to implement 'wait for completion' on
|
|---|
| 72 | shutdown so that we can activate async unrefs in
|
|---|
| 73 | bonobo.
|
|---|
| 74 |
|
|---|
| 75 | * Dragons:
|
|---|
| 76 | + giop_recv_buffer.c (giop_connection_handle_input)
|
|---|
| 77 | manages to go through the error condition a whole
|
|---|
| 78 | load; _why_ is this ? - is it a recursive effect on
|
|---|
| 79 | the method ? why are we called if there is no data ?
|
|---|
| 80 |
|
|---|
| 81 | + We need a typecast in an _allocbuf which we know is
|
|---|
| 82 | aliased, eg. PortableServer_POAList_allocbuf, since
|
|---|
| 83 | it gets assigned to a CORBA_Object *, and the cast
|
|---|
| 84 | is bogus.
|
|---|
