From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Subject: Re: [PATCH 3/4] ksysfs: export VMCSINFO via sysfs Date: Mon, 16 Apr 2012 19:30:53 -0700 Message-ID: <20120417023053.GA19491@kroah.com> References: <4F84E0DF.8040206@cn.fujitsu.com> <4F84E50E.2090008@cn.fujitsu.com> <20120412230058.GA23292@kroah.com> <4F8CCCEA.2090706@cn.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: avi@redhat.com, mtosatti@redhat.com, ebiederm@xmission.com, luto@mit.edu, joerg.roedel@amd.com, dzickus@redhat.com, paul.gortmaker@windriver.com, ludwig.nussel@suse.de, linux-kernel@vger.kernel.org, kvm@vger.kernel.org, kexec@lists.infradead.org To: zhangyanfei Return-path: Content-Disposition: inline In-Reply-To: <4F8CCCEA.2090706@cn.fujitsu.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org On Tue, Apr 17, 2012 at 09:52:42AM +0800, zhangyanfei wrote: > =E4=BA=8E 2012=E5=B9=B404=E6=9C=8813=E6=97=A5 07:00, Greg KH =E5=86=99= =E9=81=93: > > On Wed, Apr 11, 2012 at 09:57:34AM +0800, zhangyanfei wrote: > >> This patch creates sysfs file to export where VMCSINFO is allocate= d, > >> 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. > >=20 > > Ick, why do you have 2 values in one sysfs file, that's not nice, o= r > > good. > >=20 > > What's wrong with 2 different files? > >=20 >=20 > The reason why I put the 2 values in one sysfs file is that there is = a similar > file 'vmcoreinfo' in sysfs. > # cat /sys/kernel/vmcoreinfo=20 > 1d75380 1000 Then that should be fixed as well, using two different file names now := ( thanks, greg k-h