Re: [Xen-devel] [V2] xen-netback notify DomU to send ARP.

[Jason Luan <luanjianhai@xxxxxxx>]

ä 2013å01æ21æ 20:23, Ian Campbell åé:
> Which Linux did you test? pvops or something based on the classic-Xen
> patches?
Which Linux, you meaning is which distribution or which version? I have 
tested Unbreakable Linux
on v2.6.18, v2.6.32(need one frontend patch) and v2.6.39, Ubuntu 11.04 
on v3.2.0-35.
i misunderstand "pvops or something based on the classic-Xen patches?" , 
Would you like to give
me detailed explanation?
> On Mon, 2013-01-21 at 11:55 +0000, Jan Beulich wrote:
> > > > > On 21.01.13 at 08:26, jianhai luan <jianhai.luan@xxxxxxxxxx> wrote:
> > > +static void notify_front_arping(struct xenbus_device *dev)
> > > +{
> > > +       int err;
> > > +
> > > +       if (dev->state != XenbusStateConnected)
> > > +               return;
> > > +
> > > +       err = xenbus_printf(XBT_NIL, dev->nodename, "state", "%d", dev->state);
> > > +       if (err) {
> > > +               pr_fmt("Error writing the state");
> > What's this? pr_fmt() alone makes no sense at all, and I'd be
> > pretty surprised if the compiler didn't warn about this construct.
> >
> > Further, you probably want to say "re-writing" and include the
> > error code in the message. And of course you want a \n at the
> > end.
> >
> > Finally - no need for the braces ...
> >
> > > +       }
> > > +
> > > +       return;
> > ... nor this "return".
> Agreed on all counts.
>
> Jason, when you resend with these fixes please CC the netdev list and
> folks listed by ./scripts/get-maintainers.pl for the patch.
>
> Ian.
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> http://lists.xen.org/xen-devel


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel