[openib-general] Re: Latest kdapl load problem

James Lentini jlentini at netapp.com
Thu May 12 12:03:41 PDT 2005


tduffy> On Thu, 2005-05-12 at 12:02 -0400, Hal Rosenstock wrote:
tduffy> > On Thu, 2005-05-12 at 11:52, James Lentini wrote:
tduffy> > > Did you rebuild the kdapltest module?
tduffy> > 
tduffy> > Thanks. I forgot to do that.
tduffy> > 
tduffy> > Does kdapltest build fine for you ? I'm getting some errors on building
tduffy> > the user portion.
tduffy> 
tduffy> Sorry, that was my fault.  Forgot to check that my changes still allowed
tduffy> user to build.
tduffy> 
tduffy> > Modified: gen2/users/jlentini/linux-kernel/test/dapltest/mdep/linux/dapl_mdep_user.h
tduffy> > ===================================================================
tduffy> > --- gen2/users/jlentini/linux-kernel/test/dapltest/mdep/linux/dapl_mdep_user.h  2005-05-12 15:55:55 UTC (rev 2316)
tduffy> > +++ gen2/users/jlentini/linux-kernel/test/dapltest/mdep/linux/dapl_mdep_user.h  2005-05-12 15:58:00 UTC (rev 2317)
tduffy> > @@ -50,7 +50,15 @@
tduffy> >  #endif
tduffy> >  #endif
tduffy> >  
tduffy> > +typedef enum {
tduffy> > +#undef FALSE
tduffy> > +       FALSE = 0,
tduffy> > +#undef TRUE
tduffy> > +       TRUE = 1
tduffy> > +} boolean_t;
tduffy> >  
tduffy> > +typedef uint32_t u32;
tduffy> > +
tduffy> >  /* Default Device Name */
tduffy> >  #define DT_MdepDeviceName    "ia0a"
tduffy> >  
tduffy> > @@ -72,7 +80,7 @@
tduffy> 
tduffy> BTW, u32 should not be used in user stuff.  For that we should use the
tduffy> standard uint32_t.

This is partially a result of the user and kernel portions of 
kdapltest being intertwined. Ultimately I'd like to see the directory 
structure of kdapltest updated to seperate the kernel and user code.

I'll go through the user code and change the u32 to uint32_t.



More information about the general mailing list