linux-kernel - RE: [PATCH 4/5] staging: fsl-mc: bus rescan attribute to sync kernel with MC

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <BY1PR03MB14525BA21E8FD3E6B9111D10FC200@BY1PR03MB1452.namprd03.prod.outlook.com>
Date:	Thu, 29 Oct 2015 17:13:16 +0000
From:	Lijun Pan <Lijun.Pan@...escale.com>
To:	Greg KH <gregkh@...uxfoundation.org>
CC:	"arnd@...db.de" <arnd@...db.de>,
	"devel@...verdev.osuosl.org" <devel@...verdev.osuosl.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Hamciuc Bogdan <bhamciu1@...escale.com>,
	Sharma Bhupesh <bhupesh.sharma@...escale.com>,
	Jose Rivera <German.Rivera@...escale.com>,
	"agraf@...e.de" <agraf@...e.de>,
	Stuart Yoder <stuart.yoder@...escale.com>,
	Erez Nir <nir.erez@...escale.com>,
	Katz Itai <itai.katz@...escale.com>,
	Scott Wood <scottwood@...escale.com>,
	Li Leo <LeoLi@...escale.com>,
	Marginean Alexandru <R89243@...escale.com>,
	"dan.carpenter@...cle.com" <dan.carpenter@...cle.com>,
	Richard Schmitt <richard.schmitt@...escale.com>
Subject: RE: [PATCH 4/5] staging: fsl-mc: bus rescan attribute to sync kernel
 with MC



> -----Original Message-----
> From: Greg KH [mailto:gregkh@...uxfoundation.org]
> Sent: Tuesday, October 27, 2015 12:40 AM
> To: Pan Lijun-B44306 <Lijun.Pan@...escale.com>
> Cc: arnd@...db.de; devel@...verdev.osuosl.org; linux-
> kernel@...r.kernel.org; Hamciuc Bogdan-BHAMCIU1
> <bhamciu1@...escale.com>; Sharma Bhupesh-B45370
> <bhupesh.sharma@...escale.com>; Rivera Jose-B46482
> <German.Rivera@...escale.com>; agraf@...e.de; Yoder Stuart-B08248
> <stuart.yoder@...escale.com>; Erez Nir-RM30794 <nir.erez@...escale.com>;
> katz Itai-RM05202 <itai.katz@...escale.com>; Wood Scott-B07421
> <scottwood@...escale.com>; Li Yang-Leo-R58472 <LeoLi@...escale.com>;
> Marginean Alexandru-R89243 <R89243@...escale.com>;
> dan.carpenter@...cle.com; Schmitt Richard-B43082
> <richard.schmitt@...escale.com>
> Subject: Re: [PATCH 4/5] staging: fsl-mc: bus rescan attribute to sync kernel
> with MC
> 
> On Sun, Oct 25, 2015 at 05:41:22PM -0500, Lijun Pan wrote:
> > Introduce the rescan attribute as a bus attribute to synchronize the
> > fsl-mc bus objects and the MC firmware.
> >
> > To rescan the fsl-mc bus, e.g.,
> > echo 1 > /sys/bus/fsl-mc/rescan
> >
> > Signed-off-by: Lijun Pan <Lijun.Pan@...escale.com>
> > ---
> >  drivers/staging/fsl-mc/bus/mc-bus.c | 46
> > +++++++++++++++++++++++++++++++++++++
> >  1 file changed, 46 insertions(+)
> >
> > diff --git a/drivers/staging/fsl-mc/bus/mc-bus.c
> > b/drivers/staging/fsl-mc/bus/mc-bus.c
> > index 33a56ad..f1baad7 100644
> > --- a/drivers/staging/fsl-mc/bus/mc-bus.c
> > +++ b/drivers/staging/fsl-mc/bus/mc-bus.c
> > @@ -144,11 +144,57 @@ static const struct attribute_group
> *fsl_mc_dev_groups[] = {
> >  	NULL,
> >  };
> >
> > +static int scan_fsl_mc_bus(struct device *dev, void *data) {
> > +	struct fsl_mc_device *root_mc_dev;
> > +	struct fsl_mc_bus *root_mc_bus;
> > +
> > +	if (is_root_dprc(dev)) {
> > +		root_mc_dev = to_fsl_mc_device(dev);
> > +		root_mc_bus = to_fsl_mc_bus(root_mc_dev);
> > +		mutex_lock(&root_mc_bus->scan_mutex);
> > +		dprc_scan_objects(root_mc_dev);
> > +		mutex_unlock(&root_mc_bus->scan_mutex);
> > +	}
> > +
> > +	return 0;
> > +}
> > +
> > +static ssize_t bus_rescan_store(struct bus_type *bus,
> > +			    const char *buf, size_t count) {
> > +	unsigned long val;
> > +
> > +	if (kstrtoul(buf, 0, &val) < 0)
> > +		return -EINVAL;
> > +
> > +	if (val)
> > +		bus_for_each_dev(bus, NULL, NULL, scan_fsl_mc_bus);
> > +
> > +	return count;
> > +}
> > +static BUS_ATTR(rescan, (S_IWUSR|S_IWGRP), NULL, bus_rescan_store);
> > +
> > +static struct attribute *fsl_mc_bus_attrs[] = {
> > +	&bus_attr_rescan.attr,
> > +	NULL,
> > +};
> > +
> > +static const struct attribute_group fsl_mc_bus_group = {
> > +	.attrs = fsl_mc_bus_attrs,
> > +};
> > +
> > +static const struct attribute_group *fsl_mc_bus_groups[] = {
> > +	&fsl_mc_bus_group,
> > +	NULL,
> > +};
> > +
> >  struct bus_type fsl_mc_bus_type = {
> >  	.name = "fsl-mc",
> >  	.match = fsl_mc_bus_match,
> >  	.uevent = fsl_mc_bus_uevent,
> >  	.dev_groups = fsl_mc_dev_groups,
> > +	.bus_groups = fsl_mc_bus_groups,
> >  };
> >  EXPORT_SYMBOL_GPL(fsl_mc_bus_type);
> 
> No documentation update as well?

Hi Greg,

Are you saying something like adding
Documentation/ABI/testing/sysfs-bus-fsl-mc? For the fsl-mc bus
Documentation/ABI/testing/sysfs-devices-dprc? For the dprc device?
Or 
drivers/staging/fsl-mc/[README.txt | TODO]?

Thanks
Lijun


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/