[SLOF] [PATCH v2 10/10] obp-tftp: Remove the ciregs-buffer

Alexey Kardashevskiy aik at ozlabs.ru
Mon Oct 10 14:06:56 AEDT 2016


On 20/09/16 05:08, Thomas Huth wrote:
> Since we're not messing up with the client interface now anymore,

Out of curiosity - what was messing with CI and why/when did this stop?


> we don't need to buffer the register contents of the client anymore.
> 
> Signed-off-by: Thomas Huth <thuth at redhat.com>
> ---
>  slof/fs/packages/obp-tftp.fs | 11 -----------
>  1 file changed, 11 deletions(-)
> 
> diff --git a/slof/fs/packages/obp-tftp.fs b/slof/fs/packages/obp-tftp.fs
> index 2e6bcfc..63171d0 100644
> --- a/slof/fs/packages/obp-tftp.fs
> +++ b/slof/fs/packages/obp-tftp.fs
> @@ -14,18 +14,11 @@ s" obp-tftp" device-name
>  
>  VARIABLE huge-tftp-load 1 huge-tftp-load !
>  
> -INSTANCE VARIABLE ciregs-buffer
> -
>  : open ( -- okay? ) 
> -    ciregs-size alloc-mem ciregs-buffer ! 
>      true
>  ;
>  
>  : load ( addr -- size )
> -
> -    \ Save old client interface register 
> -    ciregs ciregs-buffer @ ciregs-size move
> -
>      s" bootargs" get-chosen 0= IF 0 0 THEN >r >r
>      s" bootpath" get-chosen 0= IF 0 0 THEN >r >r
>  
> @@ -42,9 +35,6 @@ INSTANCE VARIABLE ciregs-buffer
>      my-args
>      net-load dup 0< IF drop 0 THEN
>  
> -    \ Restore to old client interface register 
> -    ciregs-buffer @ ciregs ciregs-size move
> -
>      \ Recover buffer address of BOOTP-REPLY packet
>      r>
>  
> @@ -59,7 +49,6 @@ INSTANCE VARIABLE ciregs-buffer
>  ;
>  
>  : close ( -- )
> -   ciregs-buffer @ ciregs-size free-mem 
>  ;
>  
>  : ping  ( -- )
> 


-- 
Alexey


More information about the SLOF mailing list