ea/04c7df7027a81c4b051592f7ebd16f4befc0e0

   1 Return-Path: <tomi.ollila@nixu.com>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5         by olra.theworths.org (Postfix) with ESMTP id 6AABE431FD0\r
   6         for <notmuch@notmuchmail.org>; Fri, 23 Dec 2011 04:31:03 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id 9lhO1VQ57QtB for <notmuch@notmuchmail.org>;\r
  16         Fri, 23 Dec 2011 04:31:01 -0800 (PST)\r
  17 Received: from mail-gw3.nixu.fi (mail-gw3.nixu.fi [193.209.237.7])\r
  18         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 050EB431FB6\r
  21         for <notmuch@notmuchmail.org>; Fri, 23 Dec 2011 04:31:00 -0800 (PST)\r
  22 Received: from pps.filterd (mail-gw3 [127.0.0.1])\r
  23         by mail-gw3.nixu.fi (8.14.4/8.14.4) with SMTP id pBNCUKhf017815;\r
  24         Fri, 23 Dec 2011 14:30:52 +0200\r
  25 Received: from taco2.nixu.fi (taco2.nixu.fi [194.197.118.31])\r
  26         by mail-gw3.nixu.fi with ESMTP id 114cs0v5nn-1\r
  27         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NOT);\r
  28         Fri, 23 Dec 2011 14:30:52 +0200\r
  29 Received: from taco2.nixu.fi (taco2.nixu.fi [194.197.118.31])\r
  30         by taco2.nixu.fi (8.14.3/8.14.3/Debian-5+lenny1) with ESMTP id\r
  31         pBNCUoUY026817; Fri, 23 Dec 2011 14:30:51 +0200\r
  32 From: Tomi Ollila <tomi.ollila@iki.fi>\r
  33 To: David Edmondson <dme@dme.org>, Austin Clements <amdragon@MIT.EDU>,\r
  34         notmuch@notmuchmail.org\r
  35 Subject: Re: [PATCH] Properly handle short writes in sigint handlers\r
  36 In-Reply-To: <cunhb0rvhc6.fsf@hotblack-desiato.hh.sledj.net>\r
  37 References: <20111222201553.GK10376@mit.edu>\r
  38         <1324584948-8009-1-git-send-email-amdragon@mit.edu>\r
  39         <cunhb0rvhc6.fsf@hotblack-desiato.hh.sledj.net>\r
  40 User-Agent: Notmuch/0.10.2+123~g49af79c (http://notmuchmail.org) Emacs/23.3.1\r
  41         (i686-pc-linux-gnu)\r
  42 X-Face: HhBM'cA~<r"^Xv\KRN0P{vn'Y"Kd;zg_y3S[4)KSN~s?O\"QPoL\r
  43         $[Xv_BD:i/F$WiEWax}R(MPS`^UaptOGD`*/=@\1lKoVa9tnrg0TW?"r7aRtgk[F\r
  44         !)g;OY^,BjTbr)Np:%c_o'jj,Z\r
  45 Date: Fri, 23 Dec 2011 14:30:50 +0200\r
  46 Message-ID: <yf6fwgba2rp.fsf@taco2.nixu.fi>\r
  47 MIME-Version: 1.0\r
  48 Content-Type: text/plain; charset=us-ascii\r
  49 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.5.7110,\r
  50  1.0.211,       0.0.0000        definitions=2011-12-23_03:2011-12-23, 2011-12-23,\r
  51         1970-01-01 signatures=0\r
  52 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 spamscore=0\r
  53         ipscore=0 suspectscore=0\r
  54         phishscore=0 bulkscore=0 adultscore=0 classifier=spam adjust=0\r
  55         reason=mlx\r
  56         scancount=1 engine=6.0.2-1012030000 definitions=main-1112230066\r
  57 X-BeenThere: notmuch@notmuchmail.org\r
  58 X-Mailman-Version: 2.1.13\r
  59 Precedence: list\r
  60 List-Id: "Use and development of the notmuch mail system."\r
  61         <notmuch.notmuchmail.org>\r
  62 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  63         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  64 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  65 List-Post: <mailto:notmuch@notmuchmail.org>\r
  66 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  67 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  68         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  69 X-List-Received-Date: Fri, 23 Dec 2011 12:31:03 -0000\r
  70 \r
  71 On Fri, 23 Dec 2011 08:10:33 +0000, David Edmondson <dme@dme.org> wrote:\r
  72 > Sorry for being slow.\r
  73 > \r
  74 > Can you describe the situation in which you expect a write to stderr to\r
  75 > be a short write? (Without error.)\r
  76 \r
  77 In the following hypothetical case (correct me if I'm wrong :):\r
  78 \r
  79 * There is 4096 byte buffer in tty driver.\r
  80 * Stderr is in blocking-mode (the usual case).\r
  81 * There is already 4090 bytes in that buffer that has not been read.\r
  82 * One attemtps to write "Stopping...         \n" there (blocks).\r
  83 * Somehow the system call is interrupted (and SA_RESTART not set)\r
  84   -- write() should return 6 bytes written.\r
  85 \r
  86 But, if the buffer is full already, does the write() system call return\r
  87 with -1 and EINTR set ?\r
  88 \r
  89 If there is enough space for all data in that buffer to begin with, \r
  90 write() should be atomic.\r
  91 \r
  92 > In that situation, what guarantee is there that the loop you've written\r
  93 > will terminate?\r
  94 \r
  95 If write() keeps returning 0 then it will not terminate (I guess this never\r
  96 happens). Also, it never terminates if write blocks indefinitely \r
  97 (with or without that loop).\r
  98 \r
  99 > We're not talking about safeguarding a users' data here - this is a\r
 100 > short message to indicate that a tool is terminating due to a signal.\r
 101 > I'm concerned that the solution is worse than the problem.\r
 102 \r
 103 I'm also in favor of "opportunistic" write *in this particular case*\r
 104 \r
 105 In case that write fails there is most probably more serious things going\r
 106 on (all resources eaten, hardware problem, etc) and trying to push these\r
 107 writes forward doesn't help.\r
 108 \r
 109 Tomi\r