linux-kernel - Re: [PATCH 04/10] mpt3sas: Fix removal and addition of vSES device during host reset

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1507742391.3168.7.camel@HansenPartnership.com>
Date:   Wed, 11 Oct 2017 10:19:51 -0700
From:   James Bottomley <James.Bottomley@...senPartnership.com>
To:     Tomas Henzl <thenzl@...hat.com>,
        Sreekanth Reddy <sreekanth.reddy@...adcom.com>,
        linux-scsi@...r.kernel.org, hch@...radead.org
Cc:     "Martin K. Petersen" <martin.petersen@...cle.com>,
        Sathya.Prakash@...adcom.com, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 04/10] mpt3sas: Fix removal and addition of vSES device
 during host reset

On Wed, 2017-10-11 at 18:12 +0200, Tomas Henzl wrote:
> On 10/11/2017 05:56 PM, James Bottomley wrote:
> > 
> > On Wed, 2017-10-11 at 17:35 +0200, Tomas Henzl wrote:
> > > 
> > > On 10/10/2017 03:11 PM, Sreekanth Reddy wrote:
> > > > 
> > > > For Dev Handles who value is less than hba's phys count number
> > > >  driver will return HBA sas address value as a sas address.
> > > > So for Virtual SES device also driver was returning HBA sas
> > > > address
> > > > instead
> > > >  of Virtual SES sas address. So now updated the driver to
> > > > return
> > > >  Virtual SES's sas address for Virtual SES device instead of
> > > >  HBA's sas address.
> > > > 
> > > > Signed-off-by: Sreekanth Reddy <Sreekanth.Reddy@...adcom.com>
> > > Signed-off-by: Tomas Henzl <thenzl@...hat.com>
> > You mean Reviewed-by: I think?
> 
> Sure, sorry for that, corrected too late in 8/10
> should I resend with  "Reviewed-by" ?

It's probably OK, but please do so if there's another resend.
 Patchwork blindly captures what it sees, so the application has to be
hand modified to change Signed-off-by to Reviewed-by even if you resend
with Reviewed-by.

James