Hi,
that can be a way, but then it is not possible to connect to different servers,
when they have different ports.
e.g. one server for large files, one (intelligent) server for script-files
Why different ports? - For testing it is very nice to have all the servers on
one machine.
Dietrich
-----Original Message-----
Hi,
Am 03.08.2018 um 13:07 schrieb Toews, Dietrich:
> Hi all,
>
> we are using barebox for a while now, but we need different tftp-server with
> different ports.
> Here a proposition for a patch, if someone else needs it also.
>
> Usage: tftp -o port=1234 ....
Good point. I assume, barebox way, would be to make it:
global.net.tftp_port
nv.net.tftp_port
nv.dev.<devname>.tftp_port
> Sincirely
> Dietrich
>
>
> From 062a4c8148c2d31c7ce1d405632051b297b64ce0 Mon Sep 17 00:00:00 2001
> From: Dietrich Toews <dietrich.to...@heidelberg.com>
> Date: Fri, 3 Aug 2018 12:55:53 +0200
> Subject: [PATCH] Added tftp port to fsoptions
>
> ---
> fs/tftp.c | 20 ++++++++++++++++++--
> 1 file changed, 18 insertions(+), 2 deletions(-)
>
> diff --git a/fs/tftp.c b/fs/tftp.c
> index 847921a..13efa20 100644
> --- a/fs/tftp.c
> +++ b/fs/tftp.c
> @@ -87,6 +87,7 @@ struct file_priv {
>
> struct tftp_priv {
> IPaddr_t server;
> + uint16_t port;
> };
>
> static int tftp_create(struct device_d *dev, const char *pathname,
> mode_t mode) @@ -423,7 +424,7 @@ static struct file_priv *tftp_do_open(struct
> device_d *dev,
> goto out;
> }
>
> - priv->tftp_con = net_udp_new(tpriv->server, TFTP_PORT, tftp_handler,
> + priv->tftp_con = net_udp_new(tpriv->server, tpriv->port,
> + tftp_handler,
> priv);
> if (IS_ERR(priv->tftp_con)) {
> ret = PTR_ERR(priv->tftp_con); @@ -651,7 +652,22 @@
> static int tftp_probe(struct device_d *dev)
> dev->priv = priv;
>
> priv->server = resolv(fsdev->backingstore);
> -
> + priv->port = TFTP_PORT;
> + if( fsdev->options ) {
> + // ToDo: if needed check other options.
> + // Assumption here: the only option used is 'port=nxxxx'
> + if( strncmp( fsdev->options, "port=", 5 ) == 0 ) {
> + uint16_t p = 0;
> + for( uint8_t i=5; i<10; i++ ) {
> + // portnumber can only contain ascii chars
> 0..9
> + if( (fsdev->options[i] >= 48) &&
> (fsdev->options[i] < 58) )
> + p = (p * 10) + (fsdev->options[i] -
> 48);
> + else
> + break; // break on first char that
> is not a number
> + }
> + if( p > 0 ) priv->port = p;
> + }
> + }
> return 0;
> }
>
> --
> 1.9.1
>
>
> ________________________________
>
> Confidentiality note:
> The information in this email and any attachment may contain confidential and
> proprietary information of Heidelberger Druckmaschinen AG and/or its
> affiliates and may be privileged or otherwise protected from disclosure. If
> you are not the intended recipient, you are hereby notified that any review,
> reliance or distribution by others or forwarding without express permission
> is strictly prohibited and may cause liability. In case you have received
> this message due to an error in transmission, we kindly ask you to notify the
> sender immediately and to delete this email and any attachment from your
> system.
>
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
>
--
Regards,
Oleksij
________________________________
Confidentiality note:
The information in this email and any attachment may contain confidential and
proprietary information of Heidelberger Druckmaschinen AG and/or its affiliates
and may be privileged or otherwise protected from disclosure. If you are not
the intended recipient, you are hereby notified that any review, reliance or
distribution by others or forwarding without express permission is strictly
prohibited and may cause liability. In case you have received this message due
to an error in transmission, we kindly ask you to notify the sender immediately
and to delete this email and any attachment from your system.
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
