From owner-freebsd-hackers@FreeBSD.ORG Thu Apr 29 20:56:59 2010 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 89A35106566C for ; Thu, 29 Apr 2010 20:56:59 +0000 (UTC) (envelope-from jhb@freebsd.org) Received: from cyrus.watson.org (cyrus.watson.org [65.122.17.42]) by mx1.freebsd.org (Postfix) with ESMTP id 593788FC14 for ; Thu, 29 Apr 2010 20:56:59 +0000 (UTC) Received: from bigwig.baldwin.cx (66.111.2.69.static.nyinternet.net [66.111.2.69]) by cyrus.watson.org (Postfix) with ESMTPSA id D847346B0C; Thu, 29 Apr 2010 16:56:58 -0400 (EDT) Received: from jhbbsd.localnet (smtp.hudson-trading.com [209.249.190.9]) by bigwig.baldwin.cx (Postfix) with ESMTPA id 2FF828A01F; Thu, 29 Apr 2010 16:56:58 -0400 (EDT) From: John Baldwin To: "=?iso-8859-15?q?Luk=E1=A8?= Czerner" Date: Thu, 29 Apr 2010 16:56:12 -0400 User-Agent: KMail/1.12.1 (FreeBSD/7.3-CBSD-20100217; KDE/4.3.1; amd64; ; ) References: <201004291606.35899.jhb@freebsd.org> In-Reply-To: MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-15" Content-Transfer-Encoding: quoted-printable Message-Id: <201004291656.12565.jhb@freebsd.org> X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.0.1 (bigwig.baldwin.cx); Thu, 29 Apr 2010 16:56:58 -0400 (EDT) X-Virus-Scanned: clamav-milter 0.95.1 at bigwig.baldwin.cx X-Virus-Status: Clean X-Spam-Status: No, score=-2.6 required=4.2 tests=AWL,BAYES_00 autolearn=ham version=3.2.5 X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on bigwig.baldwin.cx Cc: freebsd-hackers@freebsd.org Subject: Re: ioctl, copy string from user X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 29 Apr 2010 20:56:59 -0000 On Thursday 29 April 2010 4:28:31 pm Luk=E1=A8 Czerner wrote: > On Thu, 29 Apr 2010, John Baldwin wrote: > > > >=20 > > > > On Thursday 29 April 2010 1:52:45 pm Luk=E1=A8 Czerner wrote: > > > > > Hi, > > > > >=20 > > > > > I know that there are plenty of examples in the kernel code, but I > > > > > just can not get it working, so maybe I am doing some stupid mist= ake > > > > > I am not aware of. Please give me a hint if you can. > > > > >=20 > > > > > What I want to do is simply call the ioctl from the userspace with > > > > > (char *) argument. Then, in kernel ioctl handling function copy t= he > > > > > string argument into the kernel space. I have tried it various wa= ys, > > > > > everything without any success. > > > > >=20 > > > > > *** Userspace *** > > > > > char name[MAXLEN]; > > > > >=20 > > > > > strncpy(name, argv[1], MAXLEN); > > > > > fprintf(stdout,"Name: %s\n",name); > > > > >=20 > > > > > if (ioctl(fd, MYIOCTL, name)) { > > > >=20 > > > > On BSD systems, ioctl() copies the data into the kernel for you ahe= ad of=20 > > time. =20 > > > > What does the definition of MYIOCTL look like? > > >=20 > > > #define MYIOCTL _IOW('M', 0, char *) > >=20 > > Ok. In that case the argument to ioctl needs to be a pointer to a char= *, > > not the raw char * itself. Try doing 'ioctl(fd, MYIOCTL, &name)' from= =20 > > userland to see if that fixes it. >=20 > I have already tried that, but still without any success. The buffer > remains unchanged (which is weird IMO). Can you print out the value of 'ap->a_data' from the kernel and the value of '&name' in userland? > > Generally yes, but it depends on the lock. If it is the vn_lock lock t= hen it=20 > > is ok to do a blocking malloc(). As a general rule I do try to call ma= lloc() > > before acquiring locks (basically preallocating) whenever possible. >=20 > So I suppose M_NOWAIT will do the trick when there is no other way > (preallocations etc..) ? Of course I should test if it does not > return NULL then. Yes, but in a VOP_IOCTL() handler it should be safe to M_WAITOK malloc() as long as you do it before you acquire any mutexes. =2D-=20 John Baldwin