<font face="Default Sans Serif,Verdana,Arial,Helvetica,sans-serif" size="2"><br><br><font color="#990099">-----Andrew Jeffery <<a target="_blank" href="mailto:andrew@aj.id.au">andrew@aj.id.au</a>> wrote: -----</font><br><br>>To: OpenBMC Patches <<a target="_blank" href="mailto:openbmc-patches@stwcx.xyz">openbmc-patches@stwcx.xyz</a>>,<br>><a target="_blank" href="mailto:openbmc@lists.ozlabs.org">openbmc@lists.ozlabs.org</a><br>>From: Andrew Jeffery <<a target="_blank" href="mailto:andrew@aj.id.au">andrew@aj.id.au</a>><br>>Date: 02/24/2016 10:14PM<br>>Cc: Milton Miller II/Austin/IBM@IBMUS<br>>Subject: Re: [PATCH openbmc 9/9] initfs: Use stty to drain output<br>>before reboot<br>><br>>On Wed, 2016-02-24 at 12:40 -0600, OpenBMC Patches wrote:<br>>> From: "Milton D. Miller II" <<a target="_blank" href="mailto:miltonm@us.ibm.com">miltonm@us.ibm.com</a>><br>>> <br>>> To see all the messages during shutdown call stty cooked.<br>>> <br>>> A side effect of setting any mode is it will call ioctl(TIOC_DRAIN)<br>><br>>So due to past indiscretions and increasing my own understanding I<br>>tried to follow this up. Seems that:<br>><br>>* TIOC_DRAIN doesn't appear to be a thing?<br>>* Google knows more about TIOCDRAIN, but not a lot?<br>>* TCSADRAIN might be more useful to mention here?<br>><br>>If I were to follow it up the TIOC_DRAIN , where should I look?<br>><br>>Should this part of the commit message be changed?<br><br>Yes it should have, but while you raised it on irc last night I didn't <br>read this message until several hours after I told Norm to merge <br>the code.  Sorry.<br><br>><br>>However, I've followed the busybox stty implementation to the point<br>>of<br>>calling tcsetattr(..., TCSADRAIN, ...), so the change looks like it<br>>should do what we want. Also you've documented testing it below, so<br>>that helps :)<br>><br><br>Yes. I also looked at the code to figure out what was needed.<br><br>>> to drain the buffers first without regard to any change in the<br>>mode.<br>>> <br>>> Shutdown and flash update now show the final ouptut<br>><br>>typo: output<br>><br><br>Yes that it is :-).<br><br>>Otherwise,<br>><br>>Acked-by: Andrew Jeffery <<a target="_blank" href="mailto:andrew@aj.id.au">andrew@aj.id.au</a>><br>><br>>>  and "Rebooting"<br>>> message before the reboot system call.<br>>> <br><br>Milton<br></font><BR>