Re: [PATCH 1/1] Drivers: scsi: storvsc: Get rid of warning messages

2014-09-18 Thread Olaf Hering
On Wed, Sep 17, Christoph Hellwig wrote: Also the two patches at http://thread.gmane.org/gmane.linux.drivers.driver-project.devel/56242/ Wasnt the outcome that this was a bad idea? Or at least doing it globally is bad. In any case, its not something I can help with. Olaf

Re: [PATCH 1/1] Drivers: scsi: storvsc: Get rid of warning messages

2014-09-18 Thread Olaf Hering
On Tue, Sep 02, K. Y. Srinivasan wrote: Get rid of the warning messages since they will clutter up various system logs and are of questionable value to the end user. For debugging purposes, this information can be gotten by setting the scsi log level appropriately. Signed-off-by: K. Y.

Re: [PATCH 1/1] Drivers: scsi: storvsc: Get rid of warning messages

2014-09-18 Thread Christoph Hellwig
On Thu, Sep 18, 2014 at 01:50:17PM +0200, Olaf Hering wrote: On Wed, Sep 17, Christoph Hellwig wrote: Also the two patches at http://thread.gmane.org/gmane.linux.drivers.driver-project.devel/56242/ Wasnt the outcome that this was a bad idea? Or at least doing it globally is bad. In any

Re: [PATCH 1/1] Drivers: scsi: storvsc: Get rid of warning messages

2014-09-17 Thread Olaf Hering
On Mon, Sep 08, KY Srinivasan wrote: -Original Message- From: Christoph Hellwig [mailto:h...@infradead.org] Sent: Thursday, September 4, 2014 10:40 PM To: KY Srinivasan Subject: Re: [PATCH 1/1] Drivers: scsi: storvsc: Get rid of warning messages Looks good to me. Olaf

Re: [PATCH 1/1] Drivers: scsi: storvsc: Get rid of warning messages

2014-09-17 Thread Christoph Hellwig
On Wed, Sep 17, 2014 at 09:33:24AM +0200, Olaf Hering wrote: Christoph, what does review of this mean? There are many hv related patches floating around. The removal of the warning in msgid 1409701396-20226-1-git-send-email-...@microsoft.com looks ok to me. Thanks, can you give me a

RE: [PATCH 1/1] Drivers: scsi: storvsc: Get rid of warning messages

2014-09-08 Thread KY Srinivasan
: [PATCH 1/1] Drivers: scsi: storvsc: Get rid of warning messages Looks good to me. Olaf, Hannes - can I get another review for this (and the older hyperv scanning patch set)? Olaf, Hannes, Ping. K. Y ___ devel mailing list de

Re: [PATCH 1/1] Drivers: scsi: storvsc: Get rid of warning messages

2014-09-05 Thread Sitsofe Wheeler
On Thu, Sep 04, 2014 at 10:40:14PM -0700, Christoph Hellwig wrote: Looks good to me. Olaf, Hannes - can I get another review for this (and the older hyperv scanning patch set)? I agree this looks useful because on a 59753a805499f1ffbca4ac0a24b3dff67bf1 3.17rc2 kernel with 92578ea

Re: [PATCH 1/1] Drivers: scsi: storvsc: Get rid of warning messages

2014-09-04 Thread Christoph Hellwig
Looks good to me. Olaf, Hannes - can I get another review for this (and the older hyperv scanning patch set)? ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel