Re: [PATCH 01/10] conf: DHCP - add state for DHCP Relay and On/Off switch

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 02/27/2013 07:18 PM, TJ wrote:
> Signed-off-by: TJ <linux@xxxxxx>

Can you use a full legal name, instead of a two-letter pseudonym?

Your commit message is missing some substance, such as a summary of what
is being added.

> ---
>  src/conf/network_conf.h | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/src/conf/network_conf.h b/src/conf/network_conf.h
> index c509915..8400eab 100644
> --- a/src/conf/network_conf.h
> +++ b/src/conf/network_conf.h
> @@ -144,6 +144,10 @@ struct _virNetworkIpDef {
>      char *bootfile;
>      virSocketAddr bootserver;
>  +    /* when false no DHCP server is started */
> +    bool dhcp_enabled;

Your patch is whitespace-damaged.  Did you sent it with 'git send-email'?

> +    /* when true ranges are ignored and a DHCP relay-agent started */
> +    bool dhcp_relay;
>     };
>   typedef struct _virNetworkForwardIfDef virNetworkForwardIfDef;
> @@ -234,6 +238,7 @@ typedef virNetworkObj *virNetworkObjPtr;
>  struct _virNetworkObj {
>      virMutex lock;
>  +    pid_t dhcprelayPid;
>      pid_t dnsmasqPid;

Does your series ever allow dnsmasq and dhcprelay to run at the same
time, or can we use a single pid_t field that covers the mutually
exclusive choice of which helper is running based on the rest of the config?

>      pid_t radvdPid;
>      unsigned int active : 1;

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]