From: "Tomáš Golembiovský" <tgolembi@redhat.com>
To: Eric Blake <eblake@redhat.com>
Cc: qemu-devel@nongnu.org,
"Marc-André Lureau" <marcandre.lureau@redhat.com>,
"Michael Roth" <mdroth@linux.vnet.ibm.com>
Subject: Re: [Qemu-devel] [PATCH v2 4/4] qga: report serial ID and device node
Date: Thu, 9 Aug 2018 13:50:03 +0200 [thread overview]
Message-ID: <20180809135003.75fe8fa3@fiorina> (raw)
In-Reply-To: <e91a6eb7-d5b2-93e6-05bf-65270ccb8736@redhat.com>
On Tue, 7 Aug 2018 08:52:37 -0500
Eric Blake <eblake@redhat.com> wrote:
> On 08/07/2018 05:51 AM, Tomáš Golembiovský wrote:
> > On Linux the functionality depends on libudev.
> >
> > Example from Linux:
> >
> > {
> > "name": "dm-2",
> > "mountpoint": "/",
> > ...
> > "disk": [
> > {
> > "serial": "SAMSUNG_MZ7LN512HCHP-000L1_S1ZKNXAG822493",
> > "dev": "/dev/sda2",
> > ...
> > }
> > ],
> > }
> >
> > Signed-off-by: Tomáš Golembiovský <tgolembi@redhat.com>
> > ---
>
> > +++ b/qga/commands-posix.c
> > @@ -47,6 +47,7 @@ extern char **environ;
> > #include <sys/socket.h>
> > #include <net/if.h>
> > #include <sys/statvfs.h>
> > +#include <libudev.h>
>
> Is libudev universally available on BSD systems and always preinstalled
> on Linux systems, or does this need a configure probe?
The code is specific to Linux. I couldn't find portable way of querying
the serial number.
libudev is part of systemd, so it should generally be there. But I
suppose we should not break on systems that don't use it. I'll add
configure check for it.
Tomas
--
Tomáš Golembiovský <tgolembi@redhat.com>
prev parent reply other threads:[~2018-08-09 11:50 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-07 10:51 [Qemu-devel] [PATCH v2 0/4] qga: report serial number and disk node Tomáš Golembiovský
2018-08-07 10:51 ` [Qemu-devel] [PATCH v2 1/4] build: rename CONFIG_QGA_NTDDDISK to CONFIG_QGA_NTDDSCSI Tomáš Golembiovský
2018-08-07 10:51 ` [Qemu-devel] [PATCH v2 2/4] qga: win32: add debugging information Tomáš Golembiovský
2018-08-07 10:51 ` [Qemu-devel] [PATCH v2 3/4] qga: win32: fix crashes when PCI info cannot be retrived Tomáš Golembiovský
2018-09-05 23:21 ` Michael Roth
2018-09-07 11:24 ` Tomáš Golembiovský
2018-08-07 10:51 ` [Qemu-devel] [PATCH v2 4/4] qga: report serial ID and device node Tomáš Golembiovský
2018-08-07 13:52 ` Eric Blake
2018-08-09 11:50 ` Tomáš Golembiovský [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180809135003.75fe8fa3@fiorina \
--to=tgolembi@redhat.com \
--cc=eblake@redhat.com \
--cc=marcandre.lureau@redhat.com \
--cc=mdroth@linux.vnet.ibm.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.