[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/8] qemu-ga: move channel/transport function
From: |
Michael Roth |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/8] qemu-ga: move channel/transport functionalit |
Date: |
Fri, 03 Feb 2012 11:07:38 -0600 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.24) Gecko/20111109 Thunderbird/3.1.16 |
On 02/02/2012 10:25 PM, MATSUDA, Daiki wrote:
Hi, Michael!
Thank you for your working.
And I have a question the process id written in pid file.
If qemu-ga is ran as daemon, the parent process id not child is written
in pid file. So, id gotten by 'ps' command is different. Is it correct
work? Many other daemon writes child process id.
Regards
MATSUDA Daiki
Hi Matsuda,
Thank you for testing!
In the become_daemon() function, the parent exits immediately after the
fork(), so only the child has the opportunity to write to the pid file.
It calls getpid() to get the pid to write, which should be it's own
lwpid. So I'm not seeing where there's an opportunity for the parent pid
to be written.
Can you confirm? It seems to behave as expected for me:
address@hidden ~]# /home/mdroth/w/qemu-build/qemu-ga -d
** (process:7441): DEBUG: starting daemon
address@hidden ~]# ps aux | grep qemu-ga
root 7442 0.0 0.0 13792 348 ? Ss 10:56 0:00
/home/mdroth/w/qemu-build/qemu-ga -d
root 7471 0.0 0.1 109108 816 pts/2 R+ 11:00 0:00 grep
--color=auto qemu-ga
address@hidden ~]# cat /var/run/qemu-ga.pid
7442
- Re: [Qemu-devel] [PATCH v2 2/8] qemu-ga: move channel/transport functionalit, MATSUDA, Daiki, 2012/02/03
- Re: [Qemu-devel] [PATCH v2 2/8] qemu-ga: move channel/transport functionalit,
Michael Roth <=
- Re: [Qemu-devel] [PATCH v2 2/8] qemu-ga: move channel/transport functionalit, MATSUDA, Daiki, 2012/02/09
- Re: [Qemu-devel] [PATCH v2 2/8] qemu-ga: move channel/transport functionalit, Michael Roth, 2012/02/09
- Re: [Qemu-devel] [PATCH v2 2/8] qemu-ga: move channel/transport functionalit, MATSUDA, Daiki, 2012/02/12
- Re: [Qemu-devel] [PATCH v2 2/8] qemu-ga: move channel/transport functionalit, Michael Roth, 2012/02/13
- Re: [Qemu-devel] [PATCH v2 2/8] qemu-ga: move channel/transport functionalit, MATSUDA, Daiki, 2012/02/13