| To: | "David S. Miller" <davem@xxxxxxxxxx> |
|---|---|
| Subject: | Re: [BK PATCH 2.6] repost, fix sysctl breakage during network device renaming, for ipv4 |
| From: | Dan Aloni <da-x@xxxxxxx> |
| Date: | Mon, 1 Sep 2003 21:55:32 +0300 |
| Cc: | netdev@xxxxxxxxxxx |
| In-reply-to: | <20030901113143.1ba34464.davem@xxxxxxxxxx> |
| References: | <20030901164624.GA26886@xxxxxxxxxxxxxxx> <20030901094127.6a0f6878.davem@xxxxxxxxxx> <20030901165559.GA27099@xxxxxxxxxxxxxxx> <20030901113143.1ba34464.davem@xxxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | Mutt/1.5.4i |
On Mon, Sep 01, 2003 at 11:31:43AM -0700, David S. Miller wrote: > On Mon, 1 Sep 2003 19:55:59 +0300 > Dan Aloni <da-x@xxxxxxx> wrote: > > > The private copy of strdup() is something that I'd also wouldn't > > want, but I don't have much choice. Please read a recent > > thread in lkml regarding strdup() consolidation. > > Ugh, I'm afraid to look... I guess the controversy is > over the allocation function, what GFP_* flags it should > use etc.? Yes, this is the one reason against consolidation, though I don't know why anyone would want to call strdup() from a context other than GPF_KERNEL. Copying strings around in the kernel is done as a result of userspace interaction. > For now just call the thing in your patch netdev_name_dup() or > something like that. I can't handle having something named > "strdup()" in there :-) Sure, I'll do it and resend. -- Dan Aloni da-x@xxxxxxx |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [BK PATCH 2.6] repost, fix sysctl breakage during network device renaming, for ipv4, David S. Miller |
|---|---|
| Next by Date: | [patch 2.4][fyi] s/blog_dev/backlog_dev/ in process_backlog, Jeff Garzik |
| Previous by Thread: | Re: [BK PATCH 2.6] repost, fix sysctl breakage during network device renaming, for ipv4, David S. Miller |
| Next by Thread: | [patch 1/2][fyi] alloc_ei_netdev() function for 8390 devices, Jeff Garzik |
| Indexes: | [Date] [Thread] [Top] [All Lists] |