Запуск параллельного Task из другого Task.

[New man]

Собственно, есть у меня приложение, которое выступает в качестве сервера. Есть некоторая куча клиентов, которые подключаются к нему по TCP и отсылают каждый как минимум раз в секунду хертбит (сигнал с некоторой информацией о своём состоянии).

Насколько нормально обработку хертбитов сделать в виде асинхронного таска, который после обработки каждого хертбита запускает самого себя?
Может, есть какие-то грабли?

Собственно, тут отмечено:

    class Program
    {
        public delegate void MessageReceivedHandler(object sender, EventArgs args);
 
        public event MessageReceivedHandler OnReceived;
        public event MessageReceivedHandler OnLost;
 
        private TcpListener listener;
 
        private void Work()
        {
            listener = TcpListener.Create(9000);
            listener.Start();
            while (true)
            {
                TcpClient client = listener.AcceptTcpClient();
                Task.Run(async () => await HandleClient(client));
            }
        }
 
        private long call_count;
 
        private async Task HandleClient(TcpClient client)
        {
            try
            {
                byte[] buffer = new byte[1024];
                int read = await client.GetStream().ReadAsync(buffer, 0, buffer.Length);
                OnReceived?.Invoke(this, new EventArgs());
                await client.GetStream().WriteAsync(buffer, 0, read);
                Task.Run(async () => await HandleClient(client));   // Насколько вот это хорошо?
            }
            catch
            {
                OnLost?.Invoke(this, null);
                client?.Close();
            }
        }
 
        static void Main(string[] args)
        {
            new Program().Work();
        }
    }

Второе: что поменяется, если я изменю эту строчку вот так?

await Task.Run(async () => await HandleClient(client));

[New man]

Я понял, что можно было написать так:

        private async Task HandleClient(TcpClient client)
        {
            try
            {
                byte[] buffer = new byte[1024];
                while (true)
                {
                    int read = await client.GetStream().ReadAsync(buffer, 0, buffer.Length);
                    OnReceived?.Invoke(this, new EventArgs());
                    await client.GetStream().WriteAsync(buffer, 0, read);
                }
            }
            catch
            {
                OnLost?.Invoke(this, null);
                client?.Close();
            }
        }

Этот вариант потратил в 2 раза меньше Total CPU time, если верить профилировщику.