public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>
To: KY Srinivasan <kys@microsoft.com>
Cc: Vitaly Kuznetsov <vkuznets@redhat.com>,
	"devel@linuxdriverproject.org" <devel@linuxdriverproject.org>,
	Haiyang Zhang <haiyangz@microsoft.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 0/5] Tools: hv: fix compiler warnings and do minor cleanup
Date: Fri, 9 Jan 2015 12:47:35 -0800	[thread overview]
Message-ID: <20150109204735.GA1194@kroah.com> (raw)
In-Reply-To: <BN1PR0301MB0707072CBA0483E06C6FE6E1A0470@BN1PR0301MB0707.namprd03.prod.outlook.com>

On Thu, Jan 08, 2015 at 05:04:20PM +0000, KY Srinivasan wrote:
> 
> 
> > -----Original Message-----
> > From: Vitaly Kuznetsov [mailto:vkuznets@redhat.com]
> > Sent: Thursday, January 8, 2015 8:02 AM
> > To: KY Srinivasan
> > Cc: gregkh@linuxfoundation.org; devel@linuxdriverproject.org; Haiyang
> > Zhang; linux-kernel@vger.kernel.org
> > Subject: Re: [PATCH 0/5] Tools: hv: fix compiler warnings and do minor
> > cleanup
> > 
> > KY Srinivasan <kys@microsoft.com> writes:
> > 
> > >> -----Original Message-----
> > >> From: devel [mailto:driverdev-devel-bounces@linuxdriverproject.org]
> > >> On Behalf Of gregkh@linuxfoundation.org
> > >> Sent: Wednesday, December 10, 2014 6:48 AM
> > >> To: Vitaly Kuznetsov
> > >> Cc: devel@linuxdriverproject.org; Haiyang Zhang; linux-
> > >> kernel@vger.kernel.org
> > >> Subject: Re: [PATCH 0/5] Tools: hv: fix compiler warnings and do
> > >> minor cleanup
> > >>
> > >> On Wed, Dec 10, 2014 at 10:22:40AM +0100, Vitaly Kuznetsov wrote:
> > >> > Dexuan Cui <decui@microsoft.com> writes:
> > >> >
> > >> > >> -----Original Message-----
> > >> > >> From: Vitaly Kuznetsov [mailto:vkuznets@redhat.com]
> > >> > >> Sent: Tuesday, December 9, 2014 23:48 PM
> > >> > >> To: KY Srinivasan
> > >> > >> Cc: Haiyang Zhang; devel@linuxdriverproject.org; linux-
> > >> > >> kernel@vger.kernel.org; Dexuan Cui
> > >> > >> Subject: [PATCH 0/5] Tools: hv: fix compiler warnings and do
> > >> > >> minor cleanup
> > >> > >>
> > >> > >> When someone does 'make' in tools/hv/ issues appear:
> > >> > >> - hv_fcopy_daemon is not being built;
> > >> > >> - lots of compiler warnings.
> > >> > >>
> > >> > >> This is just a cleanup. Compile-tested by myself on top of
> > >> > >> linux-
> > >> next/master.
> > >> > >>
> > >> > >> Piggyback this series and send "[PATCH 5/5] Tools: hv: do not
> > >> > >> add redundant '/'
> > >> > >>  in hv_start_fcopy()"
> > >> > >>
> > >> > >> Vitaly Kuznetsov (5):
> > >> > >>   Tools: hv: add mising fcopyd to the Makefile
> > >> > >>   Tools: hv: remove unused bytes_written from kvp_update_file()
> > >> > >>   Tools: hv: address compiler warnings for hv_kvp_daemon.c
> > >> > >>   Tools: hv: address compiler warnings for hv_fcopy_daemon.c
> > >> > >>   Tools: hv: do not add redundant '/' in hv_start_fcopy()
> > >> > >>
> > >> > >>  tools/hv/Makefile          |  4 ++--
> > >> > >>  tools/hv/hv_fcopy_daemon.c | 10 ++--------
> > >> > >>  tools/hv/hv_kvp_daemon.c   | 29 +++++++++++++----------------
> > >> > >>  3 files changed, 17 insertions(+), 26 deletions(-)
> > >> > >>
> > >> > >> --
> > >> > >> 1.9.3
> > >> > >
> > >> > > Hi Vitaly,
> > >> > > Thanks for the patchset!
> > >> > >
> > >> > > Acked-by: Dexuan Cui <decui@microsoft.com>
> > >> > >
> > >> > > PS, I added Greg into the TO list.
> > >> > > The hv code in drivers/hv/ and tools/hv/ usually has to go into
> > >> > > Greg's tree first.
> > >> >
> > >> > Well, I don't mind spamming Greg but he's not on the
> > >> > scripts/get_maintainer.pl output. In case he's not monitoring the
> > >> > list for patches by some other tool (patchwork?) a patch adding him
> > >> > to MAINTAINERS would do the job.
> > >> >
> > >> > Greg, do you want to become an official Hyper-V maintainer in
> > >> > MAINTAINERS? I can send a patch then :-)
> > >>
> > >> No.  It's up to the "real" maintainers to take the patches and then
> > >> forward them on to me for inclusion in the kernel tree.  KY knows this...
> > >
> > > I will take care of this.
> > >
> > 
> > Hi KY,
> > 
> > I just found out these patches never made it to any repo, at least I can't see
> > them in char-misc-next. Do I need to resend?
> 
> Greg,
> 
> I have signed off on these patches. Do you want us to resend them. On a different note, I too have a
> Few patches that I sent some weeks ago. Should I be resending them as well.

Please resend everything, there has been a mess of different patches and
discussions and I can't figure out what should be applied and what
should not.  I'll guess at a few easy ones, but getting the "correct"
ones from you is the best thing.

thanks,

greg k-h

  reply	other threads:[~2015-01-09 20:47 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-09 15:48 [PATCH 0/5] Tools: hv: fix compiler warnings and do minor cleanup Vitaly Kuznetsov
2014-12-09 15:48 ` [PATCH 1/5] Tools: hv: add mising fcopyd to the Makefile Vitaly Kuznetsov
2014-12-10 18:52   ` KY Srinivasan
2014-12-09 15:48 ` [PATCH 2/5] Tools: hv: remove unused bytes_written from kvp_update_file() Vitaly Kuznetsov
2014-12-10 18:49   ` KY Srinivasan
2014-12-09 15:48 ` [PATCH 3/5] Tools: hv: address compiler warnings for hv_kvp_daemon.c Vitaly Kuznetsov
2014-12-10 18:51   ` KY Srinivasan
2014-12-09 15:48 ` [PATCH 4/5] Tools: hv: address compiler warnings for hv_fcopy_daemon.c Vitaly Kuznetsov
2014-12-10 18:55   ` KY Srinivasan
2014-12-09 15:48 ` [PATCH 5/5] Tools: hv: do not add redundant '/' in hv_start_fcopy() Vitaly Kuznetsov
2014-12-10 18:54   ` KY Srinivasan
2014-12-10  2:58 ` [PATCH 0/5] Tools: hv: fix compiler warnings and do minor cleanup Dexuan Cui
2014-12-10  9:22   ` Vitaly Kuznetsov
2014-12-10 14:48     ` gregkh
2014-12-10 17:50       ` KY Srinivasan
2015-01-08 16:02         ` Vitaly Kuznetsov
2015-01-08 17:04           ` KY Srinivasan
2015-01-09 20:47             ` gregkh [this message]
2015-01-09 20:58               ` KY Srinivasan
2015-01-09 21:02                 ` gregkh
2015-01-09 21:08                   ` KY Srinivasan

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=20150109204735.GA1194@kroah.com \
    --to=gregkh@linuxfoundation.org \
    --cc=devel@linuxdriverproject.org \
    --cc=haiyangz@microsoft.com \
    --cc=kys@microsoft.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=vkuznets@redhat.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox