summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authornobody <nobody@ae88bc3d-4319-0410-8dbf-d08b4c9d3795>2001-06-25 12:51:33 +0000
committernobody <nobody@ae88bc3d-4319-0410-8dbf-d08b4c9d3795>2001-06-25 12:51:33 +0000
commit9b22c3f061566c6860df94d7929297582a0c8b01 (patch)
tree93b8d478613022a1f1c62c9a4cdbae6e8b061cb3
parent9102f3eb96dbfe7310e953fe9369bebe2600f619 (diff)
downloadATCD-9b22c3f061566c6860df94d7929297582a0c8b01.tar.gz
This commit was manufactured by cvs2svn to create branch 'bug_575'.
-rw-r--r--TAO/ChangeLog_bug_575260
1 files changed, 260 insertions, 0 deletions
diff --git a/TAO/ChangeLog_bug_575 b/TAO/ChangeLog_bug_575
new file mode 100644
index 00000000000..6e8dac4a2bb
--- /dev/null
+++ b/TAO/ChangeLog_bug_575
@@ -0,0 +1,260 @@
+Mon Jun 25 07:45:38 2001 Balachandran Natarajan <bala@cs.wustl.edu>
+
+ * Created this file.
+
+Fri Jun 22 17:00:38 2001 Balachandran Natarajan <bala@cs.wustl.edu>
+
+ * tao/LIST_OF_TODO: Updated the list of TODO's.
+
+Mon Jun 18 13:31:38 2001 Balachandran Natarajan <bala@cs.wustl.edu>
+
+ * tao/Asynch_Reply_Dispatcher.cpp (dispatch_reply):
+ * tao/Synch_Reply_Dispatcher.cpp:
+ * tao/GIOP_Message_Base.cpp:
+ * tao/Transport.cpp:
+ * tao/DynamicInterface/DII_Reply_Dispatcher.cpp: Fixed warnings in
+ g++ builds.
+
+Sat Jun 17 17:46:23 2001 Balachandran Natarajan <bala@cs.wustl.edu>
+
+ This set of changes comes with complete revamping of the previous
+ design. The flaws with the previous design were as follows
+ (1) We were unnecessarily penalising large data blocks. We were
+ trying to read a particular size of data till the data was
+ completely removed from the socket. This was totally
+ ridiculous because we were doing more reads than required.
+
+ (2) The message block that was constructed on the stack with a
+ buffer from stack never did what we wanted. It was allocating
+ a data block on the heap and was thus spoiling whatever
+ optimization that we had tried putting in.
+
+ (3) The incoming message Queue is now managed by the TAO_Transport
+ object instead of the GIOP classes.
+
+ * tao/GIOP_Message_Base.cpp:
+ * tao/GIOP_Message_Base.h: Removed the references to the incoming
+ message queue. Implemention for two methods missing_data () and
+ byte_order (). Added an extra argument to the methods
+ process_request_message () and process_reply_message (). Used
+ the incoming message block to create a input CDR with the
+ DONT_DELETE flag so that the data block is not deleted after
+ request processing.
+
+ * tao/GIOP_Message_State.cpp: Removed the inclusion of
+ Transport.h.
+
+ * tao/Incoming_Message_Queue.h:
+ * tao/Incoming_Message_Queue.cpp:
+ * tao/Incoming_Message_Queue.inl: Added an argument to the
+ add_message (). Further the implementation of add_message () has
+ changed a bit. It now adds only a new message to the queue. It
+ doesn't modify a half filled queue. The TAO_Transport object
+ does that job. So declared the TAO_Transport as the friend class
+ of the Incoming_Message_Queue.
+
+ Changed the name of the methods complete_message () as
+ is_complete_message (). Removed the methods current_message ()&
+ current_byte_order ().
+
+ Added a new method copy_message () which copies messages into
+ the half empty nodes.
+
+ Added a method wr_ptr () to access the write pointer of the tail
+ node that has halfempty message.
+
+ * tao/Transport.cpp:
+ * tao/Transport.h: The Incoming Message Queue is now managed by
+ this class. Added the following methods
+ - missing_data ()
+ - parse_incoming_messages ()
+ - check_message_integrity ()
+ - consolidate_message ()
+ - conslodate_message_queue ()
+
+ * tao/Pluggable_Messaging.h: Added two new virtual functions
+ missing_data () and byte_order ().
+
+ * tao/ORB_Core.h:
+ * tao/ORB_Core.cpp:
+ * tao/ORB_Core.i: Added an accessor for the locking_strategy used
+ for the CDR blocks.
+
+ * tao/Synch_Reply_Dispatcher.cpp:
+ * tao/Asynch_Reply_Dispatcher.cpp:
+ * tao/DynamicInterface/DII_Reply_Dispatcher.cpp: Changed the
+ exchange_data_block () to clone_from () which is a new method in
+ ACE_InputCDR.
+
+ * tao/LIST_OF_TODO: Updated the list
+
+Sat Jun 16 15:49:23 2001 Balachandran Natarajan <bala@cs.wustl.edu>
+
+ * tao/Any.cpp:
+ * tao/Asynch_Reply_Dispatcher.cpp:
+ * tao/CDR.cpp:
+ * tao/CDR.h:
+ * tao/CDR.i:
+ * tao/GIOP_Message_Lite.cpp:
+ * tao/IIOP_Profile.cpp:
+ * tao/Invocation.cpp:
+ * tao/ORB.cpp:
+ * tao/Pluggable_Messaging_Utils.cpp:
+ * tao/Synch_Reply_Dispatcher.cpp:
+ * tao/TAO_Server_Request.cpp:
+ * tao/DynamicInterface/DII_Reply_Dispatcher.cpp: Integrated some
+ of the changes from the main trunk in this branch.
+
+Wed Jun 13 17:55:24 2001 Balachandran Natarajan <bala@cs.wustl.edu>
+
+ * tao/Incoming_Message_Queue.cpp:
+ * tao/Incoming_Message_Queue.inl: Fixed a bad way to use the tail
+ of the circular linked list. This is now replaced by the size of
+ the linked list. This fixes quite a few errors.
+
+ * tao/Transport.cpp: On read () if we get errno === EWOULDBLOCK we
+ were closing the connection prematurely. This is not right. We
+ should only return a 0 to the reactor, so that it can call us
+ back when there is data in the socket.
+
+Wed Jun 13 10:45:24 2001 Balachandran Natarajan <bala@cs.wustl.edu>
+
+ * tao/GIOP_Message_Base.cpp (process_reply_message): Fixed the
+ reply parsing and generation of the CDR stream that is passed on
+ to the higher layers of the ORB.
+
+Wed Jun 13 07:25:24 2001 Balachandran Natarajan <bala@cs.wustl.edu>
+
+ * tao/Makefile: Added new files.
+ * tao/Exclusive_TMS.cpp: Removed some commented code.
+
+Tue Jun 12 18:42:55 2001 Balachandran Natarajan <bala@cs.wustl.edu>
+
+ * tao/GIOP_Message_State.h:
+ * tao/GIOP_Message_State.inl: The message_size () now returns the
+ payload size + the GIOP header length. Added a new method
+ payload_size () that returns the payload size alone.
+
+ * tao/GIOP_Message_Base.cpp: Fixed a warning with g++ builds.
+
+ * tao/Connection_Handler.h:
+ * tao/Connection_Handler.cpp: The svc_i () method now calls
+ handle_input_i () on the transport instead of the method in the
+ same class. The handle_input_i () in the TAO_Connection_Handler
+ class has been removed as it is no longer used.
+
+ * tao/IIOP_Connection_Handler.h:
+ * tao/IIOP_Connection_Handler.cpp: Removed the implementation of
+ handle_input_i ().
+
+ * tao/Incoming_Message_Queue.cpp: When trying to make a new data
+ block we dont add the size of GIOP header. The message_size ()
+ now returns with that value.
+
+ * tao/Incoming_Message_Queue.inl: Fixed a link error in g++
+ builds.
+
+Tue Jun 12 17:42:55 2001 Balachandran Natarajan <bala@cs.wustl.edu>
+
+ First set of checkins for big two ways.
+
+ * tao/Connection_Handler.h: Added #define for the default
+ buffer size . Not sure yet whether this is the right place for
+ it.
+
+ * tao/Incoming_Message_Queue.h:
+ * tao/Incoming_Message_Queue.cpp:
+ * tao/Incoming_Message_Queue.inl: Queue up the incoming
+ messages. We form a circular linked list of messages that are
+ bigger than a particular buffer size. During processing we take
+ messages of the queue to pass it onto the higher layers of the
+ ORB.
+
+ * tao/GIOP_Message_Base.cpp:
+ * tao/GIOP_Message_Base.h: Added support for big two way
+ requests. If the message has only been partially read we add the
+ message in the queue and then go for the next read. During
+ processing we check if the queue has messages before processing
+ the message on hand.
+
+ * tao/GIOP_Message_State.h:
+ * tao/GIOP_Message_State.cpp:
+ * tao/GIOP_Message_State.inl: Added accessor methods.
+
+ * tao/Transport.cpp: Added support for bigger two ways. If we
+ receive a two way bigger than a particular size we read and
+ queue the message and do a further read to retrieve the rest of
+ the message.
+
+ * tao/TAO.dsp: Added new files.
+ * tao/default_resource.cpp: For single threaded configuration we
+ have been creating a locked data block and that seems to defy
+ logic. We now should create a lock free datablock for single
+ threaded configurations.
+
+Sat Jun 2 12:02:55 2001 Balachandran Natarajan <bala@cs.wustl.edu>
+
+ * tao/* : Merged with the main trunk.
+
+Fri Jun 1 17:22:29 2001 Balachandran Natarajan <bala@cs.wustl.edu>
+
+ * tao/GIOP_Message_State.cpp (parse_message_header_i):
+ * tao/Connection_Handler.cpp:
+ * tao/GIOP_Message_Base.cpp:
+ * tao/Makefile:
+ * tao/Transport.cpp: Fixed warnings and errors in Linux g++.
+
+Thu Jun 01 13:39:02 2001 Balachandran Natarajan <bala@cs.wustl.edu>
+
+ * tao/Transport.cpp:
+ * tao/Transport.h: Added the following methods -- handle_input_i
+ (), process_parsed_message (). The handle_input_i () creates a
+ buffer on the stack. It uses the buffer to read the
+ messages. Any errors in read () or dealt here directly. The read
+ message is then parsed and processed by the GIOP classes. Moved
+ most of the code for processing here as they seem to be common
+ between different transport protocols. Once we have received a
+ full message, we just resume the handler and go ahead with
+ processing the message.
+
+ * tao/IIOP_Connection_Handler.cpp (handle_input ()): Calls
+ handle_input_i () on the transport.
+
+ * tao/Connection_Handler.h:
+ * tao/Connection_Handler.cpp: The svc_i () calls the
+ handle_input_i () on the transport instead of the same call on
+ the connection handler. This way we should be able to share the
+ same code among different protocol objects.
+
+ * tao/GIOP_Message_State.h:
+ * tao/GIOP_Message_State.cpp: Much of the message parsing and
+ state information is stored here.
+
+ * tao/GIOP_Message_Base.h:
+ * tao/GIOP_Message_Base.cpp:
+ * tao/Pluggable_Messaging.h: made some changes to suit the
+ above. But more changes could come in, to suit different
+ protocols. We may want to change the interfaces in
+ Pluggable_Messaging to give a much cleaner interface to the
+ world.
+
+ * tao/Wait_On_Read.cpp: Used the handle_input_i () on the
+ transport for processing the incoming data.
+
+ * tao/TAO.dsp:
+ * tao/PortableServer/TAO_PortableServer.dsp: Added new files and
+ removed files from GIOP_Lite in the first round.
+
+ * tao/GIOP_Message_Reactive_Handler.cpp:
+ * tao/GIOP_Message_Reactive_Handler.h: Looks like these files
+ would be removed during the final merge.
+
+ The above checkins works for simple two way calls. The
+ BiDirectional_NestedUpcall test works with the TP_Reactor. This
+ works on Win32.
+
+
+Thu May 24 12:19:02 2001 Balachandran Natarajan <bala@cs.wustl.edu>
+
+ * tao/Transport.h: Added some design forces for the input data
+ path. \ No newline at end of file