linux-kernel - RE: [EXT] Re: [PATCH] scsi: qedf: Add port

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: <MN2PR18MB2527C8FC1902CA98123E2231D2840@MN2PR18MB2527.namprd18.prod.outlook.com>
Date:   Tue, 24 Sep 2019 09:10:23 +0000
From:   Saurav Kashyap <skashyap@...vell.com>
To:     Daniel Wagner <dwagner@...e.de>
CC:     "QLogic-Storage-Upstream@...ium.com" 
        <QLogic-Storage-Upstream@...ium.com>,
        "linux-scsi@...r.kernel.org" <linux-scsi@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [EXT] Re: [PATCH] scsi: qedf: Add port_id getter

Hi Daniel,

> -----Original Message-----
> From: Daniel Wagner <dwagner@...e.de>
> Sent: Tuesday, September 24, 2019 12:54 PM
> To: Saurav Kashyap <skashyap@...vell.com>
> Cc: QLogic-Storage-Upstream@...ium.com; linux-scsi@...r.kernel.org; linux-
> kernel@...r.kernel.org
> Subject: [EXT] Re: [PATCH] scsi: qedf: Add port_id getter
> 
> External Email
> 
> ----------------------------------------------------------------------
> Hi Saurav,
> 
> On Tue, Sep 24, 2019 at 09:11:38AM +0200, Daniel Wagner wrote:
> > On Tue, Sep 24, 2019 at 06:08:09AM +0000, Saurav Kashyap wrote:
> > > > +static void qedf_get_host_port_id(struct Scsi_Host *shost) {
> > > > +	struct fc_lport *lport = shost_priv(shost);
> > > > +
> > > > +	fc_host_port_id(shost) = lport->port_id; }
> > >
> > > Minor stuff, the closing brace should be in next line. Please submit v2.
> >
> > Oops, sorry about that.
> 
> The patch I sent out had the closing brace on a new
> line:
> 
> https://lore.kernel.org/linux-scsi/20190923103738.67749-1-dwagner@suse.de/
> 
> Now I am a bit confused how I screwed it up. Anyway, I'll send it out
> again after with hexdump there is not a special ASCII character
> hidden :)

Sorry about that, I changed my email client and it remove the extra lines.

Thanks,
~Saurav
> 
> Thanks,
> Daniel