Re: [Intel-wired-lan] [PATCH net] iavf: missing unlocks in iavf_watchdog_task()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
Subject: Re: [Intel-wired-lan] [PATCH net] iavf: missing unlocks in iavf_watchdog_task()
From: Paul Menzel <pmenzel@xxxxxxxxxxxxx>
Date: Wed, 10 Nov 2021 09:53:50 +0100
Cc: Jakub Pawlak <jakub.pawlak@xxxxxxxxx>, netdev@xxxxxxxxxxxxxxx, kernel-janitors@xxxxxxxxxxxxxxx, Jakub Kicinski <kuba@xxxxxxxxxx>, intel-wired-lan@xxxxxxxxxxxxxxxx, "David S. Miller" <davem@xxxxxxxxxxxxx>, Jesse Brandeburg <jesse.brandeburg@xxxxxxxxx>, Mateusz Palczewski <mateusz.palczewski@xxxxxxxxx>
In-reply-to: <20211110081350.GI5176@kili>
References: <20211110081350.GI5176@kili>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.3.0

Dear Dan,


Thank you for your patch.

For the future, just a nit for the commit message summary. Could youmake it a statement by adding a verb (in imperative mood) [1].


> iavf: Add missing unlocks in iavf_watchdog_task()


Kind regards,

Paul


[1]: https://chris.beams.io/posts/git-commit/

Follow-Ups:
- Re: [Intel-wired-lan] [PATCH net] iavf: missing unlocks in iavf_watchdog_task()
  - From: Dan Carpenter

References:
- [PATCH net] iavf: missing unlocks in iavf_watchdog_task()
  - From: Dan Carpenter

Prev by Date: RE: [PATCH] remoteproc: imx_rproc: Fix a resource leak in the remove function
Next by Date: RE: [PATCH] remoteproc: imx_rproc: Fix a resource leak in the remove function
Previous by thread: [PATCH net] iavf: missing unlocks in iavf_watchdog_task()
Next by thread: Re: [Intel-wired-lan] [PATCH net] iavf: missing unlocks in iavf_watchdog_task()
Index(es):
- Date
- Thread

[Index of Archives] [Kernel Development] [Kernel Announce] [Kernel Newbies] [Linux Networking Development] [Share Photos] [IDE] [Security] [Git] [Netfilter] [Yosemite News] [MIPS Linux] [ARM Linux] [Device Mapper]

Powered by Linux