[ofa-general] Re: [opensm patch][2/2] verify config inputs when config file is rescanned

Sasha Khapyorsky sashak at voltaire.com
Tue Nov 11 12:26:48 PST 2008


On 13:41 Mon 10 Nov     , Al Chu wrote:
> Hey Sasha,
> 
> Sorry, repost, w/ the right Author.
> 
> Al
> 
> On Mon, 2008-11-10 at 13:15 -0800, Al Chu wrote:
> > On Mon, 2008-11-10 at 23:02 +0200, Sasha Khapyorsky wrote:
> > > Hi Al,
> > > 
> > > On 15:01 Thu 30 Oct     , Al Chu wrote:
> > > > Hey Sasha,
> > > > 
> > > > I noticed that after the config file is rescanned, the new potential
> > > > inputs aren't checked for validity.  Patch is attached.
> > > > 
> > > > Al
> > > > 
> > > > -- 
> > > > Albert Chu
> > > > chu11 at llnl.gov
> > > > Computer Scientist
> > > > High Performance Systems Division
> > > > Lawrence Livermore National Laboratory
> > > 
> > > > From edfcd2de96c3525d1609b4c0f03c17ecc0495c18 Mon Sep 17 00:00:00 2001
> > > > From: root <root at wopri.(none)>
> > > > Date: Thu, 30 Oct 2008 13:58:55 -0700
> > > > Subject: [PATCH] verify rescanned config input
> > > > 
> > > > 
> > > > Signed-off-by: root <root at wopri.(none)>
> > >                  ^^^^^^^^^^^^^^^^^^^^^^^^
> > > 
> > > I'm fine with this patch, but could you fix S-O-B line? Thanks.
> > 
> > Oops.  New one is attached (I'll repost the [1/2] patch too).
> > 
> > Al
> > 
> > > Sasha
> > _______________________________________________
> > general mailing list
> > general at lists.openfabrics.org
> > http:// lists.openfabrics.org/cgi-bin/mailman/listinfo/general
> > 
> > To unsubscribe, please visit http:// openib.org/mailman/listinfo/openib-general
> -- 
> Albert Chu
> chu11 at llnl.gov
> Computer Scientist
> High Performance Systems Division
> Lawrence Livermore National Laboratory

> From a9f7ea0b667ff32a029593e954286c349fe499e7 Mon Sep 17 00:00:00 2001
> From: Albert Chu <chu11 at llnl.gov>
> Date: Mon, 10 Nov 2008 13:10:25 -0800
> Subject: [PATCH] verify rescanned config input
> 
> 
> Signed-off-by: Albert Chu <chu11 at llnl.gov>

Applied. Thanks.

Sasha



More information about the general mailing list