- Subject: Re: [PATCH 3/4] ksysfs: export VMCSINFO via sysfs
- From: Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx>
- Date: Thu, 12 Apr 2012 16:00:58 -0700
- Cc: avi@xxxxxxxxxx, mtosatti@xxxxxxxxxx, ebiederm@xxxxxxxxxxxx, luto@xxxxxxx, joerg.roedel@xxxxxxx, dzickus@xxxxxxxxxx, paul.gortmaker@xxxxxxxxxxxxx, gregkh@xxxxxxx, ludwig.nussel@xxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, kvm@xxxxxxxxxxxxxxx, kexec@xxxxxxxxxxxxxxxxxxx
- In-reply-to: <4F84E50E.2090008@cn.fujitsu.com>
- References: <4F84E0DF.8040206@cn.fujitsu.com> <4F84E50E.2090008@cn.fujitsu.com>
- User-agent: Mutt/1.5.21 (2010-09-15)
On Wed, Apr 11, 2012 at 09:57:34AM +0800, zhangyanfei wrote:
> This patch creates sysfs file to export where VMCSINFO is allocated,
> as below:
> $ cat /sys/kernel/vmcsinfo
> 1cb88a0 2000
> number on the left-hand side is the physical address of VMCSINFO,
> while the one on the right-hand side is the max size of VMCSINFO.
Ick, why do you have 2 values in one sysfs file, that's not nice, or
good.
What's wrong with 2 different files?
Also, any new sysfs file you add needs to also have a Documentation/ABI
entry added as well.
But we can't accept this as-is, sorry, please split it up into 2 files.
greg k-h
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
[KVM ARM]
[KVM ia64]
[KVM ppc]
[Spice Development]
[Libvirt]
[Libvirt Users]
[Linux USB Devel]
[Video for Linux]
[Linux Audio Users]
[Photo]
[Yosemite News]
[Yosemite Photos]
[Linux Kernel]
[Linux SCSI]
[XFree86]