Re: [PATCH v4 02/10] x86/hyper-v: stash the max number of virtual/logical processor

From	Stephen Hemminger <stephen@networkplumber.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH v4 02/10] x86/hyper-v: stash the max number of virtual/logical processor
Date	2017-06-05 17:40 +0200
Message-ID	<tP2gO-4G7-5@gated-at.bofh.it> (permalink)
References	<tKDh0-2cD-7@gated-at.bofh.it> <tKDh1-2cD-27@gated-at.bofh.it> <tLO0G-77h-15@gated-at.bofh.it> <tP278-4D2-15@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Mon, 5 Jun 2017 11:24:27 -0400
Steven Rostedt <rostedt@goodmis.org> wrote:

> On Sat, 27 May 2017 20:43:58 +0300
> Andy Shevchenko <andy.shevchenko@gmail.com> wrote:
> 
> > On Wed, May 24, 2017 at 3:03 PM, Vitaly Kuznetsov <vkuznets@redhat.com> wrote:  
> > > Max virtual processor will be needed for 'extended' hypercalls supporting
> > > more than 64 vCPUs. While on it, unify on 'Hyper-V' in mshyperv.c as we
> > > currently have a mix, report acquired misc features as well.
> > >
> > > Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com>
> > > Acked-by: K. Y. Srinivasan <kys@microsoft.com>
> > > Tested-by: Simon Xiao <sixiao@microsoft.com>
> > > Tested-by: Srikanth Myakam <v-srm@microsoft.com>    
> >   
> > > +       u32 max_vp_index;
> > > +       u32 max_lp_index;    
> >   
> > > +       pr_info("Hyper-V: max %d virtual processors, %d logical processors\n",
> > > +               ms_hyperv.max_vp_index, ms_hyperv.max_lp_index);    
> > 
> > And surprisingly no-one from the above list did not get a warning?!  

Gcc 6.3 does not warn when %d is used on unsigned values.


> Begs to question how many other warnings are they ignoring?
None.

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH v4 02/10] x86/hyper-v: stash the max number of  virtual/logical processor Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-05-27 19:50 +0200
  Re: [PATCH v4 02/10] x86/hyper-v: stash the max number of  virtual/logical processor Steven Rostedt <rostedt@goodmis.org> - 2017-06-05 17:30 +0200
    Re: [PATCH v4 02/10] x86/hyper-v: stash the max number of  virtual/logical processor Stephen Hemminger <stephen@networkplumber.org> - 2017-06-05 17:40 +0200

csiph-web