[Devel] [PATCH rh7] ve: introduce ve.veid for setting id from userspace
Cyrill Gorcunov
gorcunov at virtuozzo.com
Sat Jun 20 10:34:51 PDT 2015
On Sat, Jun 20, 2015 at 08:18:42PM +0300, Cyrill Gorcunov wrote:
> On Sat, Jun 20, 2015 at 07:27:30PM +0300, Vladimir Davydov wrote:
> ...
> > +static int ve_id_write(struct cgroup *cg, struct cftype *cft, u64 value)
> > +{
> > + struct ve_struct *ve = cgroup_ve(cg);
> > + int veid;
> > + int err = 0;
> > +
> > + if (value <= 0 || value > INT_MAX)
> > + return -EINVAL;
>
> A small nit -- @value is unsigned no need for <= test.
> Other than that, looks great, thank you!
I mean since 0 is prohibited here, <= won't hurt but
might be a bit confusing.
More information about the Devel
mailing list