Bug#782400: multipath-tools: libmultipath: fix discovery of devices with empty rev sysfs attribute

2015-08-11 Thread Ritesh Raj Sarraf
On Tue, 2015-08-11 at 07:53 -0300, Mauricio Faria de Oliveira wrote: Sure, thanks. BTW, Hannes authored himself a patch like the one proposed [1], but it seems not posted to dm-devel (I didn't notice it earlier). I'd expect this to give some reassurance that something like it is more

Bug#782400: multipath-tools: libmultipath: fix discovery of devices with empty rev sysfs attribute

2015-08-11 Thread Mauricio Faria de Oliveira
Hi Ritesh, On 08/10/2015 09:04 AM, Ritesh Raj Sarraf wrote: I'll look into this soon. But lowering the severity because, from the description, it is again specific to one family of controllers. This week, I intend to work on both the patches. Sure, thanks. BTW, Hannes authored himself a

Bug#782400: multipath-tools: libmultipath: fix discovery of devices with empty rev sysfs attribute

2015-08-10 Thread Ritesh Raj Sarraf
Control: severity -1 important HI Mauricio, On Mon, 2015-06-22 at 08:33 -0300, Mauricio Faria de Oliveira wrote: On 06/21/2015 06:16 AM, Ritesh Raj Sarraf wrote: Given that mulitpath doesn't do a release very often, I plan on rebasing on the latest git repo (which includes many of

Bug#782400: multipath-tools: libmultipath: fix discovery of devices with empty rev sysfs attribute

2015-06-22 Thread Mauricio Faria de Oliveira
Hi Ritesh, On 06/21/2015 06:16 AM, Ritesh Raj Sarraf wrote: Given that mulitpath doesn't do a release very often, I plan on rebasing on the latest git repo (which includes many of Hannes' fixes). They only reason I'm procrastinating is because neither of our patches have been accepted/pushed by

Bug#782400: multipath-tools: libmultipath: fix discovery of devices with empty rev sysfs attribute

2015-06-21 Thread Ritesh Raj Sarraf
On Thursday 28 May 2015 05:31 PM, Mauricio Faria de Oliveira wrote: These days I don't work on storage, i.e. don't have a team that spends the time on these tools. So I keep the upstream first policy, for now. Even my patch (kpartx) is very trivial but 've put it on hold until Christoph

Bug#782400: multipath-tools: libmultipath: fix discovery of devices with empty rev sysfs attribute

2015-05-28 Thread Ritesh Raj Sarraf
On Thursday 28 May 2015 12:37 AM, Mauricio Faria de Oliveira wrote: Thanks! Do you think it would be a problem to include the patch? As explained the commit message, I think it's an obvious fix for the regression introduced (with 'chomp', or similar name) for the case when the sysfs

Bug#782400: multipath-tools: libmultipath: fix discovery of devices with empty rev sysfs attribute

2015-05-28 Thread Mauricio Faria de Oliveira
On Thu, 2015-05-28 at 12:21 +0530, Ritesh Raj Sarraf wrote: No. It has nothing to do with the patch. These days I don't work on storage, i.e. don't have a team that spends the time on these tools. So I keep the upstream first policy, for now. Even my patch (kpartx) is very trivial but 've

Bug#782400: multipath-tools: libmultipath: fix discovery of devices with empty rev sysfs attribute

2015-05-27 Thread Ritesh Raj Sarraf
On Wednesday 22 April 2015 05:12 PM, Mauricio Faria de Oliveira wrote: Hey Ritesh, On 04/21/2015 10:23 AM, Ritesh Raj Sarraf wrote: Is this been merged upstream ? No, I haven't seen any reply to it on dm-devel nor commits. I have the impression there's some delay between those 2 points.

Bug#782400: multipath-tools: libmultipath: fix discovery of devices with empty rev sysfs attribute

2015-05-27 Thread Mauricio Faria de Oliveira
Hi Ritesh, On Wed, 2015-05-27 at 23:40 +0530, Ritesh Raj Sarraf wrote: OKay!! THanks. I'll put this one on hold then. The other one, is already ready for Unstable. And as I write, I'm also preparing the same for Jessie+U1. Thanks! Do you think it would be a problem to include the patch? As

Bug#782400: multipath-tools: libmultipath: fix discovery of devices with empty rev sysfs attribute

2015-04-22 Thread Mauricio Faria de Oliveira
Hey Ritesh, On 04/21/2015 10:23 AM, Ritesh Raj Sarraf wrote: Is this been merged upstream ? No, I haven't seen any reply to it on dm-devel nor commits. I have the impression there's some delay between those 2 points. I just pinged Hannes on the mailing list, in case he has some time to

Bug#782400: multipath-tools: libmultipath: fix discovery of devices with empty rev sysfs attribute

2015-04-21 Thread Ritesh Raj Sarraf
On Monday 13 April 2015 05:52 AM, Mauricio Faria de Oliveira wrote: Oops, please use this version (-v2). When the forwarded patch is applied in the source package, the change ends up in cciss_sysfs_pathinfo() rather than scsi_sysfs_pathinfo(): Hunk #1 succeeded at 783 (offset -312

Bug#782400: multipath-tools: libmultipath: fix discovery of devices with empty rev sysfs attribute

2015-04-12 Thread Mauricio Faria de Oliveira
Oops, please use this version (-v2). When the forwarded patch is applied in the source package, the change ends up in cciss_sysfs_pathinfo() rather than scsi_sysfs_pathinfo(): Hunk #1 succeeded at 783 (offset -312 lines). This version has the change in the right place, plus quilt refresh.

Bug#782400: multipath-tools: libmultipath: fix discovery of devices with empty rev sysfs attribute

2015-04-11 Thread Mauricio Faria de Oliveira
Package: src:multipath-tools Version: 0.5.0-6 Severity: serious Tags: patch Forwarded: https://www.redhat.com/archives/dm-devel/2015-April/msg00049.html Hi Ritesh, I noticed some IPR controllers with multipath support may have only spaces in the rev sysfs attribute, which breaks a device's

Bug#782400: multipath-tools: libmultipath: fix discovery of devices with empty rev sysfs attribute

2015-04-11 Thread Mauricio Faria de Oliveira
On 04/11/2015 01:00 PM, Mauricio Faria de Oliveira wrote: Tags: patch It helps to attach the patch :) -- Mauricio Faria de Oliveira IBM Linux Technology Center diff -Nru multipath-tools-0.5.0/debian/changelog multipath-tools-0.5.0/debian/changelog --- multipath-tools-0.5.0/debian/changelog