All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Zhao, Yakui" <yakui.zhao@intel.com>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: x86@kernel.org, linux-kernel@vger.kernel.org,
	devel@driverdev.osuosl.org,
	Mingqiang Chi <mingqiang.chi@intel.com>,
	Jack Ren <jack.ren@intel.com>,
	Jason Chen CJ <jason.cj.chen@intel.com>,
	Liu Shuo <shuo.a.liu@intel.com>
Subject: Re: [RFC PATCH 04/15] drivers/acrn: add the basic framework of acrn char device driver
Date: Mon, 19 Aug 2019 12:02:33 +0800	[thread overview]
Message-ID: <cedb90e7-da98-9075-e647-075fa3a3e7ae@intel.com> (raw)
In-Reply-To: <20190816070559.GB1368@kroah.com>



On 2019年08月16日 15:05, Greg KH wrote:
> On Fri, Aug 16, 2019 at 10:25:45AM +0800, Zhao Yakui wrote:
>> ACRN hypervisor service module is the important middle layer that allows
>> the Linux kernel to communicate with the ACRN hypervisor. It includes
>> the management of virtualized CPU/memory/device/interrupt for other ACRN
>> guest. The user-space applications can use the provided ACRN ioctls to
>> interact with ACRN hypervisor through different hypercalls.
>>
>> Add one basic framework firstly and the following patches will
>> add the corresponding implementations, which includes the management of
>> virtualized CPU/memory/interrupt and the emulation of MMIO/IO/PCI access.
>> The device file of /dev/acrn_hsm can be accessed in user-space to
>> communicate with ACRN module.
>>
>> Co-developed-by: Jason Chen CJ <jason.cj.chen@intel.com>
>> Signed-off-by: Jason Chen CJ <jason.cj.chen@intel.com>
>> Co-developed-by: Jack Ren <jack.ren@intel.com>
>> Signed-off-by: Jack Ren <jack.ren@intel.com>
>> Co-developed-by: Mingqiang Chi <mingqiang.chi@intel.com>
>> Signed-off-by: Mingqiang Chi <mingqiang.chi@intel.com>
>> Co-developed-by: Liu Shuo <shuo.a.liu@intel.com>
>> Signed-off-by: Liu Shuo <shuo.a.liu@intel.com>
>> Signed-off-by: Zhao Yakui <yakui.zhao@intel.com>
>> ---
>>   drivers/staging/Kconfig         |   2 +
> 
> Also, your subject line for all of these patches are wrong, it is not
> drivers/acrn :(

Thanks for the pointing out it.

It will be fixed.

> 
> And you forgot to cc: the staging maintainer :(

Do you mean that the maintainer of staging subsystem is also added in 
the patch commit log?


> 
> As I have said with NUMEROUS Intel patches in the past, I now refuse to
> take patches from you all WITHOUT having it signed-off-by someone from
> the Intel "OTC" group (or whatever the Intel Linux group is called these
> days).  They are a resource you can not ignore, and if you do, you just
> end up making the rest of the kernel community grumpy by having us do
> their work for them :(
> 
> Please work with them.

OK. I will work with some peoples in OTC group to prepare the better 
ACRN driver.

> 
> greg k-h
> 

WARNING: multiple messages have this Message-ID (diff)
From: yakui.zhao@intel.com (Zhao, Yakui)
Subject: [RFC PATCH 04/15] drivers/acrn: add the basic framework of acrn char device driver
Date: Mon, 19 Aug 2019 12:02:33 +0800	[thread overview]
Message-ID: <cedb90e7-da98-9075-e647-075fa3a3e7ae@intel.com> (raw)
In-Reply-To: <20190816070559.GB1368@kroah.com>



On 2019?08?16? 15:05, Greg KH wrote:
> On Fri, Aug 16, 2019@10:25:45AM +0800, Zhao Yakui wrote:
>> ACRN hypervisor service module is the important middle layer that allows
>> the Linux kernel to communicate with the ACRN hypervisor. It includes
>> the management of virtualized CPU/memory/device/interrupt for other ACRN
>> guest. The user-space applications can use the provided ACRN ioctls to
>> interact with ACRN hypervisor through different hypercalls.
>>
>> Add one basic framework firstly and the following patches will
>> add the corresponding implementations, which includes the management of
>> virtualized CPU/memory/interrupt and the emulation of MMIO/IO/PCI access.
>> The device file of /dev/acrn_hsm can be accessed in user-space to
>> communicate with ACRN module.
>>
>> Co-developed-by: Jason Chen CJ <jason.cj.chen at intel.com>
>> Signed-off-by: Jason Chen CJ <jason.cj.chen at intel.com>
>> Co-developed-by: Jack Ren <jack.ren at intel.com>
>> Signed-off-by: Jack Ren <jack.ren at intel.com>
>> Co-developed-by: Mingqiang Chi <mingqiang.chi at intel.com>
>> Signed-off-by: Mingqiang Chi <mingqiang.chi at intel.com>
>> Co-developed-by: Liu Shuo <shuo.a.liu at intel.com>
>> Signed-off-by: Liu Shuo <shuo.a.liu at intel.com>
>> Signed-off-by: Zhao Yakui <yakui.zhao at intel.com>
>> ---
>>   drivers/staging/Kconfig         |   2 +
> 
> Also, your subject line for all of these patches are wrong, it is not
> drivers/acrn :(

Thanks for the pointing out it.

It will be fixed.

> 
> And you forgot to cc: the staging maintainer :(

Do you mean that the maintainer of staging subsystem is also added in 
the patch commit log?


> 
> As I have said with NUMEROUS Intel patches in the past, I now refuse to
> take patches from you all WITHOUT having it signed-off-by someone from
> the Intel "OTC" group (or whatever the Intel Linux group is called these
> days).  They are a resource you can not ignore, and if you do, you just
> end up making the rest of the kernel community grumpy by having us do
> their work for them :(
> 
> Please work with them.

OK. I will work with some peoples in OTC group to prepare the better 
ACRN driver.

> 
> greg k-h
> 

  reply	other threads:[~2019-08-19  4:09 UTC|newest]

Thread overview: 81+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-08-16  2:25 [RFC PATCH 00/15] acrn: add the ACRN driver module Zhao Yakui
2019-08-16  2:25 ` Zhao Yakui
2019-08-16  2:25 ` [RFC PATCH 01/15] x86/acrn: Report X2APIC for ACRN guest Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-16  2:25 ` [RFC PATCH 02/15] x86/acrn: Add two APIs to add/remove driver-specific upcall ISR handler Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-16  2:25 ` [RFC PATCH 03/15] x86/acrn: Add hypercall for ACRN guest Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-16  2:25 ` [RFC PATCH 04/15] drivers/acrn: add the basic framework of acrn char device driver Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-16  7:05   ` Greg KH
2019-08-16  7:05     ` Greg KH
2019-08-19  4:02     ` Zhao, Yakui [this message]
2019-08-19  4:02       ` Zhao, Yakui
2019-08-19  5:26       ` Greg KH
2019-08-19  5:26         ` Greg KH
2019-08-16 11:28   ` Dan Carpenter
2019-08-16 11:28     ` Dan Carpenter
2019-08-16  2:25 ` [RFC PATCH 05/15] drivers/acrn: add driver-specific hypercall for ACRN_HSM Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-16  2:25 ` [RFC PATCH 06/15] drivers/acrn: add the support of querying ACRN api version Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-16  2:25 ` [RFC PATCH 07/15] drivers/acrn: add acrn vm/vcpu management for ACRN_HSM char device Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-16  2:25 ` [RFC PATCH 08/15] drivers/acrn: add VM memory management for ACRN " Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-16 12:58   ` Dan Carpenter
2019-08-16 12:58     ` Dan Carpenter
2019-08-19  5:32     ` Zhao, Yakui
2019-08-19  5:32       ` Zhao, Yakui
2019-08-19  7:39       ` Dan Carpenter
2019-08-19  7:39         ` Dan Carpenter
2019-08-19  7:46         ` Borislav Petkov
2019-08-19  7:46           ` Borislav Petkov
2019-08-20  2:25         ` Zhao, Yakui
2019-08-20  2:25           ` Zhao, Yakui
2019-08-16  2:25 ` [RFC PATCH 09/15] drivers/acrn: add passthrough device support Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-16 13:05   ` Dan Carpenter
2019-08-16 13:05     ` Dan Carpenter
2019-09-02  0:18     ` Zhao, Yakui
2019-08-16  2:25 ` [RFC PATCH 10/15] drivers/acrn: add interrupt injection support Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-16 13:12   ` Dan Carpenter
2019-08-16 13:12     ` Dan Carpenter
2019-08-19  4:59     ` Zhao, Yakui
2019-08-19  4:59       ` Zhao, Yakui
2019-08-16  2:25 ` [RFC PATCH 11/15] drivers/acrn: add the support of handling emulated ioreq Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-16 13:39   ` Dan Carpenter
2019-08-16 13:39     ` Dan Carpenter
2019-08-19  4:54     ` Zhao, Yakui
2019-08-19  4:54       ` Zhao, Yakui
2019-08-16  2:25 ` [RFC PATCH 12/15] drivers/acrn: add driver-specific IRQ handle to dispatch IO_REQ request Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-16  2:25 ` [RFC PATCH 13/15] drivers/acrn: add service to obtain Power data transition Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-16  2:25 ` [RFC PATCH 14/15] drivers/acrn: add the support of irqfd and eventfd Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-16  2:25 ` [RFC PATCH 15/15] drivers/acrn: add the support of offline SOS cpu Zhao Yakui
2019-08-16  2:25   ` Zhao Yakui
2019-08-19 10:34   ` Dan Carpenter
2019-08-19 10:34     ` Dan Carpenter
2019-08-20  2:23     ` Zhao, Yakui
2019-08-20  2:23       ` Zhao, Yakui
2019-08-16  6:39 ` [RFC PATCH 00/15] acrn: add the ACRN driver module Borislav Petkov
2019-08-16  6:39   ` Borislav Petkov
2019-08-16  7:03   ` Greg KH
2019-08-16  7:03     ` Greg KH
2019-08-19  2:39     ` Zhao, Yakui
2019-08-19  2:39       ` Zhao, Yakui
2019-08-19  5:25       ` Greg KH
2019-08-19  5:25         ` Greg KH
2019-08-19  1:44   ` Zhao, Yakui
2019-08-19  1:44     ` Zhao, Yakui
2019-08-19  5:25     ` Greg KH
2019-08-19  5:25       ` Greg KH
2019-08-19  5:39       ` Zhao, Yakui
2019-08-19  5:39         ` Zhao, Yakui
2019-08-19  6:18     ` Borislav Petkov
2019-08-19  6:18       ` Borislav Petkov

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=cedb90e7-da98-9075-e647-075fa3a3e7ae@intel.com \
    --to=yakui.zhao@intel.com \
    --cc=devel@driverdev.osuosl.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=jack.ren@intel.com \
    --cc=jason.cj.chen@intel.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingqiang.chi@intel.com \
    --cc=shuo.a.liu@intel.com \
    --cc=x86@kernel.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.