Re: [libvirt PATCH v2 1/4] util: Try to get limits from /proc

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

 



On Mon, 2021-03-15 at 14:28 +0100, Michal Privoznik wrote:
> On 3/15/21 12:22 PM, Andrea Bolognani wrote:
> >       if (virFileReadAllQuiet(procfile, 2048, &buf) < 0)
> > -        return -1;
> > +        goto error;
> 
> virFileReadAllQuiet() sets errno, this would just overwrite it with 
> something less specific.

Good point. I've changed this to

  if (virFileReadAllQuiet(procfile, 2048, &buf) < 0) {
      /* This function already sets errno, so don't overwrite that
       * and return immediately instead */
      return -1;
  }

so that some explanation for the choice is retained.

> >       if (!(lines = g_strsplit(buf, "\n", 0)))
> > -        return -1;
> > +        goto error;
> 
> I think this check can be dropped. g_strsplit() doesn't ever return NULL 
> really, does it?

You're right, it doesn't. I've changed it.

> The rest looks good.

Thanks for the review and for the very good suggestions!

I'll see whether I can get some feedback from KubeVirt developers
regarding whether or not this actually improves things for them
before pushing.

-- 
Andrea Bolognani / Red Hat / Virtualization




[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]

  Powered by Linux