Erik Faye-Lund <kusmabite@xxxxxxxxx> writes: >> Hmm, loginfo() calls logreport() and adds the process information as >> necessary to the output. Wouldn't this patch give the pid information >> twice? >> > > Close, but not quite. logreport() reports the current PID, while this > call to loginfo reports the PID of the child process. So two > non-identical PIDs are reported. I know that; I was questioning if that change to the log output is really what we want. I do not deeply care myself, but people with scripts that read logs might. > > The output becomes something like this: > > [6408] [3868] Connection from [::1]:55801 > [3868] Extended attributes (16 bytes) exist <host=localhost> > [3868] Request upload-pack for '/some-repo.git' > [3868] '/some-repo.git' does not appear to be a git repository > [6408] [1876] Connection from [::1]:57311 > [1876] Extended attributes (16 bytes) exist <host=localhost> > [1876] Request upload-pack for '/some-repo.git' > [1876] '/some-repo.git' does not appear to be a git repository -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html