Re: [PATCH 2/5] nbd: Remove signal usage

From	Christoph Hellwig <hch@infradead.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH 2/5] nbd: Remove signal usage
Date	2016-01-03 13:00 +0100
Message-ID	<qMPui-4Z2-3@gated-at.bofh.it> (permalink)
References	<qMrrX-6Qc-3@gated-at.bofh.it> <qMrrY-6Qc-19@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Markus,

this looks great!

Reviewed-by: Christoph Hellwig <hch@lst.de>

One thing I noticed, which might be a good cleanup in the future:

> -	spin_lock_irqsave(&nbd->tasks_lock, flags);
>  	nbd->task_recv = current;
> -	spin_unlock_irqrestore(&nbd->tasks_lock, flags);

It seems like task_{send,recv} are only used for the files showing
the pids.  Maybe you should just store the pids directly in the
nbd_device structure and also keep the sysfs file around for the
life time of that structure.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PULL] NBD changes Markus Pargmann <mpa@pengutronix.de> - 2016-01-02 11:20 +0100
  [PATCH 1/5] nbd: Fix debugfs error handling Markus Pargmann <mpa@pengutronix.de> - 2016-01-02 11:20 +0100
    Re: [PATCH 1/5] nbd: Fix debugfs error handling kbuild test robot <lkp@intel.com> - 2016-01-02 13:00 +0100
    [PATCH] nbd: fix ifnullfree.cocci warnings kbuild test robot <lkp@intel.com> - 2016-01-02 13:00 +0100
  [PATCH 2/5] nbd: Remove signal usage Markus Pargmann <mpa@pengutronix.de> - 2016-01-02 11:20 +0100
    Re: [PATCH 2/5] nbd: Remove signal usage Christoph Hellwig <hch@infradead.org> - 2016-01-03 13:00 +0100

csiph-web