<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p><br>
</p>
<br>
<div class="moz-cite-prefix">On 29/12/17 03:31, Yan Vugenfirer
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:39FFFDD6-D32B-47C1-B657-CC43E078DD62@redhat.com">
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<br class="">
<div><br class="">
<blockquote type="cite" class="">
<div class="">On 28 Dec 2017, at 16:44, Amnon Ilan <<a
href="mailto:ailan@redhat.com" class=""
moz-do-not-send="true">ailan@redhat.com</a>> wrote:</div>
<br class="Apple-interchange-newline">
<div class=""><br style="font-family: Helvetica; font-size:
12px; font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px;" class="">
<br style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px; float: none; display:
inline !important;" class="">----- Original Message -----</span><br
style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px;" class="">
<blockquote type="cite" style="font-family: Helvetica;
font-size: 12px; font-style: normal; font-variant-caps:
normal; font-weight: normal; letter-spacing: normal;
orphans: auto; text-align: start; text-indent: 0px;
text-transform: none; white-space: normal; widows: auto;
word-spacing: 0px; -webkit-text-size-adjust: auto;
-webkit-text-stroke-width: 0px;" class="">From: "Lev
Veyde" <<a href="mailto:lveyde@redhat.com" class=""
moz-do-not-send="true">lveyde@redhat.com</a>><br
class="">
To: "Yedidyah Bar David" <<a
href="mailto:didi@redhat.com" class=""
moz-do-not-send="true">didi@redhat.com</a>><br
class="">
Cc: "Frediano Ziglio" <<a
href="mailto:fziglio@redhat.com" class=""
moz-do-not-send="true">fziglio@redhat.com</a>>,
"Christophe Fergeau" <<a
href="mailto:cfergeau@redhat.com" class=""
moz-do-not-send="true">cfergeau@redhat.com</a>>,<br
class="">
<a href="mailto:spice-devel@lists.freedesktop.org"
class="" moz-do-not-send="true">spice-devel@lists.freedesktop.org</a>,
"Amnon Ilan" <<a href="mailto:ailan@redhat.com"
class="" moz-do-not-send="true">ailan@redhat.com</a>><br
class="">
Sent: Thursday, December 28, 2017 1:13:24 PM<br class="">
Subject: Re: [Spice-devel] [nsis v3 2/2] build: Don't add
.pdb debug files to the installer<br class="">
<br class="">
Hi Didi (and the guys),<br class="">
<br class="">
We don't ship debug information in our d/s WGT, and
personally I'm not sure<br class="">
how useful it really may be, e.g. if the customers will
get BSOD if that<br class="">
will show any more useful information.<br class="">
<br class="">
Luckily our VirtIO-Win drivers are quite stable, so we
don't have many such<br class="">
issues anyhow.<br class="">
But I think it's best to see what VirtIO-Win guys think
about it.<br class="">
Amnon, what is your view on this?<br class="">
</blockquote>
<br style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px; float: none; display:
inline !important;" class="">I do not think we need to
install pdb files, but should have it accessible<span
class="Apple-converted-space"> </span></span><br
style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px; float: none; display:
inline !important;" class="">somewhere.</span><br
style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px;" class="">
<br style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px; float: none; display:
inline !important;" class="">Vadim, Yan, can you comment
on that?</span><br style="font-family: Helvetica;
font-size: 12px; font-style: normal; font-variant-caps:
normal; font-weight: normal; letter-spacing: normal;
text-align: start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px;" class="">
</div>
</blockquote>
<div><br class="">
</div>
Hello,</div>
<div><br class="">
</div>
<div>I think they should be kept together with build artefacts.
When we get bug reports from the field we use pdb files
distributed with specific build ISO.</div>
<div>But if you repackage the virtio-win package into
RHV-guest-tools installer, then I don’t see specific need to
keep PDB files as long as we can clearly connect between
RHV-guest-tools version and virtio-win version. <br>
</div>
</blockquote>
If we want to let end users running live or postmortem debug session
on their own, then shipping .pdb files is a must.<br>
<br>
Vadim.<br>
<blockquote type="cite"
cite="mid:39FFFDD6-D32B-47C1-B657-CC43E078DD62@redhat.com">
<div><br class="">
</div>
<div>Best regards,</div>
<div>Yan.<br class="">
<blockquote type="cite" class="">
<div class=""><br style="font-family: Helvetica; font-size:
12px; font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px; float: none; display:
inline !important;" class="">Thanks,</span><br
style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px;" class="">
<span style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px; float: none; display:
inline !important;" class="">Amnon</span><br
style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px;" class="">
<br style="font-family: Helvetica; font-size: 12px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal; text-align:
start; text-indent: 0px; text-transform: none;
white-space: normal; word-spacing: 0px;
-webkit-text-stroke-width: 0px;" class="">
<blockquote type="cite" style="font-family: Helvetica;
font-size: 12px; font-style: normal; font-variant-caps:
normal; font-weight: normal; letter-spacing: normal;
orphans: auto; text-align: start; text-indent: 0px;
text-transform: none; white-space: normal; widows: auto;
word-spacing: 0px; -webkit-text-size-adjust: auto;
-webkit-text-stroke-width: 0px;" class=""><br class="">
My personal opinion is that for u/s users we may provide
these files as a<br class="">
separate package or simply forward them to the u/s
VirtIO-Win ISO.<br class="">
However I don't mind to keep things as is and having them
as part of the<br class="">
base installer for simplicity of distribution.<br class="">
After all the new versions are not frequent and have to be
downloaded only<br class="">
once per oVirt instance, so the increased download size
should not be an<br class="">
issue.<br class="">
<br class="">
If anything, I'm more concerned with the waste of the disk
space of the VM,<br class="">
and even that should not be a critical issue in most
cases.<br class="">
So it would be nice to have an option to install the debug
information<br class="">
during the install, so the user could choose by himself.<br
class="">
<br class="">
Thanks in advance,<br class="">
<br class="">
On Thu, Dec 21, 2017 at 9:53 AM, Yedidyah Bar David <<a
href="mailto:didi@redhat.com" class=""
moz-do-not-send="true">didi@redhat.com</a>> wrote:<br
class="">
<br class="">
<blockquote type="cite" class="">On Wed, Dec 20, 2017 at
4:10 PM, Frediano Ziglio <<a
href="mailto:fziglio@redhat.com" class=""
moz-do-not-send="true">fziglio@redhat.com</a>><br
class="">
wrote:<br class="">
<blockquote type="cite" class="">
<blockquote type="cite" class=""><br class="">
On Tue, Dec 19, 2017 at 11:26:00AM -0500, Frediano
Ziglio wrote:<br class="">
<blockquote type="cite" class="">
<blockquote type="cite" class=""><br class="">
The .pdb files contain the debug information for
the drivers. They<br class="">
increase significantly the size of the
installer, so it's better<br class="">
</blockquote>
</blockquote>
</blockquote>
</blockquote>
not to<br class="">
<blockquote type="cite" class="">
<blockquote type="cite" class="">
<blockquote type="cite" class="">
<blockquote type="cite" class="">ship them.<br
class="">
</blockquote>
<br class="">
On the other side bug report will contain much
less informations.<br class="">
Not really sure about it.<br class="">
</blockquote>
<br class="">
Which bug reports? :) (I don't think I ever got any
report about driver<br class="">
crashes). I would reconsider when we start getting
bug reports where we<br class="">
need these symbols.<br class="">
<br class="">
Christophe<br class="">
<br class="">
</blockquote>
<br class="">
No idea, was just thinking that if somebody packaged
these files maybe<br class="">
</blockquote>
they want<br class="">
<blockquote type="cite" class="">them installed. They
are not directly in the virtio-win package but in<br
class="">
</blockquote>
the ISO.<br class="">
<blockquote type="cite" class="">Maybe we should remove
from the ISO too ?<br class="">
I had a look at the inf files and they not reference
the pdb files.<br class="">
</blockquote>
<br class="">
Adding Lev.<br class="">
<br class="">
Lev - do we ship pdb files in RHV-guest-tools?<br
class="">
Did we ever get bug reports about drivers that included
relevant<br class="">
information<br class="">
which is not available without the pdb files?<br
class="">
<br class="">
Thanks,<br class="">
--<br class="">
Didi<br class="">
<br class="">
</blockquote>
<br class="">
<br class="">
<br class="">
--<br class="">
<br class="">
Lev Veyde<br class="">
<br class="">
Software Engineer, RHCE | RHCVA | MCITP<br class="">
<br class="">
Red Hat Israel<br class="">
<br class="">
<<a href="https://www.redhat.com/" class=""
moz-do-not-send="true">https://www.redhat.com</a>><br
class="">
<br class="">
<a href="mailto:lev@redhat.com" class=""
moz-do-not-send="true">lev@redhat.com</a><span
class="Apple-converted-space"> </span>|<span
class="Apple-converted-space"> </span><a
href="mailto:lveyde@redhat.com" class=""
moz-do-not-send="true">lveyde@redhat.com</a><br class="">
<<a href="https://red.ht/sig" class=""
moz-do-not-send="true">https://red.ht/sig</a>><br
class="">
TRIED. TESTED. TRUSTED. <<a
href="https://redhat.com/trusted" class=""
moz-do-not-send="true">https://redhat.com/trusted</a>></blockquote>
</div>
</blockquote>
</div>
<br class="">
</blockquote>
<br>
</body>
</html>