Subject: Re: bug#55153: 29.0.50; Implement system_process_attributes on Cygwin

From: Ken Brown <[email-masked]>
Date: Wed, 27 Apr 2022 12:45:45 -0400

On 4/27/2022 11:53 AM, Eli Zaretskii wrote:
>> Date: Wed, 27 Apr 2022 11:16:46 -0400
>> From: Ken Brown <[email-masked]>
>>
>> +# ifdef CYGWIN
>> +  /* ttname */
>> +  strcpy (procfn_end, "/ctty");
>> +  fd = emacs_open (fn, O_RDONLY, 0);
>> +  if (fd < 0)
>> +    nread = 0;
>> +  else
>> +    {
>> +      record_unwind_protect_int (close_file_unwind, fd);
>> +      nread = emacs_read_quit (fd, procbuf, sizeof procbuf);
>> +    }
>> +  /* /proc/<pid>/ctty should always end in newline. */
>> +  if (0 < nread && procbuf[nread - 1] == '\n')
>> +    procbuf[nread - 1] = '\0';
>> +  else
>> +    procbuf[0] = '\0';
>> +  attrs = Fcons (Fcons (Qttname, build_string (procbuf)), attrs);
> 
> Is what you read from /proc/<pid>/ctty guaranteed to be pure-ASCII
> string?

Yes.  It's typically something like "/dev/pty0".

> P.S. Doesn't this warrant a NEWS entry?

Revised patch attached, with a NEWS entry.

Ken

From 5af483b3adcd475a2d544c8f23462d90055498c0 Mon Sep 17 00:00:00 2001
From: Ken Brown <[email-masked]>
Date: Wed, 27 Apr 2022 10:46:57 -0400
Subject: [PATCH] Implement system_process_attributes on Cygwin

* src/sysdep.c (system_process_attributes) [CYGWIN]: Implement,
using the /proc filesystem.  The code is identical to the
GNU/Linux code except for the 'ttname' attribute.  (Bug#55153)

* etc/NEWS: Mention the change.
---
 etc/NEWS     |  5 +++++
 src/sysdep.c | 26 +++++++++++++++++++++++++-
 2 files changed, 30 insertions(+), 1 deletion(-)

diff --git a/etc/NEWS b/etc/NEWS
index 2ecad81b11..335a55e656 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -2149,6 +2149,11 @@ where those APIs are available.
 When 'w32-use-native-image-API' is non-nil, Emacs on MS-Windows now
 has built-in support for displaying BMP images.
 
+** Cygwin
+
+---
+*** 'process-attributes' is now implemented.
+
 
 ----------------------------------------------------------------------
 This file is part of GNU Emacs.
diff --git a/src/sysdep.c b/src/sysdep.c
index 9c1e59c02b..95295e7e67 100644
--- a/src/sysdep.c
+++ b/src/sysdep.c
@@ -3193,7 +3193,7 @@ make_lisp_timeval (struct timeval t)
 
 #endif
 
-#ifdef GNU_LINUX
+#if defined (GNU_LINUX) || defined (CYGWIN)
 
 static Lisp_Object
 time_from_jiffies (unsigned long long ticks, Lisp_Object hz, Lisp_Object form)
@@ -3241,6 +3241,7 @@ get_up_time (void)
   return up;
 }
 
+# ifdef GNU_LINUX
 #define MAJOR(d) (((unsigned)(d) >> 8) & 0xfff)
 #define MINOR(d) (((unsigned)(d) & 0xff) | (((unsigned)(d) & 0xfff00000) >> 12))
 
@@ -3286,6 +3287,7 @@ procfs_ttyname (int rdev)
   unblock_input ();
   return build_string (name);
 }
+# endif	/* GNU_LINUX */
 
 static uintmax_t
 procfs_get_total_memory (void)
@@ -3434,7 +3436,9 @@ system_process_attributes (Lisp_Object pid)
 	  attrs = Fcons (Fcons (Qppid, INT_TO_INTEGER (ppid)), attrs);
 	  attrs = Fcons (Fcons (Qpgrp, INT_TO_INTEGER (pgrp)), attrs);
 	  attrs = Fcons (Fcons (Qsess, INT_TO_INTEGER (sess)), attrs);
+# ifdef GNU_LINUX
 	  attrs = Fcons (Fcons (Qttname, procfs_ttyname (tty)), attrs);
+# endif
 	  attrs = Fcons (Fcons (Qtpgid, INT_TO_INTEGER (tpgid)), attrs);
 	  attrs = Fcons (Fcons (Qminflt, INT_TO_INTEGER (minflt)), attrs);
 	  attrs = Fcons (Fcons (Qmajflt, INT_TO_INTEGER (majflt)), attrs);
@@ -3483,6 +3487,26 @@ system_process_attributes (Lisp_Object pid)
     }
   unbind_to (count, Qnil);
 
+# ifdef CYGWIN
+  /* ttname */
+  strcpy (procfn_end, "/ctty");
+  fd = emacs_open (fn, O_RDONLY, 0);
+  if (fd < 0)
+    nread = 0;
+  else
+    {
+      record_unwind_protect_int (close_file_unwind, fd);
+      nread = emacs_read_quit (fd, procbuf, sizeof procbuf);
+    }
+  /* /proc/<pid>/ctty should always end in newline. */
+  if (0 < nread && procbuf[nread - 1] == '\n')
+    procbuf[nread - 1] = '\0';
+  else
+    procbuf[0] = '\0';
+  attrs = Fcons (Fcons (Qttname, build_string (procbuf)), attrs);
+  unbind_to (count, Qnil);
+# endif	/* CYGWIN */
+
   /* args */
   strcpy (procfn_end, "/cmdline");
   fd = emacs_open (fn, O_RDONLY, 0);
-- 
2.36.0