Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 30 Jun 2009 16:40:09 -0600 (MDT)
From:      "M. Warner Losh" <imp@bsdimp.com>
To:        marc@msys.ch
Cc:        svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, mbr@FreeBSD.org, src-committers@FreeBSD.org
Subject:   Re: svn commit: r195200 - in head/usr.sbin: . wake
Message-ID:  <20090630.164009.2130804684.imp@bsdimp.com>
In-Reply-To: <7B9C309F-63E3-4CB8-9871-DE2DEE010096@msys.ch>
References:  <0E6D4FB2-A485-40ED-A856-ACC311A90EFE@msys.ch> <20090630.141340.289711551.imp@bsdimp.com> <7B9C309F-63E3-4CB8-9871-DE2DEE010096@msys.ch>

next in thread | previous in thread | raw e-mail | index | archive | help
In message: <7B9C309F-63E3-4CB8-9871-DE2DEE010096@msys.ch>
            Marc Balmer <marc@msys.ch> writes:
: 
: Am 30.06.2009 um 22:13 schrieb M. Warner Losh:
: 
: > In message: <0E6D4FB2-A485-40ED-A856-ACC311A90EFE@msys.ch>
: >            Marc Balmer <marc@msys.ch> writes:
: > :
: > : Am 30.06.2009 um 21:36 schrieb M. Warner Losh:
: > :
: > : > wake really is too generic a name for this.  Why didn't the wol  
: > port
: > : > get committed anyway, it seems to be better than this...
: > :
: > : wake is a short, mnemonic and imperative name that describes what  
: > the
: > : command does. It is exactly the same command as in NetBSD.
: >
: > And the discussions in the NetBSD src discussion mailing lists have
: > resolved the issue with finality?
: 
: well, yes.  the name stays,  as martin put it so correctly "the one  
: who does
: the work chooses the name".  And we use this name since 2006 on various
: OSs.  And I think it is not a weak name, I can not think of any other  
: sensible
: use of this command name, really.

That's a crock.  You're rushing things into FreeBSD before they have
even been settled in NetBSD using NetBSD as a justification?  I
wouldn't call that playing well with others.  Especially since there
was no socialization of this issue within FreeBSD before the commit.
I didn't see any discussion about this at all.  You can't dictate to
the community: you have to convince them through sound technical
argument.  One that you've not made.

I hate to be blunt, but this leaves a bad taste in everybody's mouth,
even if at the end of the day you get your way.

Warner



: > Warner
: >
: >
: > : > Warner
: > : >
: > : >
: > : > In message: <200906301851.n5UIpNJQ089171@svn.freebsd.org>
: > : >            Martin Blapp <mbr@FreeBSD.org> writes:
: > : > : Author: mbr
: > : > : Date: Tue Jun 30 18:51:22 2009
: > : > : New Revision: 195200
: > : > : URL: http://svn.freebsd.org/changeset/base/195200
: > : > :
: > : > : Log:
: > : > :   Add wake, a tool to send Wake on LAN frames to hosts on a  
: > local
: > : > Ethernet network
: > : > :
: > : > :   Submitted by:   Marc Balmer <marc@msys.ch>
: > : > :   Reviewed by:    rwatson
: > : > :   Approved by:	re
: > : > :
: > : > :   M    usr.sbin/Makefile
: > : > :   A    usr.sbin/wake
: > : > :   AM   usr.sbin/wake/wake.c
: > : > :   AM   usr.sbin/wake/Makefile
: > : > :   AM   usr.sbin/wake/wake.8
: > : > :
: > : > : Added:
: > : > :   head/usr.sbin/wake/
: > : > :   head/usr.sbin/wake/Makefile   (contents, props changed)
: > : > :   head/usr.sbin/wake/wake.8   (contents, props changed)
: > : > :   head/usr.sbin/wake/wake.c   (contents, props changed)
: > : > : Modified:
: > : > :   head/usr.sbin/Makefile
: > : > :
: > : > : Modified: head/usr.sbin/Makefile
: > : > :
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : >  
: > ======================================================================
: > : > : --- head/usr.sbin/Makefile	Tue Jun 30 17:27:53 2009	(r195199)
: > : > : +++ head/usr.sbin/Makefile	Tue Jun 30 18:51:22 2009	(r195200)
: > : > : @@ -190,6 +190,7 @@ SUBDIR=	${_ac} \
: > : > :  	${_usbconfig} \
: > : > :  	${_vidcontrol} \
: > : > :  	vipw \
: > : > : +	wake \
: > : > :  	watch \
: > : > :  	watchdogd \
: > : > :  	${_wlandebug} \
: > : > :
: > : > : Added: head/usr.sbin/wake/Makefile
: > : > :
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : >  
: > ======================================================================
: > : > : --- /dev/null	00:00:00 1970	(empty, because file is newly added)
: > : > : +++ head/usr.sbin/wake/Makefile	Tue Jun 30 18:51:22 2009	 
: > (r195200)
: > : > : @@ -0,0 +1,8 @@
: > : > : +# $FreeBSD$
: > : > : +
: > : > : +PROG=	wake
: > : > : +MAN=	wake.8
: > : > : +
: > : > : +WARNS?= 2
: > : > : +
: > : > : +.include <bsd.prog.mk>
: > : > :
: > : > : Added: head/usr.sbin/wake/wake.8
: > : > :
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : >  
: > ======================================================================
: > : > : --- /dev/null	00:00:00 1970	(empty, because file is newly added)
: > : > : +++ head/usr.sbin/wake/wake.8	Tue Jun 30 18:51:22 2009	(r195200)
: > : > : @@ -0,0 +1,61 @@
: > : > : +.\"
: > : > : +.\" $FreeBSD$
: > : > : +.\"
: > : > : +.\" Copyright (c) 2009 Marc Balmer <marc@msys.ch>
: > : > : +.\"
: > : > : +.\" Permission to use, copy, modify, and distribute this  
: > software
: > : > for any
: > : > : +.\" purpose with or without fee is hereby granted, provided  
: > that
: > : > the above
: > : > : +.\" copyright notice and this permission notice appear in all
: > : > copies.
: > : > : +.\"
: > : > : +.\" THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS  
: > ALL
: > : > WARRANTIES
: > : > : +.\" WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED  
: > WARRANTIES
: > : > OF
: > : > : +.\" MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR  
: > BE
: > : > LIABLE FOR
: > : > : +.\" ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR
: > : > ANY DAMAGES
: > : > : +.\" WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS,
: > : > WHETHER IN AN
: > : > : +.\" ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION,
: > : > ARISING OUT OF
: > : > : +.\" OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS  
: > SOFTWARE.
: > : > : +.\"
: > : > : +.Dd June 27 2009
: > : > : +.Dt WAKE 8
: > : > : +.Os
: > : > : +.Sh NAME
: > : > : +.Nm wake
: > : > : +.Nd send Wake on LAN frames to hosts on a local Ethernet  
: > network
: > : > : +.Sh SYNOPSIS
: > : > : +.Nm
: > : > : +.Ar interface
: > : > : +.Op Ar lladdr ...
: > : > : +.Sh DESCRIPTION
: > : > : +The
: > : > : +.Nm
: > : > : +program is used to send Wake on LAN (WoL) frames over a local
: > : > : +Ethernet network to one or more hosts using their link layer
: > : > (hardware)
: > : > : +addresses.
: > : > : +WoL functionality is generally enabled in a machine's BIOS
: > : > : +and can be used to power on machines from a remote system  
: > without
: > : > : +having physical access to them.
: > : > : +.Pp
: > : > : +.Ar interface
: > : > : +is the network interface of the local machine.
: > : > : +.Ar lladdr
: > : > : +are the link layer addresses of the remote machines
: > : > : +and can be specified as the actual hardware address
: > : > : +(six hexadecimal numbers separated by colons)
: > : > : +or a hostname entry in
: > : > : +.Pa /etc/ethers .
: > : > : +Link layer addresses can be determined and set on
: > : > : +.Fx
: > : > : +machines using
: > : > : +.Xr ifconfig 8 .
: > : > : +.Sh FILES
: > : > : +.Bl -tag -width "/etc/ethers" -compact
: > : > : +.It /etc/ethers
: > : > : +Ethernet host name database.
: > : > : +.El
: > : > : +.Sh SEE ALSO
: > : > : +.Xr ethers 5 ,
: > : > : +.Xr ifconfig 8
: > : > : +.Sh AUTHORS
: > : > : +.Nm
: > : > : +was written by
: > : > : +.Ar Marc Balmer Aq marc@msys.ch .
: > : > :
: > : > : Added: head/usr.sbin/wake/wake.c
: > : > :
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : > =
: > : >  
: > ======================================================================
: > : > : --- /dev/null	00:00:00 1970	(empty, because file is newly added)
: > : > : +++ head/usr.sbin/wake/wake.c	Tue Jun 30 18:51:22 2009	(r195200)
: > : > : @@ -0,0 +1,174 @@
: > : > : +/*
: > : > : + * Copyright (C) 2006, 2007, 2008, 2009 Marc Balmer <marc@msys.ch 
: > >
: > : > : + * Copyright (C) 2000 Eugene M. Kim.  All rights reserved.
: > : > : + *
: > : > : + * Redistribution and use in source and binary forms, with or
: > : > without
: > : > : + * modification, are permitted provided that the following
: > : > conditions
: > : > : + * are met:
: > : > : + *
: > : > : + * 1. Redistributions of source code must retain the above
: > : > copyright
: > : > : + *    notice, this list of conditions and the following  
: > disclaimer.
: > : > : + * 2. Author's name may not be used endorse or promote products
: > : > derived
: > : > : + *    from this software without specific prior written  
: > permission.
: > : > : + *
: > : > : + * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY
: > : > EXPRESS OR
: > : > : + * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE  
: > IMPLIED
: > : > : + * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
: > : > PURPOSE ARE
: > : > : + * DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY
: > : > DIRECT,
: > : > : + * INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
: > : > DAMAGES
: > : > : + * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE
: > : > GOODS OR
: > : > : + * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
: > : > INTERRUPTION)
: > : > : + * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
: > : > CONTRACT,
: > : > : + * STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR  
: > OTHERWISE)
: > : > ARISING
: > : > : + * IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF  
: > ADVISED OF
: > : > THE
: > : > : + * POSSIBILITY OF SUCH DAMAGE.
: > : > : + */
: > : > : +
: > : > : +#include <sys/cdefs.h>
: > : > : +__FBSDID("$FreeBSD$");
: > : > : +
: > : > : +#include <sys/types.h>
: > : > : +#include <sys/param.h>
: > : > : +#include <sys/queue.h>
: > : > : +#include <sys/ioctl.h>
: > : > : +#include <sys/socket.h>
: > : > : +#include <sys/time.h>
: > : > : +#include <net/bpf.h>
: > : > : +#include <net/if.h>
: > : > : +
: > : > : +#include <netinet/in.h>
: > : > : +#include <netinet/if_ether.h>
: > : > : +
: > : > : +#include <err.h>
: > : > : +#include <errno.h>
: > : > : +#include <fcntl.h>
: > : > : +#include <limits.h>
: > : > : +#include <stdarg.h>
: > : > : +#include <stdio.h>
: > : > : +#include <stdlib.h>
: > : > : +#include <string.h>
: > : > : +#include <sysexits.h>
: > : > : +#include <unistd.h>
: > : > : +
: > : > : +#define _PATH_BPF	"/dev/bpf"
: > : > : +
: > : > : +#ifndef SYNC_LEN
: > : > : +#define SYNC_LEN 6
: > : > : +#endif
: > : > : +
: > : > : +#ifndef DESTADDR_COUNT
: > : > : +#define DESTADDR_COUNT 16
: > : > : +#endif
: > : > : +
: > : > : +void usage(void);
: > : > : +
: > : > : +int wake(const char *iface, const char *host);
: > : > : +int bind_if_to_bpf(char const *ifname, int bpf);
: > : > : +int get_ether(char const *text, struct ether_addr *addr);
: > : > : +int send_wakeup(int bpf, struct ether_addr const *addr);
: > : > : +
: > : > : +void
: > : > : +usage(void)
: > : > : +{
: > : > : +	(void)fprintf(stderr, "usage: wake interface lladdr\n");
: > : > : +	exit(0);
: > : > : +}
: > : > : +
: > : > : +int
: > : > : +wake(const char *iface, const char *host)
: > : > : +{
: > : > : +	int res, bpf;
: > : > : +	struct ether_addr macaddr;
: > : > : +
: > : > : +	bpf = open(_PATH_BPF, O_RDWR);
: > : > : +	if (bpf == -1) {
: > : > : +		printf("no bpf\n");
: > : > : +		return -1;
: > : > : +	}
: > : > : +	if (bind_if_to_bpf(iface, bpf) == -1 ||
: > : > : +	    get_ether(host, &macaddr) == -1) {
: > : > : +		(void)close(bpf);
: > : > : +		return -1;
: > : > : +	}
: > : > : +	res = send_wakeup(bpf, &macaddr);
: > : > : +	(void)close(bpf);
: > : > : +	return res;
: > : > : +}
: > : > : +
: > : > : +int
: > : > : +bind_if_to_bpf(char const *ifname, int bpf)
: > : > : +{
: > : > : +	struct ifreq ifr;
: > : > : +	u_int dlt;
: > : > : +
: > : > : +	if (strlcpy(ifr.ifr_name, ifname, sizeof(ifr.ifr_name)) >=
: > : > : +	    sizeof(ifr.ifr_name))
: > : > : +		return -1;
: > : > : +	if (ioctl(bpf, BIOCSETIF, &ifr) == -1)
: > : > : +		return -1;
: > : > : +	if (ioctl(bpf, BIOCGDLT, &dlt) == -1)
: > : > : +		return -1;
: > : > : +	if (dlt != DLT_EN10MB)
: > : > : +		return -1;
: > : > : +	return 0;
: > : > : +}
: > : > : +
: > : > : +int
: > : > : +get_ether(char const *text, struct ether_addr *addr)
: > : > : +{
: > : > : +	struct ether_addr *paddr;
: > : > : +
: > : > : +	paddr = ether_aton(text);
: > : > : +	if (paddr != NULL) {
: > : > : +		*addr = *paddr;
: > : > : +		return 0;
: > : > : +	}
: > : > : +	if (ether_hostton(text, addr))
: > : > : +		return -1;
: > : > : +	return 0;
: > : > : +}
: > : > : +
: > : > : +int
: > : > : +send_wakeup(int bpf, struct ether_addr const *addr)
: > : > : +{
: > : > : +	struct {
: > : > : +		struct ether_header hdr;
: > : > : +		u_char data[SYNC_LEN + ETHER_ADDR_LEN * DESTADDR_COUNT];
: > : > : +	} pkt;
: > : > : +	u_char *p;
: > : > : +	int i;
: > : > : +	ssize_t bw;
: > : > : +	ssize_t len;
: > : > : +
: > : > : +	(void)memset(pkt.hdr.ether_dhost, 0xff,
: > : > sizeof(pkt.hdr.ether_dhost));
: > : > : +	pkt.hdr.ether_type = htons(0);
: > : > : +	(void)memset(pkt.data, 0xff, SYNC_LEN);
: > : > : +	for (p = pkt.data + SYNC_LEN, i = 0; i < DESTADDR_COUNT;
: > : > : +	    p += ETHER_ADDR_LEN, i++)
: > : > : +		bcopy(addr->octet, p, ETHER_ADDR_LEN);
: > : > : +	p = (u_char *)&pkt;
: > : > : +	len = sizeof(pkt);
: > : > : +	bw = 0;
: > : > : +	while (len) {
: > : > : +		if ((bw = write(bpf, &pkt, sizeof(pkt))) == -1)
: > : > : +			return -1;
: > : > : +		len -= bw;
: > : > : +		p += bw;
: > : > : +	}
: > : > : +	return 0;
: > : > : +}
: > : > : +
: > : > : +int
: > : > : +main(int argc, char *argv[])
: > : > : +{
: > : > : +	int n;
: > : > : +
: > : > : +	if (argc < 3)
: > : > : +		usage();
: > : > : +
: > : > : +	for (n = 2; n < argc; n++)
: > : > : +		if (wake(argv[1], argv[n]))
: > : > : +			warnx("error sending Wake on LAN frame over %s to %s",
: > : > : +			    argv[1], argv[n]);
: > : > : +	return 0;
: > : > : +}
: > : > :
: > : > _______________________________________________
: > : > svn-src-all@freebsd.org mailing list
: > : > http://lists.freebsd.org/mailman/listinfo/svn-src-all
: > : > To unsubscribe, send any mail to "svn-src-all-unsubscribe@freebsd.org 
: > "
: > :
: > :
: 
: 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20090630.164009.2130804684.imp>