* Re: A small fix to davinci_emac driver [not found] ` <B85A65D85D7EB246BE421B3FB0FBB59301D829C4CE@dbde02.ent.ti.com> @ 2009-06-25 13:13 ` Pablo Bitton 2009-06-25 13:34 ` Kevin Hilman 0 siblings, 1 reply; 2+ messages in thread From: Pablo Bitton @ 2009-06-25 13:13 UTC (permalink / raw) To: Nori, Sekhar, netdev Cc: Rajashekhara, Sudhakar, Subrahmanya, Chaithrika, davinci-linux-open-source [-- Attachment #1: Type: text/plain, Size: 1846 bytes --] Hello all, I have attached a small fix for the following problem: When davinci_emac interface is down, changing MAC address results in kernel oops, due to NULL pointer dereference. I apologize for not inlining the patch into the message (since GMail is the only web client I can use, and it does not allow me to inline it correctly). Remarks are welcomed. Thanks in advance. On Thu, Jun 25, 2009 at 11:31 AM, Nori, Sekhar<nsekhar@ti.com> wrote: > > Pablo, > > Chaithrika's mails to gmail.com are bouncing, so I am replying > on her behalf. > > " > You are right there is indeed an issue. We looked at other > drivers and found that this is solved by checking for netif_running > to be true before going ahead and setting the mac address. Have a > look at how this is done in tg3.c driver > > http://git.kernel.org/?p=linux/kernel/git/khilman/linux-davinci.git;a=blob;f=drivers/net/tg3.c;h=201be425643a6fb7acb8801bbbb718897c9a6431;hb=35265abf67337f57a023ffeae5f4c492e7f8ea24#l6589 > " > > Can you please go ahead and submit a fix for this to netdev > mailing list and CC linux-davinci ML? > > Thanks, > Sekhar > >> -----Original Message----- >> From: Pablo Bitton [mailto:pablo.bitton@gmail.com] >> Sent: Wednesday, June 24, 2009 7:13 PM >> To: Subrahmanya, Chaithrika >> Subject: A small fix to davinci_emac driver >> >> Hello, >> >> In drivers/net/davinci_emac.c file, at emac_dev_setmac_addr() function, rxch >> is NULL when DaVinci EMAC is down (using "ifconfig eth0 down"). >> >> So when MAC address is changed using "ifconfig eth0 hw ether >> 00:11:22:33:44:55", >> emac_dev_setmac_addr() performs an illegal access to rxch->mac_addr, which >> causes a kernel oops. >> >> In order to fix this situation, the attached patch is applied (I've couldn't >> inline it correctly into GMail). >> Remarks are welcomed. >> >> Thanks in advance. > [-- Attachment #2: 0002-davinci_emac-fix-kernel-oops-when-changing-mac-addr.patch.txt --] [-- Type: text/plain, Size: 1571 bytes --] From ae9480af26eba3302614ea3d92077a80341cd190 Mon Sep 17 00:00:00 2001 From: Pablo Bitton <pablo.bitton@gmail.com> Date: Tue, 23 Jun 2009 15:45:43 +0300 Subject: [PATCH] davinci_emac: fix kernel oops when changing MAC address while interface is down rxch == NULL when DaVinci EMAC is down (using "ifconfig eth0 down"). So when MAC address is changed using "ifconfig eth0 hw ether 00:11:22:33:44:55", emac_dev_setmac_addr() performs an illegal access to rxch->mac_addr, which causes a kernel oops. Signed-off-by: Pablo Bitton <pablo.bitton@gmail.com> --- drivers/net/davinci_emac.c | 9 +++++++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/drivers/net/davinci_emac.c b/drivers/net/davinci_emac.c index cf689a0..cd114d9 100644 --- a/drivers/net/davinci_emac.c +++ b/drivers/net/davinci_emac.c @@ -1823,9 +1823,14 @@ static int emac_dev_setmac_addr(struct net_device *ndev, void *addr) /* Store mac addr in priv and rx channel and set it in EMAC hw */ memcpy(priv->mac_addr, sa->sa_data, ndev->addr_len); - memcpy(rxch->mac_addr, sa->sa_data, ndev->addr_len); memcpy(ndev->dev_addr, sa->sa_data, ndev->addr_len); - emac_setmac(priv, EMAC_DEF_RX_CH, rxch->mac_addr); + + /* If the interface is down - rxch is NULL. */ + /* MAC address is configured only after the interface is enabled. */ + if (netif_running(ndev)) { + memcpy(rxch->mac_addr, sa->sa_data, ndev->addr_len); + emac_setmac(priv, EMAC_DEF_RX_CH, rxch->mac_addr); + } if (netif_msg_drv(priv)) dev_notice(emac_dev, "DaVinci EMAC: emac_dev_setmac_addr %s\n", -- 1.5.4.3 ^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: A small fix to davinci_emac driver 2009-06-25 13:13 ` A small fix to davinci_emac driver Pablo Bitton @ 2009-06-25 13:34 ` Kevin Hilman 0 siblings, 0 replies; 2+ messages in thread From: Kevin Hilman @ 2009-06-25 13:34 UTC (permalink / raw) To: Pablo Bitton; +Cc: Nori, Sekhar, netdev, davinci-linux-open-source Pablo Bitton <pablo.bitton@gmail.com> writes: > Hello all, > > I have attached a small fix for the following problem: > When davinci_emac interface is down, changing MAC address results in > kernel oops, due to NULL pointer dereference. This should go linux-net after a signoff from Sekhar or Chaithrika. > I apologize for not inlining the patch into the message (since GMail > is the only web client I can use, and it does not allow me to inline > it correctly). You can send via Gmail using git-send-email directly. Read this: http://git.or.cz/gitwiki/GitTips#Mail Kevin > Remarks are welcomed. > > Thanks in advance. > > On Thu, Jun 25, 2009 at 11:31 AM, Nori, Sekhar<nsekhar@ti.com> wrote: >> >> Pablo, >> >> Chaithrika's mails to gmail.com are bouncing, so I am replying >> on her behalf. >> >> " >> You are right there is indeed an issue. We looked at other >> drivers and found that this is solved by checking for netif_running >> to be true before going ahead and setting the mac address. Have a >> look at how this is done in tg3.c driver >> >> http://git.kernel.org/?p=linux/kernel/git/khilman/linux-davinci.git;a=blob;f=drivers/net/tg3.c;h=201be425643a6fb7acb8801bbbb718897c9a6431;hb=35265abf67337f57a023ffeae5f4c492e7f8ea24#l6589 >> " >> >> Can you please go ahead and submit a fix for this to netdev >> mailing list and CC linux-davinci ML? >> >> Thanks, >> Sekhar >> >>> -----Original Message----- >>> From: Pablo Bitton [mailto:pablo.bitton@gmail.com] >>> Sent: Wednesday, June 24, 2009 7:13 PM >>> To: Subrahmanya, Chaithrika >>> Subject: A small fix to davinci_emac driver >>> >>> Hello, >>> >>> In drivers/net/davinci_emac.c file, at emac_dev_setmac_addr() function, rxch >>> is NULL when DaVinci EMAC is down (using "ifconfig eth0 down"). >>> >>> So when MAC address is changed using "ifconfig eth0 hw ether >>> 00:11:22:33:44:55", >>> emac_dev_setmac_addr() performs an illegal access to rxch->mac_addr, which >>> causes a kernel oops. >>> >>> In order to fix this situation, the attached patch is applied (I've couldn't >>> inline it correctly into GMail). >>> Remarks are welcomed. >>> >>> Thanks in advance. >> > > From ae9480af26eba3302614ea3d92077a80341cd190 Mon Sep 17 00:00:00 2001 > From: Pablo Bitton <pablo.bitton@gmail.com> > Date: Tue, 23 Jun 2009 15:45:43 +0300 > Subject: [PATCH] davinci_emac: fix kernel oops when changing MAC address while interface is down > > rxch == NULL when DaVinci EMAC is down (using "ifconfig eth0 down"). > So when MAC address is changed using "ifconfig eth0 hw ether 00:11:22:33:44:55", > emac_dev_setmac_addr() performs an illegal access to rxch->mac_addr, which > causes a kernel oops. > > Signed-off-by: Pablo Bitton <pablo.bitton@gmail.com> > --- > drivers/net/davinci_emac.c | 9 +++++++-- > 1 files changed, 7 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/davinci_emac.c b/drivers/net/davinci_emac.c > index cf689a0..cd114d9 100644 > --- a/drivers/net/davinci_emac.c > +++ b/drivers/net/davinci_emac.c > @@ -1823,9 +1823,14 @@ static int emac_dev_setmac_addr(struct net_device *ndev, void *addr) > > /* Store mac addr in priv and rx channel and set it in EMAC hw */ > memcpy(priv->mac_addr, sa->sa_data, ndev->addr_len); > - memcpy(rxch->mac_addr, sa->sa_data, ndev->addr_len); > memcpy(ndev->dev_addr, sa->sa_data, ndev->addr_len); > - emac_setmac(priv, EMAC_DEF_RX_CH, rxch->mac_addr); > + > + /* If the interface is down - rxch is NULL. */ > + /* MAC address is configured only after the interface is enabled. */ > + if (netif_running(ndev)) { > + memcpy(rxch->mac_addr, sa->sa_data, ndev->addr_len); > + emac_setmac(priv, EMAC_DEF_RX_CH, rxch->mac_addr); > + } > > if (netif_msg_drv(priv)) > dev_notice(emac_dev, "DaVinci EMAC: emac_dev_setmac_addr %s\n", > -- > 1.5.4.3 > > _______________________________________________ > Davinci-linux-open-source mailing list > Davinci-linux-open-source@linux.davincidsp.com > http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-06-25 13:34 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <028901c9f54b$4579eae0$d06dc0a0$@com>
[not found] ` <B85A65D85D7EB246BE421B3FB0FBB59301D829C4CE@dbde02.ent.ti.com>
2009-06-25 13:13 ` A small fix to davinci_emac driver Pablo Bitton
2009-06-25 13:34 ` Kevin Hilman
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).