On Wed, Oct 02, 2019 at 06:30:48PM +0800, Anand Jain wrote:
> Its very helpful if we had logged the device scanner process name
> to debug the race condition between the systemd-udevd scan and the
> user initiated device forget command.
> 
> This patch adds scanned-by process name to the scan message.

Then PID should be printed as well, otherwise ok.

> 
> Signed-off-by: Anand Jain <anand.j...@oracle.com>
> ---
>  fs/btrfs/volumes.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/fs/btrfs/volumes.c b/fs/btrfs/volumes.c
> index 788271649726..2c4c89bfafd1 100644
> --- a/fs/btrfs/volumes.c
> +++ b/fs/btrfs/volumes.c
> @@ -1011,11 +1011,11 @@ static noinline struct btrfs_device 
> *device_list_add(const char *path,
>               *new_device_added = true;
>  
>               if (disk_super->label[0])
> -                     pr_info("BTRFS: device label %s devid %llu transid %llu 
> %s\n",
> -                             disk_super->label, devid, found_transid, path);
> +                     pr_info("BTRFS: device label %s devid %llu transid %llu 
> %s scanned-by %s\n",
> +                             disk_super->label, devid, found_transid, path, 
> current->comm);
>               else
> -                     pr_info("BTRFS: device fsid %pU devid %llu transid %llu 
> %s\n",
> -                             disk_super->fsid, devid, found_transid, path);
> +                     pr_info("BTRFS: device fsid %pU devid %llu transid %llu 
> %s scanned-by %s\n",

                        pr_info("BTRFS: device fsid %pU devid %llu transid %llu 
%s scanned by %s(%d)\n",

I'm not sure if %di si the right specifier for pid, the format name(pid)
can be seen in other messages.

Reply via email to