[ofa-general] Re: [PATCH 2/2] IB/iser: add backport & kernel addons foropen-iscsiover iSER support for RHAS4 up3 and up4

Michael S. Tsirkin mst at dev.mellanox.co.il
Tue May 29 07:11:43 PDT 2007


> Quoting Erez Zilber <erezz at voltaire.com>:
> Subject: Re: [PATCH 2/2] IB/iser: add backport & kernel addons?foropen-iscsiover iSER support for RHAS4 up3 and up4
> 
> 
> >> I have the following files in backport/2.6.9_UX/include/src/:
> >>
> >> attribute_container.c - almost identical to the file on 2.6.20. I had to change one line in it.
> >>     
> >
> > could be a patch ...
> > which line?
> >
> >   
> Now, attribute_container.c, klist.c & transport_class.c are copied from
> the kernel tree. I've committed the required changes in
> ~erezz/ofabuild_iser_rh4.git & ~erezz/ofed_1_2_iser_rh4.git.

git fetch git://git.openfabrics.org/~erezz/ofed_1_2_iser_rh4.git.
fatal: The remote end hung up unexpectedly
Cannot get the repository state from
git://git.openfabrics.org/~erezz/ofed_1_2_iser_rh4.git.


> The main
> change is a new dir called "kernel_addons_patches". It contains patches
> for kernel tree files in order to create the required addons from them.

sorry, but I really don't think we can touch build scripts at this point.
Doing cp in build scripts is also a problem since it interferes with
development (there are 2 places to edit each file).
And adding kernel version dependency there is also really messy.

Suggestion: why can't these patches be part of the regular backport directory?

you copy stuff to include/src and then include it, but this just looks
like and unnecessary extra step. Can't we include the source file from
it original place directory, like this:
#include "../drivers/base/attribute_container.c"

> The rest of the files that I added to include/src are very small. I hope
> it's ok now.

Yes, the rest looks OK to me.

-- 
MST



More information about the general mailing list