* [PATCH] symlink only rules support
@ 2004-02-22 9:50 Andrey Borzenkov
2004-02-28 1:04 ` Greg KH
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Andrey Borzenkov @ 2004-02-22 9:50 UTC (permalink / raw)
To: linux-hotplug
[-- Attachment #1: Type: text/plain, Size: 483 bytes --]
On Tue, Feb 17, 2004 at 09:29:53AM -0800, Greg KH wrote:
> >
> > What about the second part (allow symlink-only rules)?
>
> Yes, I think that would be good to have too. Just split the patches up
> into two pieces, only one patch per feature/fix.
>
Slightly modified version. It is less aggressive in that it just adds
symlink collection but otherwise behaves exactly like current code.
the patch is relative to previous one (configuration directory support).
regards
-andrey
[-- Attachment #2: udev018_symlink.patch --]
[-- Type: text/plain, Size: 2289 bytes --]
--- udev-018/namedev.c.symlink 2004-02-21 21:57:19.000000000 +0300
+++ udev-018/namedev.c 2004-02-21 22:19:59.747075352 +0300
@@ -805,17 +805,30 @@ int namedev_name_device(struct sysfs_cla
list_for_each_entry(dev, &config_device_list, node) {
dbg("process rule");
if (match_rule(dev, class_dev, udev, sysfs_device) == 0) {
- if (dev->name[0] == '\0') {
+ if (dev->name[0] == '\0' && dev->symlink[0] == '\0') {
info("configured rule in '%s' at line %i applied, '%s' is ignored",
dev->config_file, dev->config_line, udev->kernel_name);
return -1;
}
- info("configured rule in '%s' at line %i applied, '%s' becomes '%s'",
- dev->config_file, dev->config_line, udev->kernel_name, dev->name);
- strfieldcpy(udev->name, dev->name);
- strfieldcpy(udev->symlink, dev->symlink);
- goto found;
+ if (dev->symlink[0] != '\0') {
+ char temp[NAME_MAX];
+
+ info("configured rule in '%s' at line %i applied, added symlink '%s'",
+ dev->config_file, dev->config_line, dev->symlink);
+ /* do not clobber dev */
+ strfieldcpy(temp, dev->symlink);
+ apply_format(udev, temp, class_dev, sysfs_device);
+ strfieldcat(udev->symlink, temp);
+ strfieldcat(udev->symlink, " ");
+ }
+
+ if (dev->name[0] != '\0') {
+ info("configured rule in '%s' at line %i applied, '%s' becomes '%s'",
+ dev->config_file, dev->config_line, udev->kernel_name, dev->name);
+ strfieldcpy(udev->name, dev->name);
+ goto found;
+ }
}
}
@@ -826,7 +839,6 @@ int namedev_name_device(struct sysfs_cla
found:
/* substitute placeholder */
apply_format(udev, udev->name, class_dev, sysfs_device);
- apply_format(udev, udev->symlink, class_dev, sysfs_device);
udev->partitions = dev->partitions;
done:
perm = find_perm(udev->name);
--- udev-018/udev.h.symlink 2004-02-19 21:38:37.000000000 +0300
+++ udev-018/udev.h 2004-02-21 22:21:44.431160952 +0300
@@ -61,6 +61,12 @@ do { \
strncpy(to, from, sizeof(to)-1); \
} while (0)
+#define strfieldcat(to, from) \
+do { \
+ to[sizeof(to)-1] = '\0'; \
+ strncat(to, from, sizeof(to)-1); \
+} while (0)
+
extern int udev_add_device(char *path, char *subsystem, int fake);
extern int udev_remove_device(char *path, char *subsystem);
extern void udev_init_config(void);
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] symlink only rules support
2004-02-22 9:50 [PATCH] symlink only rules support Andrey Borzenkov
@ 2004-02-28 1:04 ` Greg KH
2004-02-28 14:57 ` Andrey Borzenkov
2004-03-01 22:27 ` Greg KH
2 siblings, 0 replies; 4+ messages in thread
From: Greg KH @ 2004-02-28 1:04 UTC (permalink / raw)
To: linux-hotplug
On Sun, Feb 22, 2004 at 12:50:13PM +0300, Andrey Borzenkov wrote:
> On Tue, Feb 17, 2004 at 09:29:53AM -0800, Greg KH wrote:
> > >
> > > What about the second part (allow symlink-only rules)?
> >
> > Yes, I think that would be good to have too. Just split the patches up
> > into two pieces, only one patch per feature/fix.
> >
>
> Slightly modified version. It is less aggressive in that it just adds
> symlink collection but otherwise behaves exactly like current code.
>
> the patch is relative to previous one (configuration directory support).
Hm, as Kay modified that one, and things have changed enough, can you
resend this against the 019 release?
thanks,
greg k-h
-------------------------------------------------------
SF.Net is sponsored by: Speed Start Your Linux Apps Now.
Build and deploy apps & Web services for Linux with
a free DVD software kit from IBM. Click Now!
http://ads.osdn.com/?ad_id\x1356&alloc_id438&op=click
_______________________________________________
Linux-hotplug-devel mailing list http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] symlink only rules support
2004-02-22 9:50 [PATCH] symlink only rules support Andrey Borzenkov
2004-02-28 1:04 ` Greg KH
@ 2004-02-28 14:57 ` Andrey Borzenkov
2004-03-01 22:27 ` Greg KH
2 siblings, 0 replies; 4+ messages in thread
From: Andrey Borzenkov @ 2004-02-28 14:57 UTC (permalink / raw)
To: linux-hotplug
[-- Attachment #1: Type: text/plain, Size: 757 bytes --]
On Saturday 28 February 2004 04:04, Greg KH wrote:
> On Sun, Feb 22, 2004 at 12:50:13PM +0300, Andrey Borzenkov wrote:
> > On Tue, Feb 17, 2004 at 09:29:53AM -0800, Greg KH wrote:
> > > > What about the second part (allow symlink-only rules)?
> > >
> > > Yes, I think that would be good to have too. Just split the patches up
> > > into two pieces, only one patch per feature/fix.
> >
> > Slightly modified version. It is less aggressive in that it just adds
> > symlink collection but otherwise behaves exactly like current code.
> >
> > the patch is relative to previous one (configuration directory support).
>
> Hm, as Kay modified that one, and things have changed enough, can you
> resend this against the 019 release?
>
attached
regdards
-andrey
[-- Attachment #2: udev019_symlink.patch --]
[-- Type: text/x-diff, Size: 1857 bytes --]
--- udev-019/namedev.c.symlink 2004-02-28 04:11:47.000000000 +0300
+++ udev-019/namedev.c 2004-02-28 17:49:42.621962496 +0300
@@ -816,17 +816,31 @@ int namedev_name_device(struct sysfs_cla
list_for_each_entry(dev, &config_device_list, node) {
dbg("process rule");
if (match_rule(dev, class_dev, udev, sysfs_device) == 0) {
- if (dev->name[0] == '\0') {
+ if (dev->name[0] == '\0' && dev->symlink[0] == '\0') {
info("configured rule in '%s' at line %i applied, '%s' is ignored",
udev_rules_filename, dev->config_line, udev->kernel_name);
return -1;
}
- info("configured rule in '%s' at line %i applied, '%s' becomes '%s'",
- udev_rules_filename, dev->config_line, udev->kernel_name, dev->name);
- strfieldcpy(udev->name, dev->name);
- strfieldcpy(udev->symlink, dev->symlink);
- goto found;
+ if (dev->symlink[0] != '\0') {
+ char temp[NAME_MAX];
+
+ info("configured rule in '%s' at line %i applied, added symlink '%s'",
+ udev_rules_filename, dev->config_line, dev->symlink);
+ /* do not clobber dev */
+ strfieldcpy(temp, dev->symlink);
+ apply_format(udev, temp, sizeof(temp),
+ class_dev, sysfs_device);
+ strfieldcat(udev->symlink, temp);
+ strfieldcat(udev->symlink, " ");
+ }
+
+ if (dev->name[0] != '\0') {
+ info("configured rule in '%s' at line %i applied, '%s' becomes '%s'",
+ udev_rules_filename, dev->config_line, udev->kernel_name, dev->name);
+ strfieldcpy(udev->name, dev->name);
+ goto found;
+ }
}
}
@@ -838,8 +852,6 @@ found:
/* substitute placeholder */
apply_format(udev, udev->name, sizeof(udev->name),
class_dev, sysfs_device);
- apply_format(udev, udev->symlink, sizeof(udev->symlink),
- class_dev, sysfs_device);
udev->partitions = dev->partitions;
done:
perm = find_perm(udev->name);
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] symlink only rules support
2004-02-22 9:50 [PATCH] symlink only rules support Andrey Borzenkov
2004-02-28 1:04 ` Greg KH
2004-02-28 14:57 ` Andrey Borzenkov
@ 2004-03-01 22:27 ` Greg KH
2 siblings, 0 replies; 4+ messages in thread
From: Greg KH @ 2004-03-01 22:27 UTC (permalink / raw)
To: linux-hotplug
On Sat, Feb 28, 2004 at 05:57:24PM +0300, Andrey Borzenkov wrote:
> On Saturday 28 February 2004 04:04, Greg KH wrote:
> > On Sun, Feb 22, 2004 at 12:50:13PM +0300, Andrey Borzenkov wrote:
> > > On Tue, Feb 17, 2004 at 09:29:53AM -0800, Greg KH wrote:
> > > > > What about the second part (allow symlink-only rules)?
> > > >
> > > > Yes, I think that would be good to have too. Just split the patches up
> > > > into two pieces, only one patch per feature/fix.
> > >
> > > Slightly modified version. It is less aggressive in that it just adds
> > > symlink collection but otherwise behaves exactly like current code.
> > >
> > > the patch is relative to previous one (configuration directory support).
> >
> > Hm, as Kay modified that one, and things have changed enough, can you
> > resend this against the 019 release?
> >
>
> attached
Thanks for persisting, I've applied this. Care to write up a
documentation patch so that people know how to use this now?
thanks,
greg k-h
-------------------------------------------------------
SF.Net is sponsored by: Speed Start Your Linux Apps Now.
Build and deploy apps & Web services for Linux with
a free DVD software kit from IBM. Click Now!
http://ads.osdn.com/?ad_id\x1356&alloc_id438&op=click
_______________________________________________
Linux-hotplug-devel mailing list http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2004-03-01 22:27 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-02-22 9:50 [PATCH] symlink only rules support Andrey Borzenkov
2004-02-28 1:04 ` Greg KH
2004-02-28 14:57 ` Andrey Borzenkov
2004-03-01 22:27 ` Greg KH
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).