<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=ks_c_5601-1987">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:Gulim;
        panose-1:2 11 6 0 0 1 1 1 1 1;}
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:"Malgun Gothic";
        panose-1:2 11 5 3 2 0 0 2 0 4;}
@font-face
        {font-family:"\@Malgun Gothic";}
@font-face
        {font-family:"\@Gulim";
        panose-1:2 11 6 0 0 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;
        mso-fareast-language:KO;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph
        {mso-style-priority:34;
        margin-top:0in;
        margin-right:0in;
        margin-bottom:0in;
        margin-left:.5in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;
        mso-fareast-language:KO;}
p.msonormal0, li.msonormal0, div.msonormal0
        {mso-style-name:msonormal;
        mso-margin-top-alt:auto;
        margin-right:0in;
        mso-margin-bottom-alt:auto;
        margin-left:0in;
        font-size:12.0pt;
        font-family:"Times New Roman",serif;
        mso-fareast-language:KO;}
span.EmailStyle19
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
span.EmailStyle20
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
span.EmailStyle21
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
span.EmailStyle22
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
span.EmailStyle23
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
span.EmailStyle24
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
span.EmailStyle25
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
span.EmailStyle26
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
span.EmailStyle27
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
span.EmailStyle28
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
span.EmailStyle29
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
span.EmailStyle30
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
span.EmailStyle31
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
span.EmailStyle32
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
span.EmailStyle33
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
span.EmailStyle34
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
span.EmailStyle35
        {mso-style-type:personal;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
span.EmailStyle36
        {mso-style-type:personal;
        font-family:"Malgun Gothic",sans-serif;
        color:#1F497D;}
span.EmailStyle37
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="color:#1F497D;mso-fareast-language:EN-US">Hi, Cheol.  On point (1), I think the current text is better, as it is simpler and more general: we really want to say that the value of the checksum is zero if it cannot be calculated,
 no matter where the calculation is attempted (i.e., either at the sending end, for insertion into the EOF PDU, or at the receiving end for validation of the checksum in the EOF PDU).<o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D;mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D;mso-fareast-language:EN-US">On point (2), the reason to use 0 to indicate ¡°modular checksum¡± is to sustain backward compatibility with the original CFDP specification.  We want implementations of the old and new
 specifications to be able to interoperate without error, and the CFDPv1 revisions have so far been designed to achieve that.  If the new checksum type field occupies space that was defined as ¡°reserved, all zeros¡± in the original spec then a cfdpv1 entity
 receiving data from a cfdpv0 entity will use the correct checksum algorithm provided we define checksum ID zero to mean ¡°modular checksum¡±.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D;mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D;mso-fareast-language:EN-US">Scott<o:p></o:p></span></p>
<p class="MsoNormal"><span style="color:#1F497D;mso-fareast-language:EN-US"><o:p> </o:p></span></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b>From:</b> <span lang="KO" style="font-family:"Gulim",serif">
±¸Ã¶È¸</span> <chkoo@kari.re.kr> <br>
<b>Sent:</b> Sunday, May 26, 2019 6:43 PM<br>
<b>To:</b> sis-cfdpv1@mailman.ccsds.org; Burleigh, Scott C (312B) <scott.c.burleigh@jpl.nasa.gov><br>
<b>Subject:</b> [EXTERNAL] RE: updated CFDP Revisions draft<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">Hi Scott,<o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">Thank you for the changes. Please find my comments as follows;<o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">1) Section 4.1.2<o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">Is it more clear when target PDU format is specified? i.e.<o:p></o:p></span></p>
<p class="MsoNormal" style="text-autospace:none"><span style="font-size:12.0pt;font-family:"Times New Roman",serif">When an entity is required to perform checksum calculation of a given type but is for any reason unable to do so, the value of the checksum shall
 be zero.</span><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D"><o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">to<o:p></o:p></span></p>
<p class="MsoNormal" style="text-autospace:none"><span style="font-size:12.0pt;font-family:"Times New Roman",serif">When an entity is required to perform checksum calculation of a given type but is for any reason unable to do so, the value of the
<span style="color:red">file </span>checksum <span style="color:red">in the EOF PDU</span> shall be zero.<o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:12.0pt;font-family:"Times New Roman",serif"><o:p> </o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">Or I would think it is helpful if receiving entity is able to know that the file checksum in the EOF PDU to be received will
 be zero because in that case receiving entity don<span lang="KO">¡¯</span>t have to calculate checksum incrementally, which I believe some agencies perform it for time saving because it could be an issue when big file is transferred. So would it make sense?<o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:12.0pt;font-family:"Times New Roman",serif"><o:p> </o:p></span></p>
<p class="MsoNormal" style="text-autospace:none"><span style="font-size:12.0pt;font-family:"Times New Roman",serif">When an entity is required to perform checksum calculation of a given type but is for any reason unable to do so, the value of the checksum shall
 be zero.</span><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D"><o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">to<o:p></o:p></span></p>
<p class="MsoNormal" style="text-autospace:none"><span style="font-size:12.0pt;font-family:"Times New Roman",serif">When an entity is required to perform checksum calculation of a given type but is for any reason unable to do so,
<span style="color:red">the checksum type in the Metadata PDU and the value of the file checksum in the EOF PDU shall be zero.</span></span><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D"><o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">If so, the default checksum type for modular checksum should be
<span lang="KO">¡®</span>1<span lang="KO">¡¯</span> or so.<o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">2) Section 5.2.5<o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">I think that 6 for the length of checksum type is too long and some space could be requested for future use. In my opinion,
 for interoperable operation an applicable checksum algorithm should be selectable among 2 or 3, which each agency should implement as minimum and mandatory.<o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">FYI, for example checksum type could be 3 bit for length and value are as follows;<o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">   - 0 (no checksum type or checksum operation is not applicable at current transaction)  ; I think at the last CCSDS meetings,
 we discussed the checksum type of zero (0) would be no checksum, 1 for modular checksum, 2 for CRC32 and so on.<o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">   - 1 (CFDP mandatory checksum type #1)<o:p></o:p></span></p>
<p class="MsoNormal" style="text-indent:10.0pt;word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">- 2 (CFDP mandatory checksum type #2)<o:p></o:p></span></p>
<p class="MsoNormal" style="text-indent:10.0pt;word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">- 3 (CFDP mandatory checksum type #3)<o:p></o:p></span></p>
<p class="MsoNormal" style="text-indent:10.0pt;word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">- 4 ~ 7 (CFDP user defined checksum type #1 ~ #4)<o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">Best,<o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D">Cheol<o:p></o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D"><o:p> </o:p></span></p>
<p class="MsoNormal" style="word-break:break-all"><span style="font-size:10.0pt;font-family:"Malgun Gothic",sans-serif;color:#1F497D"><o:p> </o:p></span></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b>From:</b> SIS-CFDPV1 <<a href="mailto:sis-cfdpv1-bounces@mailman.ccsds.org">sis-cfdpv1-bounces@mailman.ccsds.org</a>>
<b>On Behalf Of </b>Burleigh, Scott C (312B) via SIS-CFDPV1<br>
<b>Sent:</b> Saturday, May 25, 2019 5:17 AM<br>
<b>To:</b> <a href="mailto:sis-cfdpv1@mailman.ccsds.org">sis-cfdpv1@mailman.ccsds.org</a><br>
<b>Cc:</b> <a href="mailto:osvaldo.peinado@dlr.de">osvaldo.peinado@dlr.de</a>; <a href="mailto:madalier@antarateknik.com">
madalier@antarateknik.com</a>; <a href="mailto:jens.janssen@dlr.de">jens.janssen@dlr.de</a>; Thomas Gannett <<a href="mailto:thomas.gannett@tgannett.net">thomas.gannett@tgannett.net</a>>;
<a href="mailto:Jeremy.Mayer@dlr.de">Jeremy.Mayer@dlr.de</a>; Cola, Tomaso (<a href="mailto:Tomaso.deCola@dlr.de">Tomaso.deCola@dlr.de</a>) <<a href="mailto:Tomaso.deCola@dlr.de">Tomaso.deCola@dlr.de</a>><br>
<b>Subject:</b> [SIS-CFDPV1] updated CFDP Revisions draft<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Hi, all.  I have made all the changes to the revised CFDP Blue Book that I think are needed in order to make the file checksum algorithm selectable by reference to the new SANA checksum algorithm registry.  The updated document is now in
 CWE at <a href="https://cwe.ccsds.org/sis/docs/SIS-CFDPV1/Draft%20Documents/727x0b5%20--%20Specification%20--%20Blue%20Book/727x0p42_working%202.doc?Web=1">
https://cwe.ccsds.org/sis/docs/SIS-CFDPV1/Draft%20Documents/727x0b5%20--%20Specification%20--%20Blue%20Book/727x0p42_working%202.doc?Web=1</a>.  Please take a look at it when you have a moment; I¡¯d like to send a final draft to Tom and ask for a supplementary
 Agency review soon, maybe as early as next month.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Scott<o:p></o:p></p>
</div>
</body>
</html>